Skip to content

Commit 2be01d6

Browse files
Refactor per code review suggestion
1 parent 7f8b59b commit 2be01d6

File tree

1 file changed

+4
-6
lines changed

1 file changed

+4
-6
lines changed

server/src/internalClusterTest/java/org/elasticsearch/action/admin/cluster/state/TransportClusterStateActionDisruptionIT.java

Lines changed: 4 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -21,7 +21,6 @@
2121
import org.elasticsearch.discovery.MasterNotDiscoveredException;
2222
import org.elasticsearch.gateway.GatewayService;
2323
import org.elasticsearch.plugins.Plugin;
24-
import org.elasticsearch.test.ClusterServiceUtils;
2524
import org.elasticsearch.test.ESIntegTestCase;
2625
import org.elasticsearch.test.transport.MockTransportService;
2726
import org.elasticsearch.transport.TransportService;
@@ -213,12 +212,11 @@ public void runRepeatedlyWhileChangingMaster(Runnable runnable) throws Exception
213212
}
214213

215214
final String nonMasterNode = randomValueOtherThan(masterName, () -> randomFrom(internalCluster().getNodeNames()));
216-
var newMasterNodeListener = ClusterServiceUtils.addTemporaryStateListener(
217-
internalCluster().clusterService(nonMasterNode),
218-
state -> Optional.ofNullable(state.nodes().getMasterNode()).map(m -> m.getName().equals(masterName) == false).orElse(false),
219-
TEST_REQUEST_TIMEOUT
215+
awaitClusterState(
216+
logger,
217+
nonMasterNode,
218+
state -> Optional.ofNullable(state.nodes().getMasterNode()).map(m -> m.getName().equals(masterName) == false).orElse(false)
220219
);
221-
safeAwait(newMasterNodeListener, TEST_REQUEST_TIMEOUT);
222220

223221
shutdown.set(true);
224222
assertingThread.join();

0 commit comments

Comments
 (0)