fix(instrumenter): 🐛 fix break of Karma process on error thrown in te…

…st/src

fix #226

src/instrumenter.ts

@@ -27,9 +27,11 @@ function getListOfFilesToCover(coverageReporterConfig: PublicAPI): string[] {
 
 function addFileListToCoverageData(
   filesToCover: string[],
-  coverage: CoverageData
+  coverage?: CoverageData
 ): void {
   filesToCover.forEach((filePath: string): void => {
+    if (!coverage) return;
+
     const fullFilePath: string = path.resolve(process.cwd(), filePath);
 
     if (!coverage[fullFilePath]) {

test/fixtures/typescript/test/test-error.spec.ts

@@ -0,0 +1,7 @@
+/* tslint:disable */
+
+describe('Foo', () => {
+  it('should throw error', () => {
+    throw new Error('Unexpected error in test');
+  });
+});

test/reporter.spec.ts

@@ -78,6 +78,48 @@ describe('karma-sabarivka-reporter:', () => {
         });
       });
 
+      it('should not break Karma process on thrown exception in tests', done => {
+        // given
+        const config= {
+          coverageReporter: {
+            include: 'test/**/ignored-file.ts',
+          },
+        };
+        const coverageReportDir = join(
+          OUTPUT_PATH,
+          `coverage${generate()}`
+        );
+        const server = createServer(
+          {
+            reporters: ['sabarivka', coverageReporter],
+            coverageIstanbulReporter: {
+              reports: ['json-summary'],
+              dir: coverageReportDir,
+            },
+            coverageReporter: {
+              ...config.coverageReporter,
+              type: 'json-summary',
+              dir: coverageReportDir,
+              subdir: '.',
+            },
+            files: [
+              'fixtures/typescript/test/test-error.spec.ts',
+            ],
+            preprocessors: {
+              'fixtures/typescript/test/test-error.spec.ts': ['webpack', 'sourcemap'],
+            },
+          },
+          undefined,
+          true
+        );
+
+        // when
+        const karmaStart = (server.start() as unknown) as Promise<void>;
+
+        // then
+        checkKarmaSuccessOutput(server, karmaStart, done);
+      });
+
       describe('Correct config:', () => {
         [
           {