From e4d03fbad32df8eb1b77ea4962de62b3275b67c7 Mon Sep 17 00:00:00 2001 From: Hasan Ramezani Date: Wed, 7 Feb 2018 12:16:41 +0330 Subject: [PATCH] Move test summary to after coverage report #4104 (#4512) --- 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 ecdcd06a281c..2f7ec55002c0 100644 --- a/packages/jest-cli/src/test_scheduler.js +++ b/packages/jest-cli/src/test_scheduler.js @@ -257,10 +257,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)); } @@ -279,13 +279,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)); }