langtools/test/tools/javac/modules/ExportsUnexported.java
author jjg
Tue, 22 Nov 2016 15:57:32 -0800
changeset 42269 24a766b7c106
parent 41444 b61c805c6173
child 42407 f3702cff2933
permissions -rw-r--r--
8164590: javac --inherit-runtime-environment fails with "cannot find modules: ALL-DEFAULT" Reviewed-by: mchung
Ignore whitespace changes - Everywhere: Within whitespace: At end of lines:
41444
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
     1
/*
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
     2
 * Copyright (c) 2016, Oracle and/or its affiliates. All rights reserved.
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
     3
 * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
     4
 *
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
     5
 * This code is free software; you can redistribute it and/or modify it
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
     6
 * under the terms of the GNU General Public License version 2 only, as
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
     7
 * published by the Free Software Foundation.
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
     8
 *
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
     9
 * This code is distributed in the hope that it will be useful, but WITHOUT
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
    10
 * ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
    11
 * FITNESS FOR A PARTICULAR PURPOSE.  See the GNU General Public License
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
    12
 * version 2 for more details (a copy is included in the LICENSE file that
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
    13
 * accompanied this code).
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
    14
 *
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
    15
 * You should have received a copy of the GNU General Public License version
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
    16
 * 2 along with this work; if not, write to the Free Software Foundation,
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
    17
 * Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA.
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
    18
 *
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
    19
 * Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
    20
 * or visit www.oracle.com if you need additional information or have any
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
    21
 * questions.
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
    22
 */
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
    23
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
    24
/*
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
    25
 * @test
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
    26
 * @summary tests for module declarations
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
    27
 * @library /tools/lib
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
    28
 * @modules jdk.compiler/com.sun.tools.javac.api
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
    29
 *          jdk.compiler/com.sun.tools.javac.main
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
    30
 * @build toolbox.ToolBox toolbox.JavacTask ModuleTestBase
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
    31
 * @run main ExportsUnexported
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
    32
 */
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
    33
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
    34
import java.nio.file.Path;
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
    35
import java.util.ArrayList;
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
    36
import java.util.Arrays;
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
    37
import java.util.Collections;
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
    38
import java.util.List;
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
    39
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
    40
import toolbox.JavacTask;
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
    41
import toolbox.Task;
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
    42
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
    43
public class ExportsUnexported extends ModuleTestBase {
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
    44
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
    45
    public static void main(String... args) throws Exception {
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
    46
        ExportsUnexported t = new ExportsUnexported();
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
    47
        t.runTests();
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
    48
    }
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
    49
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
    50
    @Test
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
    51
    public void testLocations(Path base) throws Exception {
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
    52
        String warningsTest = "package api;\n" +
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
    53
                      "import impl.impl.*;\n" +
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
    54
                      "@impl.impl^.DocAnn\n" +
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
    55
                      "public abstract class Api<T extends impl.impl^.Cls&impl.impl^.Intf> extends impl.impl^.Cls implements impl.impl^.Intf, impl.impl^.NonDocAnn, impl.impl^.DocAnn {\n" +
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
    56
                      "    public static <E extends impl.impl^.Cls&impl.impl^.Intf> impl.impl^.Cls m(impl.impl^.Intf i, impl.impl^.Cls c) throws impl.impl^.Exc { return null; }\n" +
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
    57
                      "    public static impl.impl^.Cls f;\n" +
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
    58
                      "}";
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
    59
        String noWarningsTest = "package api;\n" +
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
    60
                      "import impl.impl.*;\n" +
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
    61
                      "@impl.impl.NonDocAnn\n" +
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
    62
                      "public abstract class Api {\n" +
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
    63
                      "    private static abstract class I <T extends impl.impl.Cls&impl.impl.Intf> extends impl.impl.Cls implements impl.impl.Intf, impl.impl.NonDocAnn, impl.impl.DocAnn {\n" +
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
    64
                      "        public static abstract class II <T extends impl.impl.Cls&impl.impl.Intf> extends impl.impl.Cls implements impl.impl.Intf, impl.impl.NonDocAnn, impl.impl.DocAnn { }\n" +
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
    65
                      "        public static <E extends impl.impl.Cls&impl.impl.Intf> impl.impl.Cls m(impl.impl.Intf i, impl.impl.Cls c) throws impl.impl.Exc { return null; }\n" +
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
    66
                      "        public static impl.impl.Cls f;\n" +
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
    67
                      "    }\n" +
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
    68
                      "    private static <E extends impl.impl.Cls&impl.impl.Intf> impl.impl.Cls m(impl.impl.Intf i, impl.impl.Cls c) throws impl.impl.Exc { return null; }\n" +
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
    69
                      "    private static impl.impl.Cls f1;\n" +
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
    70
                      "    public static void m() { new impl.impl.Cls(); }\n" +
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
    71
                      "    public static Object f2 = new impl.impl.Cls();\n" +
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
    72
                      "}";
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
    73
        for (String genericTest : new String[] {warningsTest, noWarningsTest}) {
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
    74
            for (String test : new String[] {genericTest, genericTest.replaceAll("impl\\.impl\\^.([A-Za-z])", "^$1").replaceAll("impl\\.impl\\.([A-Za-z])", "$1")}) {
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
    75
                System.err.println("testing: " + test);
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
    76
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
    77
                Path src = base.resolve("src");
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
    78
                Path src_m1 = src.resolve("m1");
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
    79
                StringBuilder testCode = new StringBuilder();
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
    80
                List<String> expectedLog = new ArrayList<>();
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
    81
                int line = 1;
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
    82
                int col  = 1;
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
    83
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
    84
                for (int i = 0; i < test.length(); i++) {
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
    85
                    char c = test.charAt(i);
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
    86
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
    87
                    if (c == '^') {
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
    88
                        StringBuilder typeName = new StringBuilder();
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
    89
                        for (int j = i + 1 + (test.charAt(i + 1) == '.' ? 1 : 0);
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
    90
                             j < test.length() && Character.isJavaIdentifierPart(test.charAt(j));
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
    91
                             j++) {
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
    92
                            typeName.append(test.charAt(j));
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
    93
                        }
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
    94
                        String kindName;
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
    95
                        switch (typeName.toString()) {
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
    96
                            case "Exc": case "DocAnn": case "NonDocAnn":
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
    97
                            case "Cls": kindName = "kindname.class"; break;
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
    98
                            case "Intf": kindName = "kindname.interface"; break;
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
    99
                            default:
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   100
                                throw new AssertionError(typeName.toString());
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   101
                        }
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   102
                        expectedLog.add("Api.java:" + line + ":" + col + ": compiler.warn.leaks.not.accessible.unexported: " + kindName + ", impl.impl." + typeName + ", m1");
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   103
                        continue;
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   104
                    }
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   105
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   106
                    if (c == '\n') {
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   107
                        line++;
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   108
                        col = 0;
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   109
                    }
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   110
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   111
                    testCode.append(c);
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   112
                    col++;
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   113
                }
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   114
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   115
                if (!expectedLog.isEmpty()) {
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   116
                    expectedLog.add("" + expectedLog.size() + " warning" + (expectedLog.size() == 1 ? "" : "s"));
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   117
                    expectedLog.add("- compiler.err.warnings.and.werror");
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   118
                    expectedLog.add("1 error");
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   119
                }
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   120
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   121
                Collections.sort(expectedLog);
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   122
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   123
                tb.writeJavaFiles(src_m1,
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   124
                                  "module m1 { exports api; }",
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   125
                                  testCode.toString(),
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   126
                                  "package impl.impl; public class Cls { }",
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   127
                                  "package impl.impl; public class Exc extends Exception { }",
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   128
                                  "package impl.impl; public interface Intf { }",
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   129
                                  "package impl.impl; @java.lang.annotation.Documented public @interface DocAnn { }",
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   130
                                  "package impl.impl; public @interface NonDocAnn { }");
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   131
                Path classes = base.resolve("classes");
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   132
                tb.createDirectories(classes);
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   133
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   134
                List<String> log = new JavacTask(tb)
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   135
                        .options("-XDrawDiagnostics",
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   136
                                 "-Werror",
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   137
                                 "--module-source-path", src.toString(),
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   138
                                 "-Xlint:exports")
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   139
                        .outdir(classes)
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   140
                        .files(findJavaFiles(src))
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   141
                        .run(expectedLog.isEmpty() ? Task.Expect.SUCCESS : Task.Expect.FAIL)
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   142
                        .writeAll()
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   143
                        .getOutputLines(Task.OutputKind.DIRECT);
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   144
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   145
                log = new ArrayList<>(log);
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   146
                Collections.sort(log);
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   147
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   148
                if (expectedLog.isEmpty() ? !log.equals(Arrays.asList("")) : !log.equals(expectedLog)) {
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   149
                    throw new Exception("expected output not found in: " + log + "; " + expectedLog);
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   150
                }
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   151
            }
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   152
        }
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   153
    }
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   154
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   155
    @Test
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   156
    public void testAccessibleToSpecificOrAll(Path base) throws Exception {
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   157
        Path src = base.resolve("src");
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   158
        Path src_lib1 = src.resolve("lib1");
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   159
        tb.writeJavaFiles(src_lib1,
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   160
                          "module lib1 { exports lib1; }",
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   161
                          "package lib1; public class Lib1 {}");
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   162
        Path src_lib2 = src.resolve("lib2");
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   163
        tb.writeJavaFiles(src_lib2,
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   164
                          "module lib2 { exports lib2; }",
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   165
                          "package lib2; public class Lib2 {}");
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   166
        Path src_api = src.resolve("api");
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   167
        tb.writeJavaFiles(src_api,
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   168
                          "module api {\n" +
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   169
                          "    exports api;\n" +
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   170
                          "    exports qapi1 to qual1;\n" +
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   171
                          "    exports qapi2 to qual1, qual2;\n" +
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   172
                          "    requires public lib1;\n" +
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   173
                          "    requires lib2;\n" +
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   174
                          "}\n",
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   175
                          "package api;\n" +
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   176
                          "public class Api {\n" +
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   177
                          "    public lib1.Lib1 lib1;\n" +
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   178
                          "    public lib2.Lib2 lib2;\n" +
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   179
                          "    public qapi1.QApi1 qapi1;\n" +
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   180
                          "    public impl.Impl impl;\n" +
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   181
                          "}",
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   182
                          "package qapi1;\n" +
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   183
                          "public class QApi1 {\n" +
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   184
                          "    public qapi2.QApi2 qapi2;\n" +
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   185
                          "}",
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   186
                          "package qapi2;\n" +
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   187
                          "public class QApi2 {\n" +
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   188
                          "    public qapi1.QApi1 qapi1;\n" +
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   189
                          "}",
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   190
                          "package impl;\n" +
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   191
                          "public class Impl {\n" +
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   192
                          "}");
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   193
        Path src_qual1 = src.resolve("qual1");
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   194
        tb.writeJavaFiles(src_qual1, "module qual1 { }");
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   195
        Path src_qual2 = src.resolve("qual2");
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   196
        tb.writeJavaFiles(src_qual2, "module qual2 { }");
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   197
        Path classes = base.resolve("classes");
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   198
        tb.createDirectories(classes);
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   199
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   200
        List<String> log = new JavacTask(tb)
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   201
                .options("-XDrawDiagnostics",
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   202
                         "-Werror",
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   203
                         "--module-source-path", src.toString(),
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   204
                         "-Xlint:exports")
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   205
                .outdir(classes)
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   206
                .files(findJavaFiles(src))
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   207
                .run(Task.Expect.FAIL)
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   208
                .writeAll()
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   209
                .getOutputLines(Task.OutputKind.DIRECT);
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   210
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   211
        List<String> expected = Arrays.asList(
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   212
            "Api.java:4:16: compiler.warn.leaks.not.accessible.not.required.public: kindname.class, lib2.Lib2, lib2",
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   213
            "Api.java:5:17: compiler.warn.leaks.not.accessible.unexported.qualified: kindname.class, qapi1.QApi1, api",
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   214
            "Api.java:6:16: compiler.warn.leaks.not.accessible.unexported: kindname.class, impl.Impl, api",
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   215
            "- compiler.err.warnings.and.werror",
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   216
            "1 error",
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   217
            "3 warnings"
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   218
        );
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   219
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   220
        if (!log.equals(expected))
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   221
            throw new Exception("expected output not found");
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   222
    }
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   223
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   224
    @Test
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   225
    public void testNestedClasses(Path base) throws Exception {
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   226
        Path src = base.resolve("src");
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   227
        Path src_api = src.resolve("api");
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   228
        tb.writeJavaFiles(src_api,
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   229
                          "module api {\n" +
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   230
                          "    exports api;\n" +
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   231
                          "}\n",
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   232
                          "package api;\n" +
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   233
                          "import impl.Impl.Nested;\n" +
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   234
                          "public class Api {\n" +
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   235
                          "    public impl.Impl impl1;\n" +
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   236
                          "    public impl.Impl.Nested impl2;\n" +
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   237
                          "    public Nested impl3;\n" +
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   238
                          "}",
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   239
                          "package impl;\n" +
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   240
                          "public class Impl {\n" +
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   241
                          "    public static class Nested {\n" +
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   242
                          "    }\n" +
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   243
                          "}");
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   244
        Path classes = base.resolve("classes");
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   245
        tb.createDirectories(classes);
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   246
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   247
        List<String> log = new JavacTask(tb)
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   248
                .options("-XDrawDiagnostics",
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   249
                         "-Werror",
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   250
                         "--module-source-path", src.toString(),
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   251
                         "-Xlint:exports")
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   252
                .outdir(classes)
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   253
                .files(findJavaFiles(src))
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   254
                .run(Task.Expect.FAIL)
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   255
                .writeAll()
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   256
                .getOutputLines(Task.OutputKind.DIRECT);
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   257
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   258
        List<String> expected = Arrays.asList(
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   259
            "Api.java:4:16: compiler.warn.leaks.not.accessible.unexported: kindname.class, impl.Impl, api",
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   260
            "Api.java:5:16: compiler.warn.leaks.not.accessible.unexported: kindname.class, impl.Impl, api",
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   261
            "Api.java:6:12: compiler.warn.leaks.not.accessible.unexported: kindname.class, impl.Impl.Nested, api",
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   262
            "- compiler.err.warnings.and.werror",
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   263
            "1 error",
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   264
            "3 warnings"
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   265
        );
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   266
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   267
        if (!log.equals(expected))
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   268
            throw new Exception("expected output not found");
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   269
    }
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   270
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   271
    @Test
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   272
    public void testProtectedAndInaccessible(Path base) throws Exception {
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   273
        Path src = base.resolve("src");
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   274
        Path src_api = src.resolve("api");
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   275
        tb.writeJavaFiles(src_api,
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   276
                          "module api {\n" +
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   277
                          "    exports api;\n" +
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   278
                          "}\n",
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   279
                          "package api;\n" +
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   280
                          "public class Api extends PackagePrivateClass<PackagePrivateInterface> implements PackagePrivateInterface<PackagePrivateClass> {\n" +
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   281
                          "    protected PackagePrivateClass<?> f1;\n" +
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   282
                          "    protected PackagePrivateInterface<?> f2;\n" +
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   283
                          "    protected Inner f3;\n" +
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   284
                          "    protected PrivateInner f4;\n" +
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   285
                          "    protected impl.Impl f5;\n" +
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   286
                          "    public static class InnerClass extends PrivateInner {}\n" +
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   287
                          "    protected static class Inner {}\n" +
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   288
                          "    private static class PrivateInner {}\n" +
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   289
                          "}\n" +
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   290
                          "class PackagePrivateClass<T> {}\n" +
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   291
                          "interface PackagePrivateInterface<T> {}",
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   292
                          "package impl;\n" +
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   293
                          "public class Impl {\n" +
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   294
                          "}");
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   295
        Path classes = base.resolve("classes");
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   296
        tb.createDirectories(classes);
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   297
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   298
        List<String> log = new JavacTask(tb)
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   299
                .options("-XDrawDiagnostics",
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   300
                         "-Werror",
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   301
                         "--module-source-path", src.toString(),
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   302
                         "-Xlint:exports")
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   303
                .outdir(classes)
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   304
                .files(findJavaFiles(src))
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   305
                .run(Task.Expect.FAIL)
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   306
                .writeAll()
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   307
                .getOutputLines(Task.OutputKind.DIRECT);
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   308
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   309
        List<String> expected = Arrays.asList(
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   310
            "Api.java:2:46: compiler.warn.leaks.not.accessible: kindname.interface, api.PackagePrivateInterface, api",
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   311
            "Api.java:2:106: compiler.warn.leaks.not.accessible: kindname.class, api.PackagePrivateClass, api",
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   312
            "Api.java:3:15: compiler.warn.leaks.not.accessible: kindname.class, api.PackagePrivateClass, api",
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   313
            "Api.java:4:15: compiler.warn.leaks.not.accessible: kindname.interface, api.PackagePrivateInterface, api",
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   314
            "Api.java:6:15: compiler.warn.leaks.not.accessible: kindname.class, api.Api.PrivateInner, api",
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   315
            "Api.java:7:19: compiler.warn.leaks.not.accessible.unexported: kindname.class, impl.Impl, api",
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   316
            "- compiler.err.warnings.and.werror",
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   317
            "1 error",
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   318
            "6 warnings"
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   319
        );
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   320
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   321
        if (!log.equals(expected))
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   322
            throw new Exception("expected output not found");
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   323
    }
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   324
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   325
    @Test
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   326
    public void testSuppressResetProperly(Path base) throws Exception {
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   327
        Path src = base.resolve("src");
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   328
        Path src_api = src.resolve("api");
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   329
        tb.writeJavaFiles(src_api,
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   330
                          "module api {\n" +
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   331
                          "    exports api;\n" +
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   332
                          "}\n",
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   333
                          "package api;\n" +
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   334
                          "public class Api {\n" +
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   335
                          "    @SuppressWarnings(\"exports\")\n" +
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   336
                          "    public PackagePrivateClass f1;\n" +
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   337
                          "    public PackagePrivateClass f2;\n" +
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   338
                          "    @SuppressWarnings(\"exports\")\n" +
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   339
                          "    public void t() {}\n" +
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   340
                          "    public PackagePrivateClass f3;\n" +
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   341
                          "    @SuppressWarnings(\"exports\")\n" +
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   342
                          "    public static class C {\n" +
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   343
                          "        public PackagePrivateClass f4;\n" +
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   344
                          "    }\n" +
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   345
                          "    public PackagePrivateClass f5;\n" +
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   346
                          "}\n" +
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   347
                          "class PackagePrivateClass<T> {}\n");
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   348
        Path classes = base.resolve("classes");
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   349
        tb.createDirectories(classes);
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   350
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   351
        List<String> log = new JavacTask(tb)
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   352
                .options("-XDrawDiagnostics",
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   353
                         "-Werror",
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   354
                         "--module-source-path", src.toString(),
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   355
                         "-Xlint:exports")
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   356
                .outdir(classes)
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   357
                .files(findJavaFiles(src))
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   358
                .run(Task.Expect.FAIL)
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   359
                .writeAll()
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   360
                .getOutputLines(Task.OutputKind.DIRECT);
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   361
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   362
        List<String> expected = Arrays.asList(
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   363
            "Api.java:5:12: compiler.warn.leaks.not.accessible: kindname.class, api.PackagePrivateClass, api",
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   364
            "Api.java:8:12: compiler.warn.leaks.not.accessible: kindname.class, api.PackagePrivateClass, api",
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   365
            "Api.java:13:12: compiler.warn.leaks.not.accessible: kindname.class, api.PackagePrivateClass, api",
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   366
            "- compiler.err.warnings.and.werror",
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   367
            "1 error",
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   368
            "3 warnings"
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   369
        );
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   370
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   371
        if (!log.equals(expected))
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   372
            throw new Exception("expected output not found");
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   373
    }
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   374
b61c805c6173 8153362: Add javac -Xlint warning to list exposed types which are not accessible
jlahoda
parents:
diff changeset
   375
}