Uploaded image for project: 'Nuxeo Platform'
  1. Nuxeo Platform
  2. NXP-21941

Always exclude versions when querying for Drive synchronization roots

    XMLWordPrintable

    Details

    • Type: Bug
    • Status: Resolved
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 9.1
    • Component/s: Nuxeo Drive

      Description

      Once the issues related to NXP-21275 were merged in master we got a systematic failure at https://github.com/nuxeo/nuxeo-drive/blob/e4375bc6549b4fada44da5aced2e81154ac98011/nuxeo-drive-client/tests/test_concurrent_synchronization.py#L52

      22:22:13 [SLAVE] [INFO]      [exec] nuxeo-drive-client/tests/test_concurrent_synchronization.py F
      22:22:13 [SLAVE] [INFO]      [exec] =========================== short test summary info ============================
      22:22:13 [SLAVE] [INFO]      [exec] FAIL nuxeo-drive-client/tests/test_concurrent_synchronization.py::TestConcurrentSynchronization::test_delete_local_folder_2_clients
      22:22:13 [SLAVE] [INFO]      [exec] =================================== FAILURES ===================================
      22:22:13 [SLAVE] [INFO]      [exec] _______ TestConcurrentSynchronization.test_delete_local_folder_2_clients _______
      22:22:13 [SLAVE] [INFO]      [exec] self = <tests.test_concurrent_synchronization.TestConcurrentSynchronization testMethod=test_delete_local_folder_2_clients>
      22:22:13 [SLAVE] [INFO]      [exec]     def test_delete_local_folder_2_clients(self):
      22:22:13 [SLAVE] [INFO]      [exec]         # Get local clients for each device and remote client
      22:22:13 [SLAVE] [INFO]      [exec]         local1 = self.local_client_1
      22:22:13 [SLAVE] [INFO]      [exec]         local2 = self.local_client_2
      22:22:13 [SLAVE] [INFO]      [exec]         remote = self.remote_document_client_1
      22:22:13 [SLAVE] [INFO]      [exec]         # Check synchronization roots for drive1,
      22:22:13 [SLAVE] [INFO]      [exec]         # there should be 1, the test workspace
      22:22:13 [SLAVE] [INFO]      [exec]         sync_roots = remote.get_roots()
      22:22:13 [SLAVE] [INFO]      [exec] >       self.assertEqual(len(sync_roots), 1)
      22:22:13 [SLAVE] [INFO]      [exec] E       AssertionError: 2 != 1
      

      This is because at the time of https://github.com/nuxeo/nuxeo/tree/51ab7300b1baa28d045836624e8fe4f437a86be2 there was un unwanted behavior in Nuxeo that would version a Workspace if changed by a user different from the last contributor. Drive was also fetching the version of the Nuxeo Drive Test Workspace synchronization root, thus the failure.
      Shoud be fixed by NXP-21873.

      Nevertheless, we want to enforce all the cases where we query for synchronization roots to exclude versions.

        Attachments

          Issue Links

            Activity

              People

              • Votes:
                0 Vote for this issue
                Watchers:
                2 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: