[11u] RFR: 8221710: [TESTBUG] more configurable parameters for docker testing
Severin Gehwolf
sgehwolf at redhat.com
Mon Jul 22 15:09:36 UTC 2019
Hi,
Please review one additional docker tests backport. A pre-requisite for
JDK-8224165 to bring down which helps verifying JDK-8220672. The JDK 13
patch applies cleanly, but doesn't work without modifications pointed
out below as JDK 11 doesn't have SkippedException (which is a rather
large changeset to bring down)
Bug: https://bugs.openjdk.java.net/browse/JDK-8221710
webrev: http://cr.openjdk.java.net/~sgehwolf/webrevs/JDK-8221710/jdk11/01/webrev/
original changeset: http://hg.openjdk.java.net/jdk/jdk/rev/b354ffb03ae4
Testing: Ran docker tests on Linux x86_64
Thoughts?
Thanks,
Severin
diff --git a/test/lib/jdk/test/lib/containers/docker/DockerTestUtils.java b/test/lib/jdk/test/lib/containers/docker/DockerTestUtils.java
--- a/test/lib/jdk/test/lib/containers/docker/DockerTestUtils.java
+++ b/test/lib/jdk/test/lib/containers/docker/DockerTestUtils.java
@@ -38,7 +38,6 @@
import jdk.test.lib.Utils;
import jdk.test.lib.process.OutputAnalyzer;
import jdk.test.lib.process.ProcessTools;
-import jtreg.SkippedException;
public class DockerTestUtils {
@@ -92,7 +91,9 @@
if (isDockerEngineAvailable()) {
return true;
} else {
- throw new SkippedException("Docker engine is not available on this system");
+ System.out.println("Docker engine is not available on this system");
+ System.out.println("This test is SKIPPED");
+ return false;
}
}
More information about the jdk-updates-dev
mailing list