fix(cli): Allow absolute paths for config file - fixes #583

BrowserSync · Apr 29, 2015 · 8fcd904 · 8fcd904
1 parent 8b4d707
commit 8fcd904
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 3 deletions.
diff --git a/lib/cli/cli-info.js b/lib/cli/cli-info.js
@@ -25,7 +25,7 @@ var info = {
      * @param filePath
      */
     getConfigFile: function (filePath) {
-        return require(path.resolve(process.cwd() + "/" + filePath));
+        return require(path.resolve(process.cwd(), filePath));
     },
     /**
      * Generate an example Config file.

diff --git a/test/specs/cli/cli.get.config.js b/test/specs/cli/cli.get.config.js
@@ -1,7 +1,8 @@
 "use strict";
 
-var info           = require("../../../lib/cli/cli-info");
-var assert         = require("chai").assert;
+var info   = require("../../../lib/cli/cli-info");
+var assert = require("chai").assert;
+var path   = require("path");
 
 describe("When reading a config file from the file system", function () {
 
@@ -14,4 +15,10 @@ describe("When reading a config file from the file system", function () {
     it("should not throw if the file is found", function () {
         assert.isDefined(info.getConfigFile("test/fixtures/config/si-config.js"));
     });
+
+    it("should not throw if absolute path given", function () {
+
+        var configPath = path.resolve("test/fixtures/config/si-config.js");
+        assert.isDefined(info.getConfigFile(configPath));
+    });
 });