Gonzales 3.0: Update space-before-colon

Author: tonyganch

lib/options/space-before-colon.js

@@ -1,3 +1,5 @@
+var gonzales = require('gonzales-pe');
+
 module.exports = {
     name: 'space-before-colon',
 
@@ -13,26 +15,26 @@ module.exports = {
     /**
      * Processes tree node.
      *
-     * @param {String} nodeType
      * @param {node} node
      */
-    process: function(nodeType, node) {
-        if (nodeType !== 'declaration') return;
+    process: function(node) {
+        if (!node.is('declaration')) return;
 
         var value = this.getValue('space-before-colon');
 
-        for (var i = node.length; i--;) {
-            if (node[i][0] !== 'propertyDelim') continue;
+        for (var i = node.content.length; i--;) {
+            if (!node.content[i].is('propertyDelim')) continue;
 
-            if (this.getSyntax() === 'sass' && !node[i - 1]) break;
+            if (this.getSyntax() === 'sass' && !node.content[i - 1]) break;
 
             // Remove any spaces before colon:
-            if (node[i - 1][0] === 's') {
-                node.splice(i - 1, 1);
+            if (node.content[i - 1].is('s')) {
+                node.content.splice(i - 1, 1);
                 i--;
             }
             // If the value set in config is not empty, add spaces:
-            if (value !== '') node.splice(i, 0, ['s', value]);
+            var space = gonzales.createNode({ type: 's', content: value });
+            if (value !== '') node.content.splice(i, 0, space);
 
             break;
         }

test/options/space-before-colon-sass/test.js

@@ -1,4 +1,4 @@
-describe.skip('options/space-before-colon-sass:', function() {
+describe('options/space-before-colon-sass:', function() {
     it('Should correct space', function() {
         this.comb.configure({ 'space-before-colon': 1 });
         this.shouldBeEqual('test.sass', 'test.expected.sass');

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

@@ -1,4 +1,4 @@
-describe.skip('options/space-before-colon:', function() {
+describe('options/space-before-colon:', function() {
     it('Array value => should not change anything', function() {
         this.comb.configure({ 'space-before-colon': ['', ' '] });
         this.shouldBeEqual('test.css');
@@ -29,31 +29,31 @@ describe.skip('options/space-before-colon:', function() {
         this.shouldBeEqual('test.css', 'test-3.expected.css');
     });
 
-    it('Should detect no whitespaces', function() {
+    it.skip('Should detect no whitespaces', function() {
         this.shouldDetect(
             ['space-before-colon'],
             'a { color:red }',
             { 'space-before-colon': '' }
         );
     });
 
-    it('Should detect no space from two variants', function() {
+    it.skip('Should detect no space from two variants', function() {
         this.shouldDetect(
             ['space-before-colon'],
             'a { color: red; color :red }',
             { 'space-before-colon': '' }
         );
     });
 
-    it('Should detect no whitespaces along three variants', function() {
+    it.skip('Should detect no whitespaces along three variants', function() {
         this.shouldDetect(
             ['space-before-colon'],
             'a { color: red; background :red } b { width:10px }',
             { 'space-before-colon': '' }
         );
     });
 
-    it('Should detect space', function() {
+    it.skip('Should detect space', function() {
         this.shouldDetect(
             ['space-before-colon'],
             'a { color : red; background :red } b { width:10px }',