diff --git a/test/integration/jasmine-boot.js b/test/integration/jasmine-boot.js index 05e87cb4d..12891b8ba 100644 --- a/test/integration/jasmine-boot.js +++ b/test/integration/jasmine-boot.js @@ -52,10 +52,12 @@ async function runTests(results) { jasmineDone(suiteInfo) {}, jasmineStarted(suiteInfo) {}, specDone(result) { - // Report on the result of individual tests. - if (result.status === "excluded") { + // Ignore excluded (fit/xit) or skipped (pending) tests. + if (["excluded", "pending"].includes(result.status)) { return; } + + // Report on passed or failed tests. ++results.runs; if (result.status === "passed") { console.log(`TEST-PASSED | ${result.description}`); @@ -66,10 +68,12 @@ async function runTests(results) { }, specStarted(result) {}, suiteDone(result) { - if (result.status === "excluded") { + // Ignore excluded (fdescribe/xdescribe) or skipped (pending) suites. + if (["excluded", "pending"].includes(result.status)) { return; } - // Report on the result of `afterAll` invocations. + + // Report on failed suites only (indicates problems in setup/teardown). if (result.status === "failed") { ++results.failures; console.log(`TEST-UNEXPECTED-FAIL | ${result.description}`); diff --git a/test/reporter.js b/test/reporter.js index 5e19b9097..25d9b6ab8 100644 --- a/test/reporter.js +++ b/test/reporter.js @@ -62,10 +62,12 @@ const TestReporter = function (browser) { this.specStarted = function (result) {}; this.specDone = function (result) { - if (result.status === "excluded") { + // Ignore excluded (fit/xit) or skipped (pending) tests. + if (["excluded", "pending"].includes(result.status)) { return; } - // Report on the result of individual tests. + + // Report on passed or failed tests. if (result.status === "passed") { sendResult("TEST-PASSED", result.description); } else { @@ -78,10 +80,12 @@ const TestReporter = function (browser) { }; this.suiteDone = function (result) { - if (result.status === "excluded") { + // Ignore excluded (fdescribe/xdescribe) or skipped (pending) suites. + if (["excluded", "pending"].includes(result.status)) { return; } - // Report on the result of `afterAll` invocations. + + // Report on failed suites only (indicates problems in setup/teardown). if (result.status === "failed") { let failedMessages = ""; for (const item of result.failedExpectations) {