fix: strip resource query (#58)

* fix: strip resource query * test: resource query test * refactor: split by ? instead of url.parse * chore: update lock files Co-authored-by: Ricardo Gobbo de Souza <ricardogobbosouza@yahoo.com.br>
webpack-contrib · Jan 19, 2021 · f0a2d81 · f0a2d81
1 parent 2e6c8eb
commit f0a2d81
Show file tree

Hide file tree

Showing 4 changed files with 26 additions and 12 deletions.
diff --git a/package-lock.json b/package-lock.json
diff --git a/src/index.js b/src/index.js
@@ -80,7 +80,7 @@ export class ESLintWebpackPlugin {
 
       // @ts-ignore
       const processModule = (module) => {
-        const file = module.resource;
+        const file = module.resource.split('?')[0];
 
         if (
           file &&

diff --git a/test/fixtures/query-entry.js b/test/fixtures/query-entry.js
@@ -0,0 +1,2 @@
+// eslint-disable-next-line import/no-unresolved
+require('./good.js?some-query=1234.js');
diff --git a/test/query.test.js b/test/query.test.js
@@ -0,0 +1,15 @@
+import pack from './utils/pack';
+
+describe('query', () => {
+  it('should correctly resolve file despite query path', (done) => {
+    const compiler = pack('query');
+
+    compiler.run((err, stats) => {
+      expect(err).toBeNull();
+      expect(stats.hasWarnings()).toBe(false);
+      expect(stats.hasErrors()).toBe(false);
+
+      done();
+    });
+  });
+});