Some refactoring — setters updated

Readme and tests fixed:
* block-indent
* rule-indent
* stick-brace

Author: mishanga

README.md

@@ -61,7 +61,7 @@ Example configuration:
 
 ### verbose
 
-Available value: `{Boolean}` true
+Available value: `{Boolean}` `true`
 
 Config mode: `{ "verbose": true }`
 ```bash
@@ -80,7 +80,7 @@ $ ./bin/csscomb ./test -v
 
 ### always-semicolon
 
-Available value: `{Boolean}` true
+Available value: `{Boolean}` `true`
 
 Example: `{ "always-semicolon": true }`
 
@@ -97,9 +97,9 @@ a { color: red; }
 **Note**: better to use with [rule-indent](#rule-indent)
 
 Available values:
-  * `{Boolean}` true (means 4 spaces)
+  * `{Boolean}` `true` (means 4 spaces)
   * `{Number}` of spaces
-  * `{String}` of whitespace characters (`/[ \t]*/`)
+  * `{String}` of whitespace characters (`/[ \t]+/`)
 
 Example: `{ "block-indent": 2 }`
 
@@ -120,7 +120,7 @@ a { color: red
 ### colon-space
 
 Available values:
-  * `{Boolean}` true (means `after`)
+  * `{Boolean}` `true` (means `after`)
   * `{String}`: `before`, `after`, `both` or any combination of whitespaces
   and/or a colon (` `, `: `, `\t:\n\t` etc.)
 
@@ -270,9 +270,9 @@ p { padding: .5em }
 **Note**: better to use with [block-indent](#block-indent)
 
 Available values:
-  * `{Boolean}` true (means 4 spaces)
+  * `{Boolean}` `true` (means 4 spaces)
   * `{Number}` of spaces
-  * `{String}` of whitespace characters (`/[ \t]*/`)
+  * `{String}` of whitespace characters (`/[ \t]+/`)
 
 Example: `{ "rule-indent": 2 }`
 
@@ -334,8 +334,9 @@ p {
 ### stick-brace
 
 Available values:
-  * `{Boolean}` true (means 1 spaces)
-  * `{String}` of whitespace characters (`/[ \t\n]*/`)
+  * `{Boolean}` `true` (means 1 space)
+  * `{Number}` of spaces
+  * `{String}` of whitespace characters (`/[ \t\n]+/`)
 
 Example: `{ "stick-brace": "\n" }`
 
@@ -350,7 +351,7 @@ a
 
 ### strip-spaces
 
-Available value: `{Boolean}` true
+Available value: `{Boolean}` `true`
 
 Example: `{ "strip-spaces": true }`
 

lib/options/block-indent.js

@@ -7,12 +7,12 @@ module.exports = {
      * @returns {Object|undefined}
      */
     setValue: function(value) {
-        this._value = false;
-        if (value === true) value = 4;
-        if (typeof value === 'number' && value === Math.abs(Math.round(value))) value = new Array(value + 1).join(' ');
-        if (typeof value === 'string' && value.match(/^[ \t]*$/)) this._value = value;
-        if (!this._value) return;
-        return this;
+        delete this._value;
+        if (value === true) this._value = '    ';
+        if (typeof value === 'number' && value === Math.abs(Math.round(value)))
+            this._value = new Array(value + 1).join(' ');
+        if (typeof value === 'string' && value.match(/^[ \t]+$/)) this._value = value;
+        if (typeof this._value === 'string') return this;
     },
 
     /**

lib/options/rule-indent.js

@@ -7,12 +7,12 @@ module.exports = {
      * @returns {Object|undefined}
      */
     setValue: function(value) {
-        this._value = false;
-        if (value === true) value = 4;
-        if (typeof value === 'number' && value === Math.abs(Math.round(value))) value = new Array(value + 1).join(' ');
-        if (typeof value === 'string' && value.match(/^[ \t]*$/)) this._value = value;
-        if (!this._value) return;
-        return this;
+        delete this._value;
+        if (value === true) this._value = '    ';
+        if (typeof value === 'number' && value === Math.abs(Math.round(value)))
+            this._value = new Array(value + 1).join(' ');
+        if (typeof value === 'string' && value.match(/^[ \t]+$/)) this._value = value;
+        if (typeof this._value === 'string') return this;
     },
 
     /**

lib/options/stick-brace.js

@@ -7,11 +7,12 @@ module.exports = {
      * @returns {Object|undefined}
      */
     setValue: function(value) {
-        this._value = false;
-        if (value === true) value = ' ';
-        if (typeof value === 'string' && value.match(/^[ \t\n]*$/)) this._value = value;
-        if (!this._value) return;
-        return this;
+        delete this._value;
+        if (value === true) this._value = ' ';
+        if (typeof value === 'number' && value === Math.abs(Math.round(value)))
+            this._value = new Array(value + 1).join(' ');
+        if (typeof value === 'string' && value.match(/^[ \t\n]+$/)) this._value = value;
+        if (typeof this._value === 'string') return this;
     },
 
     /**

test/rule-indent.js

@@ -6,14 +6,14 @@ describe('options/rule-indent', function() {
     beforeEach(function() {
         comb = new Comb();
     });
-    it('Invalid Number value should not change space after brace', function() {
+    it('Invalid Number value should not change rule indent', function() {
         comb.configure({ 'rule-indent': 3.5 });
         assert.equal(
             comb.processString('a {\n color: red }'),
             'a {\n color: red }'
         );
     });
-    it('Invalid String value should not change space after brace', function() {
+    it('Invalid String value should not change rule indent', function() {
         comb.configure({ 'rule-indent': 'foobar' });
         assert.equal(
             comb.processString('a {\n color: red }'),
@@ -27,14 +27,14 @@ describe('options/rule-indent', function() {
             'a {\n    color: red }'
         );
     });
-    it('Valid Number value should set equal space after brace', function() {
+    it('Valid Number value should set equal space indent', function() {
         comb.configure({ 'rule-indent': 3 });
         assert.equal(
             comb.processString('a {\n color: red }'),
             'a {\n   color: red }'
         );
     });
-    it('Valid String value should set equal space after brace', function() {
+    it('Valid String value should set equal indent', function() {
         comb.configure({ 'rule-indent': '\t' });
         assert.equal(
             comb.processString(

test/stick-brace.js

@@ -6,21 +6,28 @@ describe('options/stick-brace', function() {
     beforeEach(function() {
         comb = new Comb();
     });
-    it('Invalid String should not change space after brace', function() {
+    it('Invalid String should not change space before brace', function() {
         comb.configure({ 'stick-brace': 'foobar' });
         assert.equal(
             comb.processString('a { color: red }'),
             'a { color: red }'
         );
     });
-    it('True Boolean value should set 1 space after brace', function() {
+    it('True Boolean value should set 1 space before brace', function() {
         comb.configure({ 'stick-brace': true });
         assert.equal(
             comb.processString('a{color:red }'),
             'a {color:red }'
         );
     });
-    it('Valid String value should set equal space after brace', function() {
+    it('Valid Number value should set equal space before brace', function() {
+        comb.configure({ 'stick-brace': 0 });
+        assert.equal(
+            comb.processString('a {color:red }'),
+            'a{color:red }'
+        );
+    });
+    it('Valid String value should set equal space before brace', function() {
         comb.configure({ 'stick-brace': '\n' });
         assert.equal(
             comb.processString(