Raise default remediation, categorize checks

Categorize checks loosely based on the CSSLint wiki[1]. The most obvious
departure was categorizing Performance rules as Bug Risk checks.

1: https://github.com/CSSLint/csslint/wiki/Rules

Default remediation was at an abysmal 500, upped it to the "trivial"
baseline for now and built a small class allowing us to easily override
these later.

I chose keyword arguments in the CheckDetails class so we can easily
create an override object for one, the other, or both of categories and
remediation points.

Author: pbrisbin

lib/cc/engine/csslint.rb

@@ -4,6 +4,8 @@
 module CC
   module Engine
     class CSSlint
+      autoload :CheckDetails, "cc/engine/csslint/check_details"
+
       def initialize(directory: , io: , engine_config: )
         @directory = directory
         @engine_config = engine_config
@@ -18,13 +20,15 @@ def run
               next unless node.name == "error"
 
               lint = node.attributes
+              check_name = lint["source"].value
+              check_details = CheckDetails.fetch(check_name)
 
               issue = {
                 type: "issue",
-                check_name: lint["source"].value,
+                check_name: check_name,
                 description: lint["message"].value,
-                categories: ["Style"],
-                remediation_points: 500,
+                categories: check_details.categories,
+                remediation_points: check_details.remediation_points,
                 location: {
                   path: path,
                   positions: {

lib/cc/engine/csslint/check_details.rb

@@ -0,0 +1,55 @@
+module CC
+  module Engine
+    class CSSlint
+      # https://github.com/CSSLint/csslint/wiki/Rules
+      class CheckDetails
+        DEFAULT_CATEGORY = "Style".freeze
+        DEFAULT_REMEDIATION_POINTS = 50_000.freeze
+
+        attr_reader :categories, :remediation_points
+
+        def self.all
+          @all ||= {
+            "net.csslint.Adjoiningclasses" => new(categories: "Compatability"),
+            "net.csslint.Boxmodel" => new(categories: "Bug Risk"),
+            "net.csslint.Boxsizing" => new(categories: "Compatability"),
+            "net.csslint.Bulletprooffontface" => new(categories: "Compatability"),
+            "net.csslint.Compatiblevendorprefixes" => new(categories: "Compatability"),
+            "net.csslint.Displaypropertygrouping" => new(categories: "Bug Risk"),
+            "net.csslint.Duplicatebackgroundimages" => new(categories: "Bug Risk"),
+            "net.csslint.Duplicateproperties" => new(categories: "Bug Risk"),
+            "net.csslint.Emptyrules" => new(categories: "Bug Risk"),
+            "net.csslint.Fallbackcolors" => new(categories: "Compatability"),
+            "net.csslint.Fontfaces" => new(categories: "Bug Risk"),
+            "net.csslint.Gradients" => new(categories: "Compatability"),
+            "net.csslint.Import" => new(categories: "Bug Risk"),
+            "net.csslint.Knownproperties" => new(categories: "Bug Risk"),
+            "net.csslint.Overqualifiedelements" => new(categories: "Bug Risk"),
+            "net.csslint.Regexselectors" => new(categories: "Bug Risk"),
+            "net.csslint.Shorthand" => new(categories: "Bug Risk"),
+            "net.csslint.Starpropertyhack" => new(categories: "Compatability"),
+            "net.csslint.Textindent" => new(categories: "Compatability"),
+            "net.csslint.Underscorepropertyhack" => new(categories: "Compatability"),
+            "net.csslint.Uniqueheadings" => new(categories: "Duplication"),
+            "net.csslint.Universalselector" => new(categories: "Bug Risk"),
+            "net.csslint.Unqualifiedattributes" => new(categories: "Bug Risk"),
+            "net.csslint.Vendorprefix" => new(categories: "Compatability"),
+            "net.csslint.Zerounits" => new(categories: "Bug Risk"),
+          }
+        end
+
+        def self.fetch(check_name)
+          all.fetch(check_name) { new }
+        end
+
+        def initialize(
+          categories: DEFAULT_CATEGORY,
+          remediation_points: DEFAULT_REMEDIATION_POINTS
+        )
+          @categories = Array(categories)
+          @remediation_points = remediation_points
+        end
+      end
+    end
+  end
+end

spec/cc/engine/csslint/check_details_spec.rb

@@ -0,0 +1,21 @@
+require "spec_helper"
+
+class CC::Engine::CSSlint
+  describe CheckDetails do
+    describe ".fetch" do
+      it "returns details for customized checks" do
+        details = CheckDetails.fetch("net.csslint.Import")
+
+        expect(details.categories).to eq ["Bug Risk"]
+        expect(details.remediation_points).to eq 50_000
+      end
+
+      it "returns defaults for unknown checks" do
+        details = CheckDetails.fetch("made-up")
+
+        expect(details.categories).to eq ["Style"]
+        expect(details.remediation_points).to eq 50_000
+      end
+    end
+  end
+end

spec/cc/engine/csslint_spec.rb

@@ -1,5 +1,4 @@
-require 'cc/engine/csslint'
-require 'tmpdir'
+require "spec_helper"
 
 module CC
   module Engine

spec/spec_helper.rb

@@ -0,0 +1,2 @@
+require "cc/engine/csslint"
+require "tmpdir"