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 <[email protected]>

Author: privatenumber

package-lock.json

@@ -80,7 +80,7 @@ export class ESLintWebpackPlugin {
 
       // @ts-ignore
       const processModule = (module) => {
-        const file = module.resource;
+        const file = module.resource.split('?')[0];
 
         if (
           file &&

src/index.js

@@ -0,0 +1,2 @@
+// eslint-disable-next-line import/no-unresolved
+require('./good.js?some-query=1234.js');

test/fixtures/query-entry.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();
+    });
+  });
+});