langtools/test/tools/javac/main/StreamsTest.java
author jjg
Wed, 03 Aug 2016 16:01:09 -0700
changeset 40232 4995ab1a4558
permissions -rw-r--r--
8162359: javac should use stdout for --help and --version Reviewed-by: vromero
Ignore whitespace changes - Everywhere: Within whitespace: At end of lines:
40232
4995ab1a4558 8162359: javac should use stdout for --help and --version
jjg
parents:
diff changeset
     1
/*
4995ab1a4558 8162359: javac should use stdout for --help and --version
jjg
parents:
diff changeset
     2
 * Copyright (c) 2016, Oracle and/or its affiliates. All rights reserved.
4995ab1a4558 8162359: javac should use stdout for --help and --version
jjg
parents:
diff changeset
     3
 * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
4995ab1a4558 8162359: javac should use stdout for --help and --version
jjg
parents:
diff changeset
     4
 *
4995ab1a4558 8162359: javac should use stdout for --help and --version
jjg
parents:
diff changeset
     5
 * This code is free software; you can redistribute it and/or modify it
4995ab1a4558 8162359: javac should use stdout for --help and --version
jjg
parents:
diff changeset
     6
 * under the terms of the GNU General Public License version 2 only, as
4995ab1a4558 8162359: javac should use stdout for --help and --version
jjg
parents:
diff changeset
     7
 * published by the Free Software Foundation.
4995ab1a4558 8162359: javac should use stdout for --help and --version
jjg
parents:
diff changeset
     8
 *
4995ab1a4558 8162359: javac should use stdout for --help and --version
jjg
parents:
diff changeset
     9
 * This code is distributed in the hope that it will be useful, but WITHOUT
4995ab1a4558 8162359: javac should use stdout for --help and --version
jjg
parents:
diff changeset
    10
 * ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or
4995ab1a4558 8162359: javac should use stdout for --help and --version
jjg
parents:
diff changeset
    11
 * FITNESS FOR A PARTICULAR PURPOSE.  See the GNU General Public License
4995ab1a4558 8162359: javac should use stdout for --help and --version
jjg
parents:
diff changeset
    12
 * version 2 for more details (a copy is included in the LICENSE file that
4995ab1a4558 8162359: javac should use stdout for --help and --version
jjg
parents:
diff changeset
    13
 * accompanied this code).
4995ab1a4558 8162359: javac should use stdout for --help and --version
jjg
parents:
diff changeset
    14
 *
4995ab1a4558 8162359: javac should use stdout for --help and --version
jjg
parents:
diff changeset
    15
 * You should have received a copy of the GNU General Public License version
4995ab1a4558 8162359: javac should use stdout for --help and --version
jjg
parents:
diff changeset
    16
 * 2 along with this work; if not, write to the Free Software Foundation,
4995ab1a4558 8162359: javac should use stdout for --help and --version
jjg
parents:
diff changeset
    17
 * Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA.
4995ab1a4558 8162359: javac should use stdout for --help and --version
jjg
parents:
diff changeset
    18
 *
4995ab1a4558 8162359: javac should use stdout for --help and --version
jjg
parents:
diff changeset
    19
 * Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA
4995ab1a4558 8162359: javac should use stdout for --help and --version
jjg
parents:
diff changeset
    20
 * or visit www.oracle.com if you need additional information or have any
4995ab1a4558 8162359: javac should use stdout for --help and --version
jjg
parents:
diff changeset
    21
 * questions.
4995ab1a4558 8162359: javac should use stdout for --help and --version
jjg
parents:
diff changeset
    22
 */
4995ab1a4558 8162359: javac should use stdout for --help and --version
jjg
parents:
diff changeset
    23
4995ab1a4558 8162359: javac should use stdout for --help and --version
jjg
parents:
diff changeset
    24
/*
4995ab1a4558 8162359: javac should use stdout for --help and --version
jjg
parents:
diff changeset
    25
 * @test
4995ab1a4558 8162359: javac should use stdout for --help and --version
jjg
parents:
diff changeset
    26
 * @bug 8162359
4995ab1a4558 8162359: javac should use stdout for --help and --version
jjg
parents:
diff changeset
    27
 * @summary extra space in javac -help for -J and @ options
4995ab1a4558 8162359: javac should use stdout for --help and --version
jjg
parents:
diff changeset
    28
 * @modules jdk.compiler
4995ab1a4558 8162359: javac should use stdout for --help and --version
jjg
parents:
diff changeset
    29
 * @library /tools/lib
4995ab1a4558 8162359: javac should use stdout for --help and --version
jjg
parents:
diff changeset
    30
 * @build toolbox.TestRunner toolbox.ToolBox
4995ab1a4558 8162359: javac should use stdout for --help and --version
jjg
parents:
diff changeset
    31
 * @run main StreamsTest
4995ab1a4558 8162359: javac should use stdout for --help and --version
jjg
parents:
diff changeset
    32
 */
4995ab1a4558 8162359: javac should use stdout for --help and --version
jjg
parents:
diff changeset
    33
4995ab1a4558 8162359: javac should use stdout for --help and --version
jjg
parents:
diff changeset
    34
import java.io.ByteArrayOutputStream;
4995ab1a4558 8162359: javac should use stdout for --help and --version
jjg
parents:
diff changeset
    35
import java.io.IOException;
4995ab1a4558 8162359: javac should use stdout for --help and --version
jjg
parents:
diff changeset
    36
import java.io.PrintStream;
4995ab1a4558 8162359: javac should use stdout for --help and --version
jjg
parents:
diff changeset
    37
import java.nio.file.Path;
4995ab1a4558 8162359: javac should use stdout for --help and --version
jjg
parents:
diff changeset
    38
import java.nio.file.Paths;
4995ab1a4558 8162359: javac should use stdout for --help and --version
jjg
parents:
diff changeset
    39
import java.util.List;
4995ab1a4558 8162359: javac should use stdout for --help and --version
jjg
parents:
diff changeset
    40
4995ab1a4558 8162359: javac should use stdout for --help and --version
jjg
parents:
diff changeset
    41
import static java.util.Arrays.asList;
4995ab1a4558 8162359: javac should use stdout for --help and --version
jjg
parents:
diff changeset
    42
4995ab1a4558 8162359: javac should use stdout for --help and --version
jjg
parents:
diff changeset
    43
import toolbox.TestRunner;
4995ab1a4558 8162359: javac should use stdout for --help and --version
jjg
parents:
diff changeset
    44
import toolbox.ToolBox;
4995ab1a4558 8162359: javac should use stdout for --help and --version
jjg
parents:
diff changeset
    45
4995ab1a4558 8162359: javac should use stdout for --help and --version
jjg
parents:
diff changeset
    46
public class StreamsTest extends TestRunner {
4995ab1a4558 8162359: javac should use stdout for --help and --version
jjg
parents:
diff changeset
    47
    public static void main(String... args) throws Exception {
4995ab1a4558 8162359: javac should use stdout for --help and --version
jjg
parents:
diff changeset
    48
        new StreamsTest().runTests(m -> new Object[] { Paths.get(m.getName()) });
4995ab1a4558 8162359: javac should use stdout for --help and --version
jjg
parents:
diff changeset
    49
    }
4995ab1a4558 8162359: javac should use stdout for --help and --version
jjg
parents:
diff changeset
    50
4995ab1a4558 8162359: javac should use stdout for --help and --version
jjg
parents:
diff changeset
    51
    StreamsTest() {
4995ab1a4558 8162359: javac should use stdout for --help and --version
jjg
parents:
diff changeset
    52
        super(System.err);
4995ab1a4558 8162359: javac should use stdout for --help and --version
jjg
parents:
diff changeset
    53
    }
4995ab1a4558 8162359: javac should use stdout for --help and --version
jjg
parents:
diff changeset
    54
4995ab1a4558 8162359: javac should use stdout for --help and --version
jjg
parents:
diff changeset
    55
    ToolBox tb = new ToolBox();
4995ab1a4558 8162359: javac should use stdout for --help and --version
jjg
parents:
diff changeset
    56
    static final String LINESEP = System.getProperty("line.separator");
4995ab1a4558 8162359: javac should use stdout for --help and --version
jjg
parents:
diff changeset
    57
4995ab1a4558 8162359: javac should use stdout for --help and --version
jjg
parents:
diff changeset
    58
    @Test // errors should be written to stderr
4995ab1a4558 8162359: javac should use stdout for --help and --version
jjg
parents:
diff changeset
    59
    public void testError(Path base) throws Exception {
4995ab1a4558 8162359: javac should use stdout for --help and --version
jjg
parents:
diff changeset
    60
        Path src = base.resolve("src");
4995ab1a4558 8162359: javac should use stdout for --help and --version
jjg
parents:
diff changeset
    61
        Path classes = base.resolve("classes");
4995ab1a4558 8162359: javac should use stdout for --help and --version
jjg
parents:
diff changeset
    62
        tb.writeJavaFiles(src,
4995ab1a4558 8162359: javac should use stdout for --help and --version
jjg
parents:
diff changeset
    63
            "import java.util.*; class C { # }");
4995ab1a4558 8162359: javac should use stdout for --help and --version
jjg
parents:
diff changeset
    64
        test(asList("-d", classes.toString(), src.resolve("C.java").toString()),
4995ab1a4558 8162359: javac should use stdout for --help and --version
jjg
parents:
diff changeset
    65
                null, "illegal character: '#'");
4995ab1a4558 8162359: javac should use stdout for --help and --version
jjg
parents:
diff changeset
    66
    }
4995ab1a4558 8162359: javac should use stdout for --help and --version
jjg
parents:
diff changeset
    67
4995ab1a4558 8162359: javac should use stdout for --help and --version
jjg
parents:
diff changeset
    68
    @Test // warnings should be written to stderr
4995ab1a4558 8162359: javac should use stdout for --help and --version
jjg
parents:
diff changeset
    69
    public void testWarning(Path base) throws Exception {
4995ab1a4558 8162359: javac should use stdout for --help and --version
jjg
parents:
diff changeset
    70
        Path src = base.resolve("src");
4995ab1a4558 8162359: javac should use stdout for --help and --version
jjg
parents:
diff changeset
    71
        Path classes = base.resolve("classes");
4995ab1a4558 8162359: javac should use stdout for --help and --version
jjg
parents:
diff changeset
    72
        tb.writeJavaFiles(src,
4995ab1a4558 8162359: javac should use stdout for --help and --version
jjg
parents:
diff changeset
    73
            "import java.util.*; class C { List list = new ArrayList(); }");
4995ab1a4558 8162359: javac should use stdout for --help and --version
jjg
parents:
diff changeset
    74
        test(asList("-d", classes.toString(), "-Xlint", src.resolve("C.java").toString()),
4995ab1a4558 8162359: javac should use stdout for --help and --version
jjg
parents:
diff changeset
    75
                null, "warning: [rawtypes]");
4995ab1a4558 8162359: javac should use stdout for --help and --version
jjg
parents:
diff changeset
    76
    }
4995ab1a4558 8162359: javac should use stdout for --help and --version
jjg
parents:
diff changeset
    77
4995ab1a4558 8162359: javac should use stdout for --help and --version
jjg
parents:
diff changeset
    78
    @Test // notes should be written to stderr
4995ab1a4558 8162359: javac should use stdout for --help and --version
jjg
parents:
diff changeset
    79
    public void testNote(Path base) throws Exception {
4995ab1a4558 8162359: javac should use stdout for --help and --version
jjg
parents:
diff changeset
    80
        Path src = base.resolve("src");
4995ab1a4558 8162359: javac should use stdout for --help and --version
jjg
parents:
diff changeset
    81
        Path classes = base.resolve("classes");
4995ab1a4558 8162359: javac should use stdout for --help and --version
jjg
parents:
diff changeset
    82
        tb.writeJavaFiles(src,
4995ab1a4558 8162359: javac should use stdout for --help and --version
jjg
parents:
diff changeset
    83
            "import java.util.*; class C { List<String> list = (List<String>) new ArrayList(); }");
4995ab1a4558 8162359: javac should use stdout for --help and --version
jjg
parents:
diff changeset
    84
        test(asList("-d", classes.toString(), src.resolve("C.java").toString()),
4995ab1a4558 8162359: javac should use stdout for --help and --version
jjg
parents:
diff changeset
    85
                null, "uses unchecked or unsafe operations.");
4995ab1a4558 8162359: javac should use stdout for --help and --version
jjg
parents:
diff changeset
    86
    }
4995ab1a4558 8162359: javac should use stdout for --help and --version
jjg
parents:
diff changeset
    87
4995ab1a4558 8162359: javac should use stdout for --help and --version
jjg
parents:
diff changeset
    88
    @Test // help output should be written to stdout
4995ab1a4558 8162359: javac should use stdout for --help and --version
jjg
parents:
diff changeset
    89
    public void testHelp(Path base) throws Exception {
4995ab1a4558 8162359: javac should use stdout for --help and --version
jjg
parents:
diff changeset
    90
        test(asList("-help"), "Usage: javac <options> <source files>", null);
4995ab1a4558 8162359: javac should use stdout for --help and --version
jjg
parents:
diff changeset
    91
    }
4995ab1a4558 8162359: javac should use stdout for --help and --version
jjg
parents:
diff changeset
    92
4995ab1a4558 8162359: javac should use stdout for --help and --version
jjg
parents:
diff changeset
    93
    @Test // version output should be written to stdout
4995ab1a4558 8162359: javac should use stdout for --help and --version
jjg
parents:
diff changeset
    94
    public void testVersion(Path base) throws Exception {
4995ab1a4558 8162359: javac should use stdout for --help and --version
jjg
parents:
diff changeset
    95
        test(asList("-version"), "javac", null);
4995ab1a4558 8162359: javac should use stdout for --help and --version
jjg
parents:
diff changeset
    96
    }
4995ab1a4558 8162359: javac should use stdout for --help and --version
jjg
parents:
diff changeset
    97
4995ab1a4558 8162359: javac should use stdout for --help and --version
jjg
parents:
diff changeset
    98
    @Test // version output should be written to stdout
4995ab1a4558 8162359: javac should use stdout for --help and --version
jjg
parents:
diff changeset
    99
    public void testFullVersion(Path base) throws Exception {
4995ab1a4558 8162359: javac should use stdout for --help and --version
jjg
parents:
diff changeset
   100
        test(asList("-fullversion"), "javac full version", null);
4995ab1a4558 8162359: javac should use stdout for --help and --version
jjg
parents:
diff changeset
   101
    }
4995ab1a4558 8162359: javac should use stdout for --help and --version
jjg
parents:
diff changeset
   102
4995ab1a4558 8162359: javac should use stdout for --help and --version
jjg
parents:
diff changeset
   103
    /**
4995ab1a4558 8162359: javac should use stdout for --help and --version
jjg
parents:
diff changeset
   104
     * Run javac as though run from the command line (but avoiding the entry point that
4995ab1a4558 8162359: javac should use stdout for --help and --version
jjg
parents:
diff changeset
   105
     * calls System.exit()), and that that expected output appears on appropriate output streams.
4995ab1a4558 8162359: javac should use stdout for --help and --version
jjg
parents:
diff changeset
   106
     * @param options the command-line options for javac
4995ab1a4558 8162359: javac should use stdout for --help and --version
jjg
parents:
diff changeset
   107
     * @param expectOut a string that should be contained in the output generated on stdout,
4995ab1a4558 8162359: javac should use stdout for --help and --version
jjg
parents:
diff changeset
   108
     *      or null, if no output should be generated to stdout
4995ab1a4558 8162359: javac should use stdout for --help and --version
jjg
parents:
diff changeset
   109
     * @param expectErra string that should be contained in the output generated on stderr,
4995ab1a4558 8162359: javac should use stdout for --help and --version
jjg
parents:
diff changeset
   110
     *      or null, if no output should be generated to stderr
4995ab1a4558 8162359: javac should use stdout for --help and --version
jjg
parents:
diff changeset
   111
     * @throws IOException if a problem occurs while setting up the streams
4995ab1a4558 8162359: javac should use stdout for --help and --version
jjg
parents:
diff changeset
   112
     */
4995ab1a4558 8162359: javac should use stdout for --help and --version
jjg
parents:
diff changeset
   113
    void test(List<String> options, String expectOut, String expectErr) throws IOException {
4995ab1a4558 8162359: javac should use stdout for --help and --version
jjg
parents:
diff changeset
   114
        out.println("test " + options);
4995ab1a4558 8162359: javac should use stdout for --help and --version
jjg
parents:
diff changeset
   115
        ByteArrayOutputStream bsOut = new ByteArrayOutputStream();
4995ab1a4558 8162359: javac should use stdout for --help and --version
jjg
parents:
diff changeset
   116
        ByteArrayOutputStream bsErr = new ByteArrayOutputStream();
4995ab1a4558 8162359: javac should use stdout for --help and --version
jjg
parents:
diff changeset
   117
        try (PrintStream psOut = new PrintStream(bsOut); PrintStream psErr = new PrintStream(bsErr)) {
4995ab1a4558 8162359: javac should use stdout for --help and --version
jjg
parents:
diff changeset
   118
            int rc;
4995ab1a4558 8162359: javac should use stdout for --help and --version
jjg
parents:
diff changeset
   119
            PrintStream saveOut = System.out;
4995ab1a4558 8162359: javac should use stdout for --help and --version
jjg
parents:
diff changeset
   120
            PrintStream saveErr = System.err;
4995ab1a4558 8162359: javac should use stdout for --help and --version
jjg
parents:
diff changeset
   121
            try {
4995ab1a4558 8162359: javac should use stdout for --help and --version
jjg
parents:
diff changeset
   122
                System.setOut(psOut);
4995ab1a4558 8162359: javac should use stdout for --help and --version
jjg
parents:
diff changeset
   123
                System.setErr(psErr);
4995ab1a4558 8162359: javac should use stdout for --help and --version
jjg
parents:
diff changeset
   124
                rc = com.sun.tools.javac.Main.compile(options.toArray(new String[0]));
4995ab1a4558 8162359: javac should use stdout for --help and --version
jjg
parents:
diff changeset
   125
            } finally {
4995ab1a4558 8162359: javac should use stdout for --help and --version
jjg
parents:
diff changeset
   126
                System.setErr(saveErr);
4995ab1a4558 8162359: javac should use stdout for --help and --version
jjg
parents:
diff changeset
   127
                System.setOut(saveOut);
4995ab1a4558 8162359: javac should use stdout for --help and --version
jjg
parents:
diff changeset
   128
            }
4995ab1a4558 8162359: javac should use stdout for --help and --version
jjg
parents:
diff changeset
   129
            System.err.println("javac exit code: " + rc);
4995ab1a4558 8162359: javac should use stdout for --help and --version
jjg
parents:
diff changeset
   130
        }
4995ab1a4558 8162359: javac should use stdout for --help and --version
jjg
parents:
diff changeset
   131
        check("stdout", bsOut.toString(), expectOut);
4995ab1a4558 8162359: javac should use stdout for --help and --version
jjg
parents:
diff changeset
   132
        check("stderr", bsErr.toString(), expectErr);
4995ab1a4558 8162359: javac should use stdout for --help and --version
jjg
parents:
diff changeset
   133
    }
4995ab1a4558 8162359: javac should use stdout for --help and --version
jjg
parents:
diff changeset
   134
4995ab1a4558 8162359: javac should use stdout for --help and --version
jjg
parents:
diff changeset
   135
    /**
4995ab1a4558 8162359: javac should use stdout for --help and --version
jjg
parents:
diff changeset
   136
     * Check that output is as expected.
4995ab1a4558 8162359: javac should use stdout for --help and --version
jjg
parents:
diff changeset
   137
     * @param name the name of the stream on which the output was found
4995ab1a4558 8162359: javac should use stdout for --help and --version
jjg
parents:
diff changeset
   138
     * @param actual the contents written to the stream
4995ab1a4558 8162359: javac should use stdout for --help and --version
jjg
parents:
diff changeset
   139
     * @param expect string that should be contained in the output, or null, if the output should be empty
4995ab1a4558 8162359: javac should use stdout for --help and --version
jjg
parents:
diff changeset
   140
     */
4995ab1a4558 8162359: javac should use stdout for --help and --version
jjg
parents:
diff changeset
   141
    void check(String name, String actual, String expect) {
4995ab1a4558 8162359: javac should use stdout for --help and --version
jjg
parents:
diff changeset
   142
        out.println("Check " + name);
4995ab1a4558 8162359: javac should use stdout for --help and --version
jjg
parents:
diff changeset
   143
        out.println("Expected: " + (expect == null ? "(nothing)" : expect));
4995ab1a4558 8162359: javac should use stdout for --help and --version
jjg
parents:
diff changeset
   144
        out.println("Actual:");
4995ab1a4558 8162359: javac should use stdout for --help and --version
jjg
parents:
diff changeset
   145
        out.println(actual.replace("\n", LINESEP));
4995ab1a4558 8162359: javac should use stdout for --help and --version
jjg
parents:
diff changeset
   146
        if (expect == null) {
4995ab1a4558 8162359: javac should use stdout for --help and --version
jjg
parents:
diff changeset
   147
            if (!actual.isEmpty()) {
4995ab1a4558 8162359: javac should use stdout for --help and --version
jjg
parents:
diff changeset
   148
                error(name + ": unexpected output");
4995ab1a4558 8162359: javac should use stdout for --help and --version
jjg
parents:
diff changeset
   149
            }
4995ab1a4558 8162359: javac should use stdout for --help and --version
jjg
parents:
diff changeset
   150
        } else if (!actual.contains(expect)) {
4995ab1a4558 8162359: javac should use stdout for --help and --version
jjg
parents:
diff changeset
   151
            error(name + ": expected output not found");
4995ab1a4558 8162359: javac should use stdout for --help and --version
jjg
parents:
diff changeset
   152
        }
4995ab1a4558 8162359: javac should use stdout for --help and --version
jjg
parents:
diff changeset
   153
    }
4995ab1a4558 8162359: javac should use stdout for --help and --version
jjg
parents:
diff changeset
   154
}
4995ab1a4558 8162359: javac should use stdout for --help and --version
jjg
parents:
diff changeset
   155