diff -r 3fd30668e9b5 -r cfadc977ca75 langtools/test/com/sun/javadoc/ValidHtml/ValidHtml.java --- a/langtools/test/com/sun/javadoc/ValidHtml/ValidHtml.java Tue Nov 30 09:38:48 2010 -0800 +++ b/langtools/test/com/sun/javadoc/ValidHtml/ValidHtml.java Wed Dec 01 11:02:38 2010 -0800 @@ -33,12 +33,10 @@ * @run main ValidHtml */ - import com.sun.javadoc.*; import java.util.*; import java.io.*; - /** * Runs javadoc and runs regression tests on the resulting HTML. * It reads each file, complete with newlines, into a string to easily @@ -66,13 +64,14 @@ String srcdir = System.getProperty("test.src", "."); // Test for all cases except the split index page - runJavadoc(new String[] {"-d", TMPDEST_DIR1, - "-doctitle", "Document Title", - "-windowtitle", "Window Title", - "-use", - "-overview", (srcdir + FS + "overview.html"), - "-sourcepath", srcdir, - "p1", "p2"}); + runJavadoc(new String[]{"-d", TMPDEST_DIR1, + "-doctitle", "Document Title", + "-windowtitle", "Window Title", + "-use", + "-overview", (srcdir + FS + "overview.html"), + "-sourcepath", srcdir, + "p1", "p2" + }); runTestsOnHTML(testArray); printSummary(); @@ -90,53 +89,52 @@ * NOTE: The standard doclet uses the same separator "\n" for all OS's */ private static final String[][] testArray = { - - // Test the proper DOCTYPE element is present: - { -"", - TMPDEST_DIR1 + "index.html" }, - - // Test the proper DOCTYPE element is present: - { -"", - TMPDEST_DIR1 + "overview-summary.html" }, - - // Test the proper DOCTYPE element is present: - { -"", - TMPDEST_DIR1 + "p1" + FS + "package-summary.html" }, - - // Test the proper DOCTYPE element is present: - { -"", - TMPDEST_DIR1 + "p1" + FS + "C.html" }, - - // Test the proper DOCTYPE element is present: - { -"", - TMPDEST_DIR1 + "overview-frame.html" }, - - // Test the proper DOCTYPE element is present: - { -"", - TMPDEST_DIR1 + "allclasses-frame.html" }, - - // Test the proper DOCTYPE element is present: - { -"", - TMPDEST_DIR1 + "p1" + FS + "package-frame.html" }, - - // Test that is inside <FRAMESET> element: - { -"" + LS + "", - TMPDEST_DIR1 + "index.html" }, - - // Test the table elements are in the correct order: - { -"" + LS + "", - TMPDEST_DIR1 + FS + "p1" + FS + "package-use.html" } - - }; + // Test the proper DOCTYPE element is present: + { + "", + TMPDEST_DIR1 + "index.html" + }, + // Test the proper DOCTYPE element is present: + { + "", + TMPDEST_DIR1 + "overview-summary.html" + }, + // Test the proper DOCTYPE element is present: + { + "", + TMPDEST_DIR1 + "p1" + FS + "package-summary.html" + }, + // Test the proper DOCTYPE element is present: + { + "", + TMPDEST_DIR1 + "p1" + FS + "C.html" + }, + // Test the proper DOCTYPE element is present: + { + "", + TMPDEST_DIR1 + "overview-frame.html" + }, + // Test the proper DOCTYPE element is present: + { + "", + TMPDEST_DIR1 + "allclasses-frame.html" + }, + // Test the proper DOCTYPE element is present: + { + "", + TMPDEST_DIR1 + "p1" + FS + "package-frame.html" + }, + // Test that is inside <FRAMESET> element: + { + "" + LS + "", + TMPDEST_DIR1 + "index.html" + }, + // Test the table elements are in the correct order: + { + "" + LS + "", + TMPDEST_DIR1 + FS + "p1" + FS + "package-use.html" + } + }; public static void runTestsOnHTML(String[][] testArray) { @@ -152,10 +150,7 @@ // Find string in file's contents if (findString(fileString, stringToFind) == -1) { - System.out.println("\nSub-test " + (subtestNum) - + " for bug " + BUGID + " (" + BUGNAME + ") FAILED\n" - + "when searching for:\n" - + stringToFind); + System.out.println("\nSub-test " + (subtestNum) + " for bug " + BUGID + " (" + BUGNAME + ") FAILED\n" + "when searching for:\n" + stringToFind); } else { numSubtestsPassed += 1; System.out.println("\nSub-test " + (subtestNum) + " passed:\n" + stringToFind); @@ -164,11 +159,10 @@ } public static void printSummary() { - if ( numSubtestsPassed == subtestNum ) { + if (numSubtestsPassed == subtestNum) { System.out.println("\nAll " + numSubtestsPassed + " subtests passed"); } else { - throw new Error("\n" + (subtestNum - numSubtestsPassed) + " of " + (subtestNum) - + " subtests failed for bug " + BUGID + " (" + BUGNAME + ")\n"); + throw new Error("\n" + (subtestNum - numSubtestsPassed) + " of " + (subtestNum) + " subtests failed for bug " + BUGID + " (" + BUGNAME + ")\n"); } } @@ -176,16 +170,16 @@ public static String readFileToString(String filename) { try { File file = new File(filename); - if ( !file.exists() ) { + if (!file.exists()) { System.out.println("\nFILE DOES NOT EXIST: " + filename); } BufferedReader in = new BufferedReader(new FileReader(file)); // Create an array of characters the size of the file - char[] allChars = new char[(int)file.length()]; + char[] allChars = new char[(int) file.length()]; // Read the characters into the allChars array - in.read(allChars, 0, (int)file.length()); + in.read(allChars, 0, (int) file.length()); in.close(); // Convert to a string