diff options
| author | Esha Maharishi <esha.maharishi@mongodb.com> | 2019-02-11 10:58:25 -0500 |
|---|---|---|
| committer | Esha Maharishi <esha.maharishi@mongodb.com> | 2019-02-11 22:30:57 -0500 |
| commit | d8cef1c71c1e9d8c59e354e08db121046af68c21 (patch) | |
| tree | 7a44b2e765c6691879468cd8f907a3f50dfd0d87 /jstests | |
| parent | e85c37fe4c1663ec2be1532b9dd94c2d19fd7cfc (diff) | |
| download | mongo-d8cef1c71c1e9d8c59e354e08db121046af68c21.tar.gz | |
SERVER-39474 Fix incorrect use of 'skip' parameter to configureFailPoint in txn_two_phase_commit_failover.js
Diffstat (limited to 'jstests')
| -rw-r--r-- | jstests/sharding/txn_two_phase_commit_failover.js | 6 | ||||
| -rw-r--r-- | jstests/sharding/txn_two_phase_commit_killop.js | 3 |
2 files changed, 5 insertions, 4 deletions
diff --git a/jstests/sharding/txn_two_phase_commit_failover.js b/jstests/sharding/txn_two_phase_commit_failover.js index ba185f33aed..85b6880fc97 100644 --- a/jstests/sharding/txn_two_phase_commit_failover.js +++ b/jstests/sharding/txn_two_phase_commit_failover.js @@ -158,8 +158,7 @@ TestData.skipCheckingUUIDsConsistentAcrossCluster = true; assert.commandWorked(coordPrimary.adminCommand({ configureFailPoint: failpointData.failpoint, - mode: "alwaysOn", - skip: (failpointData.skip ? failpointData.skip : 0), + mode: {skip: (failpointData.skip ? failpointData.skip : 0)}, })); // Run commitTransaction through a parallel shell. @@ -269,6 +268,9 @@ TestData.skipCheckingUUIDsConsistentAcrossCluster = true; failpointDataArr.forEach(function(failpointData) { if (failpointData.failpoint == "hangWhileTargetingRemoteHost") { failpointData.numTimesShouldBeHit++; + if (failpointData.skip) { + failpointData.skip++; + } } }); diff --git a/jstests/sharding/txn_two_phase_commit_killop.js b/jstests/sharding/txn_two_phase_commit_killop.js index 415b99286b5..55245306b23 100644 --- a/jstests/sharding/txn_two_phase_commit_killop.js +++ b/jstests/sharding/txn_two_phase_commit_killop.js @@ -98,8 +98,7 @@ // Turn on failpoint to make the coordinator hang at a the specified point. assert.commandWorked(coordinator.adminCommand({ configureFailPoint: failpointData.failpoint, - mode: "alwaysOn", - skip: (failpointData.skip ? failpointData.skip : 0), + mode: {skip: (failpointData.skip ? failpointData.skip : 0)}, })); // Run commitTransaction through a parallel shell. |