fix(reporter): report event order (#2311)

Report `onAllMutantsTested` before `onMutationTestReportReady`
stryker-mutator · Jul 10, 2020 · ceb73a8 · ceb73a8
1 parent 8bd634b
commit ceb73a8
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 1 deletion.
diff --git a/packages/core/src/reporters/MutationTestReportHelper.ts b/packages/core/src/reporters/MutationTestReportHelper.ts
@@ -57,8 +57,8 @@ export class MutationTestReportHelper {
 
   public reportAll(results: MutantResult[]) {
     const report = this.mutationTestReport(results);
-    this.reporter.onMutationTestReportReady(report);
     this.reporter.onAllMutantsTested(results);
+    this.reporter.onMutationTestReportReady(report);
     this.determineExitCode(report);
   }
 

diff --git a/packages/core/test/unit/reporters/MutationTestReportHelper.spec.ts b/packages/core/test/unit/reporters/MutationTestReportHelper.spec.ts
@@ -51,6 +51,16 @@ describe(MutationTestReportHelper.name, () => {
       expect(reporterMock.onMutationTestReportReady).calledOnce;
     });
 
+    it('should report "onAllMutantsTested"', () => {
+      sut.reportAll([]);
+      expect(reporterMock.onAllMutantsTested).calledOnce;
+    });
+
+    it('should report "onAllMutantsTested" before mutationTestReportReady', () => {
+      sut.reportAll([]);
+      expect(reporterMock.onAllMutantsTested).calledBefore(reporterMock.onMutationTestReportReady);
+    });
+
     it('should copy thresholds', () => {
       const actualReport = actReportAll();
       expect(actualReport.thresholds).eq(testInjector.options.thresholds);