Gonzales 3.0: Update element-case option

Author: tonyganch

lib/options/element-case.js

@@ -7,25 +7,24 @@ module.exports = {
 
     /**
      * Processes tree node.
-     * @param {String} nodeType
      * @param {node} node
      */
-    process: function(nodeType, node) {
-        if (nodeType !== 'selector' &&
-            nodeType !== 'arguments') return;
-
-        for (var x = node.length; x--;) {
-            var selector = node[x];
-            if (selector[0] !== 'simpleselector') continue;
-
-            for (var i = selector.length; i--;) {
-                var simpleselector = selector[i];
-                if (!Array.isArray(simpleselector) ||
-                    simpleselector[0] !== 'ident') continue;
-
-                simpleselector[1] = this.getValue('element-case') === 'lower' ?
-                    simpleselector[1].toLowerCase() :
-                    simpleselector[1].toUpperCase();
+    process: function(node) {
+        if (node.type !== 'selector' &&
+            node.type !== 'arguments') return;
+
+        for (var x = node.content.length; x--;) {
+            var selector = node.content[x];
+            if (selector.type !== 'simpleselector') continue;
+
+            for (var i = selector.content.length; i--;) {
+                var simpleselector = selector.content[i];
+                if (!simpleselector ||
+                    simpleselector.type !== 'ident') continue;
+
+                simpleselector.content = this.getValue('element-case') === 'lower' ?
+                    simpleselector.content.toLowerCase() :
+                    simpleselector.content.toUpperCase();
             }
         }
     },

test/options/element-case-scss/test.js

@@ -1,4 +1,4 @@
-describe.skip('options/element-case (scss):', function() {
+describe('options/element-case (scss):', function() {
     it('Should not touch mixin names', function() {
         this.comb.configure({ 'element-case': 'lower' });
         this.shouldBeEqual('mixin.scss', 'mixin.expected.scss');

test/options/element-case/test.js

@@ -1,6 +1,6 @@
 var assert = require('assert');
 
-describe.skip('options/element-case', function() {
+describe('options/element-case', function() {
     it('Invalid String should not change case of elements', function() {
         this.comb.configure({ 'element-case': 'foobar' });
         assert.equal(
@@ -41,7 +41,7 @@ describe.skip('options/element-case', function() {
         );
     });
 
-    it('Should detect lowercase elements', function() {
+    it.skip('Should detect lowercase elements', function() {
         this.shouldDetect(
             ['element-case'],
             'a { color: red }',
@@ -51,7 +51,7 @@ describe.skip('options/element-case', function() {
         );
     });
 
-    it('Should detect uppercase elements', function() {
+    it.skip('Should detect uppercase elements', function() {
         this.shouldDetect(
             ['element-case'],
             'A { color: red }',
@@ -61,7 +61,7 @@ describe.skip('options/element-case', function() {
         );
     });
 
-    it('Should detect lowercase elements in a long selector', function() {
+    it.skip('Should detect lowercase elements in a long selector', function() {
         this.shouldDetect(
             ['element-case'],
             'ul li:not(:hover) A { color: red }',
@@ -71,7 +71,7 @@ describe.skip('options/element-case', function() {
         );
     });
 
-    it('Should detect uppercase elements in a long selector', function() {
+    it.skip('Should detect uppercase elements in a long selector', function() {
         this.shouldDetect(
             ['element-case'],
             'ul .lol:not(LI) A { color: red }',
@@ -81,15 +81,15 @@ describe.skip('options/element-case', function() {
         );
     });
 
-    it('Shouldn’t detect case of elements in a mixed case', function() {
+    it.skip('Shouldn’t detect case of elements in a mixed case', function() {
         this.shouldDetect(
             ['element-case'],
             'aRtIcLe { color: red }',
             {}
         );
     });
 
-    it('Shouldn’t detect case of elements when there are no such', function() {
+    it.skip('Shouldn’t detect case of elements when there are no such', function() {
         this.shouldDetect(
             ['element-case'],
             '*.lol { color: red }',