Merge pull request #192 from SAY-5/fix/option-isscript-null

fix(Option): guard isScript and isStyle against null resource

Author: webdiscus

src/Plugin/Option.js

@@ -594,6 +594,7 @@ class Option {
    * @return {boolean}
    */
   isStyle(resource) {
+    if (resource == null) return false;
     const [file] = resource.split('?', 1);
     return this.options.css.enabled && this.options.css.test.test(file);
   }
@@ -603,6 +604,7 @@ class Option {
    * @return {boolean}
    */
   isScript(resource) {
+    if (resource == null) return false;
     const [file] = resource.split('?', 1);
     return this.options.js.enabled && this.options.js.test.test(file);
   }

test/unit.test.js

@@ -1481,6 +1481,18 @@ describe('plugin options unit tests', () => {
     return expect(received).toEqual(true);
   });
 
+  test('isScript null resource', () => {
+    option.options = {
+      js: {
+        enabled: true,
+        test: option.js.test,
+      },
+    };
+
+    const received = option.isScript(null);
+    return expect(received).toEqual(false);
+  });
+
   test('getEntryPath', () => {
     option.dynamicEntry = 'src/views/';
     option.options.entry = option.dynamicEntry;