style_cop 0.0.1 → 0.0.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: cad754b549725a014e392f631d97c12dbb12dfc8
-  data.tar.gz: aba58cb8317eacd9a5d90b97ee02e5e4352f8974
+  metadata.gz: 8ccd9358deff944371ff06fcc84ad663b589c577
+  data.tar.gz: e293278f633c29acef072db79cac6880e4f7b9cd
 SHA512:
-  metadata.gz: a2a236e98c7fc36371c3c468d6f25e8315f5473326085d7a10a289dc8bbbe51e300f5b7cfcc5312191163aeeaf07f3fb4f8ede8cad7dfe54e7548bded8a99225
-  data.tar.gz: a88d3a48dd18d12f03e416393c32d728b6f068f6a524b5dfc25c1888c05e6e849ba0a66f66ded8a59bc93066fd22cc3bad02a2da2aa145adff436fc038efece7
+  metadata.gz: e6005e11b8f026750264a673e4943c3c01e5c91553949c7c7257633ed9d5ba3932a53645110789b3e810aef7ad307bb3bfa89893dd91cc7eecf15695054f1b87
+  data.tar.gz: f532b4948694b8b5215b206c136bf5701ec6637c817db760ea78a202453ea337737fdc70e5d37b13de86d8c255463026ec295f173b9fe72f8b8ce935309a11ff

data/.gitignore

@@ -0,0 +1,3 @@
+style_cop-0.0.1.gem
+.DS_Store
+Gemfile.lock

data/.pairs

@@ -0,0 +1,10 @@
+pairs:
+  dt: David Tegdin; dtegdin
+  wp: Ward Penney; wpenney
+  jn: Jordi Noguera; jordi
+
+email:
+  prefix: style_cop+pair
+  domain: pivotallabs.com
+
+global: true

data/.rspec

@@ -0,0 +1,2 @@
+--color
+--format progress

data/README.md

@@ -1,2 +1,46 @@
 # style_cop
-Police your style!
+Style Cop helps you enforce markup structure for front-end patterns via Rspec and Capybara. You spent all that time making your CSS play nicely. If all developers use the markup correctly, then it will render beautifully! 
+
+## Hypothesis
+
+1. We believe developers / designers have a problem sticiking w established DOM patterns, rather than page-overriding to tweak style.
+2. We can fix it by creating momentum to "use or improve" front-end CSS patterns by breaking the build if a pattern is used incorrectly
+3. We know we are right if a team thinks this is a good idea and uses it on a project, and they tweet a link to the repo.
+
+## MVP
+
+What's the least amount of work we can do to in/vadidate this hypothesis?
+
+* A gem that targets Rails projects that use Hologram
+* It automagically "enforces" DOM and style as defined in the Hologram live style guide
+
+## Installation
+
+Note: StyleCop depends on CapybaraWebkit so you will need to make sure that installs first: https://github.com/thoughtbot/capybara-webkit
+
+Put into your gemfile
+
+    gem style_cop
+    
+Bundle
+
+    $ bundle
+    
+## Getting Started
+
+To test a selector on your page matches the same selector on your styleguide
+you will need to do two things:
+
+1. Add the class 'style-cop-pattern' to the selector in your styleguide
+
+2. Add code like this somewhere in an integration test:
+
+'''
+  context "some context", style_cop: true do
+    it "tests something" do
+      visit "/some/path"
+      selector = page.find(".your-selector")
+      expect(selector).to match_styleguide(styleguide_page("/your/styleguide/path"))
+    end
+  end
+'''

data/lib/style_cop/capybara_webkit.rb

@@ -0,0 +1,9 @@
+RSpec.configure do |config|
+  config.before(:each, style_cop: true) do
+    Capybara.current_driver = :webkit
+  end
+
+  config.after(:each, style_cop: true) do
+    Capybara.use_default_driver
+  end
+end

data/lib/style_cop/helpers.rb

@@ -0,0 +1,14 @@
+module StyleCop
+  module Helpers
+    def styleguide_page(path)
+      old_session_name = Capybara.session_name
+      Capybara.session_name = "styleguide"
+      Capybara.visit path
+      Capybara.page.tap { Capybara.session_name = old_session_name }
+    end
+  end
+end
+
+RSpec.configure do |config|
+  config.include(StyleCop::Helpers, style_cop: true)
+end

data/lib/style_cop/match_styleguide_matcher.rb

@@ -0,0 +1,16 @@
+RSpec::Matchers.define :match_styleguide do |capybara_styleguide|
+  match do |capybara_selector|
+    selector = StyleCop::Selector.new(capybara_selector)
+    styleguide = StyleCop::StyleGuide.new(capybara_styleguide)
+    styleguide_selector = styleguide.find(selector.key)
+    StyleCop::SelectorDifference.new(selector, styleguide_selector).empty?
+  end
+
+  failure_message_for_should do |capybara_selector|
+    selector = StyleCop::Selector.new(capybara_selector)
+    styleguide = StyleCop::StyleGuide.new(capybara_styleguide)
+    styleguide_selector = styleguide.find(selector.key)
+    StyleCop::SelectorDifference.new(selector, styleguide_selector).error_message
+  end
+end
+

data/lib/style_cop/selector.rb

@@ -0,0 +1,68 @@
+module StyleCop
+  class Selector
+    EXCLUDED_KEYS = ["width", "height", "top", "bottom", "right", "left"]
+
+    def initialize(selector)
+      @selector = selector
+    end
+
+    def key
+      if selector['class']
+        ".#{selector['class'].gsub(' ', '.')}"
+      elsif selector['id']
+        "##{selector['id']}"
+      else
+        selector.tag_name
+      end
+    end
+
+    def representation
+      clean_key = key.gsub(".style-cop-pattern", "")
+      return { clean_key => computed_style } if children.empty?
+      children_hash = children.map(&:representation).inject({}) { |hash, h| hash.merge!(h) }
+      Hash[children_hash.map { |key, value| ["#{clean_key} #{key}", value] }].merge(
+        clean_key => computed_style
+      )
+    end
+
+    private
+
+    attr_reader :selector
+
+    def computed_style
+      style_hash.tap do |hash|
+        EXCLUDED_KEYS.each { |key| hash.delete(key) }
+      end
+    end
+
+    def children
+      selector.all(:xpath, "#{selector.path}/*").map do |child|
+        Selector.new(child)
+      end
+    end
+
+    def style_hash
+      Hash[css.split(/\s*;\s*/).map { |s| s.split(/\s*:\s*/) }]
+    end
+
+    def css
+      if computed_style = session.evaluate_script(computed_style_script)
+        computed_style["cssText"]
+      else
+        raise RuntimeError.new("Can't find css for #{selector.key}")
+      end
+    end
+
+    def session
+      selector.session
+    end
+
+    def computed_style_script
+      %{
+        var node = document.evaluate("/#{selector.path}",
+            document, null, XPathResult.FIRST_ORDERED_NODE_TYPE, null ).singleNodeValue
+        window.getComputedStyle(node);
+      }
+    end
+  end
+end

data/lib/style_cop/selector_difference.rb

@@ -0,0 +1,56 @@
+module StyleCop
+  class SelectorDifference
+    MISSING_CSS_TEXT = "element is missing the following css"
+    EXTRA_CSS_TEXT = "element has the following extra css"
+    MISSING_STRUCTURE_TEXT = "element is missing the following structure piece"
+    EXTRA_STRUCTURE_TEXT = "element has the following extra structure piece"
+
+    def initialize(selector, other_selector)
+      @selector = selector
+      @other_selector = other_selector
+    end
+
+    def empty?
+      selector.representation == other_selector.representation
+    end
+
+    def error_message
+      (css_errors + structure_errors).join(", ")
+    end
+
+    private
+
+    attr_reader :selector, :other_selector
+
+    def structure_errors
+      errors = []
+      extra_elements = selector.representation.keys - other_selector.representation.keys
+      missing_elements = other_selector.representation.keys - selector.representation.keys
+      errors << "The #{selector.key} #{EXTRA_STRUCTURE_TEXT}: #{extra_elements.join(", ")}" if extra_elements.any?
+      errors << "The #{selector.key} #{MISSING_STRUCTURE_TEXT}: #{missing_elements.join(", ")}" if missing_elements.any?
+      errors
+    end
+
+    def css_errors
+      errors = []
+      css_difference(other_selector, selector).each do |path, extra_css|
+        errors << "The #{selector.key} #{EXTRA_CSS_TEXT}: #{extra_css.map{|k,v| "#{k}: #{v}"}.join(', ')}" unless extra_css.empty?
+      end
+      css_difference(selector, other_selector).each do |path, extra_css|
+        errors << "The #{selector.key} #{MISSING_CSS_TEXT}: #{extra_css.map{|k,v| "#{k}: #{v}"}.join(', ')}" unless extra_css.empty?
+      end
+      errors
+    end
+
+    def css_difference(selector1, selector2)
+      difference = {}
+      selector2_representation = selector2.representation
+      selector1.representation.each do |path, selector1_css|
+        if selector2_css = selector2_representation[path]
+          difference[path] = Hash[selector2_css.to_a - selector1_css.to_a]
+        end
+      end
+      difference
+    end
+  end
+end

data/lib/style_cop/style_guide.rb

@@ -0,0 +1,15 @@
+module StyleCop
+  class StyleGuide
+    def initialize(styleguide)
+      @styleguide = styleguide
+    end
+
+    def find(selector_key)
+      Selector.new(styleguide.find("#{selector_key}.style-cop-pattern"))
+    end
+
+    private
+
+    attr_reader :styleguide
+  end
+end

data/lib/style_cop.rb

@@ -1,10 +1,9 @@
-module StyleCop
-  require 'style_cop/assert_style'
-  require 'style_cop/register_style'
-  require 'style_cop/public_methods'
-end
+require "style_cop/capybara_webkit"
+require "style_cop/selector"
+require "style_cop/style_guide"
+require "style_cop/match_styleguide_matcher"
+require "style_cop/selector_difference"
+require "style_cop/helpers"
 
-RSpec.configure do |c|
-  c.include StyleCop::PublicMethods, type: :feature
+module StyleCop
 end
-

data/spec/lib/match_styleguide_matcher_spec.rb

@@ -0,0 +1,56 @@
+require 'spec_helper'
+
+module StyleCop
+  describe "MatchStyleguideMatcher" do
+    it "includes the matcher" do
+      expect(self).to respond_to(:match_styleguide)
+    end
+
+    context "when the selectors matches the styleguide" do
+      let(:html) do
+        create_html({
+          body: %(
+            <div class='selector'></div>
+            <div class='selector style-cop-pattern'></div>
+          )
+        })
+      end
+      let(:page) { FakePage.new(html) }
+      let(:selector) { page.all(".selector").first }
+
+      it "passes" do
+        expect(selector).to match_styleguide(page)
+      end
+    end
+
+    context "when the selector doesn't match the styleguide" do
+      let(:html) do
+        create_html({
+          body: %(
+            <div class='selector'></div>
+            <div class='selector style-cop-pattern'><div class='wrong'></div></div>
+          )
+        })
+      end
+
+      let(:page) { FakePage.new(html) }
+      let(:selector) { page.all(".selector").first }
+      let(:styleguide_selector) { page.find(".selector.style-cop-pattern") }
+      let(:selector_difference) { double(SelectorDifference, empty?: false) }
+
+      it "doesn't pass" do
+        expect(selector).to_not match_styleguide(page)
+      end
+
+      it "displays error messages when matcher fails" do
+        allow(SelectorDifference).to receive(:new).
+          and_return(selector_difference)
+        expect(selector_difference).to receive(:error_message) { "selector to match" }
+
+        expect {
+          expect(selector).to match_styleguide(page)
+        }.to raise_error(/selector to match/)
+      end
+    end
+  end
+end

data/spec/lib/selector_difference_spec.rb

@@ -0,0 +1,168 @@
+require 'spec_helper'
+
+module StyleCop
+  describe SelectorDifference do
+    let(:selector) { double }
+    let(:styleguide_selector) { double }
+    subject { SelectorDifference.new(selector, styleguide_selector) }
+
+    describe "#error_message" do
+      before do
+        allow(selector).to receive(:key).and_return(".structure")
+        allow(selector).to receive(:representation).and_return(selector_representation)
+        allow(styleguide_selector).to receive(:representation).and_return(styleguide_representation)
+      end
+
+      context "css" do
+        context "missing styleguide css" do
+          let(:selector_representation) { { ".selector" => {"a"=>"b"} } }
+          let(:styleguide_representation) { { ".selector" => {"a"=>"b", "font-size"=>"100px"} } }
+
+          it "shows what css the element is missing" do
+            message = "The .structure element is missing the following css: font-size: 100px"
+            expect(subject.error_message).to eq message
+          end
+        end
+
+        context "extra css not in styleguide" do
+          let(:selector_representation) { { ".selector" => {"a"=>"b", "font-size"=>"100px"} } }
+          let(:styleguide_representation) { { ".selector" => {"a"=>"b"} } }
+
+          it "shows what extra css is present" do
+            message = "The .structure element has the following extra css: font-size: 100px"
+            expect(subject.error_message).to eq message
+          end
+        end
+
+        context "missing css in the styleguide and has extra css not in the styleguide" do
+          let(:selector_representation) { {".selector" => { "font-size"=>"100px"} } }
+          let(:styleguide_representation) { {".selector" => { "font-size"=>"80px"} } }
+
+          it "shows what extra css is present" do
+            message = "The .structure element has the following extra css: font-size: 100px, The .structure element is missing the following css: font-size: 80px"
+            expect(subject.error_message).to include message
+          end
+        end
+      end
+
+      context "structure" do
+        context "missing styleguide structure" do
+          let(:selector_representation) { { ".selector .child" => {} } }
+          let(:styleguide_representation) { { ".selector .child" => {}, ".selector .child .missing" => {} } }
+
+          it "shows what structure is missing from the element" do
+            message = "The .structure element is missing the following structure piece: .selector .child .missing"
+            expect(subject.error_message).to eq message
+          end
+        end
+
+        context "extra structure not in styleguide" do
+          let(:selector_representation) { { ".selector .child" => {}, ".selector .child .extra" => {} } }
+          let(:styleguide_representation) { { ".selector .child" => {} } }
+
+          it "shows what extra structure is present" do
+            message = "The .structure element has the following extra structure piece: .selector .child .extra"
+            expect(subject.error_message).to eq message
+          end
+        end
+      end
+    end
+
+    describe "#empty?" do
+      let(:first_selector) { Selector.new page.all(".selector").first }
+      let(:last_selector) { Selector.new page.all(".selector").last }
+      let(:page) { FakePage.new(html) }
+      subject { SelectorDifference.new(first_selector, last_selector) }
+
+      context "when two selectors have same css" do
+        let(:html) do
+          create_html({
+            body: %{
+              <div class="selector"></div>
+              <div class="selector"></div>
+            }
+          })
+        end
+
+        it "returns true" do
+          expect(subject).to be_empty
+        end
+      end
+
+      context "when two selectors don't have same css" do
+        let(:html) do
+          create_html({
+            body: %{
+              <div class="selector"></div>
+              <div class="selector" style="font-size: 100px"></div>
+            }
+          })
+        end
+
+        it "returns false" do
+          expect(subject).to_not be_empty
+        end
+      end
+
+      context "when two selectors have same structure" do
+        let(:html) do
+          create_html({
+            body: %{
+              <div class="selector"><div class="child2"></div></div>
+              <div class="selector"><div class="child2"></div></div>
+            }
+          })
+        end
+
+        it "returns true" do
+          expect(subject).to be_empty
+        end
+      end
+
+      context "when two selectors don't have same structure" do
+        let(:html) do
+          create_html({
+            body: %{
+              <div class="selector"></div>
+              <div class="selector"><div class="child2"></div></div>
+            }
+          })
+        end
+
+        it "returns false" do
+          expect(subject).to_not be_empty
+        end
+      end
+
+      context "when two selectors children have same css" do
+        let(:html) do
+          create_html({
+            body: %{
+              <div class="selector"><div class="child2"></div></div>
+              <div class="selector"><div class="child2"></div></div>
+            }
+          })
+        end
+
+        it "returns false" do
+          expect(subject).to be_empty
+        end
+      end
+
+      context "when two selectors children don't have same css" do
+        let(:html) do
+          create_html({
+            body: %{
+              <div class="selector"><div class="child2"></div></div>
+              <div class="selector"><div class="child2" style="font-size:100px"></div></div>
+            }
+          })
+        end
+
+        it "returns false" do
+          expect(subject).to_not be_empty
+        end
+      end
+    end
+  end
+end

data/spec/lib/selector_spec.rb

@@ -0,0 +1,89 @@
+require "spec_helper"
+
+module StyleCop
+  describe Selector, style_cop: true do
+    describe "#key" do
+      let(:page) { FakePage.new(selector_html) }
+      let(:capybara_selector) { page.find("span") }
+      let(:selector) { Selector.new(capybara_selector) }
+
+      context "for a class" do
+        let(:selector_html) do
+          create_html(body: "<span class='selector other-selector' id='ignored'></span>")
+        end
+
+        it "returns the classes separated by a dot" do
+          expect(selector.key).to eq(".selector.other-selector")
+        end
+      end
+
+      context "for an id" do
+        let(:selector_html) do
+          create_html(body: "<span id='selector'></span>")
+        end
+
+        it "returns the id prefixed by a pound" do
+          expect(selector.key).to eq("#selector")
+        end
+      end
+
+      context "for an html element" do
+        let(:selector_html) do
+          create_html(body: "<span></span>")
+        end
+
+        it "returns html element" do
+          expect(selector.key).to eq("span")
+        end
+      end
+    end
+
+    describe "#representation" do
+      let(:html) do
+        create_html({
+          body: %{
+              <div class="selector style-cop-pattern" style="font-size:16px">
+                <div class="child1" style="font-size:24px">
+                  <div class="child3 style-cop-pattern" style="font-size:36px"></div>
+                </div>
+                <div class="child2"></div>
+              </div>
+          }
+        })
+      end
+
+      let(:page) { FakePage.new(html) }
+      let(:selector) { Selector.new page.find(".selector") }
+
+      it "returns a hash with structure keys andd css values" do
+        expect(selector.representation.keys.sort).to eq(
+          [".selector", ".selector .child1", ".selector .child1 .child3", ".selector .child2"]
+        )
+        expect(selector.representation[".selector"]["font-size"]).to eq("16px")
+        expect(selector.representation[".selector .child1"]["font-size"]).to eq("24px")
+        expect(selector.representation[".selector .child1 .child3"]["font-size"]).to eq("36px")
+        expect(selector.representation[".selector .child2"]["font-size"]).to eq("16px")
+      end
+
+      context "excluded keys in style" do
+        let(:html) do
+          create_html({
+            body: %{
+                <div class="selector"></div>
+            }
+          })
+        end
+
+        let(:selector) { Selector.new(page.find(".selector")) }
+        subject { selector.representation['.selector'] }
+
+        it { should_not have_key("width") }
+        it { should_not have_key("height") }
+        it { should_not have_key("top") }
+        it { should_not have_key("bottom") }
+        it { should_not have_key("right") }
+        it { should_not have_key("left") }
+      end
+    end
+  end
+end

data/spec/lib/style_cop_spec.rb

@@ -0,0 +1,27 @@
+require "spec_helper"
+
+describe StyleCop do
+  context "capybara driver" do
+    it "doesn't change the current driver by default" do
+      expect(Capybara.current_driver).to_not eq(:webkit)
+    end
+
+    it "changes to current driver for style cop specs", style_cop: true do
+      expect(Capybara.current_driver).to eq(:webkit)
+    end
+  end
+
+  context "styleguide_page" do
+    it "doesn't add the helper method for non-style cop tests" do
+      expect(self).to_not respond_to(:styleguide_page)
+    end
+
+    context "style cop tests", style_cop: true do
+      it "have a styleguide_page method" do
+        Capybara.app = FakePage::TestApp
+        Capybara.app.set_html("<html></html>")
+        expect(styleguide_page("/")).to_not be_nil
+      end
+    end
+  end
+end

data/spec/lib/style_guide_spec.rb

@@ -0,0 +1,23 @@
+require 'spec_helper'
+
+module StyleCop
+  describe StyleGuide do
+    describe "#find" do
+      let(:styleguide_html) do
+        create_html({
+          body: %{
+            <div class="selector first style-cop-pattern"></div>
+            <div class="selector second style-cop-pattern"></div>
+          }
+        })
+      end
+
+      let(:styleguide_page) { FakePage.new(styleguide_html) }
+      let(:styleguide) { StyleGuide.new(styleguide_page) }
+
+      it "finds elements in the styleguide by key" do
+        expect(styleguide.find(".selector.first").key).to eq(".selector.first.style-cop-pattern")
+      end
+    end
+  end
+end

data/spec/spec_helper.rb

@@ -1,13 +1,16 @@
-require 'rubygems'
-require 'bundler/setup'
+require "rubygems"
+require "bundler/setup"
 
-require 'pry'
-require 'style_cop'
-require 'rspec'
+require "pry"
+require "style_cop"
+require "rspec"
+require "capybara"
+require "capybara-webkit"
+require "sinatra"
 
-ENV['test_run'] = true.to_s
+ENV["test_run"] = true.to_s
 
-Dir[File.join(File.dirname(__FILE__), 'support', '**', '*.rb')].each do |file|
+Dir[File.join(File.dirname(__FILE__), "support", "**", "*.rb")].each do |file|
   require file
 end
 

data/spec/support/fake_page.rb

@@ -0,0 +1,21 @@
+class FakePage
+  def initialize(html)
+    @session = Capybara::Session.new(:webkit, TestApp)
+    @session.app.set_html(html)
+    @session.visit "/"
+  end
+
+  def method_missing(method, *args)
+    @session.public_send(method, *args)
+  end
+
+  class TestApp < Sinatra::Base
+    def self.set_html(html)
+      @@html = html
+    end
+
+    get '/' do
+      @@html
+    end
+  end
+end

data/spec/support/html_helpers.rb

@@ -0,0 +1,18 @@
+module HtmlHelpers
+  def create_html(options = {})
+    %{<html>
+      <head>
+        <style>
+    #{options[:style]}
+        </style>
+      </head>
+      <body>
+    #{options[:body]}
+      </body>
+    </html>}
+  end
+end
+
+RSpec.configure do |config|
+  config.include(HtmlHelpers)
+end

data/style_cop.gemspec

@@ -1,16 +1,18 @@
 Gem::Specification.new do |s|
   s.name        = 'style_cop'
-  s.version     = '0.0.1'
+  s.version     = '0.0.2'
   s.date        = '2014-03-03'
   s.summary     = "Gem for testing style"
   s.description = "Gem for testing style"
-  s.authors     = ["Ward Penney", "David Tengdin"]
+  s.authors     = ["Ward Penney", "David Tengdin", "Jordi Noguera"]
   s.email       = 'style-cop@googlegroups.com'
   s.files       = `git ls-files`.split("\n")
   s.homepage    = 'http://rubygems.org/gems/style_cop'
   s.license     = 'MIT'
 
-  s.add_development_dependency "rspec", "~> 2.14.0"
-  s.add_development_dependency "xpath", "~> 2.0.0"
-  s.add_development_dependency "pry", "~> 0.9.12.6"
+  s.add_dependency "rspec"
+  s.add_dependency "capybara-webkit"
+
+  s.add_development_dependency "pry"
+  s.add_development_dependency "sinatra"
 end

metadata

@@ -1,11 +1,12 @@
 --- !ruby/object:Gem::Specification
 name: style_cop
 version: !ruby/object:Gem::Version
-  version: 0.0.1
+  version: 0.0.2
 platform: ruby
 authors:
 - Ward Penney
 - David Tengdin
+- Jordi Noguera
 autorequire: 
 bindir: bin
 cert_chain: []
@@ -15,60 +16,84 @@ dependencies:
   name: rspec
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ~>
+    - - ">="
       - !ruby/object:Gem::Version
-        version: 2.14.0
-  type: :development
+        version: '0'
+  type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ~>
+    - - ">="
       - !ruby/object:Gem::Version
-        version: 2.14.0
+        version: '0'
 - !ruby/object:Gem::Dependency
-  name: xpath
+  name: capybara-webkit
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ~>
+    - - ">="
       - !ruby/object:Gem::Version
-        version: 2.0.0
-  type: :development
+        version: '0'
+  type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ~>
+    - - ">="
       - !ruby/object:Gem::Version
-        version: 2.0.0
+        version: '0'
 - !ruby/object:Gem::Dependency
   name: pry
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ~>
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+- !ruby/object:Gem::Dependency
+  name: sinatra
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
       - !ruby/object:Gem::Version
-        version: 0.9.12.6
+        version: '0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ~>
+    - - ">="
       - !ruby/object:Gem::Version
-        version: 0.9.12.6
+        version: '0'
 description: Gem for testing style
 email: style-cop@googlegroups.com
 executables: []
 extensions: []
 extra_rdoc_files: []
 files:
+- ".gitignore"
+- ".pairs"
+- ".rspec"
 - Gemfile
-- Gemfile.lock
 - README.md
 - lib/style_cop.rb
-- lib/style_cop/assert_style.rb
-- lib/style_cop/public_methods.rb
-- lib/style_cop/register_style.rb
-- spec/lib/style_cop/assert_style_spec.rb
-- spec/lib/style_cop/public_methods_spec.rb
+- lib/style_cop/capybara_webkit.rb
+- lib/style_cop/helpers.rb
+- lib/style_cop/match_styleguide_matcher.rb
+- lib/style_cop/selector.rb
+- lib/style_cop/selector_difference.rb
+- lib/style_cop/style_guide.rb
+- spec/lib/match_styleguide_matcher_spec.rb
+- spec/lib/selector_difference_spec.rb
+- spec/lib/selector_spec.rb
+- spec/lib/style_cop_spec.rb
+- spec/lib/style_guide_spec.rb
 - spec/spec_helper.rb
+- spec/support/fake_page.rb
+- spec/support/html_helpers.rb
 - style_cop.gemspec
 homepage: http://rubygems.org/gems/style_cop
 licenses:
@@ -80,17 +105,17 @@ require_paths:
 - lib
 required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
-  - - '>='
+  - - ">="
     - !ruby/object:Gem::Version
       version: '0'
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
-  - - '>='
+  - - ">="
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.1.10
+rubygems_version: 2.2.2
 signing_key: 
 specification_version: 4
 summary: Gem for testing style

data/Gemfile.lock

@@ -1,38 +0,0 @@
-PATH
-  remote: .
-  specs:
-    style_cop (0.0.1)
-
-GEM
-  remote: http://rubygems.org/
-  specs:
-    coderay (1.1.0)
-    diff-lcs (1.2.5)
-    method_source (0.8.2)
-    mini_portile (0.5.2)
-    nokogiri (1.6.1)
-      mini_portile (~> 0.5.0)
-    pry (0.9.12.6)
-      coderay (~> 1.0)
-      method_source (~> 0.8)
-      slop (~> 3.4)
-    rspec (2.14.1)
-      rspec-core (~> 2.14.0)
-      rspec-expectations (~> 2.14.0)
-      rspec-mocks (~> 2.14.0)
-    rspec-core (2.14.8)
-    rspec-expectations (2.14.5)
-      diff-lcs (>= 1.1.3, < 2.0)
-    rspec-mocks (2.14.6)
-    slop (3.4.7)
-    xpath (2.0.0)
-      nokogiri (~> 1.3)
-
-PLATFORMS
-  ruby
-
-DEPENDENCIES
-  pry (~> 0.9.12.6)
-  rspec (~> 2.14.0)
-  style_cop!
-  xpath (~> 2.0.0)

data/lib/style_cop/assert_style.rb

@@ -1,37 +0,0 @@
-module StyleCop
-  class AssertStyle
-    def initialize(selector, rules, test_context)
-      @selector = selector
-      @rules = rules
-      @test_context = test_context
-    end
-
-    def has_correct_structure
-      selector_present ? fulfills_all_rules : raise_error
-    end
-
-    private
-
-    attr_reader :selector, :test_context, :rules
-
-    def selector_present
-      test_context.has_css?(selector)
-    end
-
-    def raise_error
-      raise StandardError, "Selector: #{selector} is not present."
-    end
-
-    def fulfills_all_rules
-      test_context.page.all(selector).each do |element|
-        has(element)
-      end
-    end
-
-    def has(element)
-      rules.each do |node|
-        test_context.expect(element).to test_context.have_selector(node)
-      end
-    end
-  end
-end

data/lib/style_cop/public_methods.rb

@@ -1,23 +0,0 @@
-module StyleCop
-  module PublicMethods
-    def has_child(selector)
-      RegisterStyle.add_rule("> #{selector}")
-    end
-
-    def has_nested_children(*args)
-      rule = ""
-      args.each do |arg|
-        rule = "#{rule}> #{arg} "
-      end
-      RegisterStyle.add_rule(rule.strip)
-    end
-
-    def register_style_structure_for(selector, &block)
-      block ? RegisterStyle.new(selector, &block) : (raise(StandardError, "No block given."))
-    end
-
-    def assert_style_structure_for(selector)
-      AssertStyle.new(selector, RegisterStyle.rules, self).has_correct_structure
-    end
-  end
-end

data/lib/style_cop/register_style.rb

@@ -1,17 +0,0 @@
-module StyleCop
-  class RegisterStyle
-    @@rules = []
-
-    def initialize(selector, &block)
-      yield
-    end
-
-    def self.rules
-      @@rules
-    end
-
-    def self.add_rule(rule)
-      @@rules << rule
-    end
-  end
-end

data/spec/lib/style_cop/assert_style_spec.rb

@@ -1,52 +0,0 @@
-require 'spec_helper'
-
-describe StyleCop::AssertStyle do
-  subject{ StyleCop::AssertStyle.new(selector, rules, test_context) }
-  let(:selector) { '.selector' }
-  let(:rules) { double('rules') }
-  let(:test_context) { double('test_context') }
-
-  describe "has_correct_structure" do
-    before do
-      allow(test_context).to receive(:has_css?) { has_selector }
-    end
-
-    context "selector present" do
-      let(:has_selector) { true }
-
-      let(:rule1) { double('rule1') }
-      let(:rule2) { double('rule2') }
-      let(:element1) { double('element1') }
-      let(:element2) { double('element2') }
-      let(:node1) { double('node1') }
-      let(:node2) { double('node2') }
-      let(:rules) { [rule1, rule2] }
-
-      before do
-        test_context.stub_chain(:page, :all) { [element1, element2] }
-      end
-
-      it "calls an expectation on elements" do
-        expect(test_context).to receive(:expect).with(element1).twice { node1 }
-        expect(test_context).to receive(:expect).with(element2).twice { node2 }
-        allow(node1).to receive(:to)
-        allow(node2).to receive(:to)
-        expect(test_context).to receive(:have_selector).with(rule1).twice
-        expect(test_context).to receive(:have_selector).with(rule2).twice
-
-        subject.has_correct_structure
-      end
-    end
-
-    context "selector not present" do
-      let(:has_selector) { false }
-
-      it "raises a StandardError" do
-        expect { subject.has_correct_structure }.to raise_error { |error|
-          expect(error.message).to eq "Selector: .selector is not present."
-          error.should be_a(StandardError)
-        }
-      end
-    end
-  end
-end

data/spec/lib/style_cop/public_methods_spec.rb

@@ -1,60 +0,0 @@
-require 'spec_helper'
-
-describe StyleCop::PublicMethods do
-  include StyleCop::PublicMethods
-  let(:selector) {'.foo'}
-
-  describe 'has_child' do
-
-    it 'calls add_rule on RegisterStyle' do
-      expect(StyleCop::RegisterStyle).to receive(:add_rule).with('> .foo')
-      has_child selector
-    end
-  end
-
-  describe 'has_nested_children' do
-    let(:selector1) {'.foo'}
-    let(:selector2) {'.bar'}
-
-    it 'calls add_rule on RegisterStyle' do
-      expect(StyleCop::RegisterStyle).to receive(:add_rule).with('> .foo > .bar')
-      has_nested_children(selector1, selector2)
-    end
-  end
-
-  describe 'register_style_structure_for' do
-    let(:block) { lambda{} }
-
-    context 'block is given' do
-      it 'calls a new RegisterStyle' do
-        expect(StyleCop::RegisterStyle).to receive(:new).with(selector, &block)
-        register_style_structure_for(selector, &block)
-      end
-    end
-
-    context 'no block is given' do
-      it 'raises a no block error' do
-        expect { register_style_structure_for(selector) }.to raise_error { |error|
-          expect(error.message).to eq "No block given."
-          error.should be_a(StandardError)
-        }
-      end
-    end
-  end
-
-  describe 'assert_style_structure_for' do
-    let(:rules) { double(:rules) }
-    let(:assert_style_class) { double(:assert_style_class) }
-
-    before do
-      allow(StyleCop::RegisterStyle).to receive(:rules) { rules }
-    end
-
-    it "creates an AssertStyle object and calls has_correct_structure on it" do
-      expect(StyleCop::AssertStyle).to receive(:new).with(selector, rules, self) { assert_style_class }
-      expect(assert_style_class).to receive(:has_correct_structure)
-
-      assert_style_structure_for(selector)
-    end
-  end
-end