/* * Copyright (c) 2016, Oracle and/or its affiliates. All rights reserved. * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER. * * This code is free software; you can redistribute it and/or modify it * under the terms of the GNU General Public License version 2 only, as * published by the Free Software Foundation. * * This code is distributed in the hope that it will be useful, but WITHOUT * ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or * FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public License * version 2 for more details (a copy is included in the LICENSE file that * accompanied this code). * * You should have received a copy of the GNU General Public License version * 2 along with this work; if not, write to the Free Software Foundation, * Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA. * * Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA * or visit www.oracle.com if you need additional information or have any * questions. */ /** * @test * @bug 8145013 * @summary Javac doesn't report warnings/errors if module provides unexported service and doesn't use it itself * @library /tools/lib * @modules * jdk.compiler/com.sun.tools.javac.api * jdk.compiler/com.sun.tools.javac.main * @build toolbox.ToolBox toolbox.JavacTask ModuleTestBase * @run main ServiceProvidedButNotExportedOrUsedTest */ import java.nio.file.Files; import java.nio.file.Path; import java.util.Arrays; import java.util.List; import toolbox.JavacTask; import toolbox.Task; import toolbox.ToolBox; public class ServiceProvidedButNotExportedOrUsedTest extends ModuleTestBase { public static void main(String... args) throws Exception { ServiceProvidedButNotExportedOrUsedTest t = new ServiceProvidedButNotExportedOrUsedTest(); t.runTests(); } @Test public void testWarning(Path base) throws Exception { Path src = base.resolve("src"); tb.writeJavaFiles(src, "module m { provides p1.C1 with p2.C2; }", "package p1; public class C1 { }", "package p2; public class C2 extends p1.C1 { }"); Path classes = base.resolve("classes"); Files.createDirectories(classes); List<String> output = new JavacTask(tb) .outdir(classes) .options("-Werror", "-XDrawDiagnostics") .files(findJavaFiles(src)) .run(Task.Expect.FAIL) .writeAll() .getOutputLines(Task.OutputKind.DIRECT); List<String> expected = Arrays.asList( "module-info.java:1:12: compiler.warn.service.provided.but.not.exported.or.used: p1.C1", "- compiler.err.warnings.and.werror", "1 error", "1 warning"); if (!output.containsAll(expected)) { throw new Exception("Expected output not found"); } } @Test public void testImplementationMustBeInSameModuleAsProvidesDirective(Path base) throws Exception { Path src = base.resolve("src"); tb.writeJavaFiles(src.resolve("m1"), "module m1 { exports p1; }", "package p1; public class C1 { }"); tb.writeJavaFiles(src.resolve("m2"), "module m2 { requires m1; requires m3; provides p1.C1 with p2.C2; }"); tb.writeJavaFiles(src.resolve("m3"), "module m3 { requires m1; exports p2; }", "package p2; public class C2 extends p1.C1 { }"); Path modules = base.resolve("modules"); Files.createDirectories(modules); List<String> output = new JavacTask(tb) .options("-XDrawDiagnostics", "--module-source-path", src.toString()) .outdir(modules) .files(findJavaFiles(src)) .run(Task.Expect.FAIL) .writeAll() .getOutputLines(Task.OutputKind.DIRECT); List<String> expected = Arrays.asList( "module-info.java:1:39: compiler.err.service.implementation.not.in.right.module: m3", "1 error"); if (!output.containsAll(expected)) { throw new Exception("Expected output not found"); } } }