sucker 1.3.0 → 1.3.1

data/lib/sucker/version.rb

@@ -1,3 +1,3 @@
 module Sucker
-  VERSION = '1.3.0'
+  VERSION = '1.3.1'
 end

data/spec/sucker/response_spec.rb

@@ -48,6 +48,29 @@ module Sucker
       end
     end
 
+    describe "#each" do
+      context "when a block is given" do
+        it "yields each match to a block" do
+          has_yielded = false
+
+          response.each("ItemAttributes") do |item|
+            has_yielded = true
+            item.should be_an_instance_of Hash
+          end
+
+          has_yielded.should be_true
+        end
+      end
+
+      context "when no block is given" do
+        it "raises error" do
+          expect do
+            response.each("ItemAttributes")
+          end.to raise_error(LocalJumpError)
+        end
+      end
+    end
+
     describe '#find' do
       context 'when there are matches' do
         it 'returns an array of matching nodes' do
@@ -63,6 +86,24 @@ module Sucker
       end
     end
 
+    describe "#map" do
+      context "when a block is given" do
+        it "yields each match to a block and maps returned values" do
+          titles = response.map("ItemAttributes") { |item| item["Title"] }
+
+          titles.count.should eql 2
+        end
+      end
+
+      context "when no block is given" do
+        it "raises error" do
+          expect do
+            response.map("ItemAttributes")
+          end.to raise_error(LocalJumpError)
+        end
+      end
+    end
+
     describe '#[]' do
       it 'is an alias of #find' do
         response['Foo'].should eql response.find('Foo')

data/spec/sucker/response_spec.rbc

@@ -123,7 +123,7 @@ x
 6
 Sucker
 i
-122
+144
 5
 7
 0
@@ -245,6 +245,28 @@ i
 50
 14
 1
+15
+5
+7
+28
+64
+56
+29
+47
+50
+14
+1
+15
+5
+7
+30
+64
+56
+31
+47
+50
+14
+1
 11
 I
 7
@@ -257,7 +279,7 @@ I
 I
 -2
 p
-28
+32
 s
 20
 spec/sucker/response
@@ -1316,7 +1338,7 @@ x
 context
 s
 5
-#find
+#each
 M
 1
 p
@@ -1366,8 +1388,8 @@ I
 p
 5
 s
-22
-when there are matches
+21
+when a block is given
 M
 1
 p
@@ -1406,8 +1428,8 @@ I
 p
 3
 s
-34
-returns an array of matching nodes
+28
+yields each match to a block
 M
 1
 p
@@ -1421,32 +1443,36 @@ x
 6
 Sucker
 i
-21
+25
+3
+19
+0
+15
 5
 48
 0
 7
 1
 64
-49
+56
 2
+50
+3
 1
-5
+15
+20
+0
 5
 48
-3
-47
-49
 4
-1
 49
 5
 1
 11
 I
-4
+5
 I
-0
+1
 I
 0
 I
@@ -1459,55 +1485,8 @@ x
 8
 response
 s
-4
-ASIN
-x
-4
-find
-x
-5
-asins
-x
-3
-eql
-x
-6
-should
-p
-3
-I
-0
-I
-36
-I
-15
-x
-51
-/Users/snl/code/sucker/spec/sucker/response_spec.rb
-p
-0
-x
-2
-it
-p
-3
-I
-0
-I
-35
-I
-b
-x
-51
-/Users/snl/code/sucker/spec/sucker/response_spec.rb
-p
-0
-x
-7
-context
-s
-25
-when there are no matches
+14
+ItemAttributes
 M
 1
 p
@@ -1521,70 +1500,28 @@ x
 6
 Sucker
 i
-11
-5
-7
+23
+57
+19
 0
-64
-56
-1
-47
-50
+15
 2
-1
-11
-I
-4
-I
-0
-I
-0
-I
-0
-I
--2
-p
-3
-s
 22
-returns an empty array
-M
-1
-p
-2
-x
-9
-for_block
-t
-n
-x
-6
-Sucker
-i
-25
-5
-48
-0
-7
-1
-64
-49
-2
 1
-19
 0
 15
 20
 0
 5
-35
+45
 0
+1
 47
 49
-3
+2
 1
 49
-4
+3
 1
 11
 I
@@ -1592,40 +1529,38 @@ I
 I
 1
 I
-0
-I
-0
+1
 I
--2
+1
+n
 p
-5
-x
-8
-response
-s
-3
-Foo
+4
 x
 4
-find
+Hash
+n
 x
-3
-eql
+17
+be_an_instance_of
 x
 6
 should
 p
-5
+7
 I
 0
 I
-3c
+38
+I
+4
 I
-c
+39
 I
-3d
+9
 I
-19
+3a
+I
+17
 x
 51
 /Users/snl/code/sucker/spec/sucker/response_spec.rb
@@ -1633,43 +1568,62 @@ p
 1
 x
 4
-node
+item
 x
-2
-it
+4
+each
+x
+7
+be_true
+x
+6
+should
 p
-3
+7
 I
 0
 I
-3b
+36
 I
-b
+4
+I
+38
+I
+10
+I
+3d
+I
+19
 x
 51
 /Users/snl/code/sucker/spec/sucker/response_spec.rb
 p
-0
+1
+x
+11
+has_yielded
+x
+2
+it
 p
-5
+3
 I
 0
 I
-34
+35
 I
 b
-I
-3a
-I
-16
 x
 51
 /Users/snl/code/sucker/spec/sucker/response_spec.rb
 p
 0
+x
+7
+context
 s
-3
-#[]
+22
+when no block is given
 M
 1
 p
@@ -1708,8 +1662,8 @@ I
 p
 3
 s
-20
-is an alias of #find
+12
+raises error
 M
 1
 p
@@ -1723,14 +1677,1059 @@ x
 6
 Sucker
 i
-27
+19
 5
-48
+56
 0
-7
+47
+50
 1
-64
-49
+0
+5
+45
+2
+3
+47
+49
+4
+1
+49
+5
+1
+11
+I
+4
+I
+0
+I
+0
+I
+0
+I
+-2
+p
+6
+M
+1
+p
+2
+x
+9
+for_block
+t
+n
+x
+6
+Sucker
+i
+10
+5
+48
+0
+7
+1
+64
+49
+2
+1
+11
+I
+3
+I
+0
+I
+0
+I
+0
+I
+-2
+p
+3
+x
+8
+response
+s
+14
+ItemAttributes
+x
+4
+each
+p
+3
+I
+0
+I
+44
+I
+a
+x
+51
+/Users/snl/code/sucker/spec/sucker/response_spec.rb
+p
+0
+x
+6
+expect
+x
+14
+LocalJumpError
+n
+x
+11
+raise_error
+x
+2
+to
+p
+7
+I
+0
+I
+43
+I
+7
+I
+45
+I
+f
+I
+43
+I
+13
+x
+51
+/Users/snl/code/sucker/spec/sucker/response_spec.rb
+p
+0
+x
+2
+it
+p
+3
+I
+0
+I
+42
+I
+b
+x
+51
+/Users/snl/code/sucker/spec/sucker/response_spec.rb
+p
+0
+p
+5
+I
+0
+I
+34
+I
+b
+I
+41
+I
+16
+x
+51
+/Users/snl/code/sucker/spec/sucker/response_spec.rb
+p
+0
+s
+5
+#find
+M
+1
+p
+2
+x
+9
+for_block
+t
+n
+x
+6
+Sucker
+i
+22
+5
+7
+0
+64
+56
+1
+47
+50
+2
+1
+15
+5
+7
+3
+64
+56
+4
+47
+50
+2
+1
+11
+I
+4
+I
+0
+I
+0
+I
+0
+I
+-2
+p
+5
+s
+22
+when there are matches
+M
+1
+p
+2
+x
+9
+for_block
+t
+n
+x
+6
+Sucker
+i
+11
+5
+7
+0
+64
+56
+1
+47
+50
+2
+1
+11
+I
+4
+I
+0
+I
+0
+I
+0
+I
+-2
+p
+3
+s
+34
+returns an array of matching nodes
+M
+1
+p
+2
+x
+9
+for_block
+t
+n
+x
+6
+Sucker
+i
+21
+5
+48
+0
+7
+1
+64
+49
+2
+1
+5
+5
+48
+3
+47
+49
+4
+1
+49
+5
+1
+11
+I
+4
+I
+0
+I
+0
+I
+0
+I
+-2
+p
+6
+x
+8
+response
+s
+4
+ASIN
+x
+4
+find
+x
+5
+asins
+x
+3
+eql
+x
+6
+should
+p
+3
+I
+0
+I
+4d
+I
+15
+x
+51
+/Users/snl/code/sucker/spec/sucker/response_spec.rb
+p
+0
+x
+2
+it
+p
+3
+I
+0
+I
+4c
+I
+b
+x
+51
+/Users/snl/code/sucker/spec/sucker/response_spec.rb
+p
+0
+x
+7
+context
+s
+25
+when there are no matches
+M
+1
+p
+2
+x
+9
+for_block
+t
+n
+x
+6
+Sucker
+i
+11
+5
+7
+0
+64
+56
+1
+47
+50
+2
+1
+11
+I
+4
+I
+0
+I
+0
+I
+0
+I
+-2
+p
+3
+s
+22
+returns an empty array
+M
+1
+p
+2
+x
+9
+for_block
+t
+n
+x
+6
+Sucker
+i
+25
+5
+48
+0
+7
+1
+64
+49
+2
+1
+19
+0
+15
+20
+0
+5
+35
+0
+47
+49
+3
+1
+49
+4
+1
+11
+I
+5
+I
+1
+I
+0
+I
+0
+I
+-2
+p
+5
+x
+8
+response
+s
+3
+Foo
+x
+4
+find
+x
+3
+eql
+x
+6
+should
+p
+5
+I
+0
+I
+53
+I
+c
+I
+54
+I
+19
+x
+51
+/Users/snl/code/sucker/spec/sucker/response_spec.rb
+p
+1
+x
+4
+node
+x
+2
+it
+p
+3
+I
+0
+I
+52
+I
+b
+x
+51
+/Users/snl/code/sucker/spec/sucker/response_spec.rb
+p
+0
+p
+5
+I
+0
+I
+4b
+I
+b
+I
+51
+I
+16
+x
+51
+/Users/snl/code/sucker/spec/sucker/response_spec.rb
+p
+0
+s
+4
+#map
+M
+1
+p
+2
+x
+9
+for_block
+t
+n
+x
+6
+Sucker
+i
+22
+5
+7
+0
+64
+56
+1
+47
+50
+2
+1
+15
+5
+7
+3
+64
+56
+4
+47
+50
+2
+1
+11
+I
+4
+I
+0
+I
+0
+I
+0
+I
+-2
+p
+5
+s
+21
+when a block is given
+M
+1
+p
+2
+x
+9
+for_block
+t
+n
+x
+6
+Sucker
+i
+11
+5
+7
+0
+64
+56
+1
+47
+50
+2
+1
+11
+I
+4
+I
+0
+I
+0
+I
+0
+I
+-2
+p
+3
+s
+53
+yields each match to a block and maps returned values
+M
+1
+p
+2
+x
+9
+for_block
+t
+n
+x
+6
+Sucker
+i
+29
+5
+48
+0
+7
+1
+64
+56
+2
+50
+3
+1
+19
+0
+15
+20
+0
+49
+4
+0
+5
+80
+47
+49
+5
+1
+49
+6
+1
+11
+I
+5
+I
+1
+I
+0
+I
+0
+I
+-2
+p
+7
+x
+8
+response
+s
+14
+ItemAttributes
+M
+1
+p
+2
+x
+9
+for_block
+t
+n
+x
+6
+Sucker
+i
+13
+57
+19
+0
+15
+20
+0
+7
+0
+64
+49
+1
+1
+11
+I
+4
+I
+1
+I
+1
+I
+1
+n
+p
+2
+s
+5
+Title
+x
+2
+[]
+p
+3
+I
+0
+I
+5c
+I
+d
+x
+51
+/Users/snl/code/sucker/spec/sucker/response_spec.rb
+p
+1
+x
+4
+item
+x
+3
+map
+x
+5
+count
+x
+3
+eql
+x
+6
+should
+p
+5
+I
+0
+I
+5c
+I
+e
+I
+5e
+I
+1d
+x
+51
+/Users/snl/code/sucker/spec/sucker/response_spec.rb
+p
+1
+x
+6
+titles
+x
+2
+it
+p
+3
+I
+0
+I
+5b
+I
+b
+x
+51
+/Users/snl/code/sucker/spec/sucker/response_spec.rb
+p
+0
+x
+7
+context
+s
+22
+when no block is given
+M
+1
+p
+2
+x
+9
+for_block
+t
+n
+x
+6
+Sucker
+i
+11
+5
+7
+0
+64
+56
+1
+47
+50
+2
+1
+11
+I
+4
+I
+0
+I
+0
+I
+0
+I
+-2
+p
+3
+s
+12
+raises error
+M
+1
+p
+2
+x
+9
+for_block
+t
+n
+x
+6
+Sucker
+i
+19
+5
+56
+0
+47
+50
+1
+0
+5
+45
+2
+3
+47
+49
+4
+1
+49
+5
+1
+11
+I
+4
+I
+0
+I
+0
+I
+0
+I
+-2
+p
+6
+M
+1
+p
+2
+x
+9
+for_block
+t
+n
+x
+6
+Sucker
+i
+10
+5
+48
+0
+7
+1
+64
+49
+2
+1
+11
+I
+3
+I
+0
+I
+0
+I
+0
+I
+-2
+p
+3
+x
+8
+response
+s
+14
+ItemAttributes
+x
+3
+map
+p
+3
+I
+0
+I
+65
+I
+a
+x
+51
+/Users/snl/code/sucker/spec/sucker/response_spec.rb
+p
+0
+x
+6
+expect
+x
+14
+LocalJumpError
+n
+x
+11
+raise_error
+x
+2
+to
+p
+7
+I
+0
+I
+64
+I
+7
+I
+66
+I
+f
+I
+64
+I
+13
+x
+51
+/Users/snl/code/sucker/spec/sucker/response_spec.rb
+p
+0
+x
+2
+it
+p
+3
+I
+0
+I
+63
+I
+b
+x
+51
+/Users/snl/code/sucker/spec/sucker/response_spec.rb
+p
+0
+p
+5
+I
+0
+I
+5a
+I
+b
+I
+62
+I
+16
+x
+51
+/Users/snl/code/sucker/spec/sucker/response_spec.rb
+p
+0
+s
+3
+#[]
+M
+1
+p
+2
+x
+9
+for_block
+t
+n
+x
+6
+Sucker
+i
+11
+5
+7
+0
+64
+56
+1
+47
+50
+2
+1
+11
+I
+4
+I
+0
+I
+0
+I
+0
+I
+-2
+p
+3
+s
+20
+is an alias of #find
+M
+1
+p
+2
+x
+9
+for_block
+t
+n
+x
+6
+Sucker
+i
+27
+5
+48
+0
+7
+1
+64
+49
 2
 1
 5
@@ -1786,7 +2785,7 @@ p
 I
 0
 I
-44
+6d
 I
 1b
 x
@@ -1802,7 +2801,7 @@ p
 I
 0
 I
-43
+6c
 I
 b
 x
@@ -1929,7 +2928,7 @@ p
 I
 0
 I
-4a
+73
 I
 12
 x
@@ -2039,11 +3038,11 @@ p
 I
 0
 I
-4e
+77
 I
 e
 I
-4f
+78
 I
 26
 x
@@ -2056,11 +3055,11 @@ p
 I
 0
 I
-49
+72
 I
 b
 I
-4d
+76
 I
 16
 x
@@ -2212,7 +3211,7 @@ p
 I
 0
 I
-56
+7f
 I
 a
 x
@@ -2228,7 +3227,7 @@ p
 I
 0
 I
-55
+7e
 I
 b
 x
@@ -2350,11 +3349,11 @@ p
 I
 0
 I
-5c
+85
 I
 d
 I
-5d
+86
 I
 17
 x
@@ -2370,7 +3369,7 @@ p
 I
 0
 I
-5b
+84
 I
 b
 x
@@ -2383,11 +3382,11 @@ p
 I
 0
 I
-54
+7d
 I
 b
 I
-5a
+83
 I
 16
 x
@@ -2513,7 +3512,7 @@ p
 I
 0
 I
-64
+8d
 I
 16
 x
@@ -2529,7 +3528,7 @@ p
 I
 0
 I
-63
+8c
 I
 b
 x
@@ -2538,7 +3537,7 @@ x
 p
 0
 p
-21
+25
 I
 0
 I
@@ -2566,21 +3565,29 @@ I
 I
 4e
 I
-42
+4a
 I
 59
 I
-48
+59
 I
 64
 I
-53
+6b
 I
 6f
 I
-62
+71
 I
 7a
+I
+7c
+I
+85
+I
+8b
+I
+90
 x
 51
 /Users/snl/code/sucker/spec/sucker/response_spec.rb