Bug 1022215 - Increase the number of seconds the sync scheduler tests are willing to wait before failing from 1 to 3. r?markh draft
authorThom Chiovoloni <tchiovoloni@mozilla.com>
Tue, 20 Sep 2016 15:03:06 -0400
changeset 415615 b69af0f3af3c0c6d4a00558b67a9de5153d4711b
parent 415094 857e5699df7baaabeceafd3601ff19104507cfac
child 531651 184757a421f0d7cf3399865976f685176611d6d6
push id29915
push userbmo:tchiovoloni@mozilla.com
push dateTue, 20 Sep 2016 19:06:07 +0000
reviewersmarkh
bugs1022215
milestone51.0a1
Bug 1022215 - Increase the number of seconds the sync scheduler tests are willing to wait before failing from 1 to 3. r?markh MozReview-Commit-ID: IU2LW9xDWDS
services/sync/tests/unit/test_syncscheduler.js
--- a/services/sync/tests/unit/test_syncscheduler.js
+++ b/services/sync/tests/unit/test_syncscheduler.js
@@ -837,19 +837,19 @@ add_identity_test(this, function* test_s
   // Sanity check that we picked the right value for BACKOFF:
   do_check_true(scheduler.syncInterval < BACKOFF * 1000);
 
   // Turn on server maintenance and sync again.
   serverBackoff = true;
   Service.sync();
 
   do_check_true(Status.backoffInterval >= BACKOFF * 1000);
-  // Allowing 1 second worth of of leeway between when Status.minimumNextSync
+  // Allowing 3 seconds worth of of leeway between when Status.minimumNextSync
   // was set and when this line gets executed.
-  let minimumExpectedDelay = (BACKOFF - 1) * 1000;
+  let minimumExpectedDelay = (BACKOFF - 3) * 1000;
   do_check_true(Status.minimumNextSync >= Date.now() + minimumExpectedDelay);
 
   // Verify that the next sync is actually going to wait that long.
   do_check_true(scheduler.nextSync >= Date.now() + minimumExpectedDelay);
   do_check_true(scheduler.syncTimer.delay >= minimumExpectedDelay);
 
   yield cleanUpAndGo(server);
 });
@@ -896,19 +896,19 @@ add_identity_test(this, function* test_s
   do_check_true(scheduler.syncInterval < BACKOFF * 1000);
 
   // Turn on server maintenance and sync again.
   serverMaintenance = true;
   Service.sync();
 
   do_check_true(Status.enforceBackoff);
   do_check_true(Status.backoffInterval >= BACKOFF * 1000);
-  // Allowing 1 second worth of of leeway between when Status.minimumNextSync
+  // Allowing 3 seconds worth of of leeway between when Status.minimumNextSync
   // was set and when this line gets executed.
-  let minimumExpectedDelay = (BACKOFF - 1) * 1000;
+  let minimumExpectedDelay = (BACKOFF - 3) * 1000;
   do_check_true(Status.minimumNextSync >= Date.now() + minimumExpectedDelay);
 
   // Verify that the next sync is actually going to wait that long.
   do_check_true(scheduler.nextSync >= Date.now() + minimumExpectedDelay);
   do_check_true(scheduler.syncTimer.delay >= minimumExpectedDelay);
 
   yield cleanUpAndGo(server);
 });