Drop “Success” for TokenizerTester/TreeTester pass

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

Otherwise, without this change, in TokenizerTester/TreeTester 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 the failures).

Author: sideshowbarker

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;