Move test summary to after coverage report #4104

jestjs · Oct 7, 2017 · 74a3841 · 74a3841
1 parent 7b0e108
commit 74a3841
Showing 1 changed file with 7 additions and 3 deletions.
diff --git 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));
   }