Drop “Success” message for test passes

This change makes TokenizerTester, TreeTester, and EncodingTester stop
emitting the word “Success” to standard error every time a test passes.

Otherwise, without this change, in test output, we end up with so many
“Success” lines that the actual test failures are effectively obscured
(you have to scroll back through the output/log to hunt for failures).

Author: sideshowbarker

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

@@ -88,7 +88,6 @@ private boolean runTest() throws IOException, SAXException {
         String sniffed = charset.name();
         String expected = Encoding.forName(builder.toString()).newDecoder().charset().name();
         if (expected.equalsIgnoreCase(sniffed)) {
-            System.err.println("Success.");
             // System.err.println(stream);
         } else {
             exitStatus = 1;

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

@@ -190,9 +190,7 @@ private void runTestInner(String inputString, JSONArray expectedTokens,
         try {
             driver.tokenize(is);
             JSONArray actualTokens = tokenHandler.getArray();
-            if (jsonDeepEquals(actualTokens, expectedTokens)) {
-                writer.write("Success\n");
-            } else {
+            if (!jsonDeepEquals(actualTokens, expectedTokens)) {
                 exitStatus = 1;
                 writer.write("Failure\n");
                 writer.write(description);

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

@@ -228,7 +228,6 @@ private boolean runTest() throws Throwable {
              * && expectedErrors.size() ==
              * actualErrors.size()
              */) {
-                System.err.println("Success.");
                 // System.err.println(stream);
             } else {
                 exitStatus = 1;