Bug 1317223 (part 7) - test failure followup. r=me
☠☠ backed out by f8097ba36f5e ☠ ☠
authorMark Hammond <mhammond@skippinet.com.au>
Thu, 02 Mar 2017 19:21:53 +1100
changeset 394572 2ed30cf5eeb6a86418b440f922c3f1ab8a9c9823
parent 394571 6878846abd906e547c8b05940a0fa66281c33890
child 394573 b92b8655e634c865271a079b7709cb466fa44bdf
push id1468
push userasasaki@mozilla.com
push dateMon, 05 Jun 2017 19:31:07 +0000
treeherdermozilla-release@0641fc6ee9d1 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersme
bugs1317223
milestone54.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 1317223 (part 7) - test failure followup. r=me MozReview-Commit-ID: 6mw8e9yjrAc
services/sync/tests/unit/test_bookmark_repair_requestor.js
--- a/services/sync/tests/unit/test_bookmark_repair_requestor.js
+++ b/services/sync/tests/unit/test_bookmark_repair_requestor.js
@@ -105,17 +105,17 @@ add_task(async function test_requestor_n
     { object: "repair",
       method: "started",
       value: undefined,
       extra: { flowID, numIDs: 3 },
     },
     { object: "repair",
       method: "finished",
       value: undefined,
-      extra: { flowID, numIDs: 3, reason: undefined },
+      extra: { flowID, numIDs: 3 },
     }
   ]);
 });
 
 add_task(async function test_requestor_one_client_no_response() {
   let mockService = new MockService({ "client-a": makeClientRecord("client-a") });
   let requestor = NewBookmarkRepairRequestor(mockService);
   let validationInfo = {
@@ -166,17 +166,17 @@ add_task(async function test_requestor_o
     { object: "repair",
       method: "request",
       value: "upload",
       extra: { flowID, numIDs: 3, deviceID: "client-a" },
     },
     { object: "repair",
       method: "finished",
       value: undefined,
-      extra: { flowID, numIDs: 3, reason: undefined },
+      extra: { flowID, numIDs: 3 },
     }
   ]);
 });
 
 add_task(async function test_requestor_one_client_no_sync() {
   let mockService = new MockService({ "client-a": makeClientRecord("client-a") });
   let requestor = NewBookmarkRepairRequestor(mockService);
   let validationInfo = {
@@ -218,17 +218,17 @@ add_task(async function test_requestor_o
     { object: "repair",
       method: "abandon",
       value: "silent",
       extra: { flowID, deviceID: "client-a" },
     },
     { object: "repair",
       method: "finished",
       value: undefined,
-      extra: { flowID, numIDs: 3, reason: undefined },
+      extra: { flowID, numIDs: 3 },
     }
   ]);
 });
 
 add_task(async function test_requestor_latest_client_used() {
   let mockService = new MockService({
     "client-early": makeClientRecord("client-early", { serverLastModified: Date.now() - 10 }),
     "client-late": makeClientRecord("client-late", { serverLastModified: Date.now() }),
@@ -318,17 +318,17 @@ add_task(async function test_requestor_c
     { object: "repair",
       method: "response",
       value: "upload",
       extra: { flowID, deviceID: "client-b", numIDs: 3 },
     },
     { object: "repair",
       method: "finished",
       value: undefined,
-      extra: { flowID, numIDs: 0, reason: undefined },
+      extra: { flowID, numIDs: 0 },
     }
   ]);
 });
 
 add_task(async function test_requestor_success_responses() {
   let mockService = new MockService({
     "client-a": makeClientRecord("client-a"),
     "client-b": makeClientRecord("client-b"),
@@ -401,17 +401,17 @@ add_task(async function test_requestor_s
     { object: "repair",
       method: "response",
       value: "upload",
       extra: { flowID, deviceID: "client-b", numIDs: 1 },
     },
     { object: "repair",
       method: "finished",
       value: undefined,
-      extra: { flowID, numIDs: 0, reason: undefined },
+      extra: { flowID, numIDs: 0 },
     }
   ]);
 });
 
 add_task(async function test_client_suitability() {
   let mockService = new MockService({
     "client-a": makeClientRecord("client-a"),
     "client-b": makeClientRecord("client-b", { type: "mobile" }),