fixed bug when setting test status in reporter (#3500)

nightwatchjs · Nov 28, 2022 · 7e6fc0f · 7e6fc0f
1 parent 1c0393f
commit 7e6fc0f
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 1 deletion.
diff --git a/lib/reporter/results.js b/lib/reporter/results.js
@@ -110,7 +110,7 @@ module.exports = class Results {
   }
 
   hasFailures() {
-    return this.currentTestResult.errors === 0 && this.currentTestResult.failed === 0;
+    return this.currentTestResult.errors > 0 || this.currentTestResult.failed > 0;
   }
 
   ////////////////////////////////////////////////////////////

diff --git a/test/src/runner/testReporter.js b/test/src/runner/testReporter.js
@@ -250,6 +250,7 @@ describe('testReporter', function() {
             assert.ok(Object.keys(test).includes('status'));
             assert.ok(Object.keys(test).includes('startTimestamp'));
             assert.ok(Object.keys(test).includes('endTimestamp'));
+            assert.strictEqual(test.status, 'pass');
           }
         },
         silent: true,