Skip to content

Commit 5c7a7ca

Browse files
committed
SERVER-31512 Use awaitNodesAgreeOnPrimary instead of waitForState in replsetprio1.js to improve robustness of test.
1 parent c7286b3 commit 5c7a7ca

File tree

1 file changed

+3
-2
lines changed

1 file changed

+3
-2
lines changed

jstests/replsets/replsetprio1.js

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -16,7 +16,8 @@
1616
});
1717

1818
// 2 should be master (give this a while to happen, as other nodes might first be elected)
19-
replTest.waitForState(nodes[2], ReplSetTest.State.PRIMARY);
19+
replTest.awaitNodesAgreeOnPrimary(replTest.kDefaultTimeoutMS, nodes, 2);
20+
2021
// wait for 1 to not appear to be master (we are about to make it master and need a clean slate
2122
// here)
2223
replTest.waitForState(nodes[1], ReplSetTest.State.SECONDARY);
@@ -42,7 +43,7 @@
4243

4344
// bring 2 back up, 2 should wait until caught up and then become master
4445
replTest.restart(2);
45-
replTest.waitForState(nodes[2], ReplSetTest.State.PRIMARY);
46+
replTest.awaitNodesAgreeOnPrimary(replTest.kDefaultTimeoutMS, nodes, 2);
4647

4748
// make sure nothing was rolled back
4849
master = replTest.getPrimary();

0 commit comments

Comments
 (0)