| /* |
| * Copyright (c) 2016, 2017, 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 8160181 8176572 |
| * @summary Add lint warning for digits in module names |
| * @library /tools/lib |
| * @modules |
| * jdk.compiler/com.sun.tools.javac.api |
| * jdk.compiler/com.sun.tools.javac.code |
| * jdk.compiler/com.sun.tools.javac.main |
| * @build toolbox.ToolBox toolbox.JavacTask ModuleTestBase |
| * @run main PoorChoiceForModuleNameTest |
| */ |
| |
| |
| import java.nio.file.Path; |
| |
| import toolbox.JavacTask; |
| import toolbox.Task; |
| |
| public class PoorChoiceForModuleNameTest extends ModuleTestBase { |
| |
| public static void main(String... args) throws Exception { |
| new PoorChoiceForModuleNameTest().runTests(); |
| } |
| |
| @Test |
| public void testDigitsInModuleNames(Path base) throws Exception { |
| |
| Path src = base.resolve("src"); |
| |
| // Nitpickable module name |
| Path src_m1 = src.resolve("mango19"); |
| tb.writeJavaFiles(src_m1, "module mango19 { }"); |
| |
| // Acceptable module name. |
| Path src_m2 = src.resolve("mango20kg"); |
| tb.writeJavaFiles(src_m2, "module mango20kg { }"); |
| |
| // Nitpickable, but should not due to annotation. |
| Path src_m3 = src.resolve("mango100"); |
| tb.writeJavaFiles(src_m3, "@SuppressWarnings(\"module\") module mango100 { }"); |
| |
| // Check that there is no warning at use site. |
| Path src_m4 = src.resolve("mangouser"); |
| tb.writeJavaFiles(src_m4, "module mangouser { requires mango19; }"); |
| |
| // Check that we warn about component names ending in digit also |
| Path src_m5 = src.resolve("mango1000.mangofruit.mangomodule"); |
| tb.writeJavaFiles(src_m5, "module mango1000.mangofruit.mangomodule { }"); |
| |
| // Check that we warn about component names ending in digit also |
| Path src_m6 = src.resolve("mangofruit.mango1000.mangomodule"); |
| tb.writeJavaFiles(src_m6, "module mangofruit.mango1000.mangomodule { }"); |
| |
| // Check that we warn about component names ending in digit also |
| Path src_m7 = src.resolve("mangomodule.mangofruit.mango1000"); |
| tb.writeJavaFiles(src_m7, "module mangomodule.mangofruit.mango1000 { }"); |
| |
| Path classes = base.resolve("classes"); |
| tb.createDirectories(classes); |
| |
| String log = new JavacTask(tb) |
| .options("-XDrawDiagnostics", |
| "-Xlint:module", |
| "-Werror", |
| "--module-source-path", src.toString()) |
| .outdir(classes) |
| .files(findJavaFiles(src)) |
| .run(Task.Expect.FAIL) |
| .writeAll() |
| .getOutput(Task.OutputKind.DIRECT); |
| |
| if (!log.contains("module-info.java:1:8: compiler.warn.poor.choice.for.module.name: mango19") || |
| !log.contains("module-info.java:1:8: compiler.warn.poor.choice.for.module.name: mango1000") || |
| !log.contains("module-info.java:1:18: compiler.warn.poor.choice.for.module.name: mango1000") || |
| !log.contains("module-info.java:1:30: compiler.warn.poor.choice.for.module.name: mango1000") || |
| !log.contains("- compiler.err.warnings.and.werror") || |
| !log.contains("1 error") || |
| !log.contains("4 warning")) |
| throw new Exception("expected output not found: " + log); |
| } |
| } |