fix #231 allow square brackets in path

package.json

@@ -18,7 +18,7 @@
     "pretest": "npm run lint && npm run build && npm run build:tests",
     "test": "mocha compiled_tests/",
     "build": "babel src/ --out-dir dist/",
-    "build:tests": "babel tests/ --out-dir compiled_tests/ && ncp tests/helpers compiled_tests/helpers"
+    "build:tests": "babel tests/ --out-dir compiled_tests/ && rimraf compiled_tests/helpers && ncp tests/helpers compiled_tests/helpers && node scripts/createSpecialDirectory.js"
   },
   "dependencies": {
     "globby": "^7.1.1",
@@ -34,12 +34,14 @@
     "babel-cli": "^6.8.0",
     "babel-preset-es2015": "^6.6.0",
     "chai": "^3.4.0",
-    "eslint": "^2.9.0",
     "enhanced-resolve": "^3.4.1",
+    "eslint": "^2.9.0",
+    "is-gzip": "^2.0.0",
+    "mkdirp": "^0.5.1",
     "mocha": "^2.4.5",
     "ncp": "^2.0.0",
-    "standard-version": "^4.2.0",
-    "is-gzip": "^2.0.0"
+    "rimraf": "^2.6.2",
+    "standard-version": "^4.2.0"
   },
   "homepage": "https://github.com/webpack-contrib/copy-webpack-plugin",
   "bugs": "https://github.com/webpack-contrib/copy-webpack-plugin/issues",

scripts/createSpecialDirectory.js

@@ -0,0 +1,20 @@
+const mkdirp = require('mkdirp');
+const path = require('path');
+const fs = require('fs');
+const removeIllegalCharacterForWindows = require('./removeIllegalCharacterForWindows');
+
+const baseDir = 'compiled_tests/helpers';
+
+const specialFiles = {
+    '[special?directory]/nested/nestedfile.txt': '',
+    '[special?directory]/(special-*file).txt': 'special',
+    '[special?directory]/directoryfile.txt': 'new'
+};
+
+Object.keys(specialFiles).forEach(function (originFile) {
+    const file = removeIllegalCharacterForWindows(originFile);
+    const dir = path.dirname(file);
+    mkdirp.sync(path.join(baseDir, dir));
+    fs.writeFileSync(path.join(baseDir, file), specialFiles[originFile]);
+});
+

scripts/removeIllegalCharacterForWindows.js

@@ -0,0 +1,6 @@
+const path = require('path');
+
+module.exports = function (string) {
+    return path.sep === '/' ? string : string.replace(/[*?"<>|]/g, '');
+};
+

src/utils/escape.js

@@ -8,7 +8,7 @@ export default function escape(context, from) {
         // Handles special characters in paths
         const absoluteContext = path.resolve(context)
             .replace(/\\/, '/')
-            .replace(/[\*|\?|\!|\(|\)|\[|\]|\{|\}]/g, (substring) => `\\${substring}`);
+            .replace(/[\*|\?|\!|\(|\)|\[|\{|\}]/g, (substring) => `[${substring}]`);
 
         if (!from) {
             return absoluteContext;

tests/index.js

@@ -15,6 +15,8 @@ import cacache from 'cacache';
 import isGzip from 'is-gzip';
 import zlib from 'zlib';
 
+import removeIllegalCharacterForWindows from '../scripts/removeIllegalCharacterForWindows';
+
 const BUILD_DIR = path.join(__dirname, 'build');
 const HELPER_DIR = path.join(__dirname, 'helpers');
 const TEMP_DIR = path.join(__dirname, 'tempdir');
@@ -58,6 +60,13 @@ describe('apply function', () => {
     // Ideally we pass in patterns and confirm the resulting assets
     const run = (opts) => {
         return new Promise((resolve, reject) => {
+            if (Array.isArray(opts.patterns)) {
+                opts.patterns.forEach(function (pattern) {
+                    if (pattern.context) {
+                        pattern.context = removeIllegalCharacterForWindows(pattern.context);
+                    }
+                });
+            }
             const plugin = CopyWebpackPlugin(opts.patterns, opts.options);
 
             // Get a mock compiler to pass to plugin.apply
@@ -109,7 +118,7 @@ describe('apply function', () => {
         return run(opts)
             .then((compilation) => {
                 if (opts.expectedAssetKeys && opts.expectedAssetKeys.length > 0) {
-                    expect(compilation.assets).to.have.all.keys(opts.expectedAssetKeys);
+                    expect(compilation.assets).to.have.all.keys(opts.expectedAssetKeys.map(removeIllegalCharacterForWindows));
                 } else {
                     expect(compilation.assets).to.deep.equal({});
                 }
@@ -451,7 +460,7 @@ describe('apply function', () => {
                 ],
                 patterns: [{
                     from: '*/*.*',
-                    test: /([^\/]+)\/([^\/]+)\.\w+$/,
+                    test: `([^\\${path.sep}]+)\\${path.sep}([^\\${path.sep}]+)\\.\\w+$`,
                     to: '[1]-[2].[ext]'
                 }]
             })
@@ -984,7 +993,7 @@ describe('apply function', () => {
                     'nested/nestedfile.txt'
                 ],
                 patterns: [{
-                    from: '[special?directory]'
+                    from: (path.sep === '/' ? '[special?directory]' : '[specialdirectory]')
                 }]
             })
             .then(done)
@@ -1390,7 +1399,7 @@ describe('apply function', () => {
                         'noextension'
                     ],
                     options: {
-                        ignore: ['directory/**/*', '\\[special\\?directory\\]/**/*']
+                        ignore: ['directory/**/*', (path.sep === '/' ? '\\[special\\?directory\\]/**/*' : '[[]specialdirectory]/**/*')]
                     },
                     patterns: [{
                         from: '.'