Skip to content

Commit aa47a74

Browse files
committed
fix test
1 parent 766956e commit aa47a74

File tree

1 file changed

+15
-11
lines changed

1 file changed

+15
-11
lines changed

server/src/test/java/org/elasticsearch/transport/TransportServiceHandshakeTests.java

Lines changed: 15 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -9,8 +9,8 @@
99

1010
package org.elasticsearch.transport;
1111

12+
import org.apache.logging.log4j.Level;
1213
import org.elasticsearch.Build;
13-
import org.elasticsearch.ExceptionsHelper;
1414
import org.elasticsearch.TransportVersion;
1515
import org.elasticsearch.TransportVersions;
1616
import org.elasticsearch.Version;
@@ -27,6 +27,7 @@
2727
import org.elasticsearch.index.IndexVersions;
2828
import org.elasticsearch.indices.breaker.NoneCircuitBreakerService;
2929
import org.elasticsearch.test.ESTestCase;
30+
import org.elasticsearch.test.MockLog;
3031
import org.elasticsearch.test.TransportVersionUtils;
3132
import org.elasticsearch.test.VersionUtils;
3233
import org.elasticsearch.test.transport.MockTransportService;
@@ -350,23 +351,26 @@ public void testRejectsMismatchedBuildHash() {
350351
.version(Version.CURRENT.minimumCompatibilityVersion(), IndexVersions.MINIMUM_COMPATIBLE, IndexVersion.current())
351352
.build();
352353
try (
354+
MockLog mockLog = MockLog.capture(TransportService.class);
353355
Transport.Connection connection = AbstractSimpleTransportTestCase.openConnection(
354356
transportServiceA,
355357
discoveryNode,
356358
TestProfiles.LIGHT_PROFILE
357359
)
358360
) {
359-
assertThat(
360-
ExceptionsHelper.unwrap(
361-
safeAwaitFailure(
362-
TransportSerializationException.class,
363-
DiscoveryNode.class,
364-
listener -> transportServiceA.handshake(connection, timeout, listener)
365-
),
366-
IllegalArgumentException.class
367-
).getMessage(),
368-
containsString("which has an incompatible wire format")
361+
mockLog.addExpectation(
362+
new MockLog.SeenEventExpectation(
363+
"message",
364+
TransportService.class.getCanonicalName(),
365+
Level.WARN,
366+
"which has an incompatible wire format"
367+
)
369368
);
369+
370+
DiscoveryNode connectedNode = safeAwait(listener -> transportServiceA.handshake(connection, timeout, listener));
371+
assertNotNull(connectedNode);
372+
373+
mockLog.awaitAllExpectationsMatched();
370374
}
371375
assertFalse(transportServiceA.nodeConnected(discoveryNode));
372376
}

0 commit comments

Comments
 (0)