Bug 1022215 - Increase the number of seconds the sync scheduler tests are willing to wait before failing from 1 to 3. r=markh
authorThom Chiovoloni <tchiovoloni@mozilla.com>
Tue, 20 Sep 2016 15:03:06 -0400
changeset 314733 12235f039d128775f801aa7b3873011f5e82a60d
parent 314732 770a9b2e9ab6a88562f675d9787c735b51876472
child 314734 61591d5e84ba2c50f5461130d95dc57feade377a
push id30733
push usercbook@mozilla.com
push dateThu, 22 Sep 2016 09:50:59 +0000
treeherdermozilla-central@4aa20c3e211f [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersmarkh
bugs1022215
milestone52.0a1
first release with
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
last release without
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
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);
 });