Skip to content

Commit 72354a4

Browse files
committed
cleanup
1 parent b9fa7c4 commit 72354a4

File tree

1 file changed

+3
-3
lines changed

1 file changed

+3
-3
lines changed

server/src/test/java/org/elasticsearch/cluster/routing/allocation/decider/AllocationDecidersTests.java

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -44,7 +44,7 @@
4444
public class AllocationDecidersTests extends ESAllocationTestCase {
4545

4646
public void testCheckAllDecidersBeforeReturningYes() {
47-
var allDecisions = generateDecisions(() -> randomFrom(Decision.YES));
47+
var allDecisions = generateDecisions(() -> Decision.YES);
4848
var debugMode = randomFrom(RoutingAllocation.DebugMode.values());
4949
var expectedDecision = switch (debugMode) {
5050
case OFF -> Decision.YES;
@@ -57,7 +57,7 @@ public void testCheckAllDecidersBeforeReturningYes() {
5757

5858
public void testCheckAllDecidersBeforeReturningYesNotPreferred() {
5959
var debugMode = randomFrom(RoutingAllocation.DebugMode.values());
60-
var allDecisions = generateDecisions(Decision.YES_NOT_PREFERRED, () -> randomFrom(Decision.YES));
60+
var allDecisions = generateDecisions(Decision.YES_NOT_PREFERRED, () -> Decision.YES);
6161
var expectedDecision = switch (debugMode) {
6262
case OFF -> Decision.YES_NOT_PREFERRED;
6363
case EXCLUDE_YES_DECISIONS -> collectToMultiDecision(allDecisions, decision -> decision.preferred() != Decision.Preferred.YES);
@@ -67,7 +67,7 @@ public void testCheckAllDecidersBeforeReturningYesNotPreferred() {
6767
}
6868

6969
public void testCheckAllDecidersBeforeReturningThrottle() {
70-
var allDecisions = generateDecisions(Decision.THROTTLE, () -> randomFrom(Decision.YES));
70+
var allDecisions = generateDecisions(Decision.THROTTLE, () -> Decision.YES);
7171
var debugMode = randomFrom(RoutingAllocation.DebugMode.values());
7272
var expectedDecision = switch (debugMode) {
7373
case OFF -> Decision.THROTTLE;

0 commit comments

Comments
 (0)