ruby_css_lint 0.0.1

data/csslint/tests/rules/universal-selector.js

@@ -0,0 +1,31 @@
+(function(){
+
+    /*global YUITest, CSSLint*/
+    var Assert = YUITest.Assert;
+
+    YUITest.TestRunner.add(new YUITest.TestCase({
+    
+        name: "Universal Selector Errors",
+
+        "Using a universal selector alone should result in a warning": function(){
+            var result = CSSLint.verify("* { font-size: 10px; }", {"universal-selector": 1 });
+            Assert.areEqual(1, result.messages.length);
+            Assert.areEqual("warning", result.messages[0].type);
+            Assert.areEqual("The universal selector (*) is known to be slow.", result.messages[0].message);
+        },
+
+        "Using a universal selector as the right-most part should result in a warning": function(){
+            var result = CSSLint.verify("p div * { font-size: 10px; }", {"universal-selector": 1 });
+            Assert.areEqual(1, result.messages.length);
+            Assert.areEqual("warning", result.messages[0].type);
+            Assert.areEqual("The universal selector (*) is known to be slow.", result.messages[0].message);
+        },
+
+        "Using a universal selector in the middle should not result in a warning": function(){
+            var result = CSSLint.verify("* .foo { font-size: 10px; } ", {"universal-selector": 1 });
+            Assert.areEqual(0, result.messages.length);
+        }
+        
+    }));
+
+})();

data/csslint/tests/rules/unqualified-attributes.js

@@ -0,0 +1,37 @@
+(function(){
+
+    /*global YUITest, CSSLint*/
+    var Assert = YUITest.Assert;
+
+    YUITest.TestRunner.add(new YUITest.TestCase({
+    
+        name: "Unqualified Attributes Errors",
+
+        "Using an unqualified attribute selector alone should result in a warning": function(){
+            var result = CSSLint.verify("[type=text] { font-size: 10px; }", {"unqualified-attributes": 1 });
+            Assert.areEqual(1, result.messages.length);
+            Assert.areEqual("warning", result.messages[0].type);
+            Assert.areEqual("Unqualified attribute selectors are known to be slow.", result.messages[0].message);
+        },
+
+        "Using an unqualified attribute selector as the right-most part should result in a warning": function(){
+            var result = CSSLint.verify("p div [type=text] { font-size: 10px; }", {"unqualified-attributes": 1 });
+            Assert.areEqual(1, result.messages.length);
+            Assert.areEqual("warning", result.messages[0].type);
+            Assert.areEqual("Unqualified attribute selectors are known to be slow.", result.messages[0].message);
+        },
+
+        "Using an unqualified attribute selector in the middle should not result in a warning": function(){
+            var result = CSSLint.verify("[type=text] .foo { font-size: 10px; } ", {"unqualified-attributes": 1 });
+            Assert.areEqual(0, result.messages.length);
+        },
+        
+        "Using a qualified attribute selector should not result in a warning": function(){
+            var result = CSSLint.verify("input[type=text]  { font-size: 10px; } ", {"unqualified-attributes": 1 });
+            Assert.areEqual(0, result.messages.length);
+        }
+        
+        
+    }));
+
+})();

data/csslint/tests/rules/vendor-prefix.js

@@ -0,0 +1,76 @@
+(function(){
+
+    /*global YUITest, CSSLint*/
+    var Assert = YUITest.Assert;
+
+    YUITest.TestRunner.add(new YUITest.TestCase({
+
+        name: "Vendor Prefix Errors",
+
+        "Using -moz-border-radius without border-radius should result in one warning": function(){
+            var result = CSSLint.verify("h1 {\n    -moz-border-radius: 5px; \n}", { "vendor-prefix": 1 });
+            Assert.areEqual(1, result.messages.length);
+            Assert.areEqual("warning", result.messages[0].type);
+            Assert.areEqual("Missing standard property 'border-radius' to go along with '-moz-border-radius'.", result.messages[0].message);
+            Assert.areEqual(2, result.messages[0].line);
+            Assert.areEqual(5, result.messages[0].col);
+        },
+
+        "Using -webkit-border-radius without border-radius should result in one warning": function(){
+            var result = CSSLint.verify("h1 { -webkit-border-radius: 5px; }", { "vendor-prefix": 1 });
+            Assert.areEqual(1, result.messages.length);
+            Assert.areEqual("warning", result.messages[0].type);
+            Assert.areEqual("Missing standard property 'border-radius' to go along with '-webkit-border-radius'.", result.messages[0].message);
+        },
+
+        "Using -o-border-radius without border-radius should result in one warning": function(){
+            var result = CSSLint.verify("h1 { -o-border-radius: 5px; }", { "vendor-prefix": 1 });
+            Assert.areEqual(1, result.messages.length);
+            Assert.areEqual("warning", result.messages[0].type);
+            Assert.areEqual("Missing standard property 'border-radius' to go along with '-o-border-radius'.", result.messages[0].message);
+        },
+
+        "Using -moz-border-radius after  border-radius should result in one warning": function(){
+            var result = CSSLint.verify("h1 { \nborder-radius: 5px; \n    -moz-border-radius: 5px; }", { "vendor-prefix": 1 });
+            Assert.areEqual(1, result.messages.length);
+            Assert.areEqual("warning", result.messages[0].type);
+            Assert.areEqual("Standard property 'border-radius' should come after vendor-prefixed property '-moz-border-radius'.", result.messages[0].message);
+            Assert.areEqual(3, result.messages[0].line);
+            Assert.areEqual(5, result.messages[0].col);
+            
+        },
+
+        "Using -webkit-border-bottom-left-radius with border-bottom-left-radius should not result in a warning.": function(){
+            var result = CSSLint.verify("h1 { -webkit-border-bottom-left-radius: 4px; border-bottom-left-radius: 4px;  }", { "vendor-prefix": 1 });
+            Assert.areEqual(0, result.messages.length);
+        },
+        
+        "Using -moz-border-radius-bottomleft should result in a warning.": function(){
+            var result = CSSLint.verify("h1 {  -moz-border-radius-bottomleft: 5px;  }", { "vendor-prefix": 1 });
+            Assert.areEqual(1, result.messages.length);
+            Assert.areEqual("warning", result.messages[0].type);
+            Assert.areEqual("Missing standard property 'border-bottom-left-radius' to go along with '-moz-border-radius-bottomleft'.", result.messages[0].message);
+        },
+        
+        "Using -moz-box-shadow should result in a warning.": function(){
+            var result = CSSLint.verify("h1 {  -moz-box-shadow: 5px;  }", { "vendor-prefix": 1 });
+            Assert.areEqual(1, result.messages.length);
+            Assert.areEqual("warning", result.messages[0].type);
+            Assert.areEqual("Missing standard property 'box-shadow' to go along with '-moz-box-shadow'.", result.messages[0].message);
+        },
+        
+        "Using -moz-user-select should result in a warning.": function(){
+            var result = CSSLint.verify("h1 {  -moz-user-select:none;  }", { "vendor-prefix": 1 });
+            Assert.areEqual(1, result.messages.length);
+            Assert.areEqual("warning", result.messages[0].type);
+            Assert.areEqual("Missing standard property 'user-select' to go along with '-moz-user-select'.", result.messages[0].message);
+        },
+        
+        "Using @font-face should not result in an error (#90)": function(){
+            var result = CSSLint.verify("@font-face { src:url('../fonts/UniversBold.otf');font-family:Univers;advancedAntiAliasing: true;}", { "vendor-prefix": 1 });
+            Assert.areEqual(0, result.messages.length);
+        }
+
+    }));
+
+})();

data/csslint/tests/rules/zero-units.js

@@ -0,0 +1,44 @@
+(function(){
+
+    /*global YUITest, CSSLint*/
+    var Assert = YUITest.Assert;
+
+    YUITest.TestRunner.add(new YUITest.TestCase({
+
+        name: "Zero Units Errors",
+
+        "Using 0px should result in one warning": function(){
+            var result = CSSLint.verify("h1 { left: 0px; }", { "zero-units": 1 });
+            Assert.areEqual(1, result.messages.length);
+            Assert.areEqual("warning", result.messages[0].type);
+            Assert.areEqual("Values of 0 shouldn't have units specified.", result.messages[0].message);
+        },
+
+        "Using 0em should result in one warning": function(){
+            var result = CSSLint.verify("h1 { left: 0em; }", { "zero-units": 1 });
+            Assert.areEqual(1, result.messages.length);
+            Assert.areEqual("warning", result.messages[0].type);
+            Assert.areEqual("Values of 0 shouldn't have units specified.", result.messages[0].message);
+        },
+
+        "Using 0% should result in one warning": function(){
+            var result = CSSLint.verify("h1 { left: 0%; }", { "zero-units": 1 });
+            Assert.areEqual(1, result.messages.length);
+            Assert.areEqual("warning", result.messages[0].type);
+            Assert.areEqual("Values of 0 shouldn't have units specified.", result.messages[0].message);
+        },
+
+        "Using 0 should not result in a warning": function(){
+            var result = CSSLint.verify("h1 { left: 0; }", { "zero-units": 1 });
+            Assert.areEqual(0, result.messages.length);
+        },
+        
+        "Using 0s for animation-duration should not result in a warning": function(){
+            var result = CSSLint.verify("h1 { animation-duration: 0s; }", { "zero-units": 1 });
+            Assert.areEqual(0, result.messages.length);
+        }
+        
+        
+    }));
+
+})();

data/csslint/tests/testrunner.htm

@@ -0,0 +1,138 @@
+<!DOCTYPE html>
+<html>
+<head>
+<title>YUI Test</title>
+
+<!-- JS -->
+<script src="../build/csslint.js"></script>
+<script src="../lib/yuitest.js"></script>
+<script src="../build/csslint-tests.js"></script>
+<script src="all-rules.js"></script>
+<style type="text/css">
+.passed {
+    color: green;
+}
+
+.failed, .error {
+    color: red;
+}
+
+.ignored {
+    color: silver;
+}
+</style>
+
+</head>
+<body>
+    <div id="doc">
+        <h1>YUI Test - Test Runner</h1>        
+        <button id="run">Run</button>
+
+        <h2>Results</h2>
+        <ul id="results"></ul>
+
+    </div>
+
+<script >
+(function(){
+
+    //some helpful variables
+    var runButton   = document.getElementById("run"),
+        resultsList = document.getElementById("results"),
+        resultNode  = resultsList,
+        events      = [
+                        YUITest.TestRunner.TEST_CASE_BEGIN_EVENT,
+                        YUITest.TestRunner.TEST_CASE_COMPLETE_EVENT,
+                        YUITest.TestRunner.TEST_SUITE_BEGIN_EVENT,
+                        YUITest.TestRunner.TEST_SUITE_COMPLETE_EVENT,
+                        YUITest.TestRunner.TEST_PASS_EVENT,
+                        YUITest.TestRunner.TEST_FAIL_EVENT,
+                        YUITest.TestRunner.TEST_IGNORE_EVENT,
+                        YUITest.TestRunner.COMPLETE_EVENT,
+                        YUITest.TestRunner.BEGIN_EVENT,
+                        YUITest.TestRunner.ERROR_EVENT
+                    ];
+
+    for (var i=0; i < events.length; i++){
+        YUITest.TestRunner.attach(events[i], function(event){
+                var node;
+
+                switch(event.type){
+                    case this.BEGIN_EVENT:
+                        message = "Testing began at " + (new Date()).toString() + ".";
+                        messageType = "info";
+                        break;
+
+                    case this.COMPLETE_EVENT:
+                        message = "Testing completed at " + (new Date()).toString() + ".\nPassed:" + 
+                            event.results.passed + " Failed:" + event.results.failed + " Total:" + event.results.total;
+                        messageType = "info";
+                        break;
+
+                    case this.TEST_FAIL_EVENT:
+                        node = document.createElement("li");
+                        node.className = "failed";
+                        node.innerHTML = event.testName + ": " + event.error.getMessage().replace(/\n/g, "<br>");
+                        resultNode.appendChild(node);
+                        break;
+
+                    case this.ERROR_EVENT:
+                        node = document.createElement("li");
+                        node.className = "error";
+                        node.innerHTML = "ERROR: " + event.methodName + "() caused an error: " + event.error.message.replace(/\n/g, "<br>");
+                        resultNode.appendChild(node);
+                        break;
+
+                    case this.TEST_IGNORE_EVENT:
+                        node = document.createElement("li");
+                        node.className = "ignored";
+                        node.innerHTML = event.testName;
+                        resultNode.appendChild(node);
+                        break;
+
+                    case this.TEST_PASS_EVENT:
+                        node = document.createElement("li");
+                        node.className = "passed";
+                        node.innerHTML = event.testName;
+                        resultNode.appendChild(node);
+                        break;
+
+                    case this.TEST_SUITE_BEGIN_EVENT:
+                        node = document.createElement("li");
+                        node.innerHTML = event.testSuite.name;
+                        resultNode.appendChild(node);
+                        resultNode = resultNode.appendChild(document.createElement("ul"));
+                        break;
+
+                    case this.TEST_CASE_COMPLETE_EVENT:
+                    case this.TEST_SUITE_COMPLETE_EVENT:
+                        resultNode.previousSibling.innerHTML += " (passed: " + event.results.passed + ", failed: " + event.results.failed + ", total: " + event.results.total + ", errors: " + event.results.errors +  ", ignored: " + event.results.ignored + ")";
+                        resultNode = resultNode.parentNode;
+                        break;
+
+                    case this.TEST_CASE_BEGIN_EVENT:
+                        node = document.createElement("li");
+                        node.innerHTML = event.testCase.name;
+                        resultNode.appendChild(node);
+                        resultNode = resultNode.appendChild(document.createElement("ul"));
+                        break;
+
+                }
+
+        });
+    }
+
+    runButton.onclick = function(){
+
+         //reset the interface
+        resultsList.innerHTML = "";
+        resultNode = resultsList;
+
+        YUITest.TestRunner.run();
+    };
+
+})();
+
+</script>
+</body>
+</html>

data/lib/ruby_css_lint.rb

@@ -0,0 +1,168 @@
+require 'tempfile'
+
+if defined?(Rails)
+  require 'rake'
+  module RubyCssLint
+    class Railtie < Rails::Railtie
+      extend Rake::DSL
+      rake_tasks do
+        namespace :css_lint do
+
+          task :run => :environment do |t|
+            RubyCssLint::construct_js_and_run_rhino(RubyCssLint::location_of_css_files(Rails.root))
+          end
+
+          task :generate_config => :environment do |t|
+            File.open("#{Rails.root.to_s}/config/initializers/css_lint.rb", "w") do |filehandle|
+              filehandle.puts <<-CSS_LINT_INIT
+module RubyCssLint
+#{RubyCssLint::DEFAULT_CONFIG}
+end
+CSS_LINT_INIT
+              
+            end
+            
+          end
+
+        end
+      end
+    end
+  end
+end
+
+module RubyCssLint
+  WARNING = 1
+  ERROR = 2
+  DONT_CARE = nil
+  
+  DEFAULT_CONFIG = <<-CSS_LINT_DEFAULT
+  def self.ruleset_classifications
+    {
+      "adjoining-classes" => RubyCssLint::WARNING,
+      "box-model" => RubyCssLint::WARNING,
+      "box-sizing" => RubyCssLint::WARNING,
+      "compatible-vendor-prefixes" => RubyCssLint::WARNING,
+      "display-property-grouping" => RubyCssLint::WARNING,
+      "duplicate-background-images" => RubyCssLint::WARNING,
+      "duplicate-properties" => RubyCssLint::WARNING,
+      "empty-rules" => RubyCssLint::WARNING,
+      "errors" => RubyCssLint::WARNING,
+      "fallback-colors" => RubyCssLint::WARNING,
+      "floats" => RubyCssLint::WARNING,
+      "font-faces" => RubyCssLint::WARNING,
+      "font-sizes" => RubyCssLint::WARNING,
+      "gradients" => RubyCssLint::WARNING,
+      "ids" => RubyCssLint::WARNING,
+      "import" => RubyCssLint::WARNING,
+      "important" => RubyCssLint::WARNING,
+      "known-properties" => RubyCssLint::WARNING,
+      "outline-none" => RubyCssLint::WARNING,
+      "overqualified-elements" => RubyCssLint::WARNING,
+      "qualified-headings" => RubyCssLint::WARNING,
+      "regex-selectors" => RubyCssLint::WARNING,
+      "rules-count" => RubyCssLint::WARNING,
+      "shorthand" => RubyCssLint::WARNING,
+      "star-property-hack" => RubyCssLint::WARNING,
+      "text-indent" => RubyCssLint::WARNING,
+      "underscore-property-hack" => RubyCssLint::WARNING,
+      "unique-headings" => RubyCssLint::WARNING,
+      "universal-selector" => RubyCssLint::WARNING,
+      "unqualified-attributes" => RubyCssLint::WARNING,
+      "vendor-prefix" => RubyCssLint::WARNING,
+      "zero-units" => RubyCssLint::WARNING,
+    }
+  end
+
+  def self.location_of_custom_rules(rails_root)
+    []
+  end
+
+  def self.location_of_css_files(rails_root)
+    [rails_root.to_s+"/public/assets/application.css"]
+  end
+
+CSS_LINT_DEFAULT
+  
+  self.class_eval(DEFAULT_CONFIG)
+  
+  
+  def self.construct_error_and_warning_options
+    rc = self.ruleset_classifications
+    warnings = rc.keys.select{|k| rc[k] == RubyCssLint::WARNING}
+    errors = rc.keys.select{|k| rc[k] == RubyCssLint::ERROR}
+    result = " "
+    result += "--warnings=#{warnings.join(',')} " if warnings.size > 0
+    result += "--errors=#{errors.join(',')} " if errors.size > 0
+    result
+  end
+  
+  def self.construct_js_and_run_rhino(css_files)
+    css_files = css_files.join(" ") if css_files.is_a?(Array)
+    
+    Tempfile.open("csslint_temp_js") do |tempfile|
+      tempfile.puts <<-HEADER
+var CSSLint = (function(){      
+HEADER
+      tempfile.puts`cat #{list_of_js_files_to_compile_step_1}`
+      tempfile.puts <<-FOOTER
+    return CSSLint;
+})();
+FOOTER
+      tempfile.puts`cat #{list_of_js_files_to_compile_step_2}`
+      tempfile.flush
+      run_rhino_with_js_file(tempfile.path, css_files)
+    end
+
+    
+  end
+  
+  def self.run_rhino_with_js_file(file, css_files)
+    rhino_jarfile = File.dirname(__FILE__) + "/../js.jar"
+    command = "java -jar #{rhino_jarfile} #{file} #{construct_error_and_warning_options} #{css_files}"
+    result = `#{command}`
+    puts result
+  end
+  
+  def self.list_of_js_files_to_compile_step_1
+    css_lint_root_directory = File.dirname(__FILE__) + "/../csslint/"
+    
+    parserlib_location = css_lint_root_directory + "/lib/parserlib.js"
+    csslint_main_location = css_lint_root_directory + "/src/core/CSSLint.js"
+    other_core_files = `ls -d #{(css_lint_root_directory+"/src/core/*.js")} | grep -v CSSLint`.split(/\n/).join(" ")
+    built_in_rules_file = css_lint_root_directory + "/src/rules/*.js"
+
+    custom_rules_files = location_of_custom_rules(Rails.root).join(" ")
+
+    formatters = css_lint_root_directory + "/src/formatters/*.js"
+    
+    [
+      parserlib_location, 
+      csslint_main_location, 
+      other_core_files, 
+      built_in_rules_file, 
+      custom_rules_files, 
+      formatters
+    ].join(" ")
+  end
+  
+  def self.list_of_js_files_to_compile_step_2
+    css_lint_root_directory = File.dirname(__FILE__) + "/../csslint/"
+    
+    cli_common = css_lint_root_directory + "/src/cli/common.js"
+    cli_rhino = css_lint_root_directory + "/src/cli/rhino.js"
+    [
+      cli_common,
+      cli_rhino 
+    ].join(" ")
+  end
+  
+  
+  def self.run
+    rhino_jarfile = File.dirname(__FILE__) + "../js.jar"
+    rhino_csslint_file = File.dirname(__FILE__) + "../csslint/"
+    command = "java -jar js.jar "
+  end
+  
+end
+
+