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 374622 2ed30cf5eeb6a86418b440f922c3f1ab8a9c9823
parent 374621 6878846abd906e547c8b05940a0fa66281c33890
child 374623 b92b8655e634c865271a079b7709cb466fa44bdf
push id10863
push userjlorenzo@mozilla.com
push dateMon, 06 Mar 2017 23:02:23 +0000
treeherdermozilla-aurora@0931190cd725 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersme
bugs1317223
milestone54.0a1
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" }),