CLI flags override config file options

Author: giuseppeg

CHANGELOG.md

@@ -1,5 +1,6 @@
 ### HEAD
 
+* CLI flags override config file options
 * Input file is now linted (previously only imported files were passed through
   the linting tools)
 * Add `--throw-error` (`-e`) to CLI

bin/suitcss

@@ -109,11 +109,11 @@ function run() {
   read(input, function(err, buffer) {
     if (err) logger.throw(err);
     var css = buffer.toString();
-    var opts = assign({}, {
+    var opts = assign({}, config, {
       minify: program.minify,
       root: program.importRoot,
       lint: program.lint
-    }, config);
+    });
 
     if (program.throwError) {
       assign(opts, {

test/cli.js

@@ -86,6 +86,16 @@ describe('cli', function() {
     });
   });
 
+  it('should allow to override config options via cli flags', function(done) {
+    exec('node bin/suitcss -L -c test/config/test.js test/fixtures/import.css test/fixtures/cli/output.css', function(err) {
+      if (err) return done(err);
+      var res = util.read('fixtures/cli/output');
+      var commentsPattern = /\/\*[^]*?\*\//g;
+      expect(res).to.match(commentsPattern);
+      done();
+    });
+  });
+
   it('should output an error to stderr on conformance failure when --throw-error is set', function(done) {
     exec('node bin/suitcss -i test/fixtures -e test/fixtures/import-error.css test/fixtures/cli/output.css', function(err, stdout, stderr) {
       expect(err).to.be.an('error');

test/config/test.js

@@ -1,4 +1,5 @@
 module.exports = {
+  lint: true,
   use: [
     "postcss-property-lookup"
   ],