fix(files): Fix regression where strings from the CLI files option ar…

…e no-longer exploded if they enter as an array - fixes #1076
BrowserSync · Apr 25, 2016 · 5333848 · 5333848
1 parent f1baae1
commit 5333848
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 1 deletion.
diff --git a/lib/cli/cli-options.js b/lib/cli/cli-options.js
@@ -297,7 +297,9 @@ opts.makeFilesArg = function (value) {
     if (isList(value) && value.size) {
         value.forEach(function (value) {
             if (_.isString(value)) {
-                globs.push(value);
+                globs = globs.concat(
+                    opts.utils.explodeFilesArg(value)
+                );
             } else {
                 if (isMap(value)) {
                     objs.push(value);

diff --git a/test/specs/cli/cli.options.files.js b/test/specs/cli/cli.options.files.js
@@ -13,6 +13,13 @@ describe("CLI: Options: Merging Options: Files", function () {
             objs: []
         });
     });
+    it("should return the files property from Array given with strings in legacy format", function () {
+        var imm = merge({files: ["css/*.css,*.html"]});
+        assert.deepEqual(imm.get("files").get("core").toJS(), {
+            globs: ["css/*.css", "*.html"],
+            objs: []
+        });
+    });
     it("should return the files property from array given", function () {
         var imm = merge({files: ["css/*.css", "*.html"]});
         assert.deepEqual(imm.get("files").get("core").toJS(), {