see-less-ess-ess 0.0.1 → 0.0.2

data/.travis.yml

@@ -0,0 +1,6 @@
+language: ruby
+rvm:
+  - 1.9.3
+  - 1.9.2
+  - 1.8.7
+script: bundle exec rspec

data/README.md

@@ -1,6 +1,9 @@
 # See-less-ess-ess
 
-TODO: Write a gem description
+[![Build Status](https://travis-ci.org/glenjamin/see-less-ess-ess.png)](https://travis-ci.org/glenjamin/see-less-ess-ess)
+
+A compass extension that scans HTML files, makes a note of which tags and
+classes are in use - and then removes anything else from the CSS file
 
 ## Installation
 

data/lib/see-less-ess-ess.rb

@@ -1,41 +1,53 @@
+require 'set'
+
 require 'sass'
 require 'nokogiri'
 
 module SeeLessEssEss
-  VERSION = '0.0.1'
+  VERSION = '0.0.2'
 
   class Extractor
-    def initialize(location)
-      @location = location
+    def initialize(glob)
+      @glob = glob
+      @extracted = false
     end
 
     def css_classes
-      @css_classes ||= extract_css_classes!
+      extract_if_needed
+      @css_classes
+    end
+    def html_tags
+      extract_if_needed
+      @html_tags
     end
 
     protected
 
     def files
-      Dir.glob("#{@location}/*.html")
+      Dir.glob(@glob)
     end
-    def extract_css_classes!
-      files.map do |file|
-        collector = Collector.new()
+    def extract_if_needed
+      collector = Collector.new()
+      files.each do |file|
         parser = Nokogiri::HTML::SAX::Parser.new(collector)
-        contents = open(file, 'rb').read
-        parser.parse(contents)
-        collector.css_classes
-      end.flatten
+        parser.parse(open(file, 'rb').read)
+      end
+      @css_classes = collector.css_classes
+      @html_tags = collector.html_tags
     end
 
     class Collector < Nokogiri::XML::SAX::Document
       def css_classes
-        @css_classes ||= []
+        @css_classes ||= Set.new
       end
-      def start_element(name, attrs = [])
+      def html_tags
+        @html_tags ||= Set.new
+      end
+      def start_element(tag, attrs = [])
+        html_tags << tag
         attrs.each do |name, value|
           if name == "class"
-            css_classes << value.split
+            css_classes.merge(value.split)
           end
         end
       end
@@ -61,13 +73,18 @@ module SeeLessEssEss
     def unused(simple)
       if simple.is_a?(Sass::Selector::Class)
         !css_classes_whitelist.include?(simple.name[0].to_s)
+      elsif simple.is_a?(Sass::Selector::Element)
+        !html_tags_whitelist.include?(simple.name[0].to_s)
       else
         false
       end
     end
 
     def css_classes_whitelist
-      @whitelist ||= @extractor.css_classes + @used_classes
+      @class_whitelist ||= @extractor.css_classes + @used_classes
+    end
+    def html_tags_whitelist
+      @tag_whitelist ||= @extractor.html_tags
     end
   end
 
@@ -126,9 +143,9 @@ module SeeLessEssEss
     @@initialised = true
 
     Compass::Configuration.add_configuration_property(
-      :templates_location, "Directory to scan for template files") do
+      :templates_glob, "Glob to scan for template files") do
 
-      project_path
+      "#{project_path}/**/*.html"
     end
     Compass::Configuration.add_configuration_property(
       :used_css_classes, "CSS classes to explicity declare as 'used'") do
@@ -140,7 +157,7 @@ module SeeLessEssEss
       alias_method :_slss_to_tree, :_to_tree
       def _to_tree
         conf = Compass.configuration
-        extractor = SeeLessEssEss::Extractor.new(conf.templates_location)
+        extractor = SeeLessEssEss::Extractor.new(conf.templates_glob)
         checker = SeeLessEssEss::Checker.new(extractor, conf.used_css_classes)
 
         tree = _slss_to_tree

data/see-less-ess-ess.gemspec

@@ -1,8 +1,10 @@
 # -*- encoding: utf-8 -*-
 
+require File.expand_path('../lib/see-less-ess-ess', __FILE__)
+
 Gem::Specification.new do |gem|
   gem.name          = "see-less-ess-ess"
-  gem.version       = '0.0.1'
+  gem.version       = SeeLessEssEss::VERSION
   gem.authors       = ["Glen Mailer"]
   gem.email         = ["glenjamin@gmail.com"]
   gem.description   = %q{Remove unused CSS rules from a compass project by scanning template files}

data/spec/checker_spec.rb

@@ -5,8 +5,10 @@ module SeeLessEssEss
     let(:extractor) do
       mock("Extractor").tap do |m|
         m.stub!(:css_classes).and_return(parsed_css_classes)
+        m.stub!(:html_tags).and_return(parsed_html_tags)
       end
     end
+    let(:parsed_html_tags) { %w(html head title body h1 p td) }
     let(:parsed_css_classes) { %w(row columns one two top-bar) }
     let(:used_css_classes) { %w(zebra) }
 
@@ -48,6 +50,16 @@ module SeeLessEssEss
           sequence(simple_sequence(className('class')))
         ).should be_true
       end
+      it "should reject ul" do
+        subject.does_not_use(
+          sequence(simple_sequence(tag('ul')))
+        ).should be_true
+      end
+      it "should reject ul.row" do
+        subject.does_not_use(
+          sequence(simple_sequence(tag('ul'),className('class')))
+        ).should be_true
+      end
       it "should reject .row .class" do
         subject.does_not_use(
           sequence(

data/spec/extractor_spec.rb

@@ -3,12 +3,18 @@ require File.expand_path('../../lib/see-less-ess-ess.rb', __FILE__)
 module SeeLessEssEss
   describe "Extractor" do
 
-    let(:subject) { Extractor.new(location) }
-    let(:location) { File.expand_path('../fixtures', __FILE__) }
+    let(:subject) { Extractor.new(glob) }
+    let(:glob) { "#{File.expand_path('../fixtures', __FILE__)}/**/*.html" }
 
     it "should read all css classes from html files in location" do
+      subject.should have(6).css_classes
       subject.css_classes.should include(*%w(body1 body2 title main left lead))
     end
 
+    it "should read all tags from html files in location" do
+      subject.should have(6).html_tags
+      subject.html_tags.should include(*%w(html head title body h1 p))
+    end
+
   end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: see-less-ess-ess
 version: !ruby/object:Gem::Version
-  version: 0.0.1
+  version: 0.0.2
   prerelease: 
 platform: ruby
 authors:
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2012-12-20 00:00:00.000000000 Z
+date: 2012-12-21 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: sass
@@ -68,6 +68,7 @@ extra_rdoc_files: []
 files:
 - .gitignore
 - .rspec
+- .travis.yml
 - Gemfile
 - LICENSE.txt
 - README.md