From 74a3841e4fb721ae782706bdb93cd52123cf0974 Mon Sep 17 00:00:00 2001 From: Hasan Ramezani Date: Tue, 19 Sep 2017 20:24:13 +0430 Subject: [PATCH] Move test summary to after coverage report #4104 --- packages/jest-cli/src/test_scheduler.js | 10 +++++++--- 1 file changed, 7 insertions(+), 3 deletions(-) diff --git a/packages/jest-cli/src/test_scheduler.js b/packages/jest-cli/src/test_scheduler.js index 291174290daf..b4334146c048 100644 --- a/packages/jest-cli/src/test_scheduler.js +++ b/packages/jest-cli/src/test_scheduler.js @@ -226,10 +226,10 @@ export default class TestScheduler { const isDefault = this._shouldAddDefaultReporters(reporters); if (isDefault) { - this._setupDefaultReporters(); + this._setupDefaultReporters(collectCoverage); } - if (collectCoverage) { + if (!isDefault && collectCoverage) { this.addReporter(new CoverageReporter(this._globalConfig)); } @@ -244,13 +244,17 @@ export default class TestScheduler { } } - _setupDefaultReporters() { + _setupDefaultReporters(collectCoverage: boolean) { this.addReporter( this._globalConfig.verbose ? new VerboseReporter(this._globalConfig) : new DefaultReporter(this._globalConfig), ); + if (collectCoverage) { + this.addReporter(new CoverageReporter(this._globalConfig)); + } + this.addReporter(new SummaryReporter(this._globalConfig)); }