stylesheet 0.1.0 → 0.1.6

checksums.yaml

@@ -0,0 +1,7 @@
+---
+SHA256:
+  metadata.gz: d8731f136d13c2aafa5686bc348ed721890e8d8186ccf7b3d07b04b5e4b1d286
+  data.tar.gz: 5e1a9e9c982d43c12c2e7bed8c43703b7fae491282d1c17e24f25567bbbec08a
+SHA512:
+  metadata.gz: 73c42acfa16d26fe5c48dffaaf738cf21452dcdc8e1c2be9ae050663a70be53eaab99a87cb9cce8f7096b04bd52b828e0704f15b70ec74cb0edf93323bc4bc42
+  data.tar.gz: 696f793a09c89575152c2a31730909e76d0328bb095c540c7c405347b6bc53f3e414619e86e956bee3e51d3a10f7ce47c4bfbcc34be627f2d6f3fb8b8354e8f5

data/README.md

@@ -8,51 +8,65 @@ Get styles from a document:
 
 ```ruby
 document = Stylesheet::Document.new("http://sportspyder.com")
+=> #<Document location:http://sportspyder.com/>
+
 document.style_sheets
+=> [#<Stylesheet::CssStyleSheet:0x007fa905c58c20>, 
+    #<Stylesheet::CssStyleSheet:0x007fa905c5f430>, 
+    #<Stylesheet::CssStyleSheet:0x007fa905c5e968>]
 ```
 
 Get attributes of a stylesheet: 
 
 ```ruby
 sheet = document.style_sheets[0]
-puts sheet.href
-puts sheet.type
+=> #<Stylesheet::CssStyleSheet:0x007fa905c58c20>
+
+sheet.href
+=> "http://sportspyder.com/assets/application-26ff2c8d54ab9cd8e74af60fc650390e.css"
+
+sheet.type
+=> "text/css"
 ```
 
 Get stylesheet media definitions: 
 
 ```ruby
-sheet = document.style_sheets[0]
-sheet.media.each do |media| 
-  puts media
-end
+sheet.media.map {|medium| medium }
+=> ["screen"]
 ```
 
 Get rules defined in a stylesheet: 
 
 ```ruby
 sheet = Stylesheet::CssStyleSheet.new("http://sportspyder.com/stylesheets/screen.css")
-sheet.css_rules.each do |rule|
-  puts rule.css_text
-  puts rule.selector_text
-end
+=> #<Stylesheet::CssStyleSheet:0x007fa905c58c20>
+
+rule = sheet.css_rules[0]
+=> #<Stylesheet::CssStyleRule>
+
+rule.css_text
+=> "iframe.editor{width:580px;height:150px;border:1px solid #ccc;background-color:#fff}"
+
+rule.selector_text
+=> "iframe.editor"
 ```
 
 Get declarations defined in a style rules: 
 
 ```ruby
-sheet = document.style_sheets[0]
-rule  = sheet.css_rules[0]
-puts rule.style[0]
-puts rule.style.border
+rule.style[0]
+=> "width:580px"
+
+rule.style.border
+=> "1px solid #ccc"
 ```
 
 ## Installation
 
-```
-gem install stylesheet
-```
-```
+To install Stylesheet, add the gem to your Gemfile: 
+
+```ruby
 gem "stylesheet"
 ```
 

data/lib/stylesheet.rb

@@ -21,6 +21,7 @@ require 'stylesheet/css_charset_rule'
 require 'stylesheet/css_import_rule'
 require 'stylesheet/css_media_rule'
 require 'stylesheet/css_font_face_rule'
+require 'stylesheet/css_null_rule'
 
 require 'stylesheet/css_style_declaration'
 

data/lib/stylesheet/css_import_rule.rb

@@ -12,7 +12,9 @@ module Stylesheet
     end
 
     def style_sheet
-      @style_sheet ||= CssStyleSheet.new(location.href)
+      @style_sheet ||= CssStyleSheet.new(:href       => location.href,
+                                         :owner_rule => self,
+                                         :parent     => parent_style_sheet)
     end
 
     def self.matches_rule?(text)

data/lib/stylesheet/css_null_rule.rb

@@ -0,0 +1,16 @@
+module Stylesheet
+  class CssNullRule < CssRule
+
+    def type
+      CssRule::NULL_RULE
+    end
+
+    def self.matches_rule?(text)
+      true
+    end
+
+    private
+
+    def parse_css_text; end
+  end
+end

data/lib/stylesheet/css_rule.rb

@@ -1,5 +1,6 @@
 module Stylesheet
   class CssRule
+    NULL_RULE      = 0
     STYLE_RULE     = 1
     CHARSET_RULE   = 2
     IMPORT_RULE    = 3
@@ -50,7 +51,7 @@ module Stylesheet
 
     private
 
-    def parse_css_text(css_text)
+    def parse_css_text
       raise NotImplementedError
     end
   end

data/lib/stylesheet/css_rule_list.rb

@@ -1,7 +1,7 @@
 module Stylesheet
   class CssRuleList
     extend Forwardable
-    def_delegators :@rules, :length, :size, :[], :each, :to_s
+    def_delegators :@rules, :length, :size, :[], :each, :insert, :delete_at, :to_s
     include Enumerable
 
     def initialize(rules, parent = nil)
@@ -11,15 +11,18 @@ module Stylesheet
     def item(index)
       @rules[index]
     end
-    
+
     private
     
     def parse(rules, parent)
+      # clean out comments
+      rules = rules.gsub(/\/\*[\s\S]*?\*\//, '').gsub("/*", "").gsub("*/", "")
+      
       # clean extraneous whitespace
       rules = rules.to_s.gsub(/\s+/m, " ").gsub(/([\};])\s/, '\1')
 
       directive_re = "@.+?;"
-      rules_re     = ".+?\{.+?\}"
+      rules_re     = ".+?\{.*?\}"
       split_rules = rules.scan(/(#{directive_re}|#{rules_re})/im).map {|r| r[0] }
 
       split_rules.map do |css_text| 

data/lib/stylesheet/css_style_declaration.rb

@@ -1,35 +1,39 @@
 module Stylesheet
   class CssStyleDeclaration
     extend Forwardable
-    def_delegators :@declarations, :length, :size, :[], :each, :<<, :push, :delete, :to_s
+    def_delegators :@declarations_list, :length, :size, :[], :each, :<<, :push, :delete, :to_s
     include Enumerable
 
-    attr_reader :parent_rule
+    attr_reader :declarations, :parent_rule
 
     def initialize(options={})
+      @declarations = Hash.new("")
       @parent_rule  = options[:parent_rule]
       self.css_text = options[:css_text]
     end
 
     def css_text=(css_text)
-      @declarations, @rules = [], Hash.new("")
+      @declarations_list = []
+
+      re = /((?:'(?:\\'|.)*?'|"(?:\\"|.)*?"|\([^\)]*?\)|[^};])+)\s*/
+      css_text.to_s.strip.chomp(";").scan(re).flatten.each do |declaration|
+        next unless declaration.include?(":")
 
-      css_text.to_s.strip.chomp(";").split(";").each do |declaration|
         property, value = declaration.split(":", 2)
-        @declarations << declaration.strip
-        @rules[Inflector.camelize(property.strip)] = parse_value(value.strip)
+        @declarations_list << declaration.strip
+        @declarations[property.strip] = parse_value(value.strip)
       end
     end
 
     def css_text      
-      css_text = @declarations.join("; ")
+      css_text = @declarations_list.join("; ")
       css_text += ";" if css_text != ""
     end
 
     alias_method :to_s, :css_text
 
     def method_missing(name, *args)
-      @rules[name.to_s]
+      @declarations[Inflector.dasherize(name.to_s)]
     end
 
     private

data/lib/stylesheet/css_style_sheet.rb

@@ -1,9 +1,9 @@
 module Stylesheet
   class CssStyleSheet
     attr_accessor :parent, :title
-    attr_reader :href, :media
+    attr_reader :href, :media, :owner_rule
     attr_writer :disabled, :type
-    
+
     def initialize(args)
       if args.kind_of?(String)
         init_with_url(args)
@@ -11,18 +11,19 @@ module Stylesheet
         init_with_hash(args)
       end
     end
-    
+
     def init_with_url(url)
       self.href    = url
       self.media   = ""
       self.content = nil
     end
-    
+
     def init_with_hash(args)
-      @parent  = args[:parent]
-      @title   = args[:title]
-      @type    = args[:type]
-      
+      @parent     = args[:parent]
+      @title      = args[:title]
+      @type       = args[:type]
+      @owner_rule = args[:owner_rule]
+
       self.href     = args[:href]
       self.location = args[:location]
       self.media    = args[:media]
@@ -32,25 +33,25 @@ module Stylesheet
     def disabled?
       @disabled || false
     end
-    
+
     def type
       @type || "text/css"
     end
-    
+
     def href=(url)
       return unless url
       @url  = url
       @href = location.to_s
     end
-    
+
     def media=(media)
       @media ||= MediaList.new(media)
     end
-    
+
     def content=(content)
-      @content = content if content != ""
+      @content = content if content
     end
-    
+
     def content
       @content ||= request_content
     end
@@ -61,15 +62,31 @@ module Stylesheet
 
     alias_method :rules, :css_rules 
 
-    def parent_style_sheet
-      parent
+    def import_rules
+      css_rules.select {|r| r.type == CssRule::IMPORT_RULE }
     end
     
+    def style_rules
+      css_rules.select {|r| r.type == CssRule::STYLE_RULE }
+    end
+
+    def parent_style_sheet
+      @parent if @owner_rule
+    end
+
+    def delete_rule(index)
+      @css_rules.delete_at(index)
+    end
+
+    def insert_rule(rule, index)
+      @css_rules.insert(index, rule)
+    end
+
     def location
       return if inline_css?
       @location ||= Location.new(@url, parent && parent.location)
     end
-    
+
     def location=(location)
       return unless location
 
@@ -83,13 +100,13 @@ module Stylesheet
     def standalone_css?
       !parent
     end
-    
+
     def inline_css?
       !@url || @url == ""
     end
 
     def request_content
-      raise InvalidLocationError unless location.valid?
+      raise InvalidLocationError unless location && location.valid?
       request.get(location.href)
     end
 
@@ -97,4 +114,4 @@ module Stylesheet
       @request ||= Stylesheet.request
     end
   end
-end
+end

data/lib/stylesheet/location.rb

@@ -47,7 +47,7 @@ module Stylesheet
     end
 
     def valid?
-       valid_protocol? && valid_host?
+      !!(valid_protocol? && valid_host?)
     end
     
     def expand_paths_from_parent
@@ -74,19 +74,19 @@ module Stylesheet
     def valid_protocol?
       protocol && protocol != ":"
     end
-    
+
     def valid_host?
       host && host != ""
     end
-    
+
     def valid_port?
       port && port != ""
     end
-    
+
     def standard_port?
       port_80? || port_443?
     end
-    
+
     def port_80?
       uri && uri.port == 80 && uri.scheme == "http"
     end
@@ -100,14 +100,13 @@ module Stylesheet
         URI.parse(url.strip)
       rescue URI::InvalidURIError
         URI.parse(URI.escape(url.strip))
-      end      
+      end
 
     # re-raise external library errors in our namespace
     rescue URI::InvalidURIError => error
       raise Stylesheet::InvalidLocationError.new(
         "#{error.class}: #{error.message}")
     end
-    
+
   end
 end
-

data/lib/stylesheet/request.rb

@@ -8,12 +8,12 @@ module Stylesheet
 
       curl.body_str
 
-    rescue Stylsheet::Error
+    rescue Stylesheet::Error
       raise
 
     # re-raise external library errors in our namespace
     rescue => error
-      raise Stylsheet::Error.new("#{error.class}: #{error.message}")
+      raise Stylesheet::Error.new("#{error.class}: #{error.message}")
     end
 
     def user_agent

data/lib/stylesheet/version.rb

@@ -1,3 +1,3 @@
 module Stylesheet
-  VERSION = "0.1.0"
+  VERSION = "0.1.6"
 end

data/spec/css_charset_rule_spec.rb

@@ -23,17 +23,17 @@ describe CssCharsetRule do
   describe ".matches_rule?" do 
     it "should match text starting with @charset" do 
       matches = CssCharsetRule.matches_rule?(css_text)
-      expect(matches).to be_true
+      expect(matches).to eq true
     end
     
     it "should not match text without at-rule" do 
       matches = CssCharsetRule.matches_rule?("a:link { color: #357ad1; }")
-      expect(matches).to be_false
+      expect(matches).to eq false
     end
 
     it "should not match text without charset" do 
       matches = CssCharsetRule.matches_rule?("@import url(\"import1.css\");")
-      expect(matches).to be_false
+      expect(matches).to eq false
     end    
   end
-end
+end

data/spec/css_font_face_rule_spec.rb

@@ -31,17 +31,17 @@ describe CssFontFaceRule do
   describe ".matches_rule?" do 
     it "should match text starting with @font-face" do 
       matches = CssFontFaceRule.matches_rule?(css_text)
-      expect(matches).to be_true
+      expect(matches).to eq true
     end
     
     it "should not match text without at-rule" do 
       matches = CssFontFaceRule.matches_rule?("a:link { color: #357ad1; }")
-      expect(matches).to be_false
+      expect(matches).to eq false
     end
 
     it "should not match text without font-face" do 
       matches = CssFontFaceRule.matches_rule?("@import url(\"import1.css\");")
-      expect(matches).to be_false
+      expect(matches).to eq false
     end
   end
-end
+end

data/spec/css_import_rule_spec.rb

@@ -21,9 +21,9 @@ describe CssImportRule do
     let(:parent) do 
       CssStyleSheet.new("http://example.com/css_import/stylesheets/screen.css")
     end
-    
+
     let(:rule_url) { "http://example.com/css_import/stylesheets/import1.css"  }
-    
+
     it "parses an url from the style rule" do 
       css_text = "@import url(\"http://example.com/css_import/stylesheets/import1.css\");" 
     
@@ -162,17 +162,17 @@ describe CssImportRule do
   describe ".matches_rule?" do 
     it "should match text starting with @import" do 
       matches = CssImportRule.matches_rule?(css_text)
-      expect(matches).to be_true
+      expect(matches).to eq true
     end
     
     it "should not match text without at-rule" do 
       matches = CssImportRule.matches_rule?("a:link { color: #357ad1; }")
-      expect(matches).to be_false
+      expect(matches).to eq false
     end
 
     it "should not match text without import" do 
       matches = CssImportRule.matches_rule?("@charset \"UTF-8\";")
-      expect(matches).to be_false
+      expect(matches).to eq false
     end
   end
-end
+end

data/spec/css_media_rule_spec.rb

@@ -41,17 +41,17 @@ describe CssMediaRule do
   describe ".matches_rule?" do 
     it "should match text starting with @media" do 
       matches = CssMediaRule.matches_rule?(css_text)
-      expect(matches).to be_true
+      expect(matches).to eq true
     end
 
     it "should not match text without at-rule" do 
       matches = CssMediaRule.matches_rule?("a:link { color: #357ad1; }")
-      expect(matches).to be_false
+      expect(matches).to eq false
     end
 
     it "should not match text without media" do 
       matches = CssMediaRule.matches_rule?("@charset \"UTF-8\";")
-      expect(matches).to be_false
+      expect(matches).to eq false
     end
   end
-end
+end

data/spec/css_rule_list_spec.rb

@@ -23,7 +23,34 @@ describe CssRuleList do
       src: url(\"http://example.com/fonts/VeraSeBd.ttf\");
     }"
   end
+
+  let(:style_w_comments) do     
+    "body {
+      color: #444;
+      background-color: #535353;
+    }
+
+    /* style paragraphs */
+    p {
+      font-size: 90%;
+    }"
+  end
+  
+  let(:style_w_mismatched_comments) do 
+    "body {
+      color: #444;
+      background-color: #535353;
+    }
+
+    */
+    p {
+      font-size: 90%;
+    }"  end
   
+  let(:style_w_empty_rules) do 
+    "#cboxOverlay{color:#ccc}#colorbox{}#cboxTopLeft{width:21px;}"
+  end
+
   describe ".new" do 
     it "parses charset rules" do 
       rules = CssRuleList.new(styles)
@@ -49,6 +76,28 @@ describe CssRuleList do
       rules = CssRuleList.new(styles)
       expect(rules[4]).to be_kind_of(CssFontFaceRule)
     end
+
+    it "removes comments" do 
+      rules = CssRuleList.new(style_w_comments)
+      expect(rules.length).to eq 2
+      expect(rules[0].css_text).to eq "body { color: #444;background-color: #535353;}"
+      expect(rules[1].css_text).to eq "p { font-size: 90%;}"
+    end
+    
+    it "removes mismatched comments" do 
+      rules = CssRuleList.new(style_w_mismatched_comments)
+      expect(rules.length).to eq 2
+      expect(rules[0].css_text).to eq "body { color: #444;background-color: #535353;}"
+      expect(rules[1].css_text).to eq "p { font-size: 90%;}"
+    end
+    
+    it "parses empty rules" do 
+      rules = CssRuleList.new(style_w_empty_rules)
+      expect(rules.length).to eq 3
+      expect(rules[0].css_text).to eq "#cboxOverlay{color:#ccc}"
+      expect(rules[1].css_text).to eq "#colorbox{}"
+      expect(rules[2].css_text).to eq "#cboxTopLeft{width:21px;}"
+    end
   end
   
   describe "#[]" do 

data/spec/css_rule_spec.rb

@@ -26,6 +26,15 @@ describe CssRule do
       src: url(\"http://example.com/fonts/VeraSeBd.ttf\");
     }"
   end
+  
+  let(:moz_doc_text) do 
+    "@-moz-document url-prefix() {
+      .rec_details .rec-ellipsis:before {
+        float:right;
+        content: url(/fade.png);
+      }
+    }"
+  end
 
   describe ".factory" do 
     it "should build an a css style rule" do 
@@ -58,6 +67,12 @@ describe CssRule do
                              :parent_style_sheet => Object.new)
       expect(rule).to be_kind_of(CssFontFaceRule)
     end
+    
+    it "should build a null rule" do 
+      rule = CssRule.factory(:css_text           => moz_doc_text, 
+                             :parent_style_sheet => Object.new)
+      expect(rule).to be_kind_of(CssNullRule)
+    end
   end
 
   describe "#css_text" do 

data/spec/css_style_declaration_spec.rb

@@ -51,6 +51,17 @@ describe CssStyleDeclaration do
     end
   end
 
+  describe "rules" do 
+    it "returns a hash of rules" do 
+      expected = {"color"              => "#444", 
+                  "font-size"          => "12px", 
+                  "font-family"        => "Arial, Verdana", 
+                  "border-left"        => "1px solid red", 
+                  "border-right-width" => "1px", 
+                  "background-color"   => "#535353"}
+      expect(declaration.declarations).to eq expected
+    end
+  end
 
   describe "#parent_rule" do 
     it "returns the parent css style rule" do 
@@ -68,4 +79,33 @@ describe CssStyleDeclaration do
       expect(declaration.overflow).to eq ""
     end
   end
+
+  describe "a declaration with base64 encoded rule" do 
+    let(:css_text) do 
+      "background:url(data:image/svg+xml;base64,PD94bWwgdmVyc2lvbj0iMS4wIi);color:#1d6299;"
+    end
+
+    it "returns the css style declaration for rules with base64 data" do 
+      rule = CssStyleRule.new(:css_text => "div.section { #{css_text} }")
+      decl = CssStyleDeclaration.new(:css_text => css_text, :parent_rule => rule)
+      
+      expected = {"background" => "url(data:image/svg+xml;base64,PD94bWwgdmVyc2lvbj0iMS4wIi)",
+                  "color"      => "#1d6299"}
+      expect(decl.declarations).to eq expected
+    end
+  end
+  
+  describe "a declaration with an invalid rule" do 
+    let(:css_text) do 
+      "padding0;color:#1d6299;"
+    end
+
+    it "skips the invalid rule" do 
+      rule = CssStyleRule.new(:css_text => "div.section { #{css_text} }")
+      decl = CssStyleDeclaration.new(:css_text => css_text, :parent_rule => rule)
+      
+      expected = {"color" => "#1d6299"}
+      expect(decl.declarations).to eq expected
+    end
+  end
 end

data/spec/css_style_rule_spec.rb

@@ -42,12 +42,12 @@ describe CssStyleRule do
   describe ".matches_rule?" do 
     it "should match text that doesn't begin with an at-rule" do 
       matches = CssStyleRule.matches_rule?(css_text)
-      expect(matches).to be_true
+      expect(matches).to eq true
     end
     
     it "should not match rules starting with at-rule" do 
       matches = CssStyleRule.matches_rule?("@import url(\"import1.css\");")
-      expect(matches).to be_false
+      expect(matches).to eq false
     end
   end
-end
+end

data/spec/css_style_sheet_spec.rb

@@ -8,10 +8,10 @@ describe CssStyleSheet do
   describe "#disabled" do 
     it "shows if style sheet is disabled" do 
       sheet = CssStyleSheet.new("http://example.com/css/stylesheets/screen.css")
-      expect(sheet.disabled?).to be_false
+      expect(sheet.disabled?).to eq false
 
       sheet.disabled = true
-      expect(sheet.disabled?).to be_true
+      expect(sheet.disabled?).to eq true
     end
   end
 
@@ -90,15 +90,20 @@ describe CssStyleSheet do
     end
   end
 
-  describe "#owner_node" do 
-    it "references owning node" do 
+  describe "#parent_style_sheet" do 
+    let(:parent) do 
+      CssStyleSheet.new("http://example.com/css_import/stylesheets/screen.css")
+    end
 
+    it "should be nil for non-imported sheets" do 
+      expect(parent.parent_style_sheet).to be_nil
     end
-  end
-  
-  describe "#parent_style_sheet" do 
+
     it "references parent style sheet" do 
+      text = "@import url(\"import1.css\");" 
+      rule = CssImportRule.new(:css_text => text, :parent_style_sheet => parent)
 
+      expect(rule.style_sheet.parent_style_sheet).to eq parent
     end
   end
 
@@ -122,7 +127,18 @@ describe CssStyleSheet do
       expect(sheet.type).to eq "text/css"
     end
   end
-  
+
+  describe "#content" do
+    it "doesn't fetch for empty inline styles" do
+      sheet = CssStyleSheet.new(content: "")
+
+      expect {
+        sheet.css_rules.map {|r| r.content }
+      }.not_to raise_error
+    end
+
+  end
+
   describe "#css_rules" do 
     it "returns a list of css rules found in the style sheet" do 
       sheet = CssStyleSheet.new("http://example.com/css/stylesheets/screen.css")
@@ -137,21 +153,69 @@ describe CssStyleSheet do
     end
   end
   
+  describe "#import_rules" do 
+    it "returns all import rules from style sheet" do 
+      sheet = CssStyleSheet.new("http://example.com/css_import/stylesheets/screen.css")
+      expect(sheet.import_rules.length).to eq 9
+      
+      sheet.import_rules.each do |rule|
+        expect(rule.type).to eq CssRule::IMPORT_RULE
+      end
+    end
+  end
+  
+  describe "#style_rules" do 
+    it "returns all style rules from style sheet" do 
+      sheet = CssStyleSheet.new("http://example.com/css_import/stylesheets/screen.css")
+      expect(sheet.style_rules.length).to eq 1
+
+      sheet.style_rules.each do |rule|
+        expect(rule.type).to eq CssRule::STYLE_RULE
+      end
+    end
+  end
+  
   describe "#owner_rule" do 
+    let(:parent) do 
+      CssStyleSheet.new("http://example.com/css_import/stylesheets/screen.css")
+    end
+
+    it "should be nil for non-imported sheets" do 
+      expect(parent.owner_rule).to be_nil
+    end
+
     it "references the owner rule" do 
+      text = "@import url(\"import1.css\");" 
+      rule = CssImportRule.new(:css_text => text, :parent_style_sheet => parent)
 
+      expect(rule.style_sheet.owner_rule).to eq rule
     end
   end
   
   describe "#delete_rule" do 
     it "deletes a rule from the css rules" do 
+      css = "div {\n  background-color: #aaa;\n} span {\n  color: #444; \n}"
+      sheet = CssStyleSheet.new(:content => css)
 
+      sheet.css_rules
+      expect(sheet.css_rules.length).to eq 2
+
+      sheet.delete_rule(0)
+      expect(sheet.css_rules.length).to eq 1
+      expect(sheet.css_rules[0].css_text).to eq "span { color: #444;}"
     end
   end
   
   describe "#insert_rule" do 
     it "adds a rule to the css rules" do 
+      css = "div {\n  background-color: #aaa;\n} span {\n  color: #444; \n}"
+      sheet = CssStyleSheet.new(:content => css)
+
+      expect(sheet.css_rules.length).to eq 2
 
+      sheet.insert_rule("#blanc { color: white }", 0); 
+      expect(sheet.css_rules.length).to eq 3
+      
     end
   end
-end
+end

data/spec/location_spec.rb

@@ -8,18 +8,14 @@ describe Location do
       location = Location.new(url)
       expect(location.host).to eq "initvisual.com"
     end
-    
+
     it "should accept relative urls" do 
       location = Location.new("/some/path.html")
       expect(location.to_s).to eq "/some/path.html"
     end
-    
-    it "should throw error for invalid host" do 
-      expect { Location.new("http://") }.to raise_error(InvalidLocationError)
-    end
   end
-  
-  
+
+
   describe "#new with parent" do 
     it "should not expand a full url" do 
       parent   = Location.new("http://example.com/css/url.html")
@@ -31,7 +27,7 @@ describe Location do
     it "should expand a relative path into a full path given the parent" do 
       parent   = Location.new("http://example.com/css/relative.html")
       location = Location.new("stylesheets/screen.css", parent)
-      
+
       expect(location.to_s).to eq "http://example.com/css/stylesheets/screen.css"
     end
 
@@ -40,9 +36,9 @@ describe Location do
       location = Location.new("/css/stylesheets/screen.css", parent)
 
       expect(location.to_s).to eq "http://example.com/css/stylesheets/screen.css"
-    end    
+    end
   end
-  
+
 
   describe "#host" do 
     it "should parse out the url host" do 
@@ -50,27 +46,27 @@ describe Location do
       expect(location.host).to eq "initvisual.com"
     end
   end
-  
-  describe "#host=" do 
-    it "should assign host to url" do 
+
+  describe "#host=" do
+    it "should assign host to url" do
       location = Location.new(url)
-      
+
       location.host = "derekdevries.com"
       expect(location.host).to eq "derekdevries.com"
     end
   end
-  
+
   describe "#hostname" do 
     it "should parse out the url hostname" do 
       location = Location.new(url)
       expect(location.hostname).to eq "initvisual.com"
     end
   end
-  
+
   describe "#hostname=" do 
     it "should assign hostname to url" do 
       location = Location.new(url)
-      
+
       location.hostname = "derekdevries.com"
       expect(location.hostname).to eq "derekdevries.com"
     end
@@ -86,14 +82,14 @@ describe Location do
   describe "#pathname=" do 
     it "should assign path to url" do 
       location = Location.new(url)
-      
+
       location.pathname = "/work"
       expect(location.pathname).to eq "/work"
     end
 
     it "add initial forward-slash if not given" do 
       location = Location.new(url)
-      
+
       location.pathname = "work"
       expect(location.pathname).to eq "/work"
     end
@@ -243,18 +239,23 @@ describe Location do
   describe "#valid?" do 
     it "should be true with a valid host and protocol" do 
       location = Location.new("http://initvisual.com")
-      expect(location.valid?).to be_true
+      expect(location.valid?).to eq true 
     end
 
-    it "should be false for an invalid host" do 
+    it "should be false for an invalid host" do
       location = Location.new("/asdf")
-      expect(location.valid?).to be_false      
+      expect(location.valid?).to eq false
+    end
+
+    it "should be false for a missing host" do
+      location = Location.new("http://")
+      expect(location.valid?).to eq false
     end
 
-    it "should be false for an invalid protocol" do 
+    it "should be false for an invalid protocol" do
       location = Location.new("foo.com/asdf")
-      expect(location.valid?).to be_false
+      expect(location.valid?).to eq false
     end
   end
   
-end
+end

data/spec/request_spec.rb

@@ -0,0 +1,12 @@
+require 'spec_helper'
+
+describe Stylesheet do
+  describe "#get" do 
+    it "must return results" do
+      request = Request.new
+
+      # body = request.get("http://google.com")
+      # expect(body).to include "<html"
+    end
+  end
+end

data/stylesheet.gemspec

@@ -6,7 +6,7 @@ require 'stylesheet/version'
 Gem::Specification.new do |gem|
   gem.name          = "stylesheet"
   gem.version       = Stylesheet::VERSION
-  gem.summary       = %q{Stylesheet is a CSS style sheet parser}
+  gem.summary       = %q{A CSS parser based on the DOM API}
   gem.description   = gem.summary
 
   gem.required_ruby_version = '>= 1.9.3'
@@ -24,6 +24,7 @@ Gem::Specification.new do |gem|
   gem.add_runtime_dependency("curb",     "~> 0.8")
   gem.add_runtime_dependency("nokogiri", "~> 1.5")
 
+  gem.add_development_dependency("rake")
   gem.add_development_dependency("rspec", "~> 2.9")
   gem.add_development_dependency("cucumber", "~> 1.2")
 

metadata

@@ -1,136 +1,135 @@
 --- !ruby/object:Gem::Specification
 name: stylesheet
 version: !ruby/object:Gem::Version
-  version: 0.1.0
-  prerelease: 
+  version: 0.1.6
 platform: ruby
 authors:
 - Derek DeVries
-autorequire: 
+autorequire:
 bindir: bin
 cert_chain: []
-date: 2013-04-29 00:00:00.000000000 Z
+date: 2020-08-24 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: curb
   requirement: !ruby/object:Gem::Requirement
-    none: false
     requirements:
-    - - ~>
+    - - "~>"
       - !ruby/object:Gem::Version
         version: '0.8'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
-    none: false
     requirements:
-    - - ~>
+    - - "~>"
       - !ruby/object:Gem::Version
         version: '0.8'
 - !ruby/object:Gem::Dependency
   name: nokogiri
   requirement: !ruby/object:Gem::Requirement
-    none: false
     requirements:
-    - - ~>
+    - - "~>"
       - !ruby/object:Gem::Version
         version: '1.5'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
-    none: false
     requirements:
-    - - ~>
+    - - "~>"
       - !ruby/object:Gem::Version
         version: '1.5'
+- !ruby/object:Gem::Dependency
+  name: rake
+  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: rspec
   requirement: !ruby/object:Gem::Requirement
-    none: false
     requirements:
-    - - ~>
+    - - "~>"
       - !ruby/object:Gem::Version
         version: '2.9'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
-    none: false
     requirements:
-    - - ~>
+    - - "~>"
       - !ruby/object:Gem::Version
         version: '2.9'
 - !ruby/object:Gem::Dependency
   name: cucumber
   requirement: !ruby/object:Gem::Requirement
-    none: false
     requirements:
-    - - ~>
+    - - "~>"
       - !ruby/object:Gem::Version
         version: '1.2'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
-    none: false
     requirements:
-    - - ~>
+    - - "~>"
       - !ruby/object:Gem::Version
         version: '1.2'
 - !ruby/object:Gem::Dependency
   name: guard
   requirement: !ruby/object:Gem::Requirement
-    none: false
     requirements:
-    - - ~>
+    - - "~>"
       - !ruby/object:Gem::Version
         version: '1.7'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
-    none: false
     requirements:
-    - - ~>
+    - - "~>"
       - !ruby/object:Gem::Version
         version: '1.7'
 - !ruby/object:Gem::Dependency
   name: guard-rspec
   requirement: !ruby/object:Gem::Requirement
-    none: false
     requirements:
-    - - ~>
+    - - "~>"
       - !ruby/object:Gem::Version
         version: '2.5'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
-    none: false
     requirements:
-    - - ~>
+    - - "~>"
       - !ruby/object:Gem::Version
         version: '2.5'
 - !ruby/object:Gem::Dependency
   name: rb-fsevent
   requirement: !ruby/object:Gem::Requirement
-    none: false
     requirements:
-    - - ~>
+    - - "~>"
       - !ruby/object:Gem::Version
         version: '0.9'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
-    none: false
     requirements:
-    - - ~>
+    - - "~>"
       - !ruby/object:Gem::Version
         version: '0.9'
-description: Stylesheet is a CSS style sheet parser
+description: A CSS parser based on the DOM API
 email:
 - derek@sportspyder.com
 executables: []
 extensions: []
 extra_rdoc_files: []
 files:
-- .gitignore
+- ".gitignore"
 - Gemfile
 - Guardfile
 - LICENSE.txt
@@ -148,6 +147,7 @@ files:
 - lib/stylesheet/css_font_face_rule.rb
 - lib/stylesheet/css_import_rule.rb
 - lib/stylesheet/css_media_rule.rb
+- lib/stylesheet/css_null_rule.rb
 - lib/stylesheet/css_rule.rb
 - lib/stylesheet/css_rule_list.rb
 - lib/stylesheet/css_style_declaration.rb
@@ -205,6 +205,7 @@ files:
 - spec/inflector_spec.rb
 - spec/location_spec.rb
 - spec/media_list_spec.rb
+- spec/request_spec.rb
 - spec/spec_helper.rb
 - spec/stubs/fake_request.rb
 - spec/style_sheet_list_spec.rb
@@ -213,28 +214,26 @@ files:
 homepage: https://github.com/devrieda/stylesheet
 licenses:
 - MIT
-post_install_message: 
+metadata: {}
+post_install_message:
 rdoc_options: []
 require_paths:
 - lib
 required_ruby_version: !ruby/object:Gem::Requirement
-  none: false
   requirements:
-  - - ! '>='
+  - - ">="
     - !ruby/object:Gem::Version
       version: 1.9.3
 required_rubygems_version: !ruby/object:Gem::Requirement
-  none: false
   requirements:
-  - - ! '>='
+  - - ">="
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubyforge_project: 
-rubygems_version: 1.8.25
-signing_key: 
-specification_version: 3
-summary: Stylesheet is a CSS style sheet parser
+rubygems_version: 3.1.2
+signing_key:
+specification_version: 4
+summary: A CSS parser based on the DOM API
 test_files:
 - features/document_styles.feature
 - features/rule_declarations.feature
@@ -287,8 +286,8 @@ test_files:
 - spec/inflector_spec.rb
 - spec/location_spec.rb
 - spec/media_list_spec.rb
+- spec/request_spec.rb
 - spec/spec_helper.rb
 - spec/stubs/fake_request.rb
 - spec/style_sheet_list_spec.rb
 - spec/version_spec.rb
-has_rdoc: