diff -r d0cb66d8cbf1 -r 8fee80b92e65 test/langtools/tools/javac/options/release/ReleaseOptionUnsupported.java --- a/test/langtools/tools/javac/options/release/ReleaseOptionUnsupported.java Thu Nov 02 10:09:34 2017 +0100 +++ b/test/langtools/tools/javac/options/release/ReleaseOptionUnsupported.java Thu Nov 02 11:16:27 2017 +0100 @@ -23,7 +23,7 @@ /** * @test - * @bug 8178152 + * @bug 8178152 8187681 * @summary Verify unsupported modules and module options handling. * @library /tools/lib * @modules jdk.compiler/com.sun.tools.javac.api @@ -263,35 +263,19 @@ tb.createDirectories(patch); new JavacTask(tb) - .options("-XDrawDiagnostics", - "--patch-module", "java.base=" + patch) + .options("--patch-module", "java.base=" + patch) .outdir(classes) .files(tb.findJavaFiles(src)) .run(Expect.SUCCESS) - .writeAll() - .getOutputLines(Task.OutputKind.DIRECT); + .writeAll(); - List log; - List expected; - - log = new JavacTask(tb) - .options("-XDrawDiagnostics", - "--patch-module", "java.base=" + patch, + new JavacTask(tb) + .options("--patch-module", "java.base=" + patch, "--release", Target.DEFAULT.multiReleaseValue()) .outdir(classes) .files(tb.findJavaFiles(src)) - .run(Expect.FAIL) - .writeAll() - .getOutputLines(Task.OutputKind.DIRECT); - - expected = Arrays.asList( - "- compiler.err.patch.module.with.release: java.base", - "1 error" - ); - - if (!expected.equals(log)) { - throw new AssertionError("Unexpected output: " + log); - } + .run(Expect.SUCCESS) + .writeAll(); //OK to patch a non-system module: tb.createDirectories(classes);