From 5f302580f9dea18b389007b861fc06b013e94499 Mon Sep 17 00:00:00 2001 From: Daniel Mitterdorfer Date: Tue, 31 Jul 2018 16:17:28 +0200 Subject: [PATCH] Mute SSLTrustRestrictionsTests on JDK 11 This commit adds an assumption to two test methods in SSLTrustRestrictionsTests that we are not on JDK 11 as the tests currently fail there. Relates #29989 --- .../elasticsearch/xpack/ssl/SSLTrustRestrictionsTests.java | 5 +++++ 1 file changed, 5 insertions(+) diff --git a/x-pack/plugin/security/src/test/java/org/elasticsearch/xpack/ssl/SSLTrustRestrictionsTests.java b/x-pack/plugin/security/src/test/java/org/elasticsearch/xpack/ssl/SSLTrustRestrictionsTests.java index cf77ca975a4..e1896e01365 100644 --- a/x-pack/plugin/security/src/test/java/org/elasticsearch/xpack/ssl/SSLTrustRestrictionsTests.java +++ b/x-pack/plugin/security/src/test/java/org/elasticsearch/xpack/ssl/SSLTrustRestrictionsTests.java @@ -7,6 +7,7 @@ package org.elasticsearch.xpack.ssl; import org.apache.logging.log4j.message.ParameterizedMessage; import org.elasticsearch.ElasticsearchException; +import org.elasticsearch.bootstrap.JavaVersion; import org.elasticsearch.common.io.PathUtils; import org.elasticsearch.common.settings.Settings; import org.elasticsearch.common.transport.TransportAddress; @@ -173,6 +174,8 @@ public class SSLTrustRestrictionsTests extends SecurityIntegTestCase { } public void testCertificateWithUntrustedNameFails() throws Exception { + // see https://github.com/elastic/elasticsearch/issues/29989 + assumeTrue("test fails on JDK 11 currently", JavaVersion.current().compareTo(JavaVersion.parse("11")) < 0); writeRestrictions("*.trusted"); try { tryConnect(untrustedCert); @@ -183,6 +186,8 @@ public class SSLTrustRestrictionsTests extends SecurityIntegTestCase { } public void testRestrictionsAreReloaded() throws Exception { + // see https://github.com/elastic/elasticsearch/issues/29989 + assumeTrue("test fails on JDK 11 currently", JavaVersion.current().compareTo(JavaVersion.parse("11")) < 0); writeRestrictions("*"); assertBusy(() -> { try {