sucker 1.0.0.beta.3 → 1.0.0.beta.4

data/CHANGELOG.md

@@ -0,0 +1,24 @@
+Version 1.0.0.beta4
+========
+
+Release date: 2010-10-28
+
+Added
+-----
+
+* Sucker::Request#get! raises an error if response is not valid.
+
+Version 1.0.0.beta3
+========
+
+Release date: 2010-10-25
+
+Added
+-----
+
+* Sucker::Response#find yields to a block if given one.
+
+Deprecated
+----------
+
+* Renamed #node to #find

data/README.md

@@ -58,7 +58,7 @@ Stubbing
 
 Use [VCR](http://github.com/myronmarston/vcr) to stub your requests.
 
-Caveat: Match URIs on host only and create a new cassette for each stubbed query. [This is how I do it](http://github.com/papercavalier/sucker/blob/master/spec/support/vcr.rb).
+Caveat: Match URIs on host only and create a new cassette for each query. [This is how I do it](http://github.com/papercavalier/sucker/blob/master/spec/support/vcr.rb).
 
 Compatibility
 -------------

data/lib/sucker.rb

@@ -1,6 +1,5 @@
 require "active_support/xml_mini/nokogiri"
 require "curb"
-require "digest/md5"
 require "sucker/request"
 require "sucker/response"
 require "uri"

data/lib/sucker/request.rb

@@ -36,7 +36,7 @@ module Sucker #:nodoc
       args.each { |k, v| send("#{k}=", v) }
     end
 
-    # A helper method that merges a hash into existing parameters
+    # Merges a hash into existing parameters
     #
     #   worker = Sucker.new
     #   worker << {
@@ -81,6 +81,17 @@ module Sucker #:nodoc
       Response.new(curl)
     end
 
+    # Similar to get but raises an error if response is not valid
+    def get!
+      response = get
+
+      unless response.valid?
+        raise ResponseError, response.inspect
+      end
+
+      response
+    end
+
     # Sets the AWS Access Key ID
     #
     #   worker = Sucker.new

data/lib/sucker/response.rb

@@ -83,4 +83,6 @@ module Sucker #:nodoc
       end
     end
   end
+
+  class ResponseError < StandardError; end
 end

data/lib/sucker/version.rb

@@ -1,3 +1,3 @@
 module Sucker #:nodoc
-  VERSION = "1.0.0.beta.3"
+  VERSION = "1.0.0.beta.4"
 end

data/spec/integration/kindle_spec.rb

@@ -0,0 +1,47 @@
+require "spec_helper"
+
+module Sucker
+  describe "Kindle" do
+    context "Book index" do
+      use_vcr_cassette "integration/kindle", :record => :new_episodes
+
+      let(:items) do
+        worker = Sucker.new(
+          :locale => "us",
+          :key    => amazon["key"],
+          :secret => amazon["secret"])
+        worker << {
+          "Operation"     => "ItemSearch",
+          "SearchIndex"   => "Books",
+          "Power"         => "deleuze binding:kindle",
+          "ResponseGroup" => "ItemAttributes" }
+        worker.get.find("Item")
+      end
+
+      it "finds Kindle books" do
+        items.first["ItemAttributes"]["Title"].should_not be_nil
+      end
+    end
+
+    context "Kindle store index" do
+      use_vcr_cassette "integration/kindle_2", :record => :new_episodes
+
+      let(:items) do
+        worker = Sucker.new(
+          :locale => "us",
+          :key    => amazon["key"],
+          :secret => amazon["secret"])
+        worker << {
+          "Operation"     => "ItemSearch",
+          "SearchIndex"   => "KindleStore",
+          "Keywords"      => "deleuze",
+          "ResponseGroup" => "ItemAttributes" }
+        worker.get.find("Item")
+      end
+
+      it "finds Kindle books" do
+        items.first["ItemAttributes"]["Title"].should_not be_nil
+      end
+    end
+  end
+end

data/spec/unit/sucker/request_spec.rb

@@ -58,8 +58,18 @@ module Sucker
     end
 
     context "#get" do
-      it "returns a Response object" do
-        worker.get.class.ancestors.should include Sucker::Response
+      it "returns a response" do
+        worker.get.class.ancestors.should include Response
+      end
+    end
+
+    context "#get!" do
+      it "raises if response is not valid" do
+        worker << {
+          "Operation"     => "ItemLookup",
+          "IdType"        => "ASIN",
+          "ItemId"        => "0816614024" }
+        lambda { worker.get! }.should raise_error ResponseError
       end
     end
 

metadata

@@ -1,15 +1,15 @@
 --- !ruby/object:Gem::Specification 
 name: sucker
 version: !ruby/object:Gem::Version 
-  hash: 62196357
+  hash: 62196363
   prerelease: true
   segments: 
   - 1
   - 0
   - 0
   - beta
-  - 3
-  version: 1.0.0.beta.3
+  - 4
+  version: 1.0.0.beta.4
 platform: ruby
 authors: 
 - Hakan Ensari
@@ -18,7 +18,7 @@ autorequire:
 bindir: bin
 cert_chain: []
 
-date: 2010-10-25 00:00:00 +01:00
+date: 2010-10-28 00:00:00 +01:00
 default_executable: 
 dependencies: 
 - !ruby/object:Gem::Dependency 
@@ -197,7 +197,7 @@ files:
 - lib/sucker.rb
 - LICENSE
 - README.md
-- History.md
+- CHANGELOG.md
 - spec/fixtures/asins.txt
 - spec/integration/alternate_versions_spec.rb
 - spec/integration/errors_spec.rb
@@ -207,6 +207,7 @@ files:
 - spec/integration/item_search_spec.rb
 - spec/integration/japan_spec.rb
 - spec/integration/keyword_search_spec.rb
+- spec/integration/kindle_spec.rb
 - spec/integration/multiple_locales_spec.rb
 - spec/integration/power_search_spec.rb
 - spec/integration/related_items_spec.rb
@@ -266,6 +267,7 @@ test_files:
 - spec/integration/item_search_spec.rb
 - spec/integration/japan_spec.rb
 - spec/integration/keyword_search_spec.rb
+- spec/integration/kindle_spec.rb
 - spec/integration/multiple_locales_spec.rb
 - spec/integration/power_search_spec.rb
 - spec/integration/related_items_spec.rb

data/History.md

@@ -1,14 +0,0 @@
-Version 1.0.0.beta3
-========
-
-Release date: 2010-10-25
-
-Added
------
-
-* #find yields to a block if given one
-
-Deprecated
-----------
-
-* Renamed #node to #find