stan/jdk-24
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

8290126: Add a check in JavadocTester for "javadoc should not crash"

Browse Source 
Reviewed-by: prappo
This commit is contained in:
Jonathan Gibbons 2022-09-01 22:41:27 +00:00
parent 0a4d0cee9f
commit 0fb9469d93
5 changed files with 148 additions and 10 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
test/langtools/jdk/javadoc/doclet/InheritDocForUserTags/DocTest.java
View File

@ -51,10 +51,6 @@ public class DocTest extends JavadocTester {
testSrc("DocTest.java")
);
checkExit(Exit.OK);
// javadoc does not report an exit code for an internal exception (!)
// so monitor stderr for stack dumps.
checkOutput(Output.STDERR, false, "at com.sun");
}
/**

19
test/langtools/jdk/javadoc/lib/javadoc/tester/JavadocTester.java
View File

@ -214,7 +214,7 @@ public abstract class JavadocTester {
NONE(null) { @Override void check(Path dir) { } };
/** The filter used to detect that files should <i>not</i> be present. */
DirectoryStream.Filter<Path> filter;
private final DirectoryStream.Filter<Path> filter;
DirectoryCheck(DirectoryStream.Filter<Path> f) {
filter = f;
@ -246,6 +246,7 @@ public abstract class JavadocTester {
private boolean automaticCheckAccessibility = true;
private boolean automaticCheckLinks = true;
private boolean automaticCheckUniqueOUT = true;
private boolean automaticCheckNoStacktrace = true;
private boolean useStandardStreams = false;
private StandardJavaFileManager fileManager = null;
@ -488,6 +489,11 @@ public abstract class JavadocTester {
}
});
if (automaticCheckNoStacktrace) {
// Any stacktrace will have javadoc near the bottom of the stack
checkOutput(Output.STDERR, false, "at jdk.javadoc/jdk.javadoc.internal.");
}
if (exitCode == Exit.OK.code && Files.exists(outputDir)) {
if (automaticCheckLinks) {
checkLinks();
@ -533,6 +539,13 @@ public abstract class JavadocTester {
automaticCheckUniqueOUT = b;
}
/**
* Sets whether or not to check for stacktraces.
*/
public void setAutomaticCheckNoStacktrace(boolean b) {
automaticCheckNoStacktrace = b;
}
/**
* Sets whether to use standard output streams (stdout and stderr)
* instead of a single temporary stream.
@ -1147,7 +1160,7 @@ public abstract class JavadocTester {
public OutputChecker check(String... strings) {
if (name == null) {
out.println("Skipping checks for:" + NL
+ List.of(strings).stream()
+ Stream.of(strings)
.map(s -> " " + toShortString(s))
.collect(Collectors.joining(NL)));
return this;
@ -1169,7 +1182,7 @@ public abstract class JavadocTester {
public OutputChecker check(Pattern... patterns) {
if (name == null) {
out.println("Skipping checks for:" + NL
+ List.of(patterns).stream()
+ Stream.of(patterns)
.map(p -> " " + toShortString(p.pattern()))
.collect(Collectors.joining(NL)));
return this;

4
test/langtools/jdk/javadoc/testJavadocTester/TestJavadocTester.java
View File

@ -63,7 +63,7 @@ public class TestJavadocTester extends JavadocTester {
tester.setup().runTests();
}
private final List<String> messages = new ArrayList<>();
protected final List<String> messages = new ArrayList<>();
private int testErrors = 0;
/**
@ -158,7 +158,7 @@ public class TestJavadocTester extends JavadocTester {
//-------------------------------------------------
private final ToolBox tb = new ToolBox();
protected final ToolBox tb = new ToolBox();
TestJavadocTester setup() throws IOException {
Path src = Path.of("src");

123
test/langtools/jdk/javadoc/testJavadocTester/TestJavadocTesterCrash.java Normal file
View File

@ -0,0 +1,123 @@
/*
* Copyright (c) 2022, Oracle and/or its affiliates. All rights reserved.
* DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
*
* This code is free software; you can redistribute it and/or modify it
* under the terms of the GNU General Public License version 2 only, as
* published by the Free Software Foundation.
*
* This code is distributed in the hope that it will be useful, but WITHOUT
* ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or
* FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public License
* version 2 for more details (a copy is included in the LICENSE file that
* accompanied this code).
*
* You should have received a copy of the GNU General Public License version
* 2 along with this work; if not, write to the Free Software Foundation,
* Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA.
*
* Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA
* or visit www.oracle.com if you need additional information or have any
* questions.
*/
/*
* @test
* @bug 8290126
* @summary Add a check in JavadocTester for "javadoc should not crash"
* @library /tools/lib/ ../lib
* @modules jdk.javadoc/jdk.javadoc.internal.tool
* @build toolbox.ToolBox javadoc.tester.* TestJavadocTester
* @run main TestJavadocTesterCrash
*/
import com.sun.source.doctree.DocTree;
import jdk.javadoc.doclet.Taglet;
import javax.lang.model.element.Element;
import java.nio.file.Path;
import java.util.EnumSet;
import java.util.List;
import java.util.Set;
/**
* Tests that {@code JavadocTester} detects and reports exceptions.
*
* It is not a direct test of the javadoc tool or the output generated by the
* Standard Doclet, although both are indirectly used as part of this test.
*
* The test uses the infrastructure of TestJavadocTester, but cannot be
* added to the tests there, which rely on checking aspects of the output
* from a single run on javadoc. This test forces a crash to occur in
* javadoc, and verifies that JavadocTester detects and reports the crash.
*
* Arguably, a crash in a user-provided taglet should not cause a full stack
* trace. If that is ever fixed, we would need to revisit the goals and mechanism
* of this test.
*/
public class TestJavadocTesterCrash extends TestJavadocTester {
public static void main(String... args) throws Exception {
TestJavadocTesterCrash tester = new TestJavadocTesterCrash();
tester.runTests();
}
/** A taglet that can throw an exception. */
public static class TestTaglet implements Taglet {
public TestTaglet() { }
@Override
public Set<Location> getAllowedLocations() {
return EnumSet.allOf(Location.class);
}
@Override
public boolean isInlineTag() {
return true;
}
@Override
public String getName() {
return "test-taglet";
}
@Override
public String toString(List<? extends DocTree> tags, Element element) {
String s = tags.toString();
if (s.contains("test")) {
throw new Error("demo error");
};
return s;
}
}
@Test
public void testDetectException(Path base) throws Exception {
messages.clear();
Path src = base.resolve("src");
tb.writeJavaFiles(src,
"""
package p;
/** Comment.
abc {@test-taglet simple} {@test-taglet test} xyz
*/
public class C { }""");
javadoc("-d", base.resolve("api").toString(),
"-tagletpath", System.getProperty("test.class.path"),
"-taglet", TestTaglet.class.getName(),
"-sourcepath", src.toString(),
"p");
checkExit(Exit.ERROR);
// verify that the taglet threw an exception as intended
new OutputChecker(Output.OUT)
.setExpectOrdered(true)
.check("Generating testDetectException/api/p/C.html...",
"error: An internal exception has occurred.",
"(java.lang.Error: demo error)",
"1 error");
// verify that JavadocTester detected the crash
checkMessages("FAILED: STDERR: following text found:");
}
}

8
test/langtools/jdk/javadoc/testTFMBuilder/TestTFMBuilder.java
View File

@ -1,5 +1,5 @@
/*
* Copyright (c) 2021, Oracle and/or its affiliates. All rights reserved.
* Copyright (c) 2021, 2022, Oracle and/or its affiliates. All rights reserved.
* DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
*
* This code is free software; you can redistribute it and/or modify it
@ -161,6 +161,7 @@ public class TestTFMBuilder extends JavadocTester {
try {
setFileManager(tfm);
setAutomaticCheckNoStacktrace(false);
javadoc("-d", base.resolve("api").toString(),
"-sourcepath", srcDir.toString(),
"p");
@ -173,6 +174,7 @@ public class TestTFMBuilder extends JavadocTester {
.replace("##EXC##", TestException.class.getName()));
} finally {
setFileManager(null);
setAutomaticCheckNoStacktrace(true);
}
}
}
@ -190,6 +192,7 @@ public class TestTFMBuilder extends JavadocTester {
try {
setFileManager(tfm);
setAutomaticCheckNoStacktrace(false);
javadoc("-d", base.resolve("api").toString(),
"-sourcepath", srcDir.toString(),
"p");
@ -203,6 +206,7 @@ public class TestTFMBuilder extends JavadocTester {
.replace("##FILE##", srcDir.resolve("p").resolve("C.java").toString()));
} finally {
setFileManager(null);
setAutomaticCheckNoStacktrace(true);
}
}
}
@ -222,6 +226,7 @@ public class TestTFMBuilder extends JavadocTester {
try {
setFileManager(tfm);
setAutomaticCheckNoStacktrace(false);
javadoc("-d", outDir.toString(),
"-sourcepath", srcDir.toString(),
"p");
@ -236,6 +241,7 @@ public class TestTFMBuilder extends JavadocTester {
.replace("##FILE##", outDir.resolve("p").resolve("C.html").toString()));
} finally {
setFileManager(null);
setAutomaticCheckNoStacktrace(true);
}
}
}