Fix 44 again (#54)

* fix: keep reporting other file messages even if the current file is message free

* fix: update test

* fix: update test - don't look for filename

Author: jsg2021

src/linter.js

@@ -76,17 +76,17 @@ export default function linter(options, compilation) {
 
     await cleanup();
 
-    // do not analyze if there are no results or eslint config
-    if (!results || results.length < 1) {
-      return {};
-    }
-
     for (const result of results) {
       crossRunResultStorage[result.filePath] = result;
     }
 
     results = Object.values(crossRunResultStorage);
 
+    // do not analyze if there are no results or eslint config
+    if (!results || results.length < 1) {
+      return {};
+    }
+
     const formatter = await loadFormatter(eslint, options.formatter);
     const { errors, warnings } = formatResults(
       formatter,

test/watch.test.js

@@ -103,7 +103,11 @@ describe('watch', () => {
     function finish(err, stats) {
       expect(err).toBeNull();
       expect(stats.hasWarnings()).toBe(false);
-      expect(stats.hasErrors()).toBe(false);
+      const { errors } = stats.compilation;
+      const [{ message }] = errors;
+      expect(stats.hasErrors()).toBe(true);
+      expect(message).toEqual(expect.stringMatching('prefer-const'));
+      expect(message).toEqual(expect.stringMatching('\\(2 errors,'));
       done();
     }
   });