Safer test suite reporter timer.

ethers-io · May 5, 2020 · 657a039 · 657a039
1 parent ab7c781
commit 657a039
Showing 1 changed file with 13 additions and 11 deletions.
diff --git a/packages/tests/src.ts/reporter.ts b/packages/tests/src.ts/reporter.ts
@@ -37,7 +37,16 @@ interface Runner {
 export function Reporter(runner: Runner) {
     let suites: Array<Suite> = [];
 
+    // Force Output; Keeps the console output alive with periodic updates
     let lastOutput = getTime();
+    function forceOutput() {
+        if (((getTime() - lastOutput) / 1000) > MAX_DELAY) {
+            const currentSuite = suites[suites.length - 1];
+            log(`[ Still running suite - test # ${ (currentSuite ? currentSuite._countTotal: "0") } ]`);
+        }
+    }
+    const timer = setInterval(forceOutput, 1000);
+
 
     function getIndent(): string {
         let result = '';
@@ -70,28 +79,21 @@ export function Reporter(runner: Runner) {
         let suite = suites.pop();
         let failure = '';
         if (suite._countTotal > suite._countPass) {
-            failure = ' (' + (suite._countTotal - suite._countPass) + ' failed)';
+            failure = ' (' + (suite._countTotal - suite._countPass) + ' failed) *****';
         }
         log('  Total Tests: ' + suite._countPass + '/' + suite._countTotal + ' passed ' + getDelta(suite._t0) + failure);
         log();
+
         if (suites.length > 0) {
             let currentSuite = suites[suites.length - 1];
             currentSuite._countFail += suite._countFail;
             currentSuite._countPass += suite._countPass;
             currentSuite._countTotal += suite._countTotal;
+        } else {
+            clearTimeout(timer);
         }
     });
 
-    function forceOutput() {
-        if (((getTime() - lastOutput) / 1000) > MAX_DELAY) {
-            const currentSuite = suites[suites.length - 1];
-            log(`[ Still running suite - test # ${ (currentSuite ? currentSuite._countTotal: "0") } ]`);
-        }
-    }
-
-    const timer = setInterval(forceOutput, 1000);
-    if (timer.unref) { timer.unref(); }
-
     runner.on('test', function(test) {
         forceOutput();
         const currentSuite = suites[suites.length - 1];