langtools/test/jdk/javadoc/tool/doclint/DocLintTest.java
author alanb
Thu, 17 Mar 2016 19:04:28 +0000
changeset 36526 3b41f1c69604
parent 35426 374342e56a56
permissions -rw-r--r--
8142968: Module System implementation Summary: Initial integration of JEP 200, JEP 260, JEP 261, and JEP 282 Reviewed-by: jjg, jlahoda, vromero, mcimadamore, bpatel, ksrini, darcy, anazarov, dfuchs Contributed-by: alan.bateman@oracle.com, alex.buckley@oracle.com, jonathan.gibbons@oracle.com, karen.kinnear@oracle.com, mandy.chung@oracle.com, mark.reinhold@oracle.com, jan.lahoda@oracle.com, vicente.romero@oracle.com, andreas.lundblad@oracle.com, andrey.x.nazarov@oracle.com, chris.hegarty@oracle.com, erik.joelsson@oracle.com, kumar.x.srinivasan@oracle.com, sundararajan.athijegannathan@oracle.com
Ignore whitespace changes - Everywhere: Within whitespace: At end of lines:
15354
52a04c670c05 8004834: Add doclint support into javadoc
jjg
parents:
diff changeset
     1
/*
30730
d3ce7619db2c 8076543: Add @modules as needed to the langtools tests
akulyakh
parents: 27319
diff changeset
     2
 * Copyright (c) 2012, 2015, Oracle and/or its affiliates. All rights reserved.
15354
52a04c670c05 8004834: Add doclint support into javadoc
jjg
parents:
diff changeset
     3
 * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
52a04c670c05 8004834: Add doclint support into javadoc
jjg
parents:
diff changeset
     4
 *
52a04c670c05 8004834: Add doclint support into javadoc
jjg
parents:
diff changeset
     5
 * This code is free software; you can redistribute it and/or modify it
52a04c670c05 8004834: Add doclint support into javadoc
jjg
parents:
diff changeset
     6
 * under the terms of the GNU General Public License version 2 only, as
52a04c670c05 8004834: Add doclint support into javadoc
jjg
parents:
diff changeset
     7
 * published by the Free Software Foundation.
52a04c670c05 8004834: Add doclint support into javadoc
jjg
parents:
diff changeset
     8
 *
52a04c670c05 8004834: Add doclint support into javadoc
jjg
parents:
diff changeset
     9
 * This code is distributed in the hope that it will be useful, but WITHOUT
52a04c670c05 8004834: Add doclint support into javadoc
jjg
parents:
diff changeset
    10
 * ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or
52a04c670c05 8004834: Add doclint support into javadoc
jjg
parents:
diff changeset
    11
 * FITNESS FOR A PARTICULAR PURPOSE.  See the GNU General Public License
52a04c670c05 8004834: Add doclint support into javadoc
jjg
parents:
diff changeset
    12
 * version 2 for more details (a copy is included in the LICENSE file that
52a04c670c05 8004834: Add doclint support into javadoc
jjg
parents:
diff changeset
    13
 * accompanied this code).
52a04c670c05 8004834: Add doclint support into javadoc
jjg
parents:
diff changeset
    14
 *
52a04c670c05 8004834: Add doclint support into javadoc
jjg
parents:
diff changeset
    15
 * You should have received a copy of the GNU General Public License version
52a04c670c05 8004834: Add doclint support into javadoc
jjg
parents:
diff changeset
    16
 * 2 along with this work; if not, write to the Free Software Foundation,
52a04c670c05 8004834: Add doclint support into javadoc
jjg
parents:
diff changeset
    17
 * Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA.
52a04c670c05 8004834: Add doclint support into javadoc
jjg
parents:
diff changeset
    18
 *
52a04c670c05 8004834: Add doclint support into javadoc
jjg
parents:
diff changeset
    19
 * Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA
52a04c670c05 8004834: Add doclint support into javadoc
jjg
parents:
diff changeset
    20
 * or visit www.oracle.com if you need additional information or have any
52a04c670c05 8004834: Add doclint support into javadoc
jjg
parents:
diff changeset
    21
 * questions.
52a04c670c05 8004834: Add doclint support into javadoc
jjg
parents:
diff changeset
    22
 */
52a04c670c05 8004834: Add doclint support into javadoc
jjg
parents:
diff changeset
    23
52a04c670c05 8004834: Add doclint support into javadoc
jjg
parents:
diff changeset
    24
/*
52a04c670c05 8004834: Add doclint support into javadoc
jjg
parents:
diff changeset
    25
 * @test
31752
a4ea4c9bce2f 8129909: Add -Xdoclint/package: to javadoc
jlahoda
parents: 30730
diff changeset
    26
 * @bug 8004834 8007610 8129909
15354
52a04c670c05 8004834: Add doclint support into javadoc
jjg
parents:
diff changeset
    27
 * @summary Add doclint support into javadoc
30730
d3ce7619db2c 8076543: Add @modules as needed to the langtools tests
akulyakh
parents: 27319
diff changeset
    28
 * @modules jdk.compiler/com.sun.tools.javac.main
15354
52a04c670c05 8004834: Add doclint support into javadoc
jjg
parents:
diff changeset
    29
 */
52a04c670c05 8004834: Add doclint support into javadoc
jjg
parents:
diff changeset
    30
52a04c670c05 8004834: Add doclint support into javadoc
jjg
parents:
diff changeset
    31
import java.io.File;
52a04c670c05 8004834: Add doclint support into javadoc
jjg
parents:
diff changeset
    32
import java.io.PrintWriter;
52a04c670c05 8004834: Add doclint support into javadoc
jjg
parents:
diff changeset
    33
import java.io.StringWriter;
52a04c670c05 8004834: Add doclint support into javadoc
jjg
parents:
diff changeset
    34
import java.net.URI;
52a04c670c05 8004834: Add doclint support into javadoc
jjg
parents:
diff changeset
    35
import java.util.Arrays;
52a04c670c05 8004834: Add doclint support into javadoc
jjg
parents:
diff changeset
    36
import java.util.Collections;
52a04c670c05 8004834: Add doclint support into javadoc
jjg
parents:
diff changeset
    37
import java.util.EnumSet;
52a04c670c05 8004834: Add doclint support into javadoc
jjg
parents:
diff changeset
    38
import java.util.List;
52a04c670c05 8004834: Add doclint support into javadoc
jjg
parents:
diff changeset
    39
import java.util.Set;
52a04c670c05 8004834: Add doclint support into javadoc
jjg
parents:
diff changeset
    40
import java.util.regex.Matcher;
52a04c670c05 8004834: Add doclint support into javadoc
jjg
parents:
diff changeset
    41
import java.util.regex.Pattern;
52a04c670c05 8004834: Add doclint support into javadoc
jjg
parents:
diff changeset
    42
52a04c670c05 8004834: Add doclint support into javadoc
jjg
parents:
diff changeset
    43
import javax.tools.Diagnostic;
52a04c670c05 8004834: Add doclint support into javadoc
jjg
parents:
diff changeset
    44
import javax.tools.DocumentationTool;
52a04c670c05 8004834: Add doclint support into javadoc
jjg
parents:
diff changeset
    45
import javax.tools.DocumentationTool.DocumentationTask;
52a04c670c05 8004834: Add doclint support into javadoc
jjg
parents:
diff changeset
    46
import javax.tools.JavaFileObject;
52a04c670c05 8004834: Add doclint support into javadoc
jjg
parents:
diff changeset
    47
import javax.tools.SimpleJavaFileObject;
52a04c670c05 8004834: Add doclint support into javadoc
jjg
parents:
diff changeset
    48
import javax.tools.StandardJavaFileManager;
52a04c670c05 8004834: Add doclint support into javadoc
jjg
parents:
diff changeset
    49
import javax.tools.StandardLocation;
52a04c670c05 8004834: Add doclint support into javadoc
jjg
parents:
diff changeset
    50
import javax.tools.ToolProvider;
52a04c670c05 8004834: Add doclint support into javadoc
jjg
parents:
diff changeset
    51
import static javax.tools.Diagnostic.Kind.*;
52a04c670c05 8004834: Add doclint support into javadoc
jjg
parents:
diff changeset
    52
52a04c670c05 8004834: Add doclint support into javadoc
jjg
parents:
diff changeset
    53
import com.sun.tools.javac.main.Main;
52a04c670c05 8004834: Add doclint support into javadoc
jjg
parents:
diff changeset
    54
52a04c670c05 8004834: Add doclint support into javadoc
jjg
parents:
diff changeset
    55
public class DocLintTest {
52a04c670c05 8004834: Add doclint support into javadoc
jjg
parents:
diff changeset
    56
    public static void main(String... args) throws Exception {
52a04c670c05 8004834: Add doclint support into javadoc
jjg
parents:
diff changeset
    57
        new DocLintTest().run();
52a04c670c05 8004834: Add doclint support into javadoc
jjg
parents:
diff changeset
    58
    }
52a04c670c05 8004834: Add doclint support into javadoc
jjg
parents:
diff changeset
    59
52a04c670c05 8004834: Add doclint support into javadoc
jjg
parents:
diff changeset
    60
    DocumentationTool javadoc;
52a04c670c05 8004834: Add doclint support into javadoc
jjg
parents:
diff changeset
    61
    StandardJavaFileManager fm;
31752
a4ea4c9bce2f 8129909: Add -Xdoclint/package: to javadoc
jlahoda
parents: 30730
diff changeset
    62
    Iterable<? extends JavaFileObject> files;
15354
52a04c670c05 8004834: Add doclint support into javadoc
jjg
parents:
diff changeset
    63
52a04c670c05 8004834: Add doclint support into javadoc
jjg
parents:
diff changeset
    64
    final String code =
52a04c670c05 8004834: Add doclint support into javadoc
jjg
parents:
diff changeset
    65
        /* 01 */    "/** Class comment. */\n" +
52a04c670c05 8004834: Add doclint support into javadoc
jjg
parents:
diff changeset
    66
        /* 02 */    "public class Test {\n" +
52a04c670c05 8004834: Add doclint support into javadoc
jjg
parents:
diff changeset
    67
        /* 03 */    "    /** Method comment. */\n" +
52a04c670c05 8004834: Add doclint support into javadoc
jjg
parents:
diff changeset
    68
        /* 04 */    "    public void method() { }\n" +
52a04c670c05 8004834: Add doclint support into javadoc
jjg
parents:
diff changeset
    69
        /* 05 */    "\n" +
52a04c670c05 8004834: Add doclint support into javadoc
jjg
parents:
diff changeset
    70
        /* 06 */    "    /** Syntax < error. */\n" +
52a04c670c05 8004834: Add doclint support into javadoc
jjg
parents:
diff changeset
    71
        /* 07 */    "    private void syntaxError() { }\n" +
52a04c670c05 8004834: Add doclint support into javadoc
jjg
parents:
diff changeset
    72
        /* 08 */    "\n" +
52a04c670c05 8004834: Add doclint support into javadoc
jjg
parents:
diff changeset
    73
        /* 09 */    "    /** @see DoesNotExist */\n" +
52a04c670c05 8004834: Add doclint support into javadoc
jjg
parents:
diff changeset
    74
        /* 10 */    "    protected void referenceError() { }\n" +
52a04c670c05 8004834: Add doclint support into javadoc
jjg
parents:
diff changeset
    75
        /* 11 */    "\n" +
52a04c670c05 8004834: Add doclint support into javadoc
jjg
parents:
diff changeset
    76
        /* 12 */    "    /** @return */\n" +
52a04c670c05 8004834: Add doclint support into javadoc
jjg
parents:
diff changeset
    77
        /* 13 */    "    public int emptyReturn() { return 0; }\n" +
52a04c670c05 8004834: Add doclint support into javadoc
jjg
parents:
diff changeset
    78
        /* 14 */    "}\n";
52a04c670c05 8004834: Add doclint support into javadoc
jjg
parents:
diff changeset
    79
31752
a4ea4c9bce2f 8129909: Add -Xdoclint/package: to javadoc
jlahoda
parents: 30730
diff changeset
    80
    final String p1Code =
a4ea4c9bce2f 8129909: Add -Xdoclint/package: to javadoc
jlahoda
parents: 30730
diff changeset
    81
        /* 01 */    "package p1;\n" +
a4ea4c9bce2f 8129909: Add -Xdoclint/package: to javadoc
jlahoda
parents: 30730
diff changeset
    82
        /* 02 */    "public class P1Test {\n" +
a4ea4c9bce2f 8129909: Add -Xdoclint/package: to javadoc
jlahoda
parents: 30730
diff changeset
    83
        /* 03 */    "    /** Syntax < error. */\n" +
a4ea4c9bce2f 8129909: Add -Xdoclint/package: to javadoc
jlahoda
parents: 30730
diff changeset
    84
        /* 04 */    "    public void method() { }\n" +
a4ea4c9bce2f 8129909: Add -Xdoclint/package: to javadoc
jlahoda
parents: 30730
diff changeset
    85
        /* 05 */    "}\n";
a4ea4c9bce2f 8129909: Add -Xdoclint/package: to javadoc
jlahoda
parents: 30730
diff changeset
    86
a4ea4c9bce2f 8129909: Add -Xdoclint/package: to javadoc
jlahoda
parents: 30730
diff changeset
    87
    final String p2Code =
a4ea4c9bce2f 8129909: Add -Xdoclint/package: to javadoc
jlahoda
parents: 30730
diff changeset
    88
        /* 01 */    "package p2;\n" +
a4ea4c9bce2f 8129909: Add -Xdoclint/package: to javadoc
jlahoda
parents: 30730
diff changeset
    89
        /* 02 */    "public class P2Test {\n" +
a4ea4c9bce2f 8129909: Add -Xdoclint/package: to javadoc
jlahoda
parents: 30730
diff changeset
    90
        /* 03 */    "    /** Syntax < error. */\n" +
a4ea4c9bce2f 8129909: Add -Xdoclint/package: to javadoc
jlahoda
parents: 30730
diff changeset
    91
        /* 04 */    "    public void method() { }\n" +
a4ea4c9bce2f 8129909: Add -Xdoclint/package: to javadoc
jlahoda
parents: 30730
diff changeset
    92
        /* 05 */    "}\n";
a4ea4c9bce2f 8129909: Add -Xdoclint/package: to javadoc
jlahoda
parents: 30730
diff changeset
    93
15354
52a04c670c05 8004834: Add doclint support into javadoc
jjg
parents:
diff changeset
    94
    private final String rawDiags = "-XDrawDiagnostics";
52a04c670c05 8004834: Add doclint support into javadoc
jjg
parents:
diff changeset
    95
52a04c670c05 8004834: Add doclint support into javadoc
jjg
parents:
diff changeset
    96
    private enum Message {
52a04c670c05 8004834: Add doclint support into javadoc
jjg
parents:
diff changeset
    97
        // doclint messages
52a04c670c05 8004834: Add doclint support into javadoc
jjg
parents:
diff changeset
    98
        DL_ERR6(ERROR, "Test.java:6:16: compiler.err.proc.messager: malformed HTML"),
52a04c670c05 8004834: Add doclint support into javadoc
jjg
parents:
diff changeset
    99
        DL_ERR9(ERROR, "Test.java:9:14: compiler.err.proc.messager: reference not found"),
52a04c670c05 8004834: Add doclint support into javadoc
jjg
parents:
diff changeset
   100
        DL_WRN12(WARNING, "Test.java:12:9: compiler.warn.proc.messager: no description for @return"),
52a04c670c05 8004834: Add doclint support into javadoc
jjg
parents:
diff changeset
   101
31752
a4ea4c9bce2f 8129909: Add -Xdoclint/package: to javadoc
jlahoda
parents: 30730
diff changeset
   102
        DL_ERR_P1TEST(ERROR, "P1Test.java:3:16: compiler.err.proc.messager: malformed HTML"),
a4ea4c9bce2f 8129909: Add -Xdoclint/package: to javadoc
jlahoda
parents: 30730
diff changeset
   103
        DL_ERR_P2TEST(ERROR, "P2Test.java:3:16: compiler.err.proc.messager: malformed HTML"),
a4ea4c9bce2f 8129909: Add -Xdoclint/package: to javadoc
jlahoda
parents: 30730
diff changeset
   104
15354
52a04c670c05 8004834: Add doclint support into javadoc
jjg
parents:
diff changeset
   105
        // doclint messages when -XDrawDiagnostics is not in effect
52a04c670c05 8004834: Add doclint support into javadoc
jjg
parents:
diff changeset
   106
        DL_ERR9A(ERROR, "Test.java:9: error: reference not found"),
52a04c670c05 8004834: Add doclint support into javadoc
jjg
parents:
diff changeset
   107
        DL_WRN12A(WARNING, "Test.java:12: warning: no description for @return"),
52a04c670c05 8004834: Add doclint support into javadoc
jjg
parents:
diff changeset
   108
52a04c670c05 8004834: Add doclint support into javadoc
jjg
parents:
diff changeset
   109
        // javadoc messages about bad content: these should only appear when doclint is disabled
52a04c670c05 8004834: Add doclint support into javadoc
jjg
parents:
diff changeset
   110
        JD_WRN10(WARNING, "Test.java:10: warning - Tag @see: reference not found: DoesNotExist"),
52a04c670c05 8004834: Add doclint support into javadoc
jjg
parents:
diff changeset
   111
        JD_WRN13(WARNING, "Test.java:13: warning - @return tag has no arguments."),
52a04c670c05 8004834: Add doclint support into javadoc
jjg
parents:
diff changeset
   112
52a04c670c05 8004834: Add doclint support into javadoc
jjg
parents:
diff changeset
   113
        // javadoc messages for bad options
52a04c670c05 8004834: Add doclint support into javadoc
jjg
parents:
diff changeset
   114
        OPT_BADARG(ERROR, "javadoc: error - Invalid argument for -Xdoclint option"),
31752
a4ea4c9bce2f 8129909: Add -Xdoclint/package: to javadoc
jlahoda
parents: 30730
diff changeset
   115
        OPT_BADQUAL(ERROR, "javadoc: error - Access qualifiers not permitted for -Xdoclint arguments"),
a4ea4c9bce2f 8129909: Add -Xdoclint/package: to javadoc
jlahoda
parents: 30730
diff changeset
   116
        OPT_BADPACKAGEARG(ERROR, "javadoc: error - Invalid argument for -Xdoclint/package option");
15354
52a04c670c05 8004834: Add doclint support into javadoc
jjg
parents:
diff changeset
   117
52a04c670c05 8004834: Add doclint support into javadoc
jjg
parents:
diff changeset
   118
        final Diagnostic.Kind kind;
52a04c670c05 8004834: Add doclint support into javadoc
jjg
parents:
diff changeset
   119
        final String text;
52a04c670c05 8004834: Add doclint support into javadoc
jjg
parents:
diff changeset
   120
52a04c670c05 8004834: Add doclint support into javadoc
jjg
parents:
diff changeset
   121
        static Message get(String text) {
52a04c670c05 8004834: Add doclint support into javadoc
jjg
parents:
diff changeset
   122
            for (Message m: values()) {
52a04c670c05 8004834: Add doclint support into javadoc
jjg
parents:
diff changeset
   123
                if (m.text.equals(text))
52a04c670c05 8004834: Add doclint support into javadoc
jjg
parents:
diff changeset
   124
                    return m;
52a04c670c05 8004834: Add doclint support into javadoc
jjg
parents:
diff changeset
   125
            }
52a04c670c05 8004834: Add doclint support into javadoc
jjg
parents:
diff changeset
   126
            return null;
52a04c670c05 8004834: Add doclint support into javadoc
jjg
parents:
diff changeset
   127
        }
52a04c670c05 8004834: Add doclint support into javadoc
jjg
parents:
diff changeset
   128
52a04c670c05 8004834: Add doclint support into javadoc
jjg
parents:
diff changeset
   129
        Message(Diagnostic.Kind kind, String text) {
52a04c670c05 8004834: Add doclint support into javadoc
jjg
parents:
diff changeset
   130
            this.kind = kind;
52a04c670c05 8004834: Add doclint support into javadoc
jjg
parents:
diff changeset
   131
            this.text = text;
52a04c670c05 8004834: Add doclint support into javadoc
jjg
parents:
diff changeset
   132
        }
52a04c670c05 8004834: Add doclint support into javadoc
jjg
parents:
diff changeset
   133
52a04c670c05 8004834: Add doclint support into javadoc
jjg
parents:
diff changeset
   134
        @Override
52a04c670c05 8004834: Add doclint support into javadoc
jjg
parents:
diff changeset
   135
        public String toString() {
52a04c670c05 8004834: Add doclint support into javadoc
jjg
parents:
diff changeset
   136
            return "[" + kind + ",\"" + text + "\"]";
52a04c670c05 8004834: Add doclint support into javadoc
jjg
parents:
diff changeset
   137
        }
52a04c670c05 8004834: Add doclint support into javadoc
jjg
parents:
diff changeset
   138
    }
52a04c670c05 8004834: Add doclint support into javadoc
jjg
parents:
diff changeset
   139
52a04c670c05 8004834: Add doclint support into javadoc
jjg
parents:
diff changeset
   140
    void run() throws Exception {
52a04c670c05 8004834: Add doclint support into javadoc
jjg
parents:
diff changeset
   141
        javadoc = ToolProvider.getSystemDocumentationTool();
52a04c670c05 8004834: Add doclint support into javadoc
jjg
parents:
diff changeset
   142
        fm = javadoc.getStandardFileManager(null, null, null);
27319
030080f03e4f 8062348: langtools tests should close file manager (group 1)
jjg
parents: 15713
diff changeset
   143
        try {
030080f03e4f 8062348: langtools tests should close file manager (group 1)
jjg
parents: 15713
diff changeset
   144
            fm.setLocation(StandardLocation.CLASS_OUTPUT, Arrays.asList(new File(".")));
36526
3b41f1c69604 8142968: Module System implementation
alanb
parents: 35426
diff changeset
   145
            fm.setLocation(StandardLocation.CLASS_PATH, Collections.<File>emptyList());
31752
a4ea4c9bce2f 8129909: Add -Xdoclint/package: to javadoc
jlahoda
parents: 30730
diff changeset
   146
            files = Arrays.asList(new TestJFO("Test.java", code));
15354
52a04c670c05 8004834: Add doclint support into javadoc
jjg
parents:
diff changeset
   147
27319
030080f03e4f 8062348: langtools tests should close file manager (group 1)
jjg
parents: 15713
diff changeset
   148
            test(Collections.<String>emptyList(),
030080f03e4f 8062348: langtools tests should close file manager (group 1)
jjg
parents: 15713
diff changeset
   149
                    Main.Result.ERROR,
030080f03e4f 8062348: langtools tests should close file manager (group 1)
jjg
parents: 15713
diff changeset
   150
                    EnumSet.of(Message.DL_ERR9A, Message.DL_WRN12A));
15354
52a04c670c05 8004834: Add doclint support into javadoc
jjg
parents:
diff changeset
   151
27319
030080f03e4f 8062348: langtools tests should close file manager (group 1)
jjg
parents: 15713
diff changeset
   152
            test(Arrays.asList(rawDiags),
030080f03e4f 8062348: langtools tests should close file manager (group 1)
jjg
parents: 15713
diff changeset
   153
                    Main.Result.ERROR,
030080f03e4f 8062348: langtools tests should close file manager (group 1)
jjg
parents: 15713
diff changeset
   154
                    EnumSet.of(Message.DL_ERR9, Message.DL_WRN12));
030080f03e4f 8062348: langtools tests should close file manager (group 1)
jjg
parents: 15713
diff changeset
   155
35426
374342e56a56 8035473: [javadoc] Revamp the existing Doclet APIs
ksrini
parents: 31752
diff changeset
   156
//            test(Arrays.asList("-Xdoclint:none"),
374342e56a56 8035473: [javadoc] Revamp the existing Doclet APIs
ksrini
parents: 31752
diff changeset
   157
//                    Main.Result.OK,
374342e56a56 8035473: [javadoc] Revamp the existing Doclet APIs
ksrini
parents: 31752
diff changeset
   158
//                    EnumSet.of(Message.JD_WRN10, Message.JD_WRN13));
15354
52a04c670c05 8004834: Add doclint support into javadoc
jjg
parents:
diff changeset
   159
27319
030080f03e4f 8062348: langtools tests should close file manager (group 1)
jjg
parents: 15713
diff changeset
   160
            test(Arrays.asList(rawDiags, "-Xdoclint"),
030080f03e4f 8062348: langtools tests should close file manager (group 1)
jjg
parents: 15713
diff changeset
   161
                    Main.Result.ERROR,
030080f03e4f 8062348: langtools tests should close file manager (group 1)
jjg
parents: 15713
diff changeset
   162
                    EnumSet.of(Message.DL_ERR9, Message.DL_WRN12));
15354
52a04c670c05 8004834: Add doclint support into javadoc
jjg
parents:
diff changeset
   163
27319
030080f03e4f 8062348: langtools tests should close file manager (group 1)
jjg
parents: 15713
diff changeset
   164
            test(Arrays.asList(rawDiags, "-Xdoclint:all/public"),
030080f03e4f 8062348: langtools tests should close file manager (group 1)
jjg
parents: 15713
diff changeset
   165
                    Main.Result.ERROR,
030080f03e4f 8062348: langtools tests should close file manager (group 1)
jjg
parents: 15713
diff changeset
   166
                    EnumSet.of(Message.OPT_BADQUAL));
15354
52a04c670c05 8004834: Add doclint support into javadoc
jjg
parents:
diff changeset
   167
27319
030080f03e4f 8062348: langtools tests should close file manager (group 1)
jjg
parents: 15713
diff changeset
   168
            test(Arrays.asList(rawDiags, "-Xdoclint:all", "-public"),
030080f03e4f 8062348: langtools tests should close file manager (group 1)
jjg
parents: 15713
diff changeset
   169
                    Main.Result.OK,
030080f03e4f 8062348: langtools tests should close file manager (group 1)
jjg
parents: 15713
diff changeset
   170
                    EnumSet.of(Message.DL_WRN12));
15354
52a04c670c05 8004834: Add doclint support into javadoc
jjg
parents:
diff changeset
   171
27319
030080f03e4f 8062348: langtools tests should close file manager (group 1)
jjg
parents: 15713
diff changeset
   172
            test(Arrays.asList(rawDiags, "-Xdoclint:syntax"),
030080f03e4f 8062348: langtools tests should close file manager (group 1)
jjg
parents: 15713
diff changeset
   173
                    Main.Result.OK,
030080f03e4f 8062348: langtools tests should close file manager (group 1)
jjg
parents: 15713
diff changeset
   174
                    EnumSet.of(Message.DL_WRN12));
15354
52a04c670c05 8004834: Add doclint support into javadoc
jjg
parents:
diff changeset
   175
27319
030080f03e4f 8062348: langtools tests should close file manager (group 1)
jjg
parents: 15713
diff changeset
   176
            test(Arrays.asList(rawDiags, "-private"),
030080f03e4f 8062348: langtools tests should close file manager (group 1)
jjg
parents: 15713
diff changeset
   177
                    Main.Result.ERROR,
030080f03e4f 8062348: langtools tests should close file manager (group 1)
jjg
parents: 15713
diff changeset
   178
                    EnumSet.of(Message.DL_ERR6, Message.DL_ERR9, Message.DL_WRN12));
15354
52a04c670c05 8004834: Add doclint support into javadoc
jjg
parents:
diff changeset
   179
27319
030080f03e4f 8062348: langtools tests should close file manager (group 1)
jjg
parents: 15713
diff changeset
   180
            test(Arrays.asList(rawDiags, "-Xdoclint:syntax", "-private"),
030080f03e4f 8062348: langtools tests should close file manager (group 1)
jjg
parents: 15713
diff changeset
   181
                    Main.Result.ERROR,
030080f03e4f 8062348: langtools tests should close file manager (group 1)
jjg
parents: 15713
diff changeset
   182
                    EnumSet.of(Message.DL_ERR6, Message.DL_WRN12));
15713
55f55d4f5f9f 8007610: javadoc doclint does not work with -private
jjg
parents: 15354
diff changeset
   183
27319
030080f03e4f 8062348: langtools tests should close file manager (group 1)
jjg
parents: 15713
diff changeset
   184
            test(Arrays.asList(rawDiags, "-Xdoclint:reference"),
030080f03e4f 8062348: langtools tests should close file manager (group 1)
jjg
parents: 15713
diff changeset
   185
                    Main.Result.ERROR,
030080f03e4f 8062348: langtools tests should close file manager (group 1)
jjg
parents: 15713
diff changeset
   186
                    EnumSet.of(Message.DL_ERR9));
15354
52a04c670c05 8004834: Add doclint support into javadoc
jjg
parents:
diff changeset
   187
27319
030080f03e4f 8062348: langtools tests should close file manager (group 1)
jjg
parents: 15713
diff changeset
   188
            test(Arrays.asList(rawDiags, "-Xdoclint:badarg"),
030080f03e4f 8062348: langtools tests should close file manager (group 1)
jjg
parents: 15713
diff changeset
   189
                    Main.Result.ERROR,
030080f03e4f 8062348: langtools tests should close file manager (group 1)
jjg
parents: 15713
diff changeset
   190
                    EnumSet.of(Message.OPT_BADARG));
15354
52a04c670c05 8004834: Add doclint support into javadoc
jjg
parents:
diff changeset
   191
31752
a4ea4c9bce2f 8129909: Add -Xdoclint/package: to javadoc
jlahoda
parents: 30730
diff changeset
   192
            files = Arrays.asList(new TestJFO("p1/P1Test.java", p1Code),
a4ea4c9bce2f 8129909: Add -Xdoclint/package: to javadoc
jlahoda
parents: 30730
diff changeset
   193
                                  new TestJFO("p2/P2Test.java", p2Code));
a4ea4c9bce2f 8129909: Add -Xdoclint/package: to javadoc
jlahoda
parents: 30730
diff changeset
   194
a4ea4c9bce2f 8129909: Add -Xdoclint/package: to javadoc
jlahoda
parents: 30730
diff changeset
   195
            test(Arrays.asList(rawDiags),
a4ea4c9bce2f 8129909: Add -Xdoclint/package: to javadoc
jlahoda
parents: 30730
diff changeset
   196
                    Main.Result.ERROR,
a4ea4c9bce2f 8129909: Add -Xdoclint/package: to javadoc
jlahoda
parents: 30730
diff changeset
   197
                    EnumSet.of(Message.DL_ERR_P1TEST, Message.DL_ERR_P2TEST));
a4ea4c9bce2f 8129909: Add -Xdoclint/package: to javadoc
jlahoda
parents: 30730
diff changeset
   198
a4ea4c9bce2f 8129909: Add -Xdoclint/package: to javadoc
jlahoda
parents: 30730
diff changeset
   199
            test(Arrays.asList(rawDiags, "-Xdoclint/package:p1"),
a4ea4c9bce2f 8129909: Add -Xdoclint/package: to javadoc
jlahoda
parents: 30730
diff changeset
   200
                    Main.Result.ERROR,
a4ea4c9bce2f 8129909: Add -Xdoclint/package: to javadoc
jlahoda
parents: 30730
diff changeset
   201
                    EnumSet.of(Message.DL_ERR_P1TEST));
a4ea4c9bce2f 8129909: Add -Xdoclint/package: to javadoc
jlahoda
parents: 30730
diff changeset
   202
a4ea4c9bce2f 8129909: Add -Xdoclint/package: to javadoc
jlahoda
parents: 30730
diff changeset
   203
            test(Arrays.asList(rawDiags, "-Xdoclint/package:*p"),
a4ea4c9bce2f 8129909: Add -Xdoclint/package: to javadoc
jlahoda
parents: 30730
diff changeset
   204
                    Main.Result.ERROR,
a4ea4c9bce2f 8129909: Add -Xdoclint/package: to javadoc
jlahoda
parents: 30730
diff changeset
   205
                    EnumSet.of(Message.OPT_BADPACKAGEARG));
a4ea4c9bce2f 8129909: Add -Xdoclint/package: to javadoc
jlahoda
parents: 30730
diff changeset
   206
27319
030080f03e4f 8062348: langtools tests should close file manager (group 1)
jjg
parents: 15713
diff changeset
   207
            if (errors > 0)
030080f03e4f 8062348: langtools tests should close file manager (group 1)
jjg
parents: 15713
diff changeset
   208
                throw new Exception(errors + " errors occurred");
030080f03e4f 8062348: langtools tests should close file manager (group 1)
jjg
parents: 15713
diff changeset
   209
        } finally {
030080f03e4f 8062348: langtools tests should close file manager (group 1)
jjg
parents: 15713
diff changeset
   210
            fm.close();
030080f03e4f 8062348: langtools tests should close file manager (group 1)
jjg
parents: 15713
diff changeset
   211
        }
15354
52a04c670c05 8004834: Add doclint support into javadoc
jjg
parents:
diff changeset
   212
    }
52a04c670c05 8004834: Add doclint support into javadoc
jjg
parents:
diff changeset
   213
52a04c670c05 8004834: Add doclint support into javadoc
jjg
parents:
diff changeset
   214
    void test(List<String> opts, Main.Result expectResult, Set<Message> expectMessages) {
52a04c670c05 8004834: Add doclint support into javadoc
jjg
parents:
diff changeset
   215
        System.err.println("test: " + opts);
52a04c670c05 8004834: Add doclint support into javadoc
jjg
parents:
diff changeset
   216
        StringWriter sw = new StringWriter();
52a04c670c05 8004834: Add doclint support into javadoc
jjg
parents:
diff changeset
   217
        PrintWriter pw = new PrintWriter(sw);
52a04c670c05 8004834: Add doclint support into javadoc
jjg
parents:
diff changeset
   218
        try {
52a04c670c05 8004834: Add doclint support into javadoc
jjg
parents:
diff changeset
   219
            DocumentationTask t = javadoc.getTask(pw, fm, null, null, opts, files);
52a04c670c05 8004834: Add doclint support into javadoc
jjg
parents:
diff changeset
   220
            boolean ok = t.call();
52a04c670c05 8004834: Add doclint support into javadoc
jjg
parents:
diff changeset
   221
            pw.close();
52a04c670c05 8004834: Add doclint support into javadoc
jjg
parents:
diff changeset
   222
            String out = sw.toString().replaceAll("[\r\n]+", "\n");
52a04c670c05 8004834: Add doclint support into javadoc
jjg
parents:
diff changeset
   223
            if (!out.isEmpty())
52a04c670c05 8004834: Add doclint support into javadoc
jjg
parents:
diff changeset
   224
                System.err.println(out);
52a04c670c05 8004834: Add doclint support into javadoc
jjg
parents:
diff changeset
   225
            if (ok && expectResult != Main.Result.OK) {
52a04c670c05 8004834: Add doclint support into javadoc
jjg
parents:
diff changeset
   226
                error("Compilation succeeded unexpectedly");
52a04c670c05 8004834: Add doclint support into javadoc
jjg
parents:
diff changeset
   227
            } else if (!ok && expectResult != Main.Result.ERROR) {
52a04c670c05 8004834: Add doclint support into javadoc
jjg
parents:
diff changeset
   228
                error("Compilation failed unexpectedly");
52a04c670c05 8004834: Add doclint support into javadoc
jjg
parents:
diff changeset
   229
            } else
52a04c670c05 8004834: Add doclint support into javadoc
jjg
parents:
diff changeset
   230
                check(out, expectMessages);
52a04c670c05 8004834: Add doclint support into javadoc
jjg
parents:
diff changeset
   231
        } catch (IllegalArgumentException e) {
52a04c670c05 8004834: Add doclint support into javadoc
jjg
parents:
diff changeset
   232
            System.err.println(e);
52a04c670c05 8004834: Add doclint support into javadoc
jjg
parents:
diff changeset
   233
            String expectOut = expectMessages.iterator().next().text;
52a04c670c05 8004834: Add doclint support into javadoc
jjg
parents:
diff changeset
   234
            if (expectResult != Main.Result.CMDERR)
52a04c670c05 8004834: Add doclint support into javadoc
jjg
parents:
diff changeset
   235
                error("unexpected exception caught");
52a04c670c05 8004834: Add doclint support into javadoc
jjg
parents:
diff changeset
   236
            else if (!e.getMessage().equals(expectOut)) {
52a04c670c05 8004834: Add doclint support into javadoc
jjg
parents:
diff changeset
   237
                error("unexpected exception message: "
52a04c670c05 8004834: Add doclint support into javadoc
jjg
parents:
diff changeset
   238
                        + e.getMessage()
52a04c670c05 8004834: Add doclint support into javadoc
jjg
parents:
diff changeset
   239
                        + " expected: " + expectOut);
52a04c670c05 8004834: Add doclint support into javadoc
jjg
parents:
diff changeset
   240
            }
52a04c670c05 8004834: Add doclint support into javadoc
jjg
parents:
diff changeset
   241
        }
52a04c670c05 8004834: Add doclint support into javadoc
jjg
parents:
diff changeset
   242
52a04c670c05 8004834: Add doclint support into javadoc
jjg
parents:
diff changeset
   243
//        if (errors > 0)
52a04c670c05 8004834: Add doclint support into javadoc
jjg
parents:
diff changeset
   244
//            throw new Error("stop");
52a04c670c05 8004834: Add doclint support into javadoc
jjg
parents:
diff changeset
   245
    }
52a04c670c05 8004834: Add doclint support into javadoc
jjg
parents:
diff changeset
   246
52a04c670c05 8004834: Add doclint support into javadoc
jjg
parents:
diff changeset
   247
    private void check(String out, Set<Message> expect) {
52a04c670c05 8004834: Add doclint support into javadoc
jjg
parents:
diff changeset
   248
        Pattern ignore = Pattern.compile("^(Building|Constructing|Generating|Loading|Standard|Starting| ) .*");
52a04c670c05 8004834: Add doclint support into javadoc
jjg
parents:
diff changeset
   249
        Pattern stats = Pattern.compile("^([1-9]+) (error|warning)(s?)");
52a04c670c05 8004834: Add doclint support into javadoc
jjg
parents:
diff changeset
   250
        Set<Message> found = EnumSet.noneOf(Message.class);
52a04c670c05 8004834: Add doclint support into javadoc
jjg
parents:
diff changeset
   251
        int e = 0, w = 0;
52a04c670c05 8004834: Add doclint support into javadoc
jjg
parents:
diff changeset
   252
        for (String line: out.split("[\r\n]+")) {
52a04c670c05 8004834: Add doclint support into javadoc
jjg
parents:
diff changeset
   253
            if (ignore.matcher(line).matches())
52a04c670c05 8004834: Add doclint support into javadoc
jjg
parents:
diff changeset
   254
                continue;
52a04c670c05 8004834: Add doclint support into javadoc
jjg
parents:
diff changeset
   255
52a04c670c05 8004834: Add doclint support into javadoc
jjg
parents:
diff changeset
   256
            Matcher s = stats.matcher(line);
52a04c670c05 8004834: Add doclint support into javadoc
jjg
parents:
diff changeset
   257
            if (s.matches()) {
52a04c670c05 8004834: Add doclint support into javadoc
jjg
parents:
diff changeset
   258
                int i = Integer.valueOf(s.group(1));
52a04c670c05 8004834: Add doclint support into javadoc
jjg
parents:
diff changeset
   259
                if (s.group(2).equals("error"))
52a04c670c05 8004834: Add doclint support into javadoc
jjg
parents:
diff changeset
   260
                    e++;
52a04c670c05 8004834: Add doclint support into javadoc
jjg
parents:
diff changeset
   261
                else
52a04c670c05 8004834: Add doclint support into javadoc
jjg
parents:
diff changeset
   262
                    w++;
52a04c670c05 8004834: Add doclint support into javadoc
jjg
parents:
diff changeset
   263
                continue;
52a04c670c05 8004834: Add doclint support into javadoc
jjg
parents:
diff changeset
   264
            }
52a04c670c05 8004834: Add doclint support into javadoc
jjg
parents:
diff changeset
   265
52a04c670c05 8004834: Add doclint support into javadoc
jjg
parents:
diff changeset
   266
            Message m = Message.get(line);
52a04c670c05 8004834: Add doclint support into javadoc
jjg
parents:
diff changeset
   267
            if (m == null)
52a04c670c05 8004834: Add doclint support into javadoc
jjg
parents:
diff changeset
   268
                error("Unexpected line: " + line);
52a04c670c05 8004834: Add doclint support into javadoc
jjg
parents:
diff changeset
   269
            else
52a04c670c05 8004834: Add doclint support into javadoc
jjg
parents:
diff changeset
   270
                found.add(m);
52a04c670c05 8004834: Add doclint support into javadoc
jjg
parents:
diff changeset
   271
        }
52a04c670c05 8004834: Add doclint support into javadoc
jjg
parents:
diff changeset
   272
        for (Message m: expect) {
52a04c670c05 8004834: Add doclint support into javadoc
jjg
parents:
diff changeset
   273
            if (!found.contains(m))
52a04c670c05 8004834: Add doclint support into javadoc
jjg
parents:
diff changeset
   274
                error("expected message not found: " + m.text);
52a04c670c05 8004834: Add doclint support into javadoc
jjg
parents:
diff changeset
   275
        }
52a04c670c05 8004834: Add doclint support into javadoc
jjg
parents:
diff changeset
   276
        for (Message m: found) {
52a04c670c05 8004834: Add doclint support into javadoc
jjg
parents:
diff changeset
   277
            if (!expect.contains(m))
52a04c670c05 8004834: Add doclint support into javadoc
jjg
parents:
diff changeset
   278
                error("unexpected message found: " + m.text);
52a04c670c05 8004834: Add doclint support into javadoc
jjg
parents:
diff changeset
   279
        }
52a04c670c05 8004834: Add doclint support into javadoc
jjg
parents:
diff changeset
   280
    }
52a04c670c05 8004834: Add doclint support into javadoc
jjg
parents:
diff changeset
   281
52a04c670c05 8004834: Add doclint support into javadoc
jjg
parents:
diff changeset
   282
    void error(String msg) {
52a04c670c05 8004834: Add doclint support into javadoc
jjg
parents:
diff changeset
   283
        System.err.println("Error: " + msg);
52a04c670c05 8004834: Add doclint support into javadoc
jjg
parents:
diff changeset
   284
        errors++;
52a04c670c05 8004834: Add doclint support into javadoc
jjg
parents:
diff changeset
   285
    }
52a04c670c05 8004834: Add doclint support into javadoc
jjg
parents:
diff changeset
   286
52a04c670c05 8004834: Add doclint support into javadoc
jjg
parents:
diff changeset
   287
    int errors;
31752
a4ea4c9bce2f 8129909: Add -Xdoclint/package: to javadoc
jlahoda
parents: 30730
diff changeset
   288
a4ea4c9bce2f 8129909: Add -Xdoclint/package: to javadoc
jlahoda
parents: 30730
diff changeset
   289
    class TestJFO extends SimpleJavaFileObject {
a4ea4c9bce2f 8129909: Add -Xdoclint/package: to javadoc
jlahoda
parents: 30730
diff changeset
   290
a4ea4c9bce2f 8129909: Add -Xdoclint/package: to javadoc
jlahoda
parents: 30730
diff changeset
   291
        private final String content;
a4ea4c9bce2f 8129909: Add -Xdoclint/package: to javadoc
jlahoda
parents: 30730
diff changeset
   292
a4ea4c9bce2f 8129909: Add -Xdoclint/package: to javadoc
jlahoda
parents: 30730
diff changeset
   293
        public TestJFO(String fileName, String content) {
a4ea4c9bce2f 8129909: Add -Xdoclint/package: to javadoc
jlahoda
parents: 30730
diff changeset
   294
            super(URI.create(fileName), JavaFileObject.Kind.SOURCE);
a4ea4c9bce2f 8129909: Add -Xdoclint/package: to javadoc
jlahoda
parents: 30730
diff changeset
   295
            this.content = content;
a4ea4c9bce2f 8129909: Add -Xdoclint/package: to javadoc
jlahoda
parents: 30730
diff changeset
   296
        }
a4ea4c9bce2f 8129909: Add -Xdoclint/package: to javadoc
jlahoda
parents: 30730
diff changeset
   297
a4ea4c9bce2f 8129909: Add -Xdoclint/package: to javadoc
jlahoda
parents: 30730
diff changeset
   298
        @Override
a4ea4c9bce2f 8129909: Add -Xdoclint/package: to javadoc
jlahoda
parents: 30730
diff changeset
   299
        public CharSequence getCharContent(boolean ignoreEncoding) {
a4ea4c9bce2f 8129909: Add -Xdoclint/package: to javadoc
jlahoda
parents: 30730
diff changeset
   300
            return content;
a4ea4c9bce2f 8129909: Add -Xdoclint/package: to javadoc
jlahoda
parents: 30730
diff changeset
   301
        }
a4ea4c9bce2f 8129909: Add -Xdoclint/package: to javadoc
jlahoda
parents: 30730
diff changeset
   302
    };
15354
52a04c670c05 8004834: Add doclint support into javadoc
jjg
parents:
diff changeset
   303
}