Skip to content

Commit e8c8553

Browse files
committed
Refactor test with different disconnect code
1 parent 49808d2 commit e8c8553

File tree

2 files changed

+4
-11
lines changed

2 files changed

+4
-11
lines changed

test/framework/src/main/java/org/elasticsearch/test/AbstractMultiClustersTestCase.java

Lines changed: 0 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -264,15 +264,6 @@ public void close() throws IOException {
264264
}
265265
}
266266

267-
protected void removeClusterFromGroup(String clusterAlias) {
268-
clusterGroup = new ClusterGroup(
269-
clusterGroup.clusters.entrySet()
270-
.stream()
271-
.filter(e -> e.getKey().equals(clusterAlias) == false)
272-
.collect(Collectors.toMap(Map.Entry::getKey, Map.Entry::getValue))
273-
);
274-
}
275-
276267
static NodeConfigurationSource nodeConfigurationSource(Settings nodeSettings, Collection<Class<? extends Plugin>> nodePlugins) {
277268
final Settings.Builder builder = Settings.builder();
278269
builder.putList(DISCOVERY_SEED_HOSTS_SETTING.getKey()); // empty list disables a port scan for other nodes

x-pack/plugin/esql/src/internalClusterTest/java/org/elasticsearch/xpack/esql/action/CrossClusterCancellationIT.java

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -304,9 +304,11 @@ public void testCloseSkipUnavailable() throws Exception {
304304
request.pragmas(randomPragmas());
305305
var requestFuture = client().execute(EsqlQueryAction.INSTANCE, request);
306306
assertTrue(SimplePauseFieldPlugin.startEmitting.await(30, TimeUnit.SECONDS));
307+
// Disconnect the remotes
308+
for (var transportService : cluster(LOCAL_CLUSTER).getInstances(TransportService.class)) {
309+
transportService.getRemoteClusterService().getRemoteClusterConnection(REMOTE_CLUSTER).close();
310+
}
307311
SimplePauseFieldPlugin.allowEmitting.countDown();
308-
cluster(REMOTE_CLUSTER).close();
309-
removeClusterFromGroup(REMOTE_CLUSTER);
310312
try (var resp = requestFuture.actionGet()) {
311313
EsqlExecutionInfo executionInfo = resp.getExecutionInfo();
312314
assertNotNull(executionInfo);

0 commit comments

Comments
 (0)