Fix test by removing unecessary mTLS

jakelandis · Jun 3, 2024 · da43174 · da43174
1 parent 5c711d0
commit da43174
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 5 deletions.
diff --git a/muted-tests.yml b/muted-tests.yml
@@ -5,9 +5,6 @@ tests:
 - class: "org.elasticsearch.cluster.coordination.CoordinatorVotingConfigurationTests"
   issue: "https://github.com/elastic/elasticsearch/issues/108729"
   method: "testClusterUUIDLogging"
-- class: "org.elasticsearch.xpack.core.ssl.SSLConfigurationReloaderTests"
-  issue: "https://github.com/elastic/elasticsearch/issues/108774"
-  method: "testReloadingKeyStore"
 - class: "org.elasticsearch.xpack.esql.ccq.MultiClusterSpecIT"
   issue: "https://github.com/elastic/elasticsearch/issues/108808"
   method: "test {k8s-metrics.MetricsWithAggs}"

diff --git a/x-pack/plugin/core/build.gradle b/x-pack/plugin/core/build.gradle
@@ -191,5 +191,6 @@ tasks.register('testAutomatonPatterns', Test) {
 
 tasks.named('test').configure {
   exclude '**/AutomatonPatternsTests.class'
-  dependsOn testAutomatonPatterns //to ensure testAutomatonPatterns are run with the test task
 }
+
+tasks.named("check").configure { dependsOn "testAutomatonPatterns" }
diff --git a/...in/core/src/test/java/org/elasticsearch/xpack/core/ssl/SSLConfigurationReloaderTests.java b/...in/core/src/test/java/org/elasticsearch/xpack/core/ssl/SSLConfigurationReloaderTests.java
@@ -130,7 +130,7 @@ public void testReloadingKeyStore() throws Exception {
         // Load HTTPClient only once. Client uses the same store as a truststore
         try (CloseableHttpClient client = getSSLClient(keystorePath, "testnode")) {
             final Consumer<SSLContext> keyMaterialPreChecks = (context) -> {
-                try (MockWebServer server = new MockWebServer(context, true)) {
+                try (MockWebServer server = new MockWebServer(context, false)) {
                     server.enqueue(new MockResponse().setResponseCode(200).setBody("body"));
                     server.start();
                     privilegedConnect(() -> client.execute(new HttpGet("https://localhost:" + server.getPort())).close());