langtools/test/tools/all/RunCodingRules.java
author jjg
Wed, 29 Oct 2014 17:25:23 -0700
changeset 27319 030080f03e4f
parent 26391 c775a09a4faf
child 30730 d3ce7619db2c
permissions -rw-r--r--
8062348: langtools tests should close file manager (group 1) Reviewed-by: darcy
Ignore whitespace changes - Everywhere: Within whitespace: At end of lines:
25848
3bc09f4676a9 8043643: Add an crules analyzer avoiding string concatenation in messages of Assert checks.
jlahoda
parents:
diff changeset
     1
/*
3bc09f4676a9 8043643: Add an crules analyzer avoiding string concatenation in messages of Assert checks.
jlahoda
parents:
diff changeset
     2
 * Copyright (c) 2014, Oracle and/or its affiliates. All rights reserved.
3bc09f4676a9 8043643: Add an crules analyzer avoiding string concatenation in messages of Assert checks.
jlahoda
parents:
diff changeset
     3
 * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
3bc09f4676a9 8043643: Add an crules analyzer avoiding string concatenation in messages of Assert checks.
jlahoda
parents:
diff changeset
     4
 *
3bc09f4676a9 8043643: Add an crules analyzer avoiding string concatenation in messages of Assert checks.
jlahoda
parents:
diff changeset
     5
 * This code is free software; you can redistribute it and/or modify it
3bc09f4676a9 8043643: Add an crules analyzer avoiding string concatenation in messages of Assert checks.
jlahoda
parents:
diff changeset
     6
 * under the terms of the GNU General Public License version 2 only, as
3bc09f4676a9 8043643: Add an crules analyzer avoiding string concatenation in messages of Assert checks.
jlahoda
parents:
diff changeset
     7
 * published by the Free Software Foundation.
3bc09f4676a9 8043643: Add an crules analyzer avoiding string concatenation in messages of Assert checks.
jlahoda
parents:
diff changeset
     8
 *
3bc09f4676a9 8043643: Add an crules analyzer avoiding string concatenation in messages of Assert checks.
jlahoda
parents:
diff changeset
     9
 * This code is distributed in the hope that it will be useful, but WITHOUT
3bc09f4676a9 8043643: Add an crules analyzer avoiding string concatenation in messages of Assert checks.
jlahoda
parents:
diff changeset
    10
 * ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or
3bc09f4676a9 8043643: Add an crules analyzer avoiding string concatenation in messages of Assert checks.
jlahoda
parents:
diff changeset
    11
 * FITNESS FOR A PARTICULAR PURPOSE.  See the GNU General Public License
3bc09f4676a9 8043643: Add an crules analyzer avoiding string concatenation in messages of Assert checks.
jlahoda
parents:
diff changeset
    12
 * version 2 for more details (a copy is included in the LICENSE file that
3bc09f4676a9 8043643: Add an crules analyzer avoiding string concatenation in messages of Assert checks.
jlahoda
parents:
diff changeset
    13
 * accompanied this code).
3bc09f4676a9 8043643: Add an crules analyzer avoiding string concatenation in messages of Assert checks.
jlahoda
parents:
diff changeset
    14
 *
3bc09f4676a9 8043643: Add an crules analyzer avoiding string concatenation in messages of Assert checks.
jlahoda
parents:
diff changeset
    15
 * You should have received a copy of the GNU General Public License version
3bc09f4676a9 8043643: Add an crules analyzer avoiding string concatenation in messages of Assert checks.
jlahoda
parents:
diff changeset
    16
 * 2 along with this work; if not, write to the Free Software Foundation,
3bc09f4676a9 8043643: Add an crules analyzer avoiding string concatenation in messages of Assert checks.
jlahoda
parents:
diff changeset
    17
 * Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA.
3bc09f4676a9 8043643: Add an crules analyzer avoiding string concatenation in messages of Assert checks.
jlahoda
parents:
diff changeset
    18
 *
3bc09f4676a9 8043643: Add an crules analyzer avoiding string concatenation in messages of Assert checks.
jlahoda
parents:
diff changeset
    19
 * Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA
3bc09f4676a9 8043643: Add an crules analyzer avoiding string concatenation in messages of Assert checks.
jlahoda
parents:
diff changeset
    20
 * or visit www.oracle.com if you need additional information or have any
3bc09f4676a9 8043643: Add an crules analyzer avoiding string concatenation in messages of Assert checks.
jlahoda
parents:
diff changeset
    21
 * questions.
3bc09f4676a9 8043643: Add an crules analyzer avoiding string concatenation in messages of Assert checks.
jlahoda
parents:
diff changeset
    22
 */
3bc09f4676a9 8043643: Add an crules analyzer avoiding string concatenation in messages of Assert checks.
jlahoda
parents:
diff changeset
    23
3bc09f4676a9 8043643: Add an crules analyzer avoiding string concatenation in messages of Assert checks.
jlahoda
parents:
diff changeset
    24
/*
3bc09f4676a9 8043643: Add an crules analyzer avoiding string concatenation in messages of Assert checks.
jlahoda
parents:
diff changeset
    25
 * @test
3bc09f4676a9 8043643: Add an crules analyzer avoiding string concatenation in messages of Assert checks.
jlahoda
parents:
diff changeset
    26
 * @bug 8043643
3bc09f4676a9 8043643: Add an crules analyzer avoiding string concatenation in messages of Assert checks.
jlahoda
parents:
diff changeset
    27
 * @summary Run the langtools coding rules over the langtools source code.
3bc09f4676a9 8043643: Add an crules analyzer avoiding string concatenation in messages of Assert checks.
jlahoda
parents:
diff changeset
    28
 */
3bc09f4676a9 8043643: Add an crules analyzer avoiding string concatenation in messages of Assert checks.
jlahoda
parents:
diff changeset
    29
3bc09f4676a9 8043643: Add an crules analyzer avoiding string concatenation in messages of Assert checks.
jlahoda
parents:
diff changeset
    30
3bc09f4676a9 8043643: Add an crules analyzer avoiding string concatenation in messages of Assert checks.
jlahoda
parents:
diff changeset
    31
import java.io.*;
3bc09f4676a9 8043643: Add an crules analyzer avoiding string concatenation in messages of Assert checks.
jlahoda
parents:
diff changeset
    32
import java.nio.file.Files;
26391
c775a09a4faf 8054563: Update RunCodingRules.java for source code reorg
jlahoda
parents: 25848
diff changeset
    33
import java.nio.file.Path;
c775a09a4faf 8054563: Update RunCodingRules.java for source code reorg
jlahoda
parents: 25848
diff changeset
    34
import java.nio.file.Paths;
c775a09a4faf 8054563: Update RunCodingRules.java for source code reorg
jlahoda
parents: 25848
diff changeset
    35
import java.nio.file.StandardOpenOption;
25848
3bc09f4676a9 8043643: Add an crules analyzer avoiding string concatenation in messages of Assert checks.
jlahoda
parents:
diff changeset
    36
import java.util.*;
3bc09f4676a9 8043643: Add an crules analyzer avoiding string concatenation in messages of Assert checks.
jlahoda
parents:
diff changeset
    37
import java.util.stream.Collectors;
26391
c775a09a4faf 8054563: Update RunCodingRules.java for source code reorg
jlahoda
parents: 25848
diff changeset
    38
import java.util.stream.Stream;
25848
3bc09f4676a9 8043643: Add an crules analyzer avoiding string concatenation in messages of Assert checks.
jlahoda
parents:
diff changeset
    39
3bc09f4676a9 8043643: Add an crules analyzer avoiding string concatenation in messages of Assert checks.
jlahoda
parents:
diff changeset
    40
import javax.tools.Diagnostic;
3bc09f4676a9 8043643: Add an crules analyzer avoiding string concatenation in messages of Assert checks.
jlahoda
parents:
diff changeset
    41
import javax.tools.DiagnosticListener;
3bc09f4676a9 8043643: Add an crules analyzer avoiding string concatenation in messages of Assert checks.
jlahoda
parents:
diff changeset
    42
import javax.tools.JavaCompiler;
3bc09f4676a9 8043643: Add an crules analyzer avoiding string concatenation in messages of Assert checks.
jlahoda
parents:
diff changeset
    43
import javax.tools.JavaFileObject;
3bc09f4676a9 8043643: Add an crules analyzer avoiding string concatenation in messages of Assert checks.
jlahoda
parents:
diff changeset
    44
import javax.tools.StandardJavaFileManager;
3bc09f4676a9 8043643: Add an crules analyzer avoiding string concatenation in messages of Assert checks.
jlahoda
parents:
diff changeset
    45
import javax.tools.ToolProvider;
3bc09f4676a9 8043643: Add an crules analyzer avoiding string concatenation in messages of Assert checks.
jlahoda
parents:
diff changeset
    46
3bc09f4676a9 8043643: Add an crules analyzer avoiding string concatenation in messages of Assert checks.
jlahoda
parents:
diff changeset
    47
import com.sun.tools.javac.util.Assert;
3bc09f4676a9 8043643: Add an crules analyzer avoiding string concatenation in messages of Assert checks.
jlahoda
parents:
diff changeset
    48
3bc09f4676a9 8043643: Add an crules analyzer avoiding string concatenation in messages of Assert checks.
jlahoda
parents:
diff changeset
    49
public class RunCodingRules {
3bc09f4676a9 8043643: Add an crules analyzer avoiding string concatenation in messages of Assert checks.
jlahoda
parents:
diff changeset
    50
    public static void main(String... args) throws Exception {
3bc09f4676a9 8043643: Add an crules analyzer avoiding string concatenation in messages of Assert checks.
jlahoda
parents:
diff changeset
    51
        new RunCodingRules().run();
3bc09f4676a9 8043643: Add an crules analyzer avoiding string concatenation in messages of Assert checks.
jlahoda
parents:
diff changeset
    52
    }
3bc09f4676a9 8043643: Add an crules analyzer avoiding string concatenation in messages of Assert checks.
jlahoda
parents:
diff changeset
    53
3bc09f4676a9 8043643: Add an crules analyzer avoiding string concatenation in messages of Assert checks.
jlahoda
parents:
diff changeset
    54
    public void run() throws Exception {
26391
c775a09a4faf 8054563: Update RunCodingRules.java for source code reorg
jlahoda
parents: 25848
diff changeset
    55
        Path testSrc = Paths.get(System.getProperty("test.src", "."));
c775a09a4faf 8054563: Update RunCodingRules.java for source code reorg
jlahoda
parents: 25848
diff changeset
    56
        Path targetDir = Paths.get(System.getProperty("test.classes", "."));
c775a09a4faf 8054563: Update RunCodingRules.java for source code reorg
jlahoda
parents: 25848
diff changeset
    57
        List<Path> sourceDirs = null;
c775a09a4faf 8054563: Update RunCodingRules.java for source code reorg
jlahoda
parents: 25848
diff changeset
    58
        Path crulesDir = null;
c775a09a4faf 8054563: Update RunCodingRules.java for source code reorg
jlahoda
parents: 25848
diff changeset
    59
        for (Path d = testSrc; d != null; d = d.getParent()) {
c775a09a4faf 8054563: Update RunCodingRules.java for source code reorg
jlahoda
parents: 25848
diff changeset
    60
            if (Files.exists(d.resolve("TEST.ROOT"))) {
c775a09a4faf 8054563: Update RunCodingRules.java for source code reorg
jlahoda
parents: 25848
diff changeset
    61
                d = d.getParent();
c775a09a4faf 8054563: Update RunCodingRules.java for source code reorg
jlahoda
parents: 25848
diff changeset
    62
                Path toolsPath = d.resolve("make/tools");
c775a09a4faf 8054563: Update RunCodingRules.java for source code reorg
jlahoda
parents: 25848
diff changeset
    63
                if (Files.exists(toolsPath)) {
c775a09a4faf 8054563: Update RunCodingRules.java for source code reorg
jlahoda
parents: 25848
diff changeset
    64
                    crulesDir = toolsPath;
c775a09a4faf 8054563: Update RunCodingRules.java for source code reorg
jlahoda
parents: 25848
diff changeset
    65
                    sourceDirs = Files.walk(d.resolve("src"), 1)
c775a09a4faf 8054563: Update RunCodingRules.java for source code reorg
jlahoda
parents: 25848
diff changeset
    66
                                      .map(p -> p.resolve("share/classes"))
c775a09a4faf 8054563: Update RunCodingRules.java for source code reorg
jlahoda
parents: 25848
diff changeset
    67
                                      .filter(p -> Files.isDirectory(p))
c775a09a4faf 8054563: Update RunCodingRules.java for source code reorg
jlahoda
parents: 25848
diff changeset
    68
                                      .collect(Collectors.toList());
25848
3bc09f4676a9 8043643: Add an crules analyzer avoiding string concatenation in messages of Assert checks.
jlahoda
parents:
diff changeset
    69
                    break;
3bc09f4676a9 8043643: Add an crules analyzer avoiding string concatenation in messages of Assert checks.
jlahoda
parents:
diff changeset
    70
                }
3bc09f4676a9 8043643: Add an crules analyzer avoiding string concatenation in messages of Assert checks.
jlahoda
parents:
diff changeset
    71
            }
3bc09f4676a9 8043643: Add an crules analyzer avoiding string concatenation in messages of Assert checks.
jlahoda
parents:
diff changeset
    72
        }
3bc09f4676a9 8043643: Add an crules analyzer avoiding string concatenation in messages of Assert checks.
jlahoda
parents:
diff changeset
    73
26391
c775a09a4faf 8054563: Update RunCodingRules.java for source code reorg
jlahoda
parents: 25848
diff changeset
    74
        if (sourceDirs == null || crulesDir == null) {
25848
3bc09f4676a9 8043643: Add an crules analyzer avoiding string concatenation in messages of Assert checks.
jlahoda
parents:
diff changeset
    75
            System.err.println("Warning: sources not found, test skipped.");
3bc09f4676a9 8043643: Add an crules analyzer avoiding string concatenation in messages of Assert checks.
jlahoda
parents:
diff changeset
    76
            return ;
3bc09f4676a9 8043643: Add an crules analyzer avoiding string concatenation in messages of Assert checks.
jlahoda
parents:
diff changeset
    77
        }
3bc09f4676a9 8043643: Add an crules analyzer avoiding string concatenation in messages of Assert checks.
jlahoda
parents:
diff changeset
    78
3bc09f4676a9 8043643: Add an crules analyzer avoiding string concatenation in messages of Assert checks.
jlahoda
parents:
diff changeset
    79
        JavaCompiler javaCompiler = ToolProvider.getSystemJavaCompiler();
27319
030080f03e4f 8062348: langtools tests should close file manager (group 1)
jjg
parents: 26391
diff changeset
    80
        try (StandardJavaFileManager fm = javaCompiler.getStandardFileManager(null, null, null)) {
030080f03e4f 8062348: langtools tests should close file manager (group 1)
jjg
parents: 26391
diff changeset
    81
            DiagnosticListener<JavaFileObject> noErrors = diagnostic -> {
030080f03e4f 8062348: langtools tests should close file manager (group 1)
jjg
parents: 26391
diff changeset
    82
                Assert.check(diagnostic.getKind() != Diagnostic.Kind.ERROR, diagnostic.toString());
030080f03e4f 8062348: langtools tests should close file manager (group 1)
jjg
parents: 26391
diff changeset
    83
            };
25848
3bc09f4676a9 8043643: Add an crules analyzer avoiding string concatenation in messages of Assert checks.
jlahoda
parents:
diff changeset
    84
27319
030080f03e4f 8062348: langtools tests should close file manager (group 1)
jjg
parents: 26391
diff changeset
    85
            List<File> crulesFiles = Files.walk(crulesDir)
030080f03e4f 8062348: langtools tests should close file manager (group 1)
jjg
parents: 26391
diff changeset
    86
                                          .filter(entry -> entry.getFileName().toString().endsWith(".java"))
030080f03e4f 8062348: langtools tests should close file manager (group 1)
jjg
parents: 26391
diff changeset
    87
                                          .filter(entry -> entry.getParent().endsWith("crules"))
030080f03e4f 8062348: langtools tests should close file manager (group 1)
jjg
parents: 26391
diff changeset
    88
                                          .map(entry -> entry.toFile())
030080f03e4f 8062348: langtools tests should close file manager (group 1)
jjg
parents: 26391
diff changeset
    89
                                          .collect(Collectors.toList());
25848
3bc09f4676a9 8043643: Add an crules analyzer avoiding string concatenation in messages of Assert checks.
jlahoda
parents:
diff changeset
    90
27319
030080f03e4f 8062348: langtools tests should close file manager (group 1)
jjg
parents: 26391
diff changeset
    91
            Path crulesTarget = targetDir.resolve("crules");
030080f03e4f 8062348: langtools tests should close file manager (group 1)
jjg
parents: 26391
diff changeset
    92
            Files.createDirectories(crulesTarget);
030080f03e4f 8062348: langtools tests should close file manager (group 1)
jjg
parents: 26391
diff changeset
    93
            List<String> crulesOptions = Arrays.asList("-d", crulesTarget.toString());
030080f03e4f 8062348: langtools tests should close file manager (group 1)
jjg
parents: 26391
diff changeset
    94
            javaCompiler.getTask(null, fm, noErrors, crulesOptions, null,
030080f03e4f 8062348: langtools tests should close file manager (group 1)
jjg
parents: 26391
diff changeset
    95
                    fm.getJavaFileObjectsFromFiles(crulesFiles)).call();
030080f03e4f 8062348: langtools tests should close file manager (group 1)
jjg
parents: 26391
diff changeset
    96
            Path registration = crulesTarget.resolve("META-INF/services/com.sun.source.util.Plugin");
030080f03e4f 8062348: langtools tests should close file manager (group 1)
jjg
parents: 26391
diff changeset
    97
            Files.createDirectories(registration.getParent());
030080f03e4f 8062348: langtools tests should close file manager (group 1)
jjg
parents: 26391
diff changeset
    98
            try (Writer metaInfServices = Files.newBufferedWriter(registration, StandardOpenOption.CREATE, StandardOpenOption.TRUNCATE_EXISTING)) {
030080f03e4f 8062348: langtools tests should close file manager (group 1)
jjg
parents: 26391
diff changeset
    99
                metaInfServices.write("crules.CodingRulesAnalyzerPlugin\n");
030080f03e4f 8062348: langtools tests should close file manager (group 1)
jjg
parents: 26391
diff changeset
   100
            }
25848
3bc09f4676a9 8043643: Add an crules analyzer avoiding string concatenation in messages of Assert checks.
jlahoda
parents:
diff changeset
   101
27319
030080f03e4f 8062348: langtools tests should close file manager (group 1)
jjg
parents: 26391
diff changeset
   102
            List<File> sources = sourceDirs.stream()
030080f03e4f 8062348: langtools tests should close file manager (group 1)
jjg
parents: 26391
diff changeset
   103
                                           .flatMap(dir -> silentFilesWalk(dir))
030080f03e4f 8062348: langtools tests should close file manager (group 1)
jjg
parents: 26391
diff changeset
   104
                                           .filter(entry -> entry.getFileName().toString().endsWith(".java"))
030080f03e4f 8062348: langtools tests should close file manager (group 1)
jjg
parents: 26391
diff changeset
   105
                                           .map(p -> p.toFile())
030080f03e4f 8062348: langtools tests should close file manager (group 1)
jjg
parents: 26391
diff changeset
   106
                                           .collect(Collectors.toList());
25848
3bc09f4676a9 8043643: Add an crules analyzer avoiding string concatenation in messages of Assert checks.
jlahoda
parents:
diff changeset
   107
27319
030080f03e4f 8062348: langtools tests should close file manager (group 1)
jjg
parents: 26391
diff changeset
   108
            Path sourceTarget = targetDir.resolve("classes");
030080f03e4f 8062348: langtools tests should close file manager (group 1)
jjg
parents: 26391
diff changeset
   109
            Files.createDirectories(sourceTarget);
030080f03e4f 8062348: langtools tests should close file manager (group 1)
jjg
parents: 26391
diff changeset
   110
            String processorPath = crulesTarget.toString() + File.pathSeparator + crulesDir.toString();
030080f03e4f 8062348: langtools tests should close file manager (group 1)
jjg
parents: 26391
diff changeset
   111
            List<String> options = Arrays.asList("-d", sourceTarget.toString(),
030080f03e4f 8062348: langtools tests should close file manager (group 1)
jjg
parents: 26391
diff changeset
   112
                    "-processorpath", processorPath, "-Xplugin:coding_rules");
030080f03e4f 8062348: langtools tests should close file manager (group 1)
jjg
parents: 26391
diff changeset
   113
            javaCompiler.getTask(null, fm, noErrors, options, null,
030080f03e4f 8062348: langtools tests should close file manager (group 1)
jjg
parents: 26391
diff changeset
   114
                    fm.getJavaFileObjectsFromFiles(sources)).call();
030080f03e4f 8062348: langtools tests should close file manager (group 1)
jjg
parents: 26391
diff changeset
   115
        }
25848
3bc09f4676a9 8043643: Add an crules analyzer avoiding string concatenation in messages of Assert checks.
jlahoda
parents:
diff changeset
   116
    }
26391
c775a09a4faf 8054563: Update RunCodingRules.java for source code reorg
jlahoda
parents: 25848
diff changeset
   117
c775a09a4faf 8054563: Update RunCodingRules.java for source code reorg
jlahoda
parents: 25848
diff changeset
   118
    Stream<Path> silentFilesWalk(Path dir) throws IllegalStateException {
c775a09a4faf 8054563: Update RunCodingRules.java for source code reorg
jlahoda
parents: 25848
diff changeset
   119
        try {
c775a09a4faf 8054563: Update RunCodingRules.java for source code reorg
jlahoda
parents: 25848
diff changeset
   120
            return Files.walk(dir);
c775a09a4faf 8054563: Update RunCodingRules.java for source code reorg
jlahoda
parents: 25848
diff changeset
   121
        } catch (IOException ex) {
c775a09a4faf 8054563: Update RunCodingRules.java for source code reorg
jlahoda
parents: 25848
diff changeset
   122
            throw new IllegalStateException(ex);
c775a09a4faf 8054563: Update RunCodingRules.java for source code reorg
jlahoda
parents: 25848
diff changeset
   123
        }
c775a09a4faf 8054563: Update RunCodingRules.java for source code reorg
jlahoda
parents: 25848
diff changeset
   124
    }
25848
3bc09f4676a9 8043643: Add an crules analyzer avoiding string concatenation in messages of Assert checks.
jlahoda
parents:
diff changeset
   125
}