Fix bug with string passed as files

Author: SimenB

CHANGELOG.md

@@ -7,6 +7,10 @@ Latest version of this document will always be available on https://github.com/S
 ## [Unreleased]
 Nothing
 
+## [2.0.1] - 2017-04-23
+### Fixes
+- Support only string as option to files
+
 ## [2.0.0] - 2017-04-23
 ### Breaking
 - Drop support for `node<4`

__snapshots__/test.js.snap

@@ -16,6 +16,15 @@ Object {
 }
 `;
 
+exports[`filter option should include some files with string option 1`] = `
+Object {
+  "css": Array [],
+  "js": Array [
+    "vendor/my-file.js",
+  ],
+}
+`;
+
 exports[`should add file missing "/" to public path 1`] = `
 Object {
   "css": Array [],

src/addAllAssetsToCompilation.js

@@ -39,10 +39,10 @@ async function addFileToAssets(
     return Promise.reject(error);
   }
 
-  const filters = Array.isArray(files) ? files : [files];
+  const fileFilters = Array.isArray(files) ? files : [files];
 
-  if (filters.length > 0) {
-    const shouldSkip = !files.some(file => minimatch(htmlPluginData.outputName, file));
+  if (fileFilters.length > 0) {
+    const shouldSkip = !fileFilters.some(file => minimatch(htmlPluginData.outputName, file));
 
     if (shouldSkip) {
       return Promise.resolve(null);

test.js

@@ -228,3 +228,21 @@ test('filter option should include some files', async () => {
   expect(callback).toHaveBeenCalledTimes(1);
   expect(callback).toHaveBeenCalledWith(null, pluginData);
 });
+
+test('filter option should include some files with string option', async () => {
+  const callback = jest.fn();
+  const compilation = { options: { output: { publicPath: 'vendor/' } } };
+  const pluginData = Object.assign({ assets: { js: [], css: [] } }, pluginMock);
+
+  await addAllAssetsToCompilation(
+    [{ filepath: path.join(__dirname, 'my-file.js'), files: 'index.*' }],
+    compilation,
+    pluginData,
+    callback
+  );
+
+  expect(pluginData.assets).toMatchSnapshot();
+
+  expect(callback).toHaveBeenCalledTimes(1);
+  expect(callback).toHaveBeenCalledWith(null, pluginData);
+});