modules_tests: additional HTML normalization in tests

- takes care of empty attribs like class="" or style=""
  left behind in certain cases
- also process output of JSON stringify necessary in some cases
  so that the tests pass

Author: amroamroamro

tests/extractSourceSpans_test.html

@@ -64,7 +64,10 @@ <h2>Log</h2>
     return JSON.stringify(s)
       .replace(/\r/g, '\\r')
       .replace(/\n/g, '\\n')
-      .replace(/\t/g, '\\t');
+      .replace(/\t/g, '\\t')
+      .replace(/\\u000d/g, '\\r')
+      .replace(/\\u000a/g, '\\n')
+      .replace(/\\u0009/g, '\\t');
   }
 
   function htmlEscape(s) {

tests/numberLines_test.html

@@ -106,7 +106,8 @@ <h2>Log</h2>
       numberLines(clone, startLineNum, isPreformatted);
       // compare result again expected
       var goldenNormalized = normListItems(testResult.innerHTML);
-      var actualNormalized = normListItems(actual.innerHTML);
+      var actualNormalized = normListItems(actual.innerHTML).replace(
+        /\s*(?:class|style)=""/i, '');  // remove some empty attribs
       var passed = goldenNormalized === actualNormalized;
       // report test result
       actual.className += passed ? ' ok' : ' failure';

tests/recombineTagsAndDecorations_test.html

@@ -89,16 +89,19 @@ <h2>Log</h2>
       };
       recombineTagsAndDecorations(job);
       // compare result against expected
-      var passed = testResult.innerHTML === actual.innerHTML;
+      var goldenNormalized = testResult.innerHTML;
+      var actualNormalized = actual.innerHTML.replace(
+        /\s*(?:class|style)=""/i, '');  // remove some empty attribs
+      var passed = goldenNormalized === actualNormalized;
       // report test result
       actual.className += passed ? ' ok' : ' failure';
       out.push((passed ? '\u2713 PASS' : '\u2717 FAIL'), '<\/div>');
       if (!passed) {
         out.push(
           '<table class="diff"><tr><th>Golden<\/th><td><code>' +
-          htmlEscape(testResult.innerHTML) +
+          htmlEscape(goldenNormalized) +
           '<\/code><\/td><\/tr><tr><th>Actual<\/th><td><code>' +
-          htmlEscape(actual.innerHTML) +
+          htmlEscape(actualNormalized) +
           '<\/code><\/td><\/tr><\/table>'
         );
       }