r2 0.2.5 → 0.2.6

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: d5ab6e294013e8786fc50aae34d70baa7a62f10b
-  data.tar.gz: a8b49f9b4b9407050145114607f23c7d391c3a6a
+  metadata.gz: f8550ffae3d9358c1257e8163f6a5d03d945d09f
+  data.tar.gz: 3a0e2808776280f8c3a87de5d1cf7baa5a18af8d
 SHA512:
-  metadata.gz: f3950af1429c22b800a0c94a297ee427b95c125dbc806f516c946f3f3c1855d3ff8a08f6adc01c7515f02e66f32e793243efe48ed558fb718ffaf65140efe086
-  data.tar.gz: b36f3c6a247163831b299922ba4114067b5052eaef920cd03f51c008fa9c94f6f76e7c60d3c9217573f1134a12c68463e1ba2d45390fb1b8b985c1dcf720585c
+  metadata.gz: d1de570c1fc501a5e278d2757bba45aacce956ef3d39660e9e3f661de8bbf7812391729407363882d9a98bb386bbff37ec8de588f85e9b49592679d3175aac1c
+  data.tar.gz: 6c80710ffed22c49310fec1766d478e4df07c10af0a9bed75ec3a1ea1c02c18f3da29d311517355fcff44c374fab99c183ca9fc39c034a18300c922938f83f96

data/README.md

@@ -23,6 +23,7 @@ Report bugs in the github project at http://github.com/mzsanford/r2rb
 
 ## Change Log
 
+ * v0.2.6 - Handle multi-row selectors (fix from [@n0nick](https://github.com/n0nick))
  * v0.2.5 - Handle `background:` shorthand
  * v0.2.4 - Handle `url()` properties better
    * [BUG] - Handle `url()` without embedded semi-colons and with trailing parameters

data/lib/r2.rb

@@ -110,7 +110,7 @@ module R2
       return '' unless css
 
       css.gsub(/\/\*[\s\S]+?\*\//, '').   # comments
-         gsub(/[\n\r]/, '').              # line breaks and carriage returns
+         gsub(/[\n\r]+/, ' ').            # line breaks and carriage returns
          gsub(/\s*([:;,\{\}])\s*/, '\1'). # space between selectors, declarations, properties and values
          gsub(/\s+/, ' ').                # replace multiple spaces with single spaces
          gsub(/(\A\s+|\s+\z)/, '')        # leading or trailing spaces

data/lib/r2/version.rb

@@ -1,3 +1,3 @@
 module R2
-  VERSION = "0.2.5"
+  VERSION = "0.2.6"
 end

data/r2.gemspec

@@ -14,8 +14,9 @@ Gem::Specification.new do |s|
 
   s.rubyforge_project = "r2"
 
-  s.add_development_dependency 'rake'
-  s.add_development_dependency 'rspec', '~> 2.13.0'
+  s.add_development_dependency 'rake', '~> 0'
+  s.add_development_dependency 'rspec', '~> 3.4', '>= 3.4.0'
+  s.add_development_dependency 'rspec-mocks', '~> 3.4.0', '>= 3.4.0'
 
   s.files         = `git ls-files`.split("\n")
   s.test_files    = `git ls-files -- {test,spec,features}/*`.split("\n")

data/spec/r2_spec.rb

@@ -8,8 +8,8 @@ describe R2 do
     let(:css) { "body { direction: rtl; }" }
 
     it "provides a shortcut to .new#r2" do
-      R2::Swapper.should_receive(:new).and_return(r2)
-      r2.should_receive(:r2).with(css)
+      allow(R2::Swapper).to receive(:new).and_return(r2)
+      expect(r2).to receive(:r2).with(css)
       R2.r2(css)
     end
   end
@@ -21,7 +21,7 @@ describe R2::Swapper do
 
   describe "#r2" do
     it "processes CSS" do
-      r2.r2("/* comment */\nbody { direction: rtl; }\nimg { padding: 4px;}").should == "body{direction:ltr;}img{padding:4px;}"
+      expect(r2.r2("/* comment */\nbody { direction: rtl; }\nimg { padding: 4px;}")).to eq("body{direction:ltr;}img{padding:4px;}")
     end
 
     it "handles media queries" do
@@ -37,7 +37,7 @@ describe R2::Swapper do
 
       flipped_css = r2.r2(css)
 
-      flipped_css.should == expected_result
+      expect(flipped_css).to eq(expected_result)
     end
 
     it "handles background-image declarations" do
@@ -52,7 +52,7 @@ describe R2::Swapper do
 
       flipped_css = r2.r2(css)
 
-      flipped_css.should == expected_result
+      expect(flipped_css).to eq(expected_result)
     end
 
     # background: #000 url("spree/frontend/cart.png") no-repeat left center;
@@ -71,7 +71,7 @@ describe R2::Swapper do
 
       flipped_css = r2.r2(css)
 
-      flipped_css.should == expected_result
+      expect(flipped_css).to eq(expected_result)
     end
 
 
@@ -90,146 +90,165 @@ describe R2::Swapper do
 
       flipped_css = r2.r2(css)
 
-      flipped_css.should == expected_result
+      expect(flipped_css).to eq(expected_result)
+    end
+
+    it "handles newline correctly" do
+      css =
+       "fieldset[disabled]\n" +
+       "input[type=\"checkbox\"], fieldset[disabled]\n" +
+       ".radio {\n" +
+       "  cursor: not-allowed; }"
+
+      expected =
+        'fieldset[disabled] input[type="checkbox"],' +
+        'fieldset[disabled] .radio' +
+        '{cursor:not-allowed;}'
+
+      expect(r2.r2(css)).to eq(expected)
     end
   end
 
   describe "#declaration_swap" do
     it "should handle nil" do
-      r2.declaration_swap(nil).should == ''
+      expect(r2.declaration_swap(nil)).to eq('')
     end
 
     it "should handle invalid declarations" do
-      r2.declaration_swap("not a decl").should == ''
+      expect(r2.declaration_swap("not a decl")).to eq('')
     end
 
     it "should swap a swappable parameter" do
-      r2.declaration_swap("padding-right:4px").should == 'padding-left:4px;'
+      expect(r2.declaration_swap("padding-right:4px")).to eq('padding-left:4px;')
     end
 
     it "should swap a swappable quad parameter" do
-      r2.declaration_swap("padding:1px 2px 3px 4px").should == 'padding:1px 4px 3px 2px;'
+      expect(r2.declaration_swap("padding:1px 2px 3px 4px")).to eq('padding:1px 4px 3px 2px;')
     end
 
     it "should ignore other parameters" do
-      r2.declaration_swap("foo:bar").should == 'foo:bar;'
+      expect(r2.declaration_swap("foo:bar")).to eq('foo:bar;')
     end
   end
 
   describe "#minimize" do
     it "should handle nil" do
-      r2.minimize(nil).should == ""
+      expect(r2.minimize(nil)).to eq("")
     end
 
     it "should strip comments" do
-      r2.minimize("/* comment */foo").should == "foo"
+      expect(r2.minimize("/* comment */foo")).to eq("foo")
+    end
+
+    it "should convert newlines to spaces" do
+      expect(r2.minimize("foo\nbar")).to eq("foo bar")
     end
 
-    it "should remove newlines" do
-      r2.minimize("foo\nbar").should == "foobar"
+    it "should convert multiple newlines to a space" do
+      expect(r2.minimize("foo\n\n\nbar")).to eq("foo bar")
     end
 
-    it "should remove carriage returns" do
-      r2.minimize("foo\rbar").should == "foobar"
+    it "should convert carriage returns to spaces" do
+      expect(r2.minimize("foo\rbar")).to eq("foo bar")
     end
 
     it "should collapse multiple spaces into one" do
-      r2.minimize("foo       bar").should == "foo bar"
+      expect(r2.minimize("foo       bar")).to eq("foo bar")
     end
   end
 
   describe "#direction_swap" do
     it "should swap 'rtl' to 'ltr'" do
-      r2.direction_swap('rtl').should == 'ltr'
+      expect(r2.direction_swap('rtl')).to eq('ltr')
     end
 
     it "should swap 'ltr' to 'rtl'" do
-      r2.direction_swap('ltr').should == 'rtl'
+      expect(r2.direction_swap('ltr')).to eq('rtl')
     end
 
     it "should ignore values other than 'ltr' and 'rtl'" do
       [nil, '', 'foo'].each do |val|
-        r2.direction_swap(val).should == val
+        expect(r2.direction_swap(val)).to eq(val)
       end
     end
   end
 
   describe "#side_swap" do
     it "should swap 'right' to 'left'" do
-      r2.side_swap('right').should == 'left'
+      expect(r2.side_swap('right')).to eq('left')
     end
 
     it "should swap 'left' to 'right'" do
-      r2.side_swap('left').should == 'right'
+      expect(r2.side_swap('left')).to eq('right')
     end
 
     it "should ignore values other than 'left' and 'right'" do
       [nil, '', 'foo'].each do |val|
-        r2.side_swap(val).should == val
+        expect(r2.side_swap(val)).to eq(val)
       end
     end
   end
 
   describe "#quad_swap" do
     it "should swap a valid quad value" do
-      r2.quad_swap("1px 2px 3px 4px").should == "1px 4px 3px 2px"
+      expect(r2.quad_swap("1px 2px 3px 4px")).to eq("1px 4px 3px 2px")
     end
 
     it "should skip a pair value" do
-      r2.quad_swap("1px 2px").should == "1px 2px"
+      expect(r2.quad_swap("1px 2px")).to eq("1px 2px")
     end
   end
 
   describe "#shadow_swap" do
     it "should swap a 2 arg value" do
-      r2.shadow_swap("1px 2px").should == "-1px 2px"
+      expect(r2.shadow_swap("1px 2px")).to eq("-1px 2px")
     end
 
     it "should swap a 2 arg value from rtl to ltr" do
-      r2.shadow_swap("-1px 2px").should == "1px 2px"
+      expect(r2.shadow_swap("-1px 2px")).to eq("1px 2px")
     end
 
     it "should swap a 3 arg value" do
-      r2.shadow_swap("1px 2px #000").should == "-1px 2px #000"
+      expect(r2.shadow_swap("1px 2px #000")).to eq("-1px 2px #000")
     end
 
     it "should swap a 4 arg value" do
-      r2.shadow_swap("1px 2px 3px 4px").should == "-1px 2px 3px 4px"
+      expect(r2.shadow_swap("1px 2px 3px 4px")).to eq("-1px 2px 3px 4px")
     end
 
     it "should swap a 5 arg value" do
-      r2.shadow_swap("1px 2px 3px 4px #000").should == "-1px 2px 3px 4px #000"
+      expect(r2.shadow_swap("1px 2px 3px 4px #000")).to eq("-1px 2px 3px 4px #000")
     end
 
     it "should swap a 6 arg value" do
-      r2.shadow_swap("1px 2px 3px 4px #000 inset").should == "-1px 2px 3px 4px #000 inset"
+      expect(r2.shadow_swap("1px 2px 3px 4px #000 inset")).to eq("-1px 2px 3px 4px #000 inset")
     end
 
     it "should swap value starting with inset" do
-      r2.shadow_swap("inset 1px 2px").should == "-1px 2px inset"
+      expect(r2.shadow_swap("inset 1px 2px")).to eq("-1px 2px inset")
     end
 
     it "should swap multiple values" do
-      r2.shadow_swap("inset 1px 2px, 1px 2px #000").should == "-1px 2px inset, -1px 2px #000"
+      expect(r2.shadow_swap("inset 1px 2px, 1px 2px #000")).to eq("-1px 2px inset, -1px 2px #000")
     end
 
     it "should swap multiple values (with rgba)" do
-      r2.shadow_swap("inset 1px 2px rgba(0,0,0,0.2), 1px 2px #000").should == "-1px 2px rgba(0,0,0,0.2) inset, -1px 2px #000"
+      expect(r2.shadow_swap("inset 1px 2px rgba(0,0,0,0.2), 1px 2px #000")).to eq("-1px 2px rgba(0,0,0,0.2) inset, -1px 2px #000")
     end
 
   end
 
   describe "#border_radius_swap" do
     it "should swap a valid quad value" do
-      r2.border_radius_swap("1px 2px 3px 4px").should == "2px 1px 4px 3px"
+      expect(r2.border_radius_swap("1px 2px 3px 4px")).to eq("2px 1px 4px 3px")
     end
 
     it "should skip a triple value" do
-      r2.border_radius_swap("1px 2px 3px").should == "2px 1px 2px 3px"
+      expect(r2.border_radius_swap("1px 2px 3px")).to eq("2px 1px 2px 3px")
     end
 
     it "should skip a pair value" do
-      r2.border_radius_swap("1px 2px").should == "2px 1px"
+      expect(r2.border_radius_swap("1px 2px")).to eq("2px 1px")
     end
   end
 
@@ -237,23 +256,23 @@ describe R2::Swapper do
 
     context "with a single value" do
       it "should ignore a named-vertical" do
-        r2.background_position_swap('top').should == 'top'
+        expect(r2.background_position_swap('top')).to eq('top')
       end
 
       it "should swap a named-horizontal 'left'" do
-        r2.background_position_swap('left').should == 'right'
+        expect(r2.background_position_swap('left')).to eq('right')
       end
 
       it "should swap a named-horizontal 'right'" do
-        r2.background_position_swap('right').should == 'left'
+        expect(r2.background_position_swap('right')).to eq('left')
       end
 
       it "should invert a percentage" do
-        r2.background_position_swap('25%').should == '75%'
+        expect(r2.background_position_swap('25%')).to eq('75%')
       end
 
       it "should convert a unit value" do
-        r2.background_position_swap('25px').should == 'right 25px center'
+        expect(r2.background_position_swap('25px')).to eq('right 25px center')
       end
     end
 
@@ -264,35 +283,35 @@ describe R2::Swapper do
       # See: http://dev.w3.org/csswg/css3-background/#background-position
 
       it "should swap named-horizontal and ignore named-vertical" do
-        r2.background_position_swap('right bottom').should == 'left bottom'
+        expect(r2.background_position_swap('right bottom')).to eq('left bottom')
       end
 
       it "should swap named-horizontal and ignore unit-vertical" do
-        r2.background_position_swap('left 100px').should == 'right 100px'
+        expect(r2.background_position_swap('left 100px')).to eq('right 100px')
       end
 
       it "should convert unit-horizontal" do
-        r2.background_position_swap('100px center').should == 'right 100px center'
+        expect(r2.background_position_swap('100px center')).to eq('right 100px center')
       end
 
       it "should swap named-horizontal and ignore percentage-vertical" do
-        r2.background_position_swap('left 0%').should == 'right 0%'
+        expect(r2.background_position_swap('left 0%')).to eq('right 0%')
       end
 
       it "should invert first percentage-horizontal value in a pair" do
-        r2.background_position_swap('25% 100%').should == '75% 100%'
+        expect(r2.background_position_swap('25% 100%')).to eq('75% 100%')
       end
     end
 
     context "with a triplet of values" do
       it "should swap named-horizontal" do
-        r2.background_position_swap('left 20px center').should == 'right 20px center'
+        expect(r2.background_position_swap('left 20px center')).to eq('right 20px center')
       end
     end
 
     context "with a quad of values" do
       it "should swap named-horizontal value" do
-        r2.background_position_swap('bottom 10px left 20px').should == 'bottom 10px right 20px'
+        expect(r2.background_position_swap('bottom 10px left 20px')).to eq('bottom 10px right 20px')
       end
     end
   end

metadata

@@ -1,43 +1,69 @@
 --- !ruby/object:Gem::Specification
 name: r2
 version: !ruby/object:Gem::Version
-  version: 0.2.5
+  version: 0.2.6
 platform: ruby
 authors:
 - Matt Sanford
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2014-05-28 00:00:00.000000000 Z
+date: 2015-11-20 00:00:00.000000000 Z
 dependencies:
 - !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
     requirements:
-    - - ~>
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: 2.13.0
+        version: '3.4'
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: 3.4.0
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ~>
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '3.4'
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: 3.4.0
+- !ruby/object:Gem::Dependency
+  name: rspec-mocks
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: 3.4.0
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: 3.4.0
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: 3.4.0
+    - - ">="
       - !ruby/object:Gem::Version
-        version: 2.13.0
+        version: 3.4.0
 description: CSS flipper for right-to-left processing. A Ruby port of https://github.com/ded/r2
 email:
 - matt@twitter.com
@@ -46,8 +72,8 @@ executables:
 extensions: []
 extra_rdoc_files: []
 files:
-- .gitignore
-- .travis.yml
+- ".gitignore"
+- ".travis.yml"
 - Gemfile
 - Guardfile
 - LICENSE
@@ -68,17 +94,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: r2
-rubygems_version: 2.0.3
+rubygems_version: 2.2.2
 signing_key: 
 specification_version: 4
 summary: CSS flipper for right-to-left processing