diff options
| author | William Schultz <william.schultz@mongodb.com> | 2017-04-21 14:40:36 -0400 |
|---|---|---|
| committer | William Schultz <william.schultz@mongodb.com> | 2017-04-21 16:36:05 -0400 |
| commit | 460fb74a52bca8b71cc2e70486df37d9f3a6ce39 (patch) | |
| tree | 990554d9bba7ba618a01a0478c070e1ea82ace38 | |
| parent | 9b7d79e1cc8b8a233ea17072fb93c8cf5c42f039 (diff) | |
| download | mongo-460fb74a52bca8b71cc2e70486df37d9f3a6ce39.tar.gz | |
SERVER-28909 double_rollback.js should assert that database commands succeed
| -rw-r--r-- | jstests/replsets/double_rollback.js | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/jstests/replsets/double_rollback.js b/jstests/replsets/double_rollback.js index 03e787343e5..c7adc95415d 100644 --- a/jstests/replsets/double_rollback.js +++ b/jstests/replsets/double_rollback.js @@ -106,8 +106,8 @@ checkLog.contains( nodes[2], "remote oplog does not contain entry with optime matching our required optime"); - var node0RBID = nodes[0].adminCommand('replSetGetRBID').rbid; - var node1RBID = nodes[1].adminCommand('replSetGetRBID').rbid; + var node0RBID = assert.commandWorked(nodes[0].adminCommand('replSetGetRBID')).rbid; + var node1RBID = assert.commandWorked(nodes[1].adminCommand('replSetGetRBID')).rbid; jsTestLog("Reconnect all nodes."); nodes[0].reconnect(nodes[1]); @@ -124,8 +124,8 @@ // Check that rollback happened on node 0, but not on node 2 since it had already rolled back // and just needed to finish applying ops to reach minValid. - assert.neq(node0RBID, nodes[0].adminCommand('replSetGetRBID').rbid); - assert.eq(node1RBID, nodes[1].adminCommand('replSetGetRBID').rbid); + assert.neq(node0RBID, assert.commandWorked(nodes[0].adminCommand('replSetGetRBID')).rbid); + assert.eq(node1RBID, assert.commandWorked(nodes[1].adminCommand('replSetGetRBID')).rbid); // Node 1 should still be primary, and should now be able to satisfy majority writes again. assert.writeOK(nodes[1].getDB(dbName).getCollection(collName + "4").insert({a: 4}, { |