Merge pull request #129 from csscomb/tg/102-chainable-methods

Pass config to `new Comb()` and make it chainable (#102)

Author: mishanga

lib/csscomb.js

@@ -11,7 +11,7 @@ var doNothing = function() {};
  * @constructor
  * @name Comb
  */
-var Comb = function() {
+var Comb = function(config) {
     this.SUPPORTED_SYNTAXES = ['css', 'scss', 'less'];
     this._options = [
         'remove-empty-rulesets',
@@ -34,6 +34,17 @@ var Comb = function() {
     ];
     this._exclude = null;
     this._detect = false;
+
+    // If config was passed, configure:
+    if (typeof config === 'string' &&
+        ['csscomb', 'zen', 'yandex'].indexOf(config) > -1) {
+        config = require('../config/' + config + '.json');
+    }
+
+    if (typeof config === 'object') this.configure(config);
+
+    // Return Comb's object to make creating new instance chainable:
+    return this;
 };
 
 Comb.prototype = {
@@ -87,6 +98,9 @@ Comb.prototype = {
         this.changed = 0;
         this._verbose = config.verbose;
         this._lint = config.lint;
+
+        // Return Comb's object to make the method chainable:
+        return this;
     },
 
     /**

test/configure.js

@@ -0,0 +1,48 @@
+var Comb = require('../lib/csscomb');
+var assert = require('assert');
+
+describe('csscomb methods', function() {
+    var comb;
+    var input;
+    var output;
+    var expected;
+
+    it('Passing no config to constructor should not configure anything', function() {
+        comb = new Comb();
+        assert.equal(undefined, comb._handlers);
+    });
+
+    it('Passing valid config name to constructor should configure using correct config', function() {
+        comb = new Comb('zen');
+        input = 'a { color: tomato; top: 0; }';
+        expected = 'a {top: 0;  color: tomato; }';
+        output = comb.processString(input);
+
+        assert.equal(expected, output);
+    });
+
+    it('Passing config object to constructor should configure using that object', function() {
+        comb = new Comb({ 'always-semicolon': true });
+        input = 'a { color: tomato }';
+        expected = 'a { color: tomato; }';
+        output = comb.processString(input);
+
+        assert.equal(expected, output);
+    });
+
+    it('new Comb() should be chainable', function() {
+        input = 'a { color: tomato; top: 0; }';
+        expected = 'a {top: 0;  color: tomato; }';
+        output = new Comb('zen').processString(input);
+
+        assert.equal(expected, output);
+    });
+
+    it('configure() should be chainable', function() {
+        input = 'a { color: tomato }';
+        expected = 'a { color: tomato; }';
+        output = new Comb().configure({ 'always-semicolon': true }).processString(input);
+
+        assert.equal(expected, output);
+    });
+});