diff --git a/lib/testem/failure-only-per-browser-reporter.js b/lib/testem/failure-only-per-browser-reporter.js deleted file mode 100644 index d4fdaa55b48..00000000000 --- a/lib/testem/failure-only-per-browser-reporter.js +++ /dev/null @@ -1,47 +0,0 @@ -const FailureOnlyReporter = require('./failure-only-reporter'); - -class FailureOnlyPerBrowserReporter extends FailureOnlyReporter { - constructor(...args) { - super(...args); - this._resultsByBrowser = {}; - } - - report(prefix, data) { - if (!this._resultsByBrowser[prefix]) { - this._resultsByBrowser[prefix] = { - total: 0, - pass: 0, - skipped: 0, - }; - } - - this._resultsByBrowser[prefix].total++; - if (data.skipped) { - this._resultsByBrowser[prefix].skipped++; - } else if (data.passed) { - this._resultsByBrowser[prefix].pass++; - } - - super.report(prefix, data); - } - - summaryDisplay() { - let originalSummary = super.summaryDisplay(); - let lines = []; - let resultsByBrowser = this._resultsByBrowser; - Object.keys(resultsByBrowser).forEach(function(browser) { - let results = resultsByBrowser[browser]; - - lines.push('#'); - lines.push('# Browser: ' + browser); - lines.push('# tests ' + results.total); - lines.push('# pass ' + results.pass); - lines.push('# skip ' + results.skipped); - lines.push('# fail ' + (results.total - results.pass - results.skipped)); - }); - lines.push('#'); - return lines.join('\n') + '\n' + originalSummary; - } -} - -module.exports = FailureOnlyPerBrowserReporter; diff --git a/lib/testem/failure-only-reporter.js b/lib/testem/failure-only-reporter.js deleted file mode 100644 index f89b9198cab..00000000000 --- a/lib/testem/failure-only-reporter.js +++ /dev/null @@ -1,23 +0,0 @@ -const TapReporter = require('testem/lib/reporters/tap_reporter'); - -class FailureOnlyReporter extends TapReporter { - constructor(...args) { - super(...args); - this._reportCount = 0; - } - - display(prefix, result) { - this._reportCount++; - - if (!result.passed) { - super.display(prefix, result); - } - - if (this._reportCount > 100) { - this.out.write('pass count: ' + this.pass + '\n'); - this._reportCount = 0; - } - } -} - -module.exports = FailureOnlyReporter; diff --git a/package.json b/package.json index cf9d57a940d..a896b920519 100644 --- a/package.json +++ b/package.json @@ -154,6 +154,7 @@ "serve-static": "^1.13.2", "simple-dom": "^1.4.0", "testem": "^2.14.0", + "testem-failure-only-reporter": "^0.0.1", "tslint": "^5.12.1", "typescript-eslint-parser": "^21.0.2" }, diff --git a/testem.browserstack.js b/testem.browserstack.js index d577d1dd98b..e9a122f885c 100644 --- a/testem.browserstack.js +++ b/testem.browserstack.js @@ -1,4 +1,4 @@ -const FailureOnlyPerBrowserReporter = require('./lib/testem/failure-only-per-browser-reporter'); +const FailureOnlyPerBrowserReporter = require('testem-failure-only-reporter/grouped-by-browser'); module.exports = { test_page: 'dist/tests/index.html?hidepassed&hideskipped&timeout=60000', diff --git a/testem.travis-browsers.js b/testem.travis-browsers.js index ca0a83104f1..9d2d02fdd5c 100644 --- a/testem.travis-browsers.js +++ b/testem.travis-browsers.js @@ -1,4 +1,4 @@ -const FailureOnlyReporter = require('./lib/testem/failure-only-reporter'); +const FailureOnlyReporter = require('testem-failure-only-reporter'); module.exports = { test_page: 'tests/index.html?hidepassed&hideskipped&timeout=60000', diff --git a/yarn.lock b/yarn.lock index e0631a8db77..65475856231 100644 --- a/yarn.lock +++ b/yarn.lock @@ -8188,6 +8188,13 @@ testdouble@^3.2.6: quibble "^0.5.1" stringify-object-es5 "^2.5.0" +testem-failure-only-reporter@^0.0.1: + version "0.0.1" + resolved "https://registry.yarnpkg.com/testem-failure-only-reporter/-/testem-failure-only-reporter-0.0.1.tgz#d096a22b3c78b4d4bdc74d4f8727d75ef8c57b7a" + integrity sha512-a3bquFhuguXcI244mAzt8ByCIEoAOsI7NVihNafi8PTYDVCnpgxm73JluduR9GITXopdOeyLAJnlI3pavY7x3w== + dependencies: + testem "^2.9.2" + testem@^2.14.0, testem@^2.9.2: version "2.14.0" resolved "https://registry.yarnpkg.com/testem/-/testem-2.14.0.tgz#418a9a15843f68381659c6a486abb4ea48d06c29"