Opush
  1. Opush
  2. OP-168

Wrong item synchronization state when many changes are discovered at the same time

    Details

    • Type: Bug Bug
    • Status: Closed Closed
    • Priority: Normal Normal
    • Resolution: Fixed
    • Affects Version/s: 3.0.6
    • Fix Version/s: 3.1.0
    • Labels:
      None
    • Rank:
      6758

      Description

      Pre-conditions

      Find the "WindowSize" of the tested device for the tested collection.
      1) synchronize the collection that you want to test, e.g. INBOX, calendar, contacts, etc
      2) search in your opush logs the <WindowSize>10</WindowSize> sent by your device.
      3) in the reproduction steps, XX is the window size of the collection. Here it's 10.

      Note that your device has a different window size for every collection type

      Reproduction steps

      For the calendar test

      1. As usera with obm-ui, create XX+5 events for tomorrow

      For the address book test

      1. As usera with obm-ui, create XX+5 contacts

      For the email test

      1. As usera I send XX+5 emails to himself

      Then

      1. As usera with a device, I synchronize my collection and receive all new items
      2. As usera from obm-ui, I delete all created items but one
      3. As usera from obm-ui, I modify the remaining item
      4. As usera from obm-ui, I create one new item
      5. As usera with a device, I synchronize my collection

      Current behavior

      1. OK, the change and the new item is received
      2. NOK, some items have not been deleted from my device

      Expected behavior

      1. OK, the change and the new item is received
      2. OK, every item is deleted from my device

      Additional informations

        Issue Links

          Activity

          Hide
          Thomas HILAIRE added a comment -

          WIP: Tests and code are available at stash-opush-thilaire/OP-168

          Show
          Thomas HILAIRE added a comment - WIP : Tests and code are available at stash-opush-thilaire/ OP-168
          Hide
          Ba Tuan NGO added a comment - - edited

          Impacts:
          _ Windowing should add all changes (new-change-delete).

          Tested on:
          _ opush_3.1.0~alpha0~git20150302.183202.5f72bfc-1 with obm 3.1.3-rc1 on Debian Wheezy. HTC One 4.4.2 and Android 4.4.2

          Tests done:
          _ Every changes from the server are synchronized.

          Show
          Ba Tuan NGO added a comment - - edited Impacts : _ Windowing should add all changes (new-change-delete). Tested on : _ opush_3.1.0~alpha0~git20150302.183202.5f72bfc-1 with obm 3.1.3-rc1 on Debian Wheezy. HTC One 4.4.2 and Android 4.4.2 Tests done : _ Every changes from the server are synchronized.
          Hide
          Jenkins Continuous Integration Server added a comment -

          ABORTED: Integrated in opush-master #206
          OP-168 Windowing adds all changes (thilaire: 5665c03e12ebda969ed8543fe57deff9a957ecdc)

          • integration-testing/opush/src/test/resources/org/obm/opush/windowing/windowing.feature
          • push-dao-testsuite/src/main/java/org/obm/push/dao/testsuite/WindowingDaoTest.java
          • push-dao-cassandra/src/main/java/org/obm/push/cassandra/dao/WindowingDaoCassandraImpl.java
          • integration-testing/opush/src/test/java/org/obm/opush/windowing/WindowingStepdefs.java
          • integration-testing/opush/src/test/java/org/obm/opush/command/sync/SyncHandlerWithBackendTest.java
          Show
          Jenkins Continuous Integration Server added a comment - ABORTED: Integrated in opush-master #206 OP-168 Windowing adds all changes (thilaire: 5665c03e12ebda969ed8543fe57deff9a957ecdc ) integration-testing/opush/src/test/resources/org/obm/opush/windowing/windowing.feature push-dao-testsuite/src/main/java/org/obm/push/dao/testsuite/WindowingDaoTest.java push-dao-cassandra/src/main/java/org/obm/push/cassandra/dao/WindowingDaoCassandraImpl.java integration-testing/opush/src/test/java/org/obm/opush/windowing/WindowingStepdefs.java integration-testing/opush/src/test/java/org/obm/opush/command/sync/SyncHandlerWithBackendTest.java
          Hide
          Jenkins Continuous Integration Server added a comment -

          ABORTED: Integrated in opush-master #207
          OP-168 Windowing adds all changes (thilaire: 2e2ccffd2c6ce6ba68168a53491aff109fe180a5)

          • integration-testing/opush/src/test/java/org/obm/opush/windowing/WindowingStepdefs.java
          • push-dao-testsuite/src/main/java/org/obm/push/dao/testsuite/WindowingDaoTest.java
          • push-dao-cassandra/src/main/java/org/obm/push/cassandra/dao/WindowingDaoCassandraImpl.java
          • integration-testing/opush/src/test/java/org/obm/opush/command/sync/SyncHandlerWithBackendTest.java
          • integration-testing/opush/src/test/resources/org/obm/opush/windowing/windowing.feature
          Show
          Jenkins Continuous Integration Server added a comment - ABORTED: Integrated in opush-master #207 OP-168 Windowing adds all changes (thilaire: 2e2ccffd2c6ce6ba68168a53491aff109fe180a5 ) integration-testing/opush/src/test/java/org/obm/opush/windowing/WindowingStepdefs.java push-dao-testsuite/src/main/java/org/obm/push/dao/testsuite/WindowingDaoTest.java push-dao-cassandra/src/main/java/org/obm/push/cassandra/dao/WindowingDaoCassandraImpl.java integration-testing/opush/src/test/java/org/obm/opush/command/sync/SyncHandlerWithBackendTest.java integration-testing/opush/src/test/resources/org/obm/opush/windowing/windowing.feature
          Hide
          Antoine DUPRAT added a comment -

          OK

          Tested on Debian wheezy with Opush 3.1.0~alpha0~git20150327.162644.3b1273a on Nexus 4 (5.0.1)

          Show
          Antoine DUPRAT added a comment - OK Tested on Debian wheezy with Opush 3.1.0~alpha0~git20150327.162644.3b1273a on Nexus 4 (5.0.1)

            People

            • Assignee:
              Ba Tuan NGO
              Reporter:
              Thomas HILAIRE
            • Votes:
              0 Vote for this issue
              Watchers:
              4 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: