Skip to content

Commit

Permalink
Fix assertion in testReturnsImmediatelyIfVoteAlreadyWithdrawn (#98570)
Browse files Browse the repository at this point in the history
This test is today asserting that the _initial_ state has all the
expected exclusions, but it should be checking the _final_ state.

Relates #98386
  • Loading branch information
DaveCTurner authored Aug 18, 2023
1 parent 9076250 commit 25a36f9
Showing 1 changed file with 3 additions and 2 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -232,8 +232,9 @@ public void testReturnsImmediatelyIfVoteAlreadyWithdrawn() {
new AddVotingConfigExclusionsRequest("other1"),
expectSuccess(r -> {
assertNotNull(r);
assertThat(clusterService.getClusterApplierService().state().getVotingConfigExclusions(), contains(otherNode1Exclusion));
assertAllExclusionsApplied(state);
final var finalState = clusterService.getClusterApplierService().state();
assertThat(finalState.getVotingConfigExclusions(), contains(otherNode1Exclusion));
assertAllExclusionsApplied(finalState);
countDownLatch.countDown();
})
);
Expand Down

0 comments on commit 25a36f9

Please sign in to comment.