tests/all-rules.js: minor refactoring.

Author: XhmikosR

tests/all-rules.js

@@ -8,8 +8,6 @@
  * to fail due to Java stack overflow. This must be run separate from other tests.
  */
 
-/* jshint loopfunc:true */
-
 (function(){
     "use strict";
     var Assert = YUITest.Assert,
@@ -18,48 +16,48 @@
         len    = rules.length,
         i;
 
-    for (i=0; i < len; i++){
-
-        (function(i, rules){
+    function testAll(i, rules){
 
-            suite.add(new YUITest.TestCase({
+        suite.add(new YUITest.TestCase({
 
-                name: "General Tests for " + rules[i].id,
+            name: "General Tests for " + rules[i].id,
 
-                setUp: function(){
-                    this.options = {};
-                    this.options[rules[i].id] = 1;
-                },
+            setUp: function(){
+                this.options = {};
+                this.options[rules[i].id] = 1;
+            },
 
-                "Using @viewport should not result in an error": function(){
-                    var result = CSSLint.verify("@viewport { width: auto; }", this.options);
-                    Assert.areEqual(0, result.messages.length);
-                },
+            "Using @viewport should not result in an error": function(){
+                var result = CSSLint.verify("@viewport { width: auto; }", this.options);
+                Assert.areEqual(0, result.messages.length);
+            },
 
-                "Using @keyframes should not result in an error": function(){
-                    var result = CSSLint.verify("@keyframes resize { 0% {padding: 0;} 50% {padding: 0;} 100% {padding: 0;}}", this.options);
-                    Assert.areEqual(0, result.messages.length);
-                },
+            "Using @keyframes should not result in an error": function(){
+                var result = CSSLint.verify("@keyframes resize { 0% {padding: 0;} 50% {padding: 0;} 100% {padding: 0;}}", this.options);
+                Assert.areEqual(0, result.messages.length);
+            },
 
-                "Using @page should not result in an error": function(){
-                    var result = CSSLint.verify("@page { width: 100px; }", this.options);
-                    Assert.areEqual(0, result.messages.length);
-                },
+            "Using @page should not result in an error": function(){
+                var result = CSSLint.verify("@page { width: 100px; }", this.options);
+                Assert.areEqual(0, result.messages.length);
+            },
 
-                "Using @page @top-left should not result in an error": function(){
-                    var result = CSSLint.verify("@page { @top-left { content: ''; } }", this.options);
-                    Assert.areEqual(0, result.messages.length);
-                },
+            "Using @page @top-left should not result in an error": function(){
+                var result = CSSLint.verify("@page { @top-left { content: ''; } }", this.options);
+                Assert.areEqual(0, result.messages.length);
+            },
 
-                "Using a regular rule should not result in an error": function(){
-                    var result = CSSLint.verify("body { margin: 0; }", this.options);
-                    Assert.areEqual(0, result.messages.length);
-                }
+            "Using a regular rule should not result in an error": function(){
+                var result = CSSLint.verify("body { margin: 0; }", this.options);
+                Assert.areEqual(0, result.messages.length);
+            }
 
-            }));
+        }));
 
-        })(i, rules);
+    }
 
+    for (i = 0; i < len; i++){
+        testAll(i, rules);
     }
 
     YUITest.TestRunner.add(suite);