Skip to content

Commit 3743dde

Browse files
moschemhl-b
andauthored
Test-fix testAcceptsMismatchedServerlessBuildHash #121869 (#122332) (#123477)
(cherry picked from commit 49ecbca) Co-authored-by: Mikhail Berezovskiy <[email protected]>
1 parent 6ed8512 commit 3743dde

File tree

2 files changed

+4
-3
lines changed

2 files changed

+4
-3
lines changed

server/src/main/java/org/elasticsearch/transport/TransportService.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -83,7 +83,7 @@ public class TransportService extends AbstractLifecycleComponent
8383
/**
8484
* A feature flag enabling transport upgrades for serverless.
8585
*/
86-
private static final String SERVERLESS_TRANSPORT_SYSTEM_PROPERTY = "es.serverless_transport";
86+
static final String SERVERLESS_TRANSPORT_SYSTEM_PROPERTY = "es.serverless_transport";
8787
private static final boolean SERVERLESS_TRANSPORT_FEATURE_FLAG = Booleans.parseBoolean(
8888
System.getProperty(SERVERLESS_TRANSPORT_SYSTEM_PROPERTY),
8989
false

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

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -377,7 +377,8 @@ public void testRejectsMismatchedBuildHash() {
377377
public void testAcceptsMismatchedServerlessBuildHash() {
378378
assumeTrue("Current build needs to be a snapshot", Build.current().isSnapshot());
379379
assumeTrue("Security manager needs to be disabled", System.getSecurityManager() == null);
380-
System.setProperty("es.serverless", Boolean.TRUE.toString()); // security manager blocks this
380+
System.setProperty(TransportService.SERVERLESS_TRANSPORT_SYSTEM_PROPERTY, Boolean.TRUE.toString()); // security manager blocks
381+
// this
381382
try {
382383
final DisruptingTransportInterceptor transportInterceptorA = new DisruptingTransportInterceptor();
383384
final DisruptingTransportInterceptor transportInterceptorB = new DisruptingTransportInterceptor();
@@ -404,7 +405,7 @@ public void testAcceptsMismatchedServerlessBuildHash() {
404405
AbstractSimpleTransportTestCase.connectToNode(transportServiceA, transportServiceB.getLocalNode(), TestProfiles.LIGHT_PROFILE);
405406
assertTrue(transportServiceA.nodeConnected(transportServiceB.getLocalNode()));
406407
} finally {
407-
System.clearProperty("es.serverless");
408+
System.clearProperty(TransportService.SERVERLESS_TRANSPORT_SYSTEM_PROPERTY);
408409
}
409410
}
410411

0 commit comments

Comments
 (0)