Gonzales 3.0: Update space-before-combinator

Author: tonyganch

lib/options/space-before-combinator.js

@@ -1,3 +1,5 @@
+var gonzales = require('gonzales-pe');
+
 module.exports = {
     name: 'space-before-combinator',
 
@@ -13,22 +15,22 @@ module.exports = {
     /**
      * Processes tree node.
      *
-     * @param {String} nodeType
      * @param {node} node
      */
-    process: function(nodeType, node) {
-        if (nodeType !== 'selector') return;
+    process: function(node) {
+        if (!node.is('selector')) return;
 
         var value = this.getValue('space-before-combinator');
 
         for (var i = node.length; i--;) {
-            var subSelector = node[i];
+            var subSelector = node.get(i);
             for (var j = subSelector.length; j--;) {
-                if (subSelector[j][0] !== 'combinator') continue;
-                if (subSelector[j - 1][0] === 's') {
-                    subSelector[j - 1][1] = value;
+                if (!subSelector.get(j).is('combinator')) continue;
+                if (subSelector.get(j - 1).is('s')) {
+                    subSelector.get(j - 1).content = value;
                 } else {
-                    subSelector.splice(j, 0, ['s', value]);
+                    var space = gonzales.createNode({ type: 's', content: value });
+                    subSelector.insert(j, space);
                 }
             }
         }

test/options/space-before-combinator/test.js

@@ -1,4 +1,4 @@
-describe.skip('options/space-before-combinator:', function() {
+describe('options/space-before-combinator:', function() {
     it('Array value => should not change anything', function() {
         this.comb.configure({ 'space-before-combinator': ['', ' '] });
         this.shouldBeEqual('test.css');
@@ -29,47 +29,47 @@ describe.skip('options/space-before-combinator:', function() {
         this.shouldBeEqual('test.css', 'test-3.expected.css');
     });
 
-    it('Should detect no whitespaces before combinator', function() {
+    it.skip('Should detect no whitespaces before combinator', function() {
         this.shouldDetect(
             ['space-before-combinator'],
             'a+b { color:red }',
             { 'space-before-combinator': '' }
         );
     });
 
-    it('Should detect a space before combinator', function() {
+    it.skip('Should detect a space before combinator', function() {
         this.shouldDetect(
             ['space-before-combinator'],
             'a + b { color:red }',
             { 'space-before-combinator': ' ' }
         );
     });
 
-    it('Should detect a space before combinator in long selector', function() {
+    it.skip('Should detect a space before combinator in long selector', function() {
         this.shouldDetect(
             ['space-before-combinator'],
             'a + b ~ c>d { color:red }',
             { 'space-before-combinator': ' ' }
         );
     });
 
-    it('Should detect a space before combinator in long selector, test 2', function() {
+    it.skip('Should detect a space before combinator in long selector, test 2', function() {
         this.shouldDetect(
             ['space-before-combinator'],
             'a>b + c + d { color:red }',
             { 'space-before-combinator': ' ' }
         );
     });
 
-    it('Should detect no whitespaces before combinator in long selector', function() {
+    it.skip('Should detect no whitespaces before combinator in long selector', function() {
         this.shouldDetect(
             ['space-before-combinator'],
             'a+b ~ c+d { color:red }',
             { 'space-before-combinator': '' }
         );
     });
 
-    it('Shouldn’t detect whitespaces before combinator in selector without combinators', function() {
+    it.skip('Shouldn’t detect whitespaces before combinator in selector without combinators', function() {
         this.shouldDetect(
             ['space-before-combinator'],
             'a { color:red }',