Gonzales 3.0: Update space-after-selector-delimiter

Author: tonyganch

lib/options/space-after-selector-delimiter.js

@@ -1,3 +1,5 @@
+var gonzales = require('gonzales-pe');
+
 module.exports = {
     name: 'space-after-selector-delimiter',
 
@@ -13,23 +15,23 @@ 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-after-selector-delimiter');
 
-        for (var i = node.length; i--;) {
-            if (node[i][0] !== 'delim') continue;
+        for (var i = node.content.length; i--;) {
+            if (!node.content[i].is('delim')) continue;
 
-            if (node[i + 1][0] === 's') {
-                node[i + 1][1] = value;
-            } else if (node[i + 1][1][0] === 's') {
-                node[i + 1][1][1] = value;
+            if (node.content[i + 1].is('s')) {
+                node.content[i + 1].content = value;
+            } else if (node.content[i + 1].content[0].is('s')) {
+                node.content[i + 1].content[0].content = value;
             } else {
-                node[i + 1].splice(1, 0, ['s', value]);
+                var space = gonzales.createNode({ type: 's', content: value });
+                node.content[i + 1].content.unshift(space);
             }
         }
     },

test/options/space-after-selector-delimiter-sass/test.js

@@ -1,4 +1,4 @@
-describe.skip('options/space-after-selector-delimiter (sass):', function() {
+describe('options/space-after-selector-delimiter (sass):', function() {
     it('Issue 238', function() {
         this.comb.configure({ 'space-after-selector-delimiter': '\n' });
         this.shouldBeEqual('issue-238.sass', 'issue-238.expected.sass');

test/options/space-after-selector-delimiter/test.js

@@ -1,4 +1,4 @@
-describe.skip('options/space-after-selector-delimiter:', function() {
+describe('options/space-after-selector-delimiter:', function() {
     it('Array value => should not change anything', function() {
         this.comb.configure({ 'space-after-selector-delimiter': ['', ' '] });
         this.shouldBeEqual('test.css');
@@ -29,47 +29,47 @@ describe.skip('options/space-after-selector-delimiter:', function() {
         this.shouldBeEqual('test.css', 'test-3.expected.css');
     });
 
-    it('Should detect no whitespace', function() {
+    it.skip('Should detect no whitespace', function() {
         this.shouldDetect(
             ['space-after-selector-delimiter'],
             'a,b{top:0}',
             { 'space-after-selector-delimiter': '' }
         );
     });
 
-    it('Should detect whitespace', function() {
+    it.skip('Should detect whitespace', function() {
         this.shouldDetect(
             ['space-after-selector-delimiter'],
             'a, \n b {top:0}',
             { 'space-after-selector-delimiter': ' \n ' }
         );
     });
 
-    it('Should detect no whitespace (2 blocks)', function() {
+    it.skip('Should detect no whitespace (2 blocks)', function() {
         this.shouldDetect(
             ['space-after-selector-delimiter'],
             'a,b{top:0} a, b{left:0}',
             { 'space-after-selector-delimiter': '' }
         );
     });
 
-    it('Should detect whitespace (2 blocks)', function() {
+    it.skip('Should detect whitespace (2 blocks)', function() {
         this.shouldDetect(
             ['space-after-selector-delimiter'],
             'a, b {top:0} b,a{left:0}',
             { 'space-after-selector-delimiter': ' ' }
         );
     });
 
-    it('Should detect no whitespace (3 blocks)', function() {
+    it.skip('Should detect no whitespace (3 blocks)', function() {
         this.shouldDetect(
             ['space-after-selector-delimiter'],
             'a, b{top:0} b,c{left:0} c,d{right:0}',
             { 'space-after-selector-delimiter': '' }
         );
     });
 
-    it('Should detect whitespace (3 blocks)', function() {
+    it.skip('Should detect whitespace (3 blocks)', function() {
         this.shouldDetect(
             ['space-after-selector-delimiter'],
             'a,b{top:0} b, c{left:0} c, sd{right:0}',