a2z 0.1.1 → 0.1.2

data/.gitignore

@@ -2,6 +2,7 @@
 *.rbc
 .bundle
 .config
+.idea
 .yardoc
 Gemfile.lock
 InstalledFiles

data/.travis.yml

@@ -0,0 +1,4 @@
+language: ruby
+rvm:
+  - 1.9.3
+  - ruby-head

data/CHANGELOG.md

@@ -1,3 +1,12 @@
+## A2z 0.1.2 (Unreleased)
+
+* Add `most_gifted`, `most_wished_for`, `new_releases`, and `top_sellers`
+  accessors on `BrowseNode`, populating them when present in a browse node
+  lookup response. *Matt Huggins*
+
+* Fix issue with offer summary parsing the wrong node for `lowest_used_price`.
+  *Matt Huggins*
+
 ## A2z 0.1.1 (Jan 3, 2012)
 
 * Add `offers` and `offer_summary` accessors on `Item`, populating them when

data/README.md

@@ -1,5 +1,8 @@
 # A2z
 
+[![Build Status](https://secure.travis-ci.org/mhuggins/a2z.png)](http://travis-ci.org/mhuggins/a2z)
+[![Code Climate](https://codeclimate.com/github/mhuggins/a2z.png)](https://codeclimate.com/github/mhuggins/a2z)
+
 A2z provides a simple Ruby DSL to retrieve product information from the
 [Amazon Product Advertising API](https://affiliate-program.amazon.com/gp/advertising/api/detail/main.html).
 
@@ -82,10 +85,22 @@ example would make the following calls possible.
     child.name                             # => "Arts & Photography"
     child.root?                            # => false
 
-For more information on interacting with `A2z::Responses::OperationRequest`
-objects, refer to its respective section below.  With regards to children, only
-the direct children are included in a response.  However, ancestors are nested
-all the way to the top-most parent.
+By default, Amazon assumes the response group to be "BrowseNodeInfo".
+Additional response groups "MostGifted", "MostWishedFor", "NewReleases", and
+"TopSellers" can be requested as well, allowing access to top items per the
+following example code.
+
+    node = response.node                   # => #<A2z::Responses::BrowseNode ...>
+    node.most_gifted.size                  # => 10
+    node.most_gifted.first                 # => #<A2z::Responses::TopItem ...>
+    node.most_wished_for.first             # => #<A2z::Responses::TopItem ...>
+    node.new_releases.first                # => #<A2z::Responses::TopItem ...>
+    node.top_sellers.first                 # => #<A2z::Responses::TopItem ...>
+
+For more information on interacting with `A2z::Responses::TopItem` and
+`A2z::Responses::OperationRequest` objects, refer to their respective sections
+below.  With regards to children, only the direct children are included in a
+response.  However, ancestors are nested all the way to the top-most parent.
 
 The following code demonstrate the full collection of methods provided when
 performing a browse node lookup.
@@ -261,6 +276,21 @@ response.
 Refer to the Image Sets and Images sections below for more information on using
 these objects.
 
+### Top Items
+
+Top items are accessible on browse nodes when performing browse node lookups
+with a response group of "MostGifted", "MostWishedFor", "NewReleases", and
+"TopSellers".
+
+    top_item = node.top_sellers.first      # => #<A2z::Responses::TopItem ...>
+    top_item.asin                          # => "B00AQ3K8IU"
+    top_item.title                         # => "Hopeless"
+    top_item.product_group                 # => "eBooks"
+    top_item.actor                         # => nil
+    top_item.artist                        # => nil
+    top_item.author                        # => "Hoover, Colleen"
+    top_item.detail_page_url               # => "http://www.amazon.com/Hopeless-ebook/dp/B00AQ3K8IU..."
+
 ### Image Sets
 
 Image sets are accessible on items when the "Images" responses group is
@@ -270,15 +300,10 @@ keyed by size.
 
 For example:
 
-    response = client.item_lookup do
-      id 'B008VVEJNE'
-      response_group 'Images'
-    end
-    
-    item_set = response.item.image_sets[:primary]  # => #<A2z::Responses::ImageSet ...>
-    item_set.category                              # => "primary"
-    item_set.images.keys                           # => [:swatch, :small, :thumbnail, :tiny, :medium, :large]
-    item_set.images[:swatch]                       # => #<A2z::Responses::Image ...>
+    item_set = item.image_sets[:primary]   # => #<A2z::Responses::ImageSet ...>
+    item_set.category                      # => "primary"
+    item_set.images.keys                   # => [:swatch, :small, :thumbnail, :tiny, :medium, :large]
+    item_set.images[:swatch]               # => #<A2z::Responses::Image ...>
 
 Refer to the Images section below for more information on using this objects.
 

data/lib/a2z/responses.rb

@@ -10,3 +10,4 @@ require 'a2z/responses/item_search'
 require 'a2z/responses/offer'
 require 'a2z/responses/offer_summary'
 require 'a2z/responses/operation_request'
+require 'a2z/responses/top_item'

data/lib/a2z/responses/browse_node.rb

@@ -3,11 +3,16 @@ module A2z
     class BrowseNode
       include Helpers
       
-      attr_accessor :id, :name, :ancestors, :children
+      attr_accessor :id, :name, :ancestors, :children, :most_gifted,
+                    :most_wished_for, :new_releases, :top_sellers
       
       def initialize
         @ancestors = []
         @children = []
+        @most_gifted = []
+        @most_wished_for = []
+        @new_releases = []
+        @top_sellers = []
         @root = false
       end
       
@@ -39,6 +44,20 @@ module A2z
               ancestors = array_wrap(data['Ancestors']['BrowseNode'])
               browse_node.ancestors = ancestors.collect { |ancestor| BrowseNode.from_response(ancestor) }
             end
+            
+            if data['TopItemSet']
+              top_item_sets = array_wrap(data['TopItemSet'])
+              top_item_sets.each do |top_item_set|
+                top_items = array_wrap(top_item_set['TopItem']).collect { |top_item| TopItem.from_response(top_item) }
+                
+                case top_item_set['Type']
+                  when 'MostGifted' then browse_node.most_gifted = top_items
+                  when 'MostWishedFor' then browse_node.most_wished_for = top_items
+                  when 'NewReleases' then browse_node.new_releases = top_items
+                  when 'TopSellers' then browse_node.top_sellers = top_items
+                end
+              end
+            end
           end
           
           browse_node.freeze

data/lib/a2z/responses/browse_node_lookup.rb

@@ -20,8 +20,8 @@ module A2z
       def self.from_response(data)
         new.tap do |browse_node_lookup|
           browse_node_lookup.operation_request = OperationRequest.from_response(data['OperationRequest']) if data['OperationRequest']
-          browse_node_lookup.node              = BrowseNode.from_response(data['BrowseNodes']['BrowseNode'])
-          browse_node_lookup.valid             = data['BrowseNodes']['Request']['IsValid'] == 'True' rescue false
+          browse_node_lookup.node = BrowseNode.from_response(data['BrowseNodes']['BrowseNode'])
+          browse_node_lookup.valid = data['BrowseNodes']['Request']['IsValid'] == 'True' rescue false
           browse_node_lookup.freeze
         end
       end

data/lib/a2z/responses/offer_summary.rb

@@ -48,7 +48,7 @@ module A2z
           end
           
           if data['LowestUsedPrice']
-            offer_summary.lowest_used_price = Money.new(data['LowestNewPrice']['Amount'].to_i, data['LowestNewPrice']['CurrencyCode'])
+            offer_summary.lowest_used_price = Money.new(data['LowestUsedPrice']['Amount'].to_i, data['LowestUsedPrice']['CurrencyCode'])
           end
           
           offer_summary.total_new = data['TotalNew']

data/lib/a2z/responses/top_item.rb

@@ -0,0 +1,21 @@
+module A2z
+  module Responses
+    class TopItem
+      attr_accessor :asin, :title, :product_group, :actor, :artist, :author,
+                    :detail_page_url
+      
+      def self.from_response(data)
+        new.tap do |top_item|
+          top_item.asin = data['ASIN']
+          top_item.title = data['Title']
+          top_item.product_group = data['ProductGroup']
+          top_item.actor = data['Actor']
+          top_item.artist = data['Artist']
+          top_item.author = data['Author']
+          top_item.detail_page_url = data['DetailPageURL']
+          top_item.freeze
+        end
+      end
+    end
+  end
+end

data/lib/a2z/version.rb

@@ -1,3 +1,3 @@
 module A2z
-  VERSION = '0.1.1'
+  VERSION = '0.1.2'
 end

data/spec/a2z/client_spec.rb

@@ -19,31 +19,113 @@ describe A2z::Client do
     end
   end
   
-  describe '#country=' do
-    it 'should succeed'
+  describe '#country' do
+    subject do
+      A2z::Client.new(country: :us)
+    end
+    
+    specify { subject.country.should eq :us }
   end
   
-  describe '#country' do
-    it 'should succeed'
+  describe '#country=' do
+    subject do
+      A2z::Client.new
+    end
+    
+    before do
+      subject.country.should_not eq :fr
+      subject.country = :fr
+    end
+    
+    specify { subject.country.should eq :fr }
   end
   
   describe '#tag' do
-    it 'should succeed'
+    subject do
+      A2z::Client.new(tag: 'MyTag')
+    end
+    
+    specify { subject.tag.should eq 'MyTag' }
   end
   
   describe '#item_search' do
     subject do
-      A2z::Client.new
+      A2z::Client.new(key: 'MyKey', secret: 'MySecret', tag: 'MyTag')
+    end
+    
+    before do
+      subject.stub(:get).and_return(item_search_response)
+    end
+    
+    let(:block) { proc {} }
+    
+    it 'forwards through item search request' do
+      A2z::Requests::ItemSearch.should_receive(:new).with(&block).and_call_original
+      subject.item_search(&block)
     end
     
-    it 'should succeed'
+    it 'parses item search response' do
+      A2z::Responses::ItemSearch.should_receive(:from_response)
+      subject.item_search(&block)
+    end
   end
   
   describe '#item_lookup' do
     subject do
-      A2z::Client.new
+      A2z::Client.new(key: 'MyKey', secret: 'MySecret', tag: 'MyTag')
     end
     
-    it 'should succeed'
+    before do
+      subject.stub(:get).and_return(item_lookup_response)
+    end
+    
+    let(:block) { proc {} }
+    
+    it 'forwards through item search request' do
+      A2z::Requests::ItemLookup.should_receive(:new).with(&block).and_call_original
+      subject.item_lookup(&block)
+    end
+    
+    it 'parses item search response' do
+      A2z::Responses::ItemLookup.should_receive(:from_response)
+      subject.item_lookup(&block)
+    end
+  end
+  
+  describe '#browse_node_lookup' do
+    subject do
+      A2z::Client.new(key: 'MyKey', secret: 'MySecret', tag: 'MyTag')
+    end
+    
+    before do
+      subject.stub(:get).and_return(browse_node_lookup_response)
+    end
+    
+    let(:id) { 1 }
+    let(:block) { proc {} }
+    
+    it 'forwards through item search request' do
+      A2z::Requests::BrowseNodeLookup.should_receive(:new).with(id, &block).and_call_original
+      subject.browse_node_lookup(id, &block)
+    end
+    
+    it 'parses item search response' do
+      A2z::Responses::BrowseNodeLookup.should_receive(:from_response)
+      subject.browse_node_lookup(id, &block)
+    end
+  end
+  
+  private
+  
+  def item_search_response
+    stub(body: '<OperationRequest/><Items/>')
+  end
+  
+  def item_lookup_response
+    stub(body: '<OperationRequest/><Items/>')
+  end
+  
+  def browse_node_lookup_response
+    stub(body: '<OperationRequest/><BrowseNodes><BrowseNode/></BrowseNodes>')
   end
 end

data/spec/a2z/requests/browse_node_lookup_spec.rb

@@ -6,9 +6,33 @@ describe A2z::Requests::BrowseNodeLookup do
   end
   
   let(:id) { 1 }
-  let(:block) { Proc.new { } }
+  let(:block) { proc { } }
   
   describe '#params' do
-    it 'should return a hash'
+    specify { subject.params.should be_a Hash }
+    
+    it 'sets the operation' do
+      subject.params['Operation'].should eq 'BrowseNodeLookup'
+    end
+    
+    it 'sets the passed node ID' do
+      subject.params['BrowseNodeId'].should eq id
+    end
+    
+    describe 'when response group is specified' do
+      let(:block) { proc { response_group 'Offers' } }
+      
+      it 'sets the response group' do
+        subject.params['ResponseGroup'].should eq 'Offers'
+      end
+    end
+    
+    describe 'when nothing is specified' do
+      let(:block) { proc { } }
+      
+      it 'does not set response group' do
+        subject.params.should_not have_key 'ResponseGroup'
+      end
+    end
   end
 end

data/spec/a2z/requests/item_lookup_spec.rb

@@ -5,9 +5,145 @@ describe A2z::Requests::ItemLookup do
     A2z::Requests::ItemLookup.new(&block)
   end
   
-  let(:block) { Proc.new { } }
+  let(:block) { proc { } }
   
   describe '#params' do
-    it 'should return a hash'
+    specify { subject.params.should be_a Hash }
+    
+    it 'sets the operation' do
+      subject.params['Operation'].should eq 'ItemLookup'
+    end
+    
+    describe 'when id is specified' do
+      describe 'as a string' do
+        let(:block) { proc { id 'ABC123' } }
+        
+        it 'sets the item ID' do
+          subject.params['ItemId'].should eq 'ABC123'
+        end
+      end
+      
+      describe 'as an array' do
+        let(:block) { proc { id %w(ABC123 DEF456) } }
+        
+        it 'sets the comma-delimited item ID' do
+          subject.params['ItemId'].should eq 'ABC123,DEF456'
+        end
+      end
+    end
+    
+    describe 'when category is specified' do
+      let(:block) { proc { category 'Books' } }
+      
+      it 'sets the search index' do
+        subject.params['SearchIndex'].should eq 'Books'
+      end
+    end
+    
+    describe 'when response group is specified' do
+      let(:block) { proc { response_group 'Offers' } }
+      
+      it 'sets the response group' do
+        subject.params['ResponseGroup'].should eq 'Offers'
+      end
+    end
+    
+    describe 'when include reviews summary is specified' do
+      describe 'as true' do
+        let(:block) { proc { include_reviews_summary true } }
+        
+        it 'sets include reviews summary' do
+          subject.params['IncludeReviewsSummary'].should eq 'True'
+        end
+      end
+      
+      describe 'as false' do
+        let(:block) { proc { include_reviews_summary false } }
+        
+        it 'sets include reviews summary' do
+          subject.params['IncludeReviewsSummary'].should eq 'False'
+        end
+      end
+    end
+    
+    describe 'when condition is specified' do
+      let(:block) { proc { condition 'New' } }
+      
+      it 'sets condition' do
+        subject.params['Condition'].should eq 'New'
+      end
+    end
+    
+    describe 'when ID type is specified' do
+      let(:block) { proc { id_type 'ASIN' } }
+      
+      it 'sets ID type' do
+        subject.params['IdType'].should eq 'ASIN'
+      end
+    end
+    
+    describe 'when merchant ID is specified' do
+      let(:block) { proc { merchant_id 'ABC123' } }
+      
+      it 'sets merchant ID' do
+        subject.params['MerchantId'].should eq 'ABC123'
+      end
+    end
+    
+    describe 'when truncate reviews at is specified' do
+      let(:block) { proc { truncate_reviews_at 10 } }
+      
+      it 'sets truncate reviews at' do
+        subject.params['TruncateReviewsAt'].should eq 10
+      end
+    end
+    
+    describe 'when variation page is specified' do
+      let(:block) { proc { variation_page 10 } }
+      
+      it 'sets variation page' do
+        subject.params['VariationPage'].should eq 10
+      end
+    end
+    
+    describe 'when nothing is specified' do
+      let(:block) { proc { } }
+      
+      it 'does not set the item ID' do
+        subject.params.should_not have_key 'ItemId'
+      end
+      
+      it 'does not set the search index' do
+        subject.params.should_not have_key 'SearchIndex'
+      end
+      
+      it 'does not set the response group' do
+        subject.params.should_not have_key 'ResponseGroup'
+      end
+      
+      it 'does not set include reviews summary' do
+        subject.params.should_not have_key 'IncludeReviewsSummary'
+      end
+      
+      it 'does not set condition' do
+        subject.params.should_not have_key 'Condition'
+      end
+      
+      it 'does not set ID type' do
+        subject.params.should_not have_key 'IdType'
+      end
+      
+      it 'does not set merchant ID' do
+        subject.params.should_not have_key 'MerchantId'
+      end
+      
+      it 'does not set truncate reviews at' do
+        subject.params.should_not have_key 'TruncateReviewsAt'
+      end
+      
+      it 'does not set variation page' do
+        subject.params.should_not have_key 'VariationPage'
+      end
+    end
   end
 end

data/spec/a2z/requests/item_search_spec.rb

@@ -8,6 +8,353 @@ describe A2z::Requests::ItemSearch do
   let(:block) { Proc.new { } }
   
   describe '#params' do
-    it 'should return a hash'
+    specify { subject.params.should be_a Hash }
+    
+    it 'sets the operation' do
+      subject.params['Operation'].should eq 'ItemSearch'
+    end
+    
+    describe 'when keywords are specified' do
+      describe 'as a string' do
+        let(:block) { proc { keywords 'Harry Potter' } }
+        
+        it 'sets the keywords' do
+          subject.params['Keywords'].should eq 'Harry Potter'
+        end
+      end
+      
+      describe 'as an array' do
+        let(:block) { proc { keywords %w(Harry Potter) } }
+        
+        it 'sets the space-delimited keywords' do
+          subject.params['Keywords'].should eq 'Harry Potter'
+        end
+      end
+    end
+    
+    describe 'when category is specified' do
+      let(:block) { proc { category 'Books' } }
+      
+      it 'sets the search index' do
+        subject.params['SearchIndex'].should eq 'Books'
+      end
+    end
+    
+    describe 'when response group is specified' do
+      let(:block) { proc { response_group 'Offers' } }
+      
+      it 'sets the response group' do
+        subject.params['ResponseGroup'].should eq 'Offers'
+      end
+    end
+    
+    describe 'when include reviews summary is specified' do
+      describe 'as true' do
+        let(:block) { proc { include_reviews_summary true } }
+        
+        it 'sets include reviews summary' do
+          subject.params['IncludeReviewsSummary'].should eq 'True'
+        end
+      end
+      
+      describe 'as false' do
+        let(:block) { proc { include_reviews_summary false } }
+        
+        it 'sets include reviews summary' do
+          subject.params['IncludeReviewsSummary'].should eq 'False'
+        end
+      end
+    end
+    
+    describe 'when actor is specified' do
+      let(:block) { proc { actor 'Matt Damon' } }
+      
+      it 'sets the actor' do
+        subject.params['Actor'].should eq 'Matt Damon'
+      end
+    end
+    
+    describe 'when artist is specified' do
+      let(:block) { proc { artist 'Grouplove' } }
+      
+      it 'sets the artist' do
+        subject.params['Artist'].should eq 'Grouplove'
+      end
+    end
+    
+    describe 'when audience rating is specified' do
+      let(:block) { proc { audience_rating 'PG' } }
+      
+      it 'sets the audience rating' do
+        subject.params['AudienceRating'].should eq 'PG'
+      end
+    end
+    
+    describe 'when author is specified' do
+      let(:block) { proc { author 'J.K. Rowling' } }
+      
+      it 'sets the author' do
+        subject.params['Author'].should eq 'J.K. Rowling'
+      end
+    end
+    
+    describe 'when brand is specified' do
+      let(:block) { proc { brand 'Timex' } }
+      
+      it 'sets the brand' do
+        subject.params['Brand'].should eq 'Timex'
+      end
+    end
+    
+    describe 'when browse node is specified' do
+      let(:block) { proc { browse_node 100 } }
+      
+      it 'sets the browse node' do
+        subject.params['BrowseNode'].should eq 100
+      end
+    end
+    
+    describe 'when composer is specified' do
+      let(:block) { proc { composer 'Bach' } }
+      
+      it 'sets the composer' do
+        subject.params['Composer'].should eq 'Bach'
+      end
+    end
+    
+    describe 'when conductor is specified' do
+      let(:block) { proc { conductor 'Gustav Mahler' } }
+      
+      it 'sets the conductor' do
+        subject.params['Conductor'].should eq 'Gustav Mahler'
+      end
+    end
+    
+    describe 'when condition is specified' do
+      let(:block) { proc { condition 'New' } }
+      
+      it 'sets the condition' do
+        subject.params['Condition'].should eq 'New'
+      end
+    end
+    
+    describe 'when director is specified' do
+      let(:block) { proc { director 'Judd Apatow' } }
+      
+      it 'sets the director' do
+        subject.params['Director'].should eq 'Judd Apatow'
+      end
+    end
+    
+    describe 'when item page is specified' do
+      let(:block) { proc { item_page 8 } }
+      
+      it 'sets the item page' do
+        subject.params['ItemPage'].should eq 8
+      end
+    end
+    
+    describe 'when manufacturer is specified' do
+      let(:block) { proc { manufacturer 'Dr Pepper' } }
+      it 'sets the manufacturer' do
+        subject.params['Manufacturer'].should eq 'Dr Pepper'
+      end
+    end
+    
+    describe 'when maximum price is specified' do
+      let(:block) { proc { maximum_price 10.99 } }
+      
+      it 'sets the maximum price' do
+        subject.params['MaximumPrice'].should eq 10.99
+      end
+    end
+    
+    describe 'when merchant ID is specified' do
+      let(:block) { proc { merchant_id 'ABC123' } }
+      
+      it 'sets the merchant ID' do
+        subject.params['MerchantId'].should eq 'ABC123'
+      end
+    end
+    
+    describe 'when minimum percentage off is specified' do
+      let(:block) { proc { min_percentage_off 20 } }
+      
+      it 'sets the minimum percentage off' do
+        subject.params['MinPercentageOff'].should eq 20
+      end
+    end
+    
+    describe 'when orchestra is specified' do
+      let(:block) { proc { orchestra 'Antonio Vivaldi' } }
+      
+      it 'sets the orchestra' do
+        subject.params['Orchestra'].should eq 'Antonio Vivaldi'
+      end
+    end
+    
+    describe 'when power search is specified' do
+      let(:block) { proc { power 'subject:history and (spain or mexico) and not military and language:spanish' } }
+      
+      it 'sets the power' do
+        subject.params['Power'].should eq 'subject:history and (spain or mexico) and not military and language:spanish'
+      end
+    end
+    
+    describe 'when publisher is specified' do
+      let(:block) { proc { publisher 'EMI' } }
+      
+      it 'sets the publisher' do
+        subject.params['Publisher'].should eq 'EMI'
+      end
+    end
+    
+    describe 'when sort is specified' do
+      let(:block) { proc { sort 'inverse-pricerank' } }
+      
+      it 'sets the sort' do
+        subject.params['Sort'].should eq 'inverse-pricerank'
+      end
+    end
+    
+    describe 'when title is specified' do
+      let(:block) { proc { title 'The Wheel of Time' } }
+      
+      it 'sets the title' do
+        subject.params['Title'].should eq 'The Wheel of Time'
+      end
+    end
+    
+    describe 'when variation page is specified' do
+      let(:block) { proc { variation_page 2 } }
+      
+      it 'sets the variation page' do
+        subject.params['VariationPage'].should eq 2
+      end
+    end
+    
+    describe 'when truncate reviews at is specified' do
+      let(:block) { proc { truncate_reviews_at 5 } }
+      
+      it 'sets the truncate reviews at' do
+        subject.params['TruncateReviewsAt'].should eq 5
+      end
+    end
+    
+    describe 'when nothing is specified' do
+      let(:block) { proc { } }
+      
+      it 'does not set the keywords' do
+        subject.params.should_not have_key 'Keywords'
+      end
+      
+      it 'does not set the search index' do
+        subject.params.should_not have_key 'SearchIndex'
+      end
+      
+      it 'does not set the response group' do
+        subject.params.should_not have_key 'ResponseGroup'
+      end
+      
+      it 'does not set include reviews summary' do
+        subject.params.should_not have_key 'IncludeReviewsSummary'
+      end
+      
+      it 'does not set actor' do
+        subject.params.should_not have_key 'Actor'
+      end
+      
+      it 'does not set artist' do
+        subject.params.should_not have_key 'Artist'
+      end
+      
+      it 'does not set audience rating' do
+        subject.params.should_not have_key 'AudienceRating'
+      end
+      
+      it 'does not set author' do
+        subject.params.should_not have_key 'Author'
+      end
+      
+      it 'does not set brand' do
+        subject.params.should_not have_key 'Brand'
+      end
+      
+      it 'does not set browse node' do
+        subject.params.should_not have_key 'BrowseNode'
+      end
+      
+      it 'does not set composer' do
+        subject.params.should_not have_key 'Composer'
+      end
+      
+      it 'does not set conductor' do
+        subject.params.should_not have_key 'Conductor'
+      end
+      
+      it 'does not set condition' do
+        subject.params.should_not have_key 'Condition'
+      end
+      
+      it 'does not set director' do
+        subject.params.should_not have_key 'Director'
+      end
+      
+      it 'does not set item page' do
+        subject.params.should_not have_key 'ItemPage'
+      end
+      
+      it 'does not set manufacturer' do
+        subject.params.should_not have_key 'Manufacturer'
+      end
+      
+      it 'does not set maximum price' do
+        subject.params.should_not have_key 'MaximumPrice'
+      end
+      
+      it 'does not set merchant ID' do
+        subject.params.should_not have_key 'MerchantId'
+      end
+      
+      it 'does not set minimum price' do
+        subject.params.should_not have_key 'MinimumPrice'
+      end
+      
+      it 'does not set minimum percentage off' do
+        subject.params.should_not have_key 'MinPercentageOff'
+      end
+      
+      it 'does not set music label' do
+        subject.params.should_not have_key 'MusicLabel'
+      end
+      
+      it 'does not set orchestra' do
+        subject.params.should_not have_key 'Orchestra'
+      end
+      
+      it 'does not set power' do
+        subject.params.should_not have_key 'Power'
+      end
+      
+      it 'does not set publisher' do
+        subject.params.should_not have_key 'Publisher'
+      end
+      
+      it 'does not set sort' do
+        subject.params.should_not have_key 'Sort'
+      end
+      
+      it 'does not set title' do
+        subject.params.should_not have_key 'Title'
+      end
+      
+      it 'does not set truncate reviews at' do
+        subject.params.should_not have_key 'TruncateReviewsAt'
+      end
+      
+      it 'does not set variation page' do
+        subject.params.should_not have_key 'VariationPage'
+      end
+    end
   end
 end

data/spec/a2z/requests/response_group_spec.rb

@@ -6,9 +6,53 @@ describe A2z::Requests::ResponseGroup do
   end
   
   let(:value) { nil }
-  let(:block) { Proc.new { } }
+  let(:block) { proc { } }
   
   describe '#params' do
-    it 'should return a hash'
+    specify { subject.params.should be_a Hash }
+    
+    describe 'when value is a string' do
+      let(:value) { 'OfferSummary' }
+      
+      it 'sets the response group' do
+        subject.params['ResponseGroup'].should eq 'OfferSummary'
+      end
+    end
+    
+    describe 'when value is an array' do
+      let(:value) { %w(OfferSummary Images) }
+      
+      it 'sets the comma-delimited response group' do
+        subject.params['ResponseGroup'].should eq 'OfferSummary,Images'
+      end
+    end
+    
+    describe 'when the related item page is specified' do
+      let(:block) { proc { related_item_page 2 } }
+      
+      it 'sets the related item page' do
+        subject.params['RelatedItemPage'].should eq 2
+      end
+    end
+    
+    describe 'when the relationship type is specified' do
+      let(:block) { proc { relationship_type 'Episode' } }
+      
+      it 'sets the relationship type' do
+        subject.params['RelationshipType'].should eq 'Episode'
+      end
+    end
+    
+    describe 'when nothing is specified' do
+      let(:block) { proc { } }
+      
+      it 'does not set the related item page' do
+        subject.params.should_not have_key 'RelatedItemPage'
+      end
+      
+      it 'does not set the relationship type' do
+        subject.params.should_not have_key 'RelationshipType'
+      end
+    end
   end
 end

data/spec/a2z/responses/top_item_spec.rb

@@ -0,0 +1,13 @@
+require 'spec_helper'
+
+describe A2z::Responses::TopItem do
+  subject do
+    A2z::Responses::TopItem.from_response(top_item_hash)
+  end
+  
+  let(:top_item_hash) { Hash.new }
+  
+  describe '.from_response' do
+    it 'should return a top item object'
+  end
+end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: a2z
 version: !ruby/object:Gem::Version
-  version: 0.1.1
+  version: 0.1.2
   prerelease: 
 platform: ruby
 authors:
@@ -9,11 +9,11 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2013-01-03 00:00:00.000000000 Z
+date: 2013-07-01 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: crack
-  requirement: &2151907560 !ruby/object:Gem::Requirement
+  requirement: &2153748040 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -21,10 +21,10 @@ dependencies:
         version: '0'
   type: :runtime
   prerelease: false
-  version_requirements: *2151907560
+  version_requirements: *2153748040
 - !ruby/object:Gem::Dependency
   name: jeff
-  requirement: &2151906660 !ruby/object:Gem::Requirement
+  requirement: &2153747620 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -32,10 +32,10 @@ dependencies:
         version: '0'
   type: :runtime
   prerelease: false
-  version_requirements: *2151906660
+  version_requirements: *2153747620
 - !ruby/object:Gem::Dependency
   name: money
-  requirement: &2151905960 !ruby/object:Gem::Requirement
+  requirement: &2153747200 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -43,10 +43,10 @@ dependencies:
         version: '0'
   type: :runtime
   prerelease: false
-  version_requirements: *2151905960
+  version_requirements: *2153747200
 - !ruby/object:Gem::Dependency
   name: rake
-  requirement: &2151904760 !ruby/object:Gem::Requirement
+  requirement: &2153746780 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -54,10 +54,10 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *2151904760
+  version_requirements: *2153746780
 - !ruby/object:Gem::Dependency
   name: rspec
-  requirement: &2151902900 !ruby/object:Gem::Requirement
+  requirement: &2153746360 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -65,7 +65,7 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *2151902900
+  version_requirements: *2153746360
 description: Ruby DSL for Amazon Product Advertising API
 email: matt.huggins@gmail.com
 executables: []
@@ -73,7 +73,7 @@ extensions: []
 extra_rdoc_files: []
 files:
 - .gitignore
-- .idea/jsLibraryMappings.xml
+- .travis.yml
 - CHANGELOG.md
 - Gemfile
 - LICENSE
@@ -102,6 +102,7 @@ files:
 - lib/a2z/responses/offer.rb
 - lib/a2z/responses/offer_summary.rb
 - lib/a2z/responses/operation_request.rb
+- lib/a2z/responses/top_item.rb
 - lib/a2z/version.rb
 - spec/a2z/client_spec.rb
 - spec/a2z/requests/browse_node_lookup_spec.rb
@@ -119,6 +120,7 @@ files:
 - spec/a2z/responses/item_spec.rb
 - spec/a2z/responses/offer_spec.rb
 - spec/a2z/responses/offer_summary_spec.rb
+- spec/a2z/responses/top_item_spec.rb
 - spec/a2z/version_spec.rb
 - spec/spec_helper.rb
 - tasks/debug.rake
@@ -164,5 +166,6 @@ test_files:
 - spec/a2z/responses/item_spec.rb
 - spec/a2z/responses/offer_spec.rb
 - spec/a2z/responses/offer_summary_spec.rb
+- spec/a2z/responses/top_item_spec.rb
 - spec/a2z/version_spec.rb
 - spec/spec_helper.rb

data/.idea/jsLibraryMappings.xml

@@ -1,18 +0,0 @@
-<?xml version="1.0" encoding="UTF-8"?>
-<project version="4">
-  <component name="JavaScriptLibraryMappings">
-    <excludedPredefinedLibrary name="AJAX" />
-    <excludedPredefinedLibrary name="DHTML" />
-    <excludedPredefinedLibrary name="DOM Core" />
-    <excludedPredefinedLibrary name="DOM Events" />
-    <excludedPredefinedLibrary name="DOM Traversal And Range" />
-    <excludedPredefinedLibrary name="DOM XPath" />
-    <excludedPredefinedLibrary name="EcmaScript" />
-    <excludedPredefinedLibrary name="EcmaScript Additional" />
-    <excludedPredefinedLibrary name="EcmaScript L5" />
-    <excludedPredefinedLibrary name="EcmaScript for XML" />
-    <excludedPredefinedLibrary name="HTML 5" />
-    <excludedPredefinedLibrary name="WebGL" />
-  </component>
-</project>
-