langtools/test/jdk/javadoc/tool/CheckResourceKeys.java
author jjg
Wed, 10 Aug 2016 15:47:46 -0700
changeset 40308 274367a99f98
parent 36526 3b41f1c69604
child 41252 058d83c9b1c7
permissions -rw-r--r--
8136930: Simplify use of module-system options by custom launchers 8160489: Multiple -Xpatch lines ignored by javac 8156998: javac should support new option -XinheritRuntimeEnvironment Reviewed-by: jlahoda, ksrini
Ignore whitespace changes - Everywhere: Within whitespace: At end of lines:
14447
6f87132c2e54 8000612: Discrepancy between resources provided in javadoc resource files and resources required by code
jjg
parents:
diff changeset
     1
/*
36526
3b41f1c69604 8142968: Module System implementation
alanb
parents: 35426
diff changeset
     2
 * Copyright (c) 2010, 2016, Oracle and/or its affiliates. All rights reserved.
14447
6f87132c2e54 8000612: Discrepancy between resources provided in javadoc resource files and resources required by code
jjg
parents:
diff changeset
     3
 * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
6f87132c2e54 8000612: Discrepancy between resources provided in javadoc resource files and resources required by code
jjg
parents:
diff changeset
     4
 *
6f87132c2e54 8000612: Discrepancy between resources provided in javadoc resource files and resources required by code
jjg
parents:
diff changeset
     5
 * This code is free software; you can redistribute it and/or modify it
6f87132c2e54 8000612: Discrepancy between resources provided in javadoc resource files and resources required by code
jjg
parents:
diff changeset
     6
 * under the terms of the GNU General Public License version 2 only, as
6f87132c2e54 8000612: Discrepancy between resources provided in javadoc resource files and resources required by code
jjg
parents:
diff changeset
     7
 * published by the Free Software Foundation.
6f87132c2e54 8000612: Discrepancy between resources provided in javadoc resource files and resources required by code
jjg
parents:
diff changeset
     8
 *
6f87132c2e54 8000612: Discrepancy between resources provided in javadoc resource files and resources required by code
jjg
parents:
diff changeset
     9
 * This code is distributed in the hope that it will be useful, but WITHOUT
6f87132c2e54 8000612: Discrepancy between resources provided in javadoc resource files and resources required by code
jjg
parents:
diff changeset
    10
 * ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or
6f87132c2e54 8000612: Discrepancy between resources provided in javadoc resource files and resources required by code
jjg
parents:
diff changeset
    11
 * FITNESS FOR A PARTICULAR PURPOSE.  See the GNU General Public License
6f87132c2e54 8000612: Discrepancy between resources provided in javadoc resource files and resources required by code
jjg
parents:
diff changeset
    12
 * version 2 for more details (a copy is included in the LICENSE file that
6f87132c2e54 8000612: Discrepancy between resources provided in javadoc resource files and resources required by code
jjg
parents:
diff changeset
    13
 * accompanied this code).
6f87132c2e54 8000612: Discrepancy between resources provided in javadoc resource files and resources required by code
jjg
parents:
diff changeset
    14
 *
6f87132c2e54 8000612: Discrepancy between resources provided in javadoc resource files and resources required by code
jjg
parents:
diff changeset
    15
 * You should have received a copy of the GNU General Public License version
6f87132c2e54 8000612: Discrepancy between resources provided in javadoc resource files and resources required by code
jjg
parents:
diff changeset
    16
 * 2 along with this work; if not, write to the Free Software Foundation,
6f87132c2e54 8000612: Discrepancy between resources provided in javadoc resource files and resources required by code
jjg
parents:
diff changeset
    17
 * Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA.
6f87132c2e54 8000612: Discrepancy between resources provided in javadoc resource files and resources required by code
jjg
parents:
diff changeset
    18
 *
6f87132c2e54 8000612: Discrepancy between resources provided in javadoc resource files and resources required by code
jjg
parents:
diff changeset
    19
 * Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA
6f87132c2e54 8000612: Discrepancy between resources provided in javadoc resource files and resources required by code
jjg
parents:
diff changeset
    20
 * or visit www.oracle.com if you need additional information or have any
6f87132c2e54 8000612: Discrepancy between resources provided in javadoc resource files and resources required by code
jjg
parents:
diff changeset
    21
 * questions.
6f87132c2e54 8000612: Discrepancy between resources provided in javadoc resource files and resources required by code
jjg
parents:
diff changeset
    22
 */
6f87132c2e54 8000612: Discrepancy between resources provided in javadoc resource files and resources required by code
jjg
parents:
diff changeset
    23
6f87132c2e54 8000612: Discrepancy between resources provided in javadoc resource files and resources required by code
jjg
parents:
diff changeset
    24
/*
6f87132c2e54 8000612: Discrepancy between resources provided in javadoc resource files and resources required by code
jjg
parents:
diff changeset
    25
 * @test
6f87132c2e54 8000612: Discrepancy between resources provided in javadoc resource files and resources required by code
jjg
parents:
diff changeset
    26
 * @bug 8000612
6f87132c2e54 8000612: Discrepancy between resources provided in javadoc resource files and resources required by code
jjg
parents:
diff changeset
    27
 * @summary need test program to validate javadoc resource bundles
36526
3b41f1c69604 8142968: Module System implementation
alanb
parents: 35426
diff changeset
    28
 * @modules jdk.javadoc/jdk.javadoc.internal.tool
3b41f1c69604 8142968: Module System implementation
alanb
parents: 35426
diff changeset
    29
 *          jdk.jdeps/com.sun.tools.classfile
14447
6f87132c2e54 8000612: Discrepancy between resources provided in javadoc resource files and resources required by code
jjg
parents:
diff changeset
    30
 */
6f87132c2e54 8000612: Discrepancy between resources provided in javadoc resource files and resources required by code
jjg
parents:
diff changeset
    31
6f87132c2e54 8000612: Discrepancy between resources provided in javadoc resource files and resources required by code
jjg
parents:
diff changeset
    32
import java.io.*;
36526
3b41f1c69604 8142968: Module System implementation
alanb
parents: 35426
diff changeset
    33
import java.lang.reflect.Layer;
3b41f1c69604 8142968: Module System implementation
alanb
parents: 35426
diff changeset
    34
import java.lang.reflect.Module;
14447
6f87132c2e54 8000612: Discrepancy between resources provided in javadoc resource files and resources required by code
jjg
parents:
diff changeset
    35
import java.util.*;
6f87132c2e54 8000612: Discrepancy between resources provided in javadoc resource files and resources required by code
jjg
parents:
diff changeset
    36
import javax.tools.*;
6f87132c2e54 8000612: Discrepancy between resources provided in javadoc resource files and resources required by code
jjg
parents:
diff changeset
    37
import com.sun.tools.classfile.*;
6f87132c2e54 8000612: Discrepancy between resources provided in javadoc resource files and resources required by code
jjg
parents:
diff changeset
    38
6f87132c2e54 8000612: Discrepancy between resources provided in javadoc resource files and resources required by code
jjg
parents:
diff changeset
    39
/**
6f87132c2e54 8000612: Discrepancy between resources provided in javadoc resource files and resources required by code
jjg
parents:
diff changeset
    40
 * Compare string constants in javadoc classes against keys in javadoc resource bundles.
6f87132c2e54 8000612: Discrepancy between resources provided in javadoc resource files and resources required by code
jjg
parents:
diff changeset
    41
 */
6f87132c2e54 8000612: Discrepancy between resources provided in javadoc resource files and resources required by code
jjg
parents:
diff changeset
    42
public class CheckResourceKeys {
6f87132c2e54 8000612: Discrepancy between resources provided in javadoc resource files and resources required by code
jjg
parents:
diff changeset
    43
    /**
6f87132c2e54 8000612: Discrepancy between resources provided in javadoc resource files and resources required by code
jjg
parents:
diff changeset
    44
     * Main program.
6f87132c2e54 8000612: Discrepancy between resources provided in javadoc resource files and resources required by code
jjg
parents:
diff changeset
    45
     * Options:
6f87132c2e54 8000612: Discrepancy between resources provided in javadoc resource files and resources required by code
jjg
parents:
diff changeset
    46
     * -finddeadkeys
6f87132c2e54 8000612: Discrepancy between resources provided in javadoc resource files and resources required by code
jjg
parents:
diff changeset
    47
     *      look for keys in resource bundles that are no longer required
6f87132c2e54 8000612: Discrepancy between resources provided in javadoc resource files and resources required by code
jjg
parents:
diff changeset
    48
     * -findmissingkeys
6f87132c2e54 8000612: Discrepancy between resources provided in javadoc resource files and resources required by code
jjg
parents:
diff changeset
    49
     *      look for keys in resource bundles that are missing
6f87132c2e54 8000612: Discrepancy between resources provided in javadoc resource files and resources required by code
jjg
parents:
diff changeset
    50
     *
6f87132c2e54 8000612: Discrepancy between resources provided in javadoc resource files and resources required by code
jjg
parents:
diff changeset
    51
     * @throws Exception if invoked by jtreg and errors occur
6f87132c2e54 8000612: Discrepancy between resources provided in javadoc resource files and resources required by code
jjg
parents:
diff changeset
    52
     */
6f87132c2e54 8000612: Discrepancy between resources provided in javadoc resource files and resources required by code
jjg
parents:
diff changeset
    53
    public static void main(String... args) throws Exception {
6f87132c2e54 8000612: Discrepancy between resources provided in javadoc resource files and resources required by code
jjg
parents:
diff changeset
    54
        CheckResourceKeys c = new CheckResourceKeys();
6f87132c2e54 8000612: Discrepancy between resources provided in javadoc resource files and resources required by code
jjg
parents:
diff changeset
    55
        if (c.run(args))
6f87132c2e54 8000612: Discrepancy between resources provided in javadoc resource files and resources required by code
jjg
parents:
diff changeset
    56
            return;
6f87132c2e54 8000612: Discrepancy between resources provided in javadoc resource files and resources required by code
jjg
parents:
diff changeset
    57
6f87132c2e54 8000612: Discrepancy between resources provided in javadoc resource files and resources required by code
jjg
parents:
diff changeset
    58
        if (is_jtreg())
6f87132c2e54 8000612: Discrepancy between resources provided in javadoc resource files and resources required by code
jjg
parents:
diff changeset
    59
            throw new Exception(c.errors + " errors occurred");
6f87132c2e54 8000612: Discrepancy between resources provided in javadoc resource files and resources required by code
jjg
parents:
diff changeset
    60
        else
6f87132c2e54 8000612: Discrepancy between resources provided in javadoc resource files and resources required by code
jjg
parents:
diff changeset
    61
            System.exit(1);
6f87132c2e54 8000612: Discrepancy between resources provided in javadoc resource files and resources required by code
jjg
parents:
diff changeset
    62
    }
6f87132c2e54 8000612: Discrepancy between resources provided in javadoc resource files and resources required by code
jjg
parents:
diff changeset
    63
6f87132c2e54 8000612: Discrepancy between resources provided in javadoc resource files and resources required by code
jjg
parents:
diff changeset
    64
    static boolean is_jtreg() {
6f87132c2e54 8000612: Discrepancy between resources provided in javadoc resource files and resources required by code
jjg
parents:
diff changeset
    65
        return (System.getProperty("test.src") != null);
6f87132c2e54 8000612: Discrepancy between resources provided in javadoc resource files and resources required by code
jjg
parents:
diff changeset
    66
    }
6f87132c2e54 8000612: Discrepancy between resources provided in javadoc resource files and resources required by code
jjg
parents:
diff changeset
    67
6f87132c2e54 8000612: Discrepancy between resources provided in javadoc resource files and resources required by code
jjg
parents:
diff changeset
    68
    /**
6f87132c2e54 8000612: Discrepancy between resources provided in javadoc resource files and resources required by code
jjg
parents:
diff changeset
    69
     * Main entry point.
6f87132c2e54 8000612: Discrepancy between resources provided in javadoc resource files and resources required by code
jjg
parents:
diff changeset
    70
     */
6f87132c2e54 8000612: Discrepancy between resources provided in javadoc resource files and resources required by code
jjg
parents:
diff changeset
    71
    boolean run(String... args) throws Exception {
6f87132c2e54 8000612: Discrepancy between resources provided in javadoc resource files and resources required by code
jjg
parents:
diff changeset
    72
        boolean findDeadKeys = false;
6f87132c2e54 8000612: Discrepancy between resources provided in javadoc resource files and resources required by code
jjg
parents:
diff changeset
    73
        boolean findMissingKeys = false;
6f87132c2e54 8000612: Discrepancy between resources provided in javadoc resource files and resources required by code
jjg
parents:
diff changeset
    74
6f87132c2e54 8000612: Discrepancy between resources provided in javadoc resource files and resources required by code
jjg
parents:
diff changeset
    75
        if (args.length == 0) {
6f87132c2e54 8000612: Discrepancy between resources provided in javadoc resource files and resources required by code
jjg
parents:
diff changeset
    76
            if (is_jtreg()) {
6f87132c2e54 8000612: Discrepancy between resources provided in javadoc resource files and resources required by code
jjg
parents:
diff changeset
    77
                findDeadKeys = true;
6f87132c2e54 8000612: Discrepancy between resources provided in javadoc resource files and resources required by code
jjg
parents:
diff changeset
    78
                findMissingKeys = true;
6f87132c2e54 8000612: Discrepancy between resources provided in javadoc resource files and resources required by code
jjg
parents:
diff changeset
    79
            } else {
6f87132c2e54 8000612: Discrepancy between resources provided in javadoc resource files and resources required by code
jjg
parents:
diff changeset
    80
                System.err.println("Usage: java CheckResourceKeys <options>");
6f87132c2e54 8000612: Discrepancy between resources provided in javadoc resource files and resources required by code
jjg
parents:
diff changeset
    81
                System.err.println("where options include");
6f87132c2e54 8000612: Discrepancy between resources provided in javadoc resource files and resources required by code
jjg
parents:
diff changeset
    82
                System.err.println("  -finddeadkeys      find keys in resource bundles which are no longer required");
6f87132c2e54 8000612: Discrepancy between resources provided in javadoc resource files and resources required by code
jjg
parents:
diff changeset
    83
                System.err.println("  -findmissingkeys   find keys in resource bundles that are required but missing");
6f87132c2e54 8000612: Discrepancy between resources provided in javadoc resource files and resources required by code
jjg
parents:
diff changeset
    84
                return true;
6f87132c2e54 8000612: Discrepancy between resources provided in javadoc resource files and resources required by code
jjg
parents:
diff changeset
    85
            }
6f87132c2e54 8000612: Discrepancy between resources provided in javadoc resource files and resources required by code
jjg
parents:
diff changeset
    86
        } else {
6f87132c2e54 8000612: Discrepancy between resources provided in javadoc resource files and resources required by code
jjg
parents:
diff changeset
    87
            for (String arg: args) {
6f87132c2e54 8000612: Discrepancy between resources provided in javadoc resource files and resources required by code
jjg
parents:
diff changeset
    88
                if (arg.equalsIgnoreCase("-finddeadkeys"))
6f87132c2e54 8000612: Discrepancy between resources provided in javadoc resource files and resources required by code
jjg
parents:
diff changeset
    89
                    findDeadKeys = true;
6f87132c2e54 8000612: Discrepancy between resources provided in javadoc resource files and resources required by code
jjg
parents:
diff changeset
    90
                else if (arg.equalsIgnoreCase("-findmissingkeys"))
6f87132c2e54 8000612: Discrepancy between resources provided in javadoc resource files and resources required by code
jjg
parents:
diff changeset
    91
                    findMissingKeys = true;
6f87132c2e54 8000612: Discrepancy between resources provided in javadoc resource files and resources required by code
jjg
parents:
diff changeset
    92
                else
6f87132c2e54 8000612: Discrepancy between resources provided in javadoc resource files and resources required by code
jjg
parents:
diff changeset
    93
                    error("bad option: " + arg);
6f87132c2e54 8000612: Discrepancy between resources provided in javadoc resource files and resources required by code
jjg
parents:
diff changeset
    94
            }
6f87132c2e54 8000612: Discrepancy between resources provided in javadoc resource files and resources required by code
jjg
parents:
diff changeset
    95
        }
6f87132c2e54 8000612: Discrepancy between resources provided in javadoc resource files and resources required by code
jjg
parents:
diff changeset
    96
6f87132c2e54 8000612: Discrepancy between resources provided in javadoc resource files and resources required by code
jjg
parents:
diff changeset
    97
        if (errors > 0)
6f87132c2e54 8000612: Discrepancy between resources provided in javadoc resource files and resources required by code
jjg
parents:
diff changeset
    98
            return false;
6f87132c2e54 8000612: Discrepancy between resources provided in javadoc resource files and resources required by code
jjg
parents:
diff changeset
    99
6f87132c2e54 8000612: Discrepancy between resources provided in javadoc resource files and resources required by code
jjg
parents:
diff changeset
   100
        Set<String> codeKeys = getCodeKeys();
6f87132c2e54 8000612: Discrepancy between resources provided in javadoc resource files and resources required by code
jjg
parents:
diff changeset
   101
        Set<String> resourceKeys = getResourceKeys();
6f87132c2e54 8000612: Discrepancy between resources provided in javadoc resource files and resources required by code
jjg
parents:
diff changeset
   102
6f87132c2e54 8000612: Discrepancy between resources provided in javadoc resource files and resources required by code
jjg
parents:
diff changeset
   103
        System.err.println("found " + codeKeys.size() + " keys in code");
6f87132c2e54 8000612: Discrepancy between resources provided in javadoc resource files and resources required by code
jjg
parents:
diff changeset
   104
        System.err.println("found " + resourceKeys.size() + " keys in resource bundles");
6f87132c2e54 8000612: Discrepancy between resources provided in javadoc resource files and resources required by code
jjg
parents:
diff changeset
   105
6f87132c2e54 8000612: Discrepancy between resources provided in javadoc resource files and resources required by code
jjg
parents:
diff changeset
   106
        if (findDeadKeys)
6f87132c2e54 8000612: Discrepancy between resources provided in javadoc resource files and resources required by code
jjg
parents:
diff changeset
   107
            findDeadKeys(codeKeys, resourceKeys);
6f87132c2e54 8000612: Discrepancy between resources provided in javadoc resource files and resources required by code
jjg
parents:
diff changeset
   108
6f87132c2e54 8000612: Discrepancy between resources provided in javadoc resource files and resources required by code
jjg
parents:
diff changeset
   109
        if (findMissingKeys)
6f87132c2e54 8000612: Discrepancy between resources provided in javadoc resource files and resources required by code
jjg
parents:
diff changeset
   110
            findMissingKeys(codeKeys, resourceKeys);
6f87132c2e54 8000612: Discrepancy between resources provided in javadoc resource files and resources required by code
jjg
parents:
diff changeset
   111
35426
374342e56a56 8035473: [javadoc] Revamp the existing Doclet APIs
ksrini
parents: 30846
diff changeset
   112
        usageTests(false);
374342e56a56 8035473: [javadoc] Revamp the existing Doclet APIs
ksrini
parents: 30846
diff changeset
   113
        usageTests(true);
374342e56a56 8035473: [javadoc] Revamp the existing Doclet APIs
ksrini
parents: 30846
diff changeset
   114
14447
6f87132c2e54 8000612: Discrepancy between resources provided in javadoc resource files and resources required by code
jjg
parents:
diff changeset
   115
        return (errors == 0);
6f87132c2e54 8000612: Discrepancy between resources provided in javadoc resource files and resources required by code
jjg
parents:
diff changeset
   116
    }
6f87132c2e54 8000612: Discrepancy between resources provided in javadoc resource files and resources required by code
jjg
parents:
diff changeset
   117
35426
374342e56a56 8035473: [javadoc] Revamp the existing Doclet APIs
ksrini
parents: 30846
diff changeset
   118
    void usageTests(boolean xflag) {
374342e56a56 8035473: [javadoc] Revamp the existing Doclet APIs
ksrini
parents: 30846
diff changeset
   119
        String[] argarray = { xflag ? "-X" : "-help" };
374342e56a56 8035473: [javadoc] Revamp the existing Doclet APIs
ksrini
parents: 30846
diff changeset
   120
        StringWriter sw = new StringWriter();
374342e56a56 8035473: [javadoc] Revamp the existing Doclet APIs
ksrini
parents: 30846
diff changeset
   121
        PrintWriter pw = new PrintWriter(sw);
374342e56a56 8035473: [javadoc] Revamp the existing Doclet APIs
ksrini
parents: 30846
diff changeset
   122
        if (jdk.javadoc.internal.tool.Main.execute(argarray, pw) == 0) {
374342e56a56 8035473: [javadoc] Revamp the existing Doclet APIs
ksrini
parents: 30846
diff changeset
   123
            pw.flush();
374342e56a56 8035473: [javadoc] Revamp the existing Doclet APIs
ksrini
parents: 30846
diff changeset
   124
            String s = sw.toString();
374342e56a56 8035473: [javadoc] Revamp the existing Doclet APIs
ksrini
parents: 30846
diff changeset
   125
            if (s.isEmpty()) {
374342e56a56 8035473: [javadoc] Revamp the existing Doclet APIs
ksrini
parents: 30846
diff changeset
   126
                error("no javadoc output ?");
374342e56a56 8035473: [javadoc] Revamp the existing Doclet APIs
ksrini
parents: 30846
diff changeset
   127
                return;
374342e56a56 8035473: [javadoc] Revamp the existing Doclet APIs
ksrini
parents: 30846
diff changeset
   128
            }
374342e56a56 8035473: [javadoc] Revamp the existing Doclet APIs
ksrini
parents: 30846
diff changeset
   129
            if (sw.toString().contains("<MISSING KEY>")) {
374342e56a56 8035473: [javadoc] Revamp the existing Doclet APIs
ksrini
parents: 30846
diff changeset
   130
                System.out.println(s);
374342e56a56 8035473: [javadoc] Revamp the existing Doclet APIs
ksrini
parents: 30846
diff changeset
   131
                error("missing resources in output ?");
374342e56a56 8035473: [javadoc] Revamp the existing Doclet APIs
ksrini
parents: 30846
diff changeset
   132
            }
374342e56a56 8035473: [javadoc] Revamp the existing Doclet APIs
ksrini
parents: 30846
diff changeset
   133
        } else {
374342e56a56 8035473: [javadoc] Revamp the existing Doclet APIs
ksrini
parents: 30846
diff changeset
   134
            error("failed to execute javadoc");
374342e56a56 8035473: [javadoc] Revamp the existing Doclet APIs
ksrini
parents: 30846
diff changeset
   135
        }
374342e56a56 8035473: [javadoc] Revamp the existing Doclet APIs
ksrini
parents: 30846
diff changeset
   136
    }
374342e56a56 8035473: [javadoc] Revamp the existing Doclet APIs
ksrini
parents: 30846
diff changeset
   137
14447
6f87132c2e54 8000612: Discrepancy between resources provided in javadoc resource files and resources required by code
jjg
parents:
diff changeset
   138
    /**
6f87132c2e54 8000612: Discrepancy between resources provided in javadoc resource files and resources required by code
jjg
parents:
diff changeset
   139
     * Find keys in resource bundles which are probably no longer required.
6f87132c2e54 8000612: Discrepancy between resources provided in javadoc resource files and resources required by code
jjg
parents:
diff changeset
   140
     * A key is required if there is a string in the code that is a resource key,
6f87132c2e54 8000612: Discrepancy between resources provided in javadoc resource files and resources required by code
jjg
parents:
diff changeset
   141
     * or if the key is well-known according to various pragmatic rules.
6f87132c2e54 8000612: Discrepancy between resources provided in javadoc resource files and resources required by code
jjg
parents:
diff changeset
   142
     */
6f87132c2e54 8000612: Discrepancy between resources provided in javadoc resource files and resources required by code
jjg
parents:
diff changeset
   143
    void findDeadKeys(Set<String> codeKeys, Set<String> resourceKeys) {
6f87132c2e54 8000612: Discrepancy between resources provided in javadoc resource files and resources required by code
jjg
parents:
diff changeset
   144
        for (String rk: resourceKeys) {
35426
374342e56a56 8035473: [javadoc] Revamp the existing Doclet APIs
ksrini
parents: 30846
diff changeset
   145
            // ignore these synthesized keys, tested by usageTests
374342e56a56 8035473: [javadoc] Revamp the existing Doclet APIs
ksrini
parents: 30846
diff changeset
   146
            if (rk.startsWith("doclet.usage.") || rk.startsWith("doclet.xusage"))
374342e56a56 8035473: [javadoc] Revamp the existing Doclet APIs
ksrini
parents: 30846
diff changeset
   147
                continue;
14447
6f87132c2e54 8000612: Discrepancy between resources provided in javadoc resource files and resources required by code
jjg
parents:
diff changeset
   148
            if (codeKeys.contains(rk))
6f87132c2e54 8000612: Discrepancy between resources provided in javadoc resource files and resources required by code
jjg
parents:
diff changeset
   149
                continue;
6f87132c2e54 8000612: Discrepancy between resources provided in javadoc resource files and resources required by code
jjg
parents:
diff changeset
   150
40308
274367a99f98 8136930: Simplify use of module-system options by custom launchers
jjg
parents: 36526
diff changeset
   151
            error("Resource key not found in code: '" + rk + '"');
14447
6f87132c2e54 8000612: Discrepancy between resources provided in javadoc resource files and resources required by code
jjg
parents:
diff changeset
   152
        }
6f87132c2e54 8000612: Discrepancy between resources provided in javadoc resource files and resources required by code
jjg
parents:
diff changeset
   153
    }
6f87132c2e54 8000612: Discrepancy between resources provided in javadoc resource files and resources required by code
jjg
parents:
diff changeset
   154
6f87132c2e54 8000612: Discrepancy between resources provided in javadoc resource files and resources required by code
jjg
parents:
diff changeset
   155
    /**
6f87132c2e54 8000612: Discrepancy between resources provided in javadoc resource files and resources required by code
jjg
parents:
diff changeset
   156
     * For all strings in the code that look like they might be
6f87132c2e54 8000612: Discrepancy between resources provided in javadoc resource files and resources required by code
jjg
parents:
diff changeset
   157
     * a resource key, verify that a key exists.
6f87132c2e54 8000612: Discrepancy between resources provided in javadoc resource files and resources required by code
jjg
parents:
diff changeset
   158
     */
6f87132c2e54 8000612: Discrepancy between resources provided in javadoc resource files and resources required by code
jjg
parents:
diff changeset
   159
    void findMissingKeys(Set<String> codeKeys, Set<String> resourceKeys) {
6f87132c2e54 8000612: Discrepancy between resources provided in javadoc resource files and resources required by code
jjg
parents:
diff changeset
   160
        for (String ck: codeKeys) {
35426
374342e56a56 8035473: [javadoc] Revamp the existing Doclet APIs
ksrini
parents: 30846
diff changeset
   161
            // ignore these synthesized keys, tested by usageTests
374342e56a56 8035473: [javadoc] Revamp the existing Doclet APIs
ksrini
parents: 30846
diff changeset
   162
            if (ck.startsWith("doclet.usage.") || ck.startsWith("doclet.xusage."))
374342e56a56 8035473: [javadoc] Revamp the existing Doclet APIs
ksrini
parents: 30846
diff changeset
   163
                continue;
14447
6f87132c2e54 8000612: Discrepancy between resources provided in javadoc resource files and resources required by code
jjg
parents:
diff changeset
   164
            if (resourceKeys.contains(ck))
6f87132c2e54 8000612: Discrepancy between resources provided in javadoc resource files and resources required by code
jjg
parents:
diff changeset
   165
                continue;
6f87132c2e54 8000612: Discrepancy between resources provided in javadoc resource files and resources required by code
jjg
parents:
diff changeset
   166
            error("No resource for \"" + ck + "\"");
6f87132c2e54 8000612: Discrepancy between resources provided in javadoc resource files and resources required by code
jjg
parents:
diff changeset
   167
        }
6f87132c2e54 8000612: Discrepancy between resources provided in javadoc resource files and resources required by code
jjg
parents:
diff changeset
   168
    }
6f87132c2e54 8000612: Discrepancy between resources provided in javadoc resource files and resources required by code
jjg
parents:
diff changeset
   169
6f87132c2e54 8000612: Discrepancy between resources provided in javadoc resource files and resources required by code
jjg
parents:
diff changeset
   170
    /**
6f87132c2e54 8000612: Discrepancy between resources provided in javadoc resource files and resources required by code
jjg
parents:
diff changeset
   171
     * Get the set of strings from (most of) the javadoc classfiles.
6f87132c2e54 8000612: Discrepancy between resources provided in javadoc resource files and resources required by code
jjg
parents:
diff changeset
   172
     */
6f87132c2e54 8000612: Discrepancy between resources provided in javadoc resource files and resources required by code
jjg
parents:
diff changeset
   173
    Set<String> getCodeKeys() throws IOException {
6f87132c2e54 8000612: Discrepancy between resources provided in javadoc resource files and resources required by code
jjg
parents:
diff changeset
   174
        Set<String> results = new TreeSet<String>();
6f87132c2e54 8000612: Discrepancy between resources provided in javadoc resource files and resources required by code
jjg
parents:
diff changeset
   175
        JavaCompiler c = ToolProvider.getSystemJavaCompiler();
27319
030080f03e4f 8062348: langtools tests should close file manager (group 1)
jjg
parents: 14545
diff changeset
   176
        try (JavaFileManager fm = c.getStandardFileManager(null, null, null)) {
030080f03e4f 8062348: langtools tests should close file manager (group 1)
jjg
parents: 14545
diff changeset
   177
            JavaFileManager.Location javadocLoc = findJavadocLocation(fm);
030080f03e4f 8062348: langtools tests should close file manager (group 1)
jjg
parents: 14545
diff changeset
   178
            String[] pkgs = {
35426
374342e56a56 8035473: [javadoc] Revamp the existing Doclet APIs
ksrini
parents: 30846
diff changeset
   179
                "jdk.javadoc.internal.doclets",
374342e56a56 8035473: [javadoc] Revamp the existing Doclet APIs
ksrini
parents: 30846
diff changeset
   180
                "jdk.javadoc.internal.tool"
27319
030080f03e4f 8062348: langtools tests should close file manager (group 1)
jjg
parents: 14545
diff changeset
   181
            };
030080f03e4f 8062348: langtools tests should close file manager (group 1)
jjg
parents: 14545
diff changeset
   182
            for (String pkg: pkgs) {
030080f03e4f 8062348: langtools tests should close file manager (group 1)
jjg
parents: 14545
diff changeset
   183
                for (JavaFileObject fo: fm.list(javadocLoc,
030080f03e4f 8062348: langtools tests should close file manager (group 1)
jjg
parents: 14545
diff changeset
   184
                        pkg, EnumSet.of(JavaFileObject.Kind.CLASS), true)) {
030080f03e4f 8062348: langtools tests should close file manager (group 1)
jjg
parents: 14545
diff changeset
   185
                    String name = fo.getName();
030080f03e4f 8062348: langtools tests should close file manager (group 1)
jjg
parents: 14545
diff changeset
   186
                    // ignore resource files
030080f03e4f 8062348: langtools tests should close file manager (group 1)
jjg
parents: 14545
diff changeset
   187
                    if (name.matches(".*resources.[A-Za-z_0-9]+\\.class.*"))
030080f03e4f 8062348: langtools tests should close file manager (group 1)
jjg
parents: 14545
diff changeset
   188
                        continue;
030080f03e4f 8062348: langtools tests should close file manager (group 1)
jjg
parents: 14545
diff changeset
   189
                    scan(fo, results);
030080f03e4f 8062348: langtools tests should close file manager (group 1)
jjg
parents: 14545
diff changeset
   190
                }
14447
6f87132c2e54 8000612: Discrepancy between resources provided in javadoc resource files and resources required by code
jjg
parents:
diff changeset
   191
            }
6f87132c2e54 8000612: Discrepancy between resources provided in javadoc resource files and resources required by code
jjg
parents:
diff changeset
   192
27319
030080f03e4f 8062348: langtools tests should close file manager (group 1)
jjg
parents: 14545
diff changeset
   193
            // special handling for code strings synthesized in
030080f03e4f 8062348: langtools tests should close file manager (group 1)
jjg
parents: 14545
diff changeset
   194
            // com.sun.tools.doclets.internal.toolkit.util.Util.getTypeName
030080f03e4f 8062348: langtools tests should close file manager (group 1)
jjg
parents: 14545
diff changeset
   195
            String[] extras = {
030080f03e4f 8062348: langtools tests should close file manager (group 1)
jjg
parents: 14545
diff changeset
   196
                "AnnotationType", "Class", "Enum", "Error", "Exception", "Interface"
030080f03e4f 8062348: langtools tests should close file manager (group 1)
jjg
parents: 14545
diff changeset
   197
            };
030080f03e4f 8062348: langtools tests should close file manager (group 1)
jjg
parents: 14545
diff changeset
   198
            for (String s: extras) {
030080f03e4f 8062348: langtools tests should close file manager (group 1)
jjg
parents: 14545
diff changeset
   199
                if (results.contains("doclet." + s))
030080f03e4f 8062348: langtools tests should close file manager (group 1)
jjg
parents: 14545
diff changeset
   200
                    results.add("doclet." + s.toLowerCase());
030080f03e4f 8062348: langtools tests should close file manager (group 1)
jjg
parents: 14545
diff changeset
   201
            }
030080f03e4f 8062348: langtools tests should close file manager (group 1)
jjg
parents: 14545
diff changeset
   202
030080f03e4f 8062348: langtools tests should close file manager (group 1)
jjg
parents: 14545
diff changeset
   203
            // special handling for code strings synthesized in
030080f03e4f 8062348: langtools tests should close file manager (group 1)
jjg
parents: 14545
diff changeset
   204
            // com.sun.tools.javadoc.Messager
030080f03e4f 8062348: langtools tests should close file manager (group 1)
jjg
parents: 14545
diff changeset
   205
            results.add("javadoc.error.msg");
030080f03e4f 8062348: langtools tests should close file manager (group 1)
jjg
parents: 14545
diff changeset
   206
            results.add("javadoc.note.msg");
030080f03e4f 8062348: langtools tests should close file manager (group 1)
jjg
parents: 14545
diff changeset
   207
            results.add("javadoc.note.pos.msg");
030080f03e4f 8062348: langtools tests should close file manager (group 1)
jjg
parents: 14545
diff changeset
   208
            results.add("javadoc.warning.msg");
030080f03e4f 8062348: langtools tests should close file manager (group 1)
jjg
parents: 14545
diff changeset
   209
030080f03e4f 8062348: langtools tests should close file manager (group 1)
jjg
parents: 14545
diff changeset
   210
            return results;
14447
6f87132c2e54 8000612: Discrepancy between resources provided in javadoc resource files and resources required by code
jjg
parents:
diff changeset
   211
        }
6f87132c2e54 8000612: Discrepancy between resources provided in javadoc resource files and resources required by code
jjg
parents:
diff changeset
   212
    }
6f87132c2e54 8000612: Discrepancy between resources provided in javadoc resource files and resources required by code
jjg
parents:
diff changeset
   213
6f87132c2e54 8000612: Discrepancy between resources provided in javadoc resource files and resources required by code
jjg
parents:
diff changeset
   214
    // depending on how the test is run, javadoc may be on bootclasspath or classpath
6f87132c2e54 8000612: Discrepancy between resources provided in javadoc resource files and resources required by code
jjg
parents:
diff changeset
   215
    JavaFileManager.Location findJavadocLocation(JavaFileManager fm) {
6f87132c2e54 8000612: Discrepancy between resources provided in javadoc resource files and resources required by code
jjg
parents:
diff changeset
   216
        JavaFileManager.Location[] locns =
6f87132c2e54 8000612: Discrepancy between resources provided in javadoc resource files and resources required by code
jjg
parents:
diff changeset
   217
            { StandardLocation.PLATFORM_CLASS_PATH, StandardLocation.CLASS_PATH };
6f87132c2e54 8000612: Discrepancy between resources provided in javadoc resource files and resources required by code
jjg
parents:
diff changeset
   218
        try {
6f87132c2e54 8000612: Discrepancy between resources provided in javadoc resource files and resources required by code
jjg
parents:
diff changeset
   219
            for (JavaFileManager.Location l: locns) {
6f87132c2e54 8000612: Discrepancy between resources provided in javadoc resource files and resources required by code
jjg
parents:
diff changeset
   220
                JavaFileObject fo = fm.getJavaFileForInput(l,
35426
374342e56a56 8035473: [javadoc] Revamp the existing Doclet APIs
ksrini
parents: 30846
diff changeset
   221
                    "jdk.javadoc.internal.tool.Main", JavaFileObject.Kind.CLASS);
14447
6f87132c2e54 8000612: Discrepancy between resources provided in javadoc resource files and resources required by code
jjg
parents:
diff changeset
   222
                if (fo != null) {
6f87132c2e54 8000612: Discrepancy between resources provided in javadoc resource files and resources required by code
jjg
parents:
diff changeset
   223
                    System.err.println("found javadoc in " + l);
6f87132c2e54 8000612: Discrepancy between resources provided in javadoc resource files and resources required by code
jjg
parents:
diff changeset
   224
                    return l;
6f87132c2e54 8000612: Discrepancy between resources provided in javadoc resource files and resources required by code
jjg
parents:
diff changeset
   225
                }
6f87132c2e54 8000612: Discrepancy between resources provided in javadoc resource files and resources required by code
jjg
parents:
diff changeset
   226
            }
6f87132c2e54 8000612: Discrepancy between resources provided in javadoc resource files and resources required by code
jjg
parents:
diff changeset
   227
        } catch (IOException e) {
6f87132c2e54 8000612: Discrepancy between resources provided in javadoc resource files and resources required by code
jjg
parents:
diff changeset
   228
            throw new Error(e);
6f87132c2e54 8000612: Discrepancy between resources provided in javadoc resource files and resources required by code
jjg
parents:
diff changeset
   229
        }
6f87132c2e54 8000612: Discrepancy between resources provided in javadoc resource files and resources required by code
jjg
parents:
diff changeset
   230
        throw new IllegalStateException("Cannot find javadoc");
6f87132c2e54 8000612: Discrepancy between resources provided in javadoc resource files and resources required by code
jjg
parents:
diff changeset
   231
    }
6f87132c2e54 8000612: Discrepancy between resources provided in javadoc resource files and resources required by code
jjg
parents:
diff changeset
   232
6f87132c2e54 8000612: Discrepancy between resources provided in javadoc resource files and resources required by code
jjg
parents:
diff changeset
   233
    /**
6f87132c2e54 8000612: Discrepancy between resources provided in javadoc resource files and resources required by code
jjg
parents:
diff changeset
   234
     * Get the set of strings from a class file.
6f87132c2e54 8000612: Discrepancy between resources provided in javadoc resource files and resources required by code
jjg
parents:
diff changeset
   235
     * Only strings that look like they might be a resource key are returned.
6f87132c2e54 8000612: Discrepancy between resources provided in javadoc resource files and resources required by code
jjg
parents:
diff changeset
   236
     */
6f87132c2e54 8000612: Discrepancy between resources provided in javadoc resource files and resources required by code
jjg
parents:
diff changeset
   237
    void scan(JavaFileObject fo, Set<String> results) throws IOException {
6f87132c2e54 8000612: Discrepancy between resources provided in javadoc resource files and resources required by code
jjg
parents:
diff changeset
   238
        //System.err.println("scan " + fo.getName());
6f87132c2e54 8000612: Discrepancy between resources provided in javadoc resource files and resources required by code
jjg
parents:
diff changeset
   239
        InputStream in = fo.openInputStream();
6f87132c2e54 8000612: Discrepancy between resources provided in javadoc resource files and resources required by code
jjg
parents:
diff changeset
   240
        try {
6f87132c2e54 8000612: Discrepancy between resources provided in javadoc resource files and resources required by code
jjg
parents:
diff changeset
   241
            ClassFile cf = ClassFile.read(in);
6f87132c2e54 8000612: Discrepancy between resources provided in javadoc resource files and resources required by code
jjg
parents:
diff changeset
   242
            for (ConstantPool.CPInfo cpinfo: cf.constant_pool.entries()) {
6f87132c2e54 8000612: Discrepancy between resources provided in javadoc resource files and resources required by code
jjg
parents:
diff changeset
   243
                if (cpinfo.getTag() == ConstantPool.CONSTANT_Utf8) {
6f87132c2e54 8000612: Discrepancy between resources provided in javadoc resource files and resources required by code
jjg
parents:
diff changeset
   244
                    String v = ((ConstantPool.CONSTANT_Utf8_info) cpinfo).value;
6f87132c2e54 8000612: Discrepancy between resources provided in javadoc resource files and resources required by code
jjg
parents:
diff changeset
   245
                    if (v.matches("(doclet|main|javadoc|tag)\\.[A-Za-z0-9-_.]+"))
6f87132c2e54 8000612: Discrepancy between resources provided in javadoc resource files and resources required by code
jjg
parents:
diff changeset
   246
                        results.add(v);
6f87132c2e54 8000612: Discrepancy between resources provided in javadoc resource files and resources required by code
jjg
parents:
diff changeset
   247
                }
6f87132c2e54 8000612: Discrepancy between resources provided in javadoc resource files and resources required by code
jjg
parents:
diff changeset
   248
            }
6f87132c2e54 8000612: Discrepancy between resources provided in javadoc resource files and resources required by code
jjg
parents:
diff changeset
   249
        } catch (ConstantPoolException ignore) {
6f87132c2e54 8000612: Discrepancy between resources provided in javadoc resource files and resources required by code
jjg
parents:
diff changeset
   250
        } finally {
6f87132c2e54 8000612: Discrepancy between resources provided in javadoc resource files and resources required by code
jjg
parents:
diff changeset
   251
            in.close();
6f87132c2e54 8000612: Discrepancy between resources provided in javadoc resource files and resources required by code
jjg
parents:
diff changeset
   252
        }
6f87132c2e54 8000612: Discrepancy between resources provided in javadoc resource files and resources required by code
jjg
parents:
diff changeset
   253
    }
6f87132c2e54 8000612: Discrepancy between resources provided in javadoc resource files and resources required by code
jjg
parents:
diff changeset
   254
6f87132c2e54 8000612: Discrepancy between resources provided in javadoc resource files and resources required by code
jjg
parents:
diff changeset
   255
    /**
6f87132c2e54 8000612: Discrepancy between resources provided in javadoc resource files and resources required by code
jjg
parents:
diff changeset
   256
     * Get the set of keys from the javadoc resource bundles.
6f87132c2e54 8000612: Discrepancy between resources provided in javadoc resource files and resources required by code
jjg
parents:
diff changeset
   257
     */
6f87132c2e54 8000612: Discrepancy between resources provided in javadoc resource files and resources required by code
jjg
parents:
diff changeset
   258
    Set<String> getResourceKeys() {
36526
3b41f1c69604 8142968: Module System implementation
alanb
parents: 35426
diff changeset
   259
        Module jdk_javadoc = Layer.boot().findModule("jdk.javadoc").get();
14447
6f87132c2e54 8000612: Discrepancy between resources provided in javadoc resource files and resources required by code
jjg
parents:
diff changeset
   260
        String[] names = {
35426
374342e56a56 8035473: [javadoc] Revamp the existing Doclet APIs
ksrini
parents: 30846
diff changeset
   261
                "jdk.javadoc.internal.doclets.formats.html.resources.standard",
374342e56a56 8035473: [javadoc] Revamp the existing Doclet APIs
ksrini
parents: 30846
diff changeset
   262
                "jdk.javadoc.internal.doclets.toolkit.resources.doclets",
374342e56a56 8035473: [javadoc] Revamp the existing Doclet APIs
ksrini
parents: 30846
diff changeset
   263
                "jdk.javadoc.internal.tool.resources.javadoc",
14447
6f87132c2e54 8000612: Discrepancy between resources provided in javadoc resource files and resources required by code
jjg
parents:
diff changeset
   264
        };
6f87132c2e54 8000612: Discrepancy between resources provided in javadoc resource files and resources required by code
jjg
parents:
diff changeset
   265
        Set<String> results = new TreeSet<String>();
6f87132c2e54 8000612: Discrepancy between resources provided in javadoc resource files and resources required by code
jjg
parents:
diff changeset
   266
        for (String name : names) {
36526
3b41f1c69604 8142968: Module System implementation
alanb
parents: 35426
diff changeset
   267
            ResourceBundle b = ResourceBundle.getBundle(name, jdk_javadoc);
14447
6f87132c2e54 8000612: Discrepancy between resources provided in javadoc resource files and resources required by code
jjg
parents:
diff changeset
   268
            results.addAll(b.keySet());
6f87132c2e54 8000612: Discrepancy between resources provided in javadoc resource files and resources required by code
jjg
parents:
diff changeset
   269
        }
6f87132c2e54 8000612: Discrepancy between resources provided in javadoc resource files and resources required by code
jjg
parents:
diff changeset
   270
        return results;
6f87132c2e54 8000612: Discrepancy between resources provided in javadoc resource files and resources required by code
jjg
parents:
diff changeset
   271
    }
6f87132c2e54 8000612: Discrepancy between resources provided in javadoc resource files and resources required by code
jjg
parents:
diff changeset
   272
6f87132c2e54 8000612: Discrepancy between resources provided in javadoc resource files and resources required by code
jjg
parents:
diff changeset
   273
    /**
6f87132c2e54 8000612: Discrepancy between resources provided in javadoc resource files and resources required by code
jjg
parents:
diff changeset
   274
     * Report an error.
6f87132c2e54 8000612: Discrepancy between resources provided in javadoc resource files and resources required by code
jjg
parents:
diff changeset
   275
     */
6f87132c2e54 8000612: Discrepancy between resources provided in javadoc resource files and resources required by code
jjg
parents:
diff changeset
   276
    void error(String msg) {
6f87132c2e54 8000612: Discrepancy between resources provided in javadoc resource files and resources required by code
jjg
parents:
diff changeset
   277
        System.err.println("Error: " + msg);
6f87132c2e54 8000612: Discrepancy between resources provided in javadoc resource files and resources required by code
jjg
parents:
diff changeset
   278
        errors++;
6f87132c2e54 8000612: Discrepancy between resources provided in javadoc resource files and resources required by code
jjg
parents:
diff changeset
   279
    }
6f87132c2e54 8000612: Discrepancy between resources provided in javadoc resource files and resources required by code
jjg
parents:
diff changeset
   280
6f87132c2e54 8000612: Discrepancy between resources provided in javadoc resource files and resources required by code
jjg
parents:
diff changeset
   281
    int errors;
6f87132c2e54 8000612: Discrepancy between resources provided in javadoc resource files and resources required by code
jjg
parents:
diff changeset
   282
}