Removed test formatting, more trouble than it's worth

Author: ctjlewis

utils/js/runtimeTests.js

@@ -29,12 +29,12 @@ const glob = require('glob');
 const path = require('path');
 
 const TEST_FILES = glob.sync(path.resolve(
-  __dirname,
-  '../../test/com/google/javascript/jscomp/runtime_tests/**/build/*_test.html'
+    __dirname,
+    '../../test/com/google/javascript/jscomp/runtime_tests/**/build/*_test.html',
 ));
 
 describe('Runtime tests', () => {
-  for (let testUrl of TEST_FILES) {
+  for (const testUrl of TEST_FILES) {
     const logs = [];
     const passed = /PASSED/i;
     const failed = /FAILED/i;
@@ -43,37 +43,36 @@ describe('Runtime tests', () => {
     const chalkMsg = (msg) => {
       const isPass = passed.test(msg);
       const isFail = failed.test(msg);
-      
+
       if (isPass || isFail) {
         return msg.replace(
-          passed, chalk.green('PASSED')
+            passed, chalk.green('PASSED'),
         ).replace(
-          failed, chalk.red('FAILED')
+            failed, chalk.red('FAILED'),
         );
-      }
-      else return msg;
-    }
-    
+      } else return msg;
+    };
+
     const testName = path.basename(testUrl);
     const TestIsFinished = new FutureEvent();
     const virtualConsole = new VirtualConsole()
-      .on('log', (msg) => {
-        logs.push(chalkMsg(msg));
-        if (/Tests complete/i.test(msg)) TestIsFinished.ready(allLogs());
-        else if (/Tests failed/i.test(msg)) TestIsFinished.cancel(allLogs());
-      });
+        .on('log', (msg) => {
+          logs.push(chalkMsg(msg));
+          if (/Tests complete/i.test(msg)) TestIsFinished.ready(allLogs());
+          else if (/Tests failed/i.test(msg)) TestIsFinished.cancel(allLogs());
+        });
 
     const TEST_DOC = fs.readFileSync(
-      path.resolve(
-        __dirname,
-        '../../',
-        testUrl
-      ), 
-      'utf-8'
+        path.resolve(
+            __dirname,
+            '../../',
+            testUrl,
+        ),
+        'utf-8',
     );
 
     it(`should pass test suite ${path.basename(testUrl)}`, async () => {
-      const { window } = new JSDOM(TEST_DOC, {
+      new JSDOM(TEST_DOC, {
         /**
          * This does not actually run a server of any kind, it only informs the
          * DOM what to put in `window.location.origin`. By default, this is
@@ -90,23 +89,15 @@ describe('Runtime tests', () => {
         /**
          * Pipe `console.log` to our virtual console.
          */
-        virtualConsole
+        virtualConsole,
       });
 
       try {
         await TestIsFinished;
       } catch (e) {
-        // Format the error a bit: first line red, rest unformatted.
-        const errorLines = e.toString().split('\n');
-        let formattedError;
-        for (let i = 0; i < errorLines.length; i++) {
-          formattedError += i === 0 
-            ? chalk.red(errorLines[i])
-            : errorLines[i];
-        }
-        fail(`Failed test in suite ${testName}: \n${formattedError}\n`);
+        fail(`Failed test in suite ${testName}: \n${e}\n`);
       }
       console.log(`Passed all tests in suite ${testName}`);
     });
   }
-});
+});