Merge pull request #1 from chlafreniere/fix/potentialzerotest

Potential fix for zero test issue
chlafreniere · Jan 31, 2020 · 920eb35 · 920eb35
2 parents 6c69c0f + c4a6ea1
commit 920eb35
Show file tree

Hide file tree

Showing 3 changed files with 8 additions and 18 deletions.
diff --git a/out/src/index.js b/out/src/index.js
diff --git a/out/src/index.js.map b/out/src/index.js.map
diff --git a/src/index.ts b/src/index.ts
@@ -198,16 +198,11 @@ function run(testsRoot, clb): any {
                 return mocha.addFile(paths.join(testsRoot, f));
             });
             // Run the tests
-            let failureCount = 0;
-
-            // Pass an empty handler to prevent NPE https://github.com/stanleyhlng/mocha-multi-reporters/issues/48
-            mocha.run(() => { })
-                .on('fail', function (test, err): void {
-                    failureCount++;
-                })
-                .on('end', function (): void {
-                    clb(undefined, failureCount);
-                });
+
+            mocha.run((failure) => { 
+                clb(undefined, failure); 
+            });
+
         } catch (error) {
             return clb(error);
         }