Exit 1 for test harness if any tests fail

This change makes TokenizerTester, TreeTester, and EncodingTester exit
with status code 1 if any test cases fail.

Otherwise, without this change, we won’t catch the test failures when
running tests under CI.

Author: sideshowbarker

test-src/nu/validator/htmlparser/test/EncodingTester.java

@@ -36,6 +36,8 @@
 
 public class EncodingTester {
 
+    private static int exitStatus = 0;
+
     private final InputStream aggregateStream;
 
     private final StringBuilder builder = new StringBuilder();
@@ -63,6 +65,7 @@ private boolean runTest() throws IOException, SAXException {
         Charset charset = reader.getCharset();
         stream.close();
         if (skipLabel()) {
+            exitStatus = 1;
             System.err.println("Premature end of test data.");
             return false;
         }
@@ -73,6 +76,7 @@ private boolean runTest() throws IOException, SAXException {
                 case '\n':
                     break loop;
                 case -1:
+                    exitStatus = 1;
                     System.err.println("Premature end of test data.");
                     return false;
                 default:
@@ -85,6 +89,7 @@ private boolean runTest() throws IOException, SAXException {
             System.err.println("Success.");
             // System.err.println(stream);
         } else {
+            exitStatus = 1;
             System.err.println("Failure. Expected: " + expected + " got "
                     + sniffed + ".");
             System.err.println(stream);
@@ -118,6 +123,7 @@ public static void main(String[] args) throws IOException, SAXException {
                     args[i]));
             tester.runTests();
         }
+        System.exit(exitStatus);
     }
 
 }

test-src/nu/validator/htmlparser/test/TokenizerTester.java

@@ -51,6 +51,8 @@
 
 public class TokenizerTester {
 
+    private static int exitStatus = 0;
+
     private static JSONString PLAINTEXT = new JSONString("PLAINTEXT state");
 
     private static JSONString PCDATA = new JSONString("Data state");
@@ -182,6 +184,7 @@ private void runTestInner(String inputString, JSONArray expectedTokens,
             if (jsonDeepEquals(actualTokens, expectedTokens)) {
                 writer.write("Success\n");
             } else {
+                exitStatus = 1;
                 writer.write("Failure\n");
                 writer.write(description);
                 writer.write("\nInput:\n");
@@ -193,6 +196,7 @@ private void runTestInner(String inputString, JSONArray expectedTokens,
                 writer.write("\n");
             }
         } catch (Throwable t) {
+            exitStatus = 1;
             writer.write("Failure\n");
             writer.write(description);
             writer.write("\nInput:\n");
@@ -216,6 +220,7 @@ public static void main(String[] args) throws TokenStreamException,
                     args[i]));
             tester.runTests();
         }
+        System.exit(exitStatus);
     }
 
 }

test-src/nu/validator/htmlparser/test/TreeTester.java

@@ -43,6 +43,8 @@ public class TreeTester {
 
     private boolean streaming = false;
 
+    private static int exitStatus = 0;
+
     /**
      * @param aggregateStream
      */
@@ -224,6 +226,7 @@ private boolean runTest() throws Throwable {
                 System.err.println("Success.");
                 // System.err.println(stream);
             } else {
+                exitStatus = 1;
                 System.err.print("Failure.\nData:\n" + stream + "\nExpected:\n"
                         + expected + "Got: \n" + actual);
                 System.err.println("Expected errors:");
@@ -236,6 +239,7 @@ private boolean runTest() throws Throwable {
                 }
             }
         } catch (Throwable t) {
+            exitStatus = 1;
             System.err.println("Failure.\nData:\n" + stream);
             throw t;
         }
@@ -266,6 +270,7 @@ public static void main(String[] args) throws Throwable {
             TreeTester tester = new TreeTester(new FileInputStream(args[i]));
             tester.runTests();
         }
+        System.exit(exitStatus);
     }
 
 }