vacuum 0.2.1 → 0.2.2

data/lib/vacuum/request/product_advertising.rb

@@ -15,27 +15,33 @@ module Vacuum
       #
       # Returns a Vacuum::Response::ProductAdvertising.
       def look_up(*item_ids)
-        given_params = item_ids.last.is_a?(Hash) ? item_ids.pop : {}
+        given = item_ids.last.is_a?(Hash) ? item_ids.pop : {}
 
-        params =
+        params = begin
           case item_ids.size
           when 1..10
             {
               'Operation' => 'ItemLookup',
               'ItemId'    => item_ids
-            }.merge given_params
+            }.merge given
           when 11..20
             default = {
-              'Operation'       => 'ItemLookup',
+              'Operation'           => 'ItemLookup',
               'ItemLookup.1.ItemId' => item_ids.shift(10),
               'ItemLookup.2.ItemId' => item_ids
             }
-            given_params.reduce(default) do |a, (k, v)|
+
+            if version = given.delete('Version') || given.delete(:version)
+              default['Version'] = version
+            end
+
+            given.reduce(default) do |a, (k, v)|
               a.merge "ItemLookup.Shared.#{Utils.camelize k.to_s}" => v
             end
           else
             raise ArgumentError, "Can't look up #{item_ids.size} items"
           end
+        end
         build! params
 
         get!

data/lib/vacuum/version.rb

@@ -1,3 +1,3 @@
 module Vacuum
-  VERSION = '0.2.1'
+  VERSION = '0.2.2'
 end

data/spec/vacuum/request/product_advertising_spec.rb

@@ -48,19 +48,25 @@ module Vacuum
 
         context 'given 11 to to 20 items' do
           before do
-            request.look_up *((1..20).to_a << { :foo => 'bar' })
+            request.look_up *((1..20).to_a << {
+              :foo => 'bar',
+              :version => 'baz'
+            })
           end
 
           it 'builds a multi-batch query' do
-            first_batch = parameters['ItemLookup.1.ItemId'].split(',')
-            first_batch.should =~ (1..10).map(&:to_s)
-            second_batch = parameters['ItemLookup.2.ItemId'].split(',')
-            second_batch.should =~ (11..20).map(&:to_s)
+            first = parameters['ItemLookup.1.ItemId'].split(',')
+            second = parameters['ItemLookup.2.ItemId'].split(',')
+            (first + second).should =~ (1..20).map(&:to_s)
           end
 
           it 'takes parameters' do
             parameters['ItemLookup.Shared.Foo'].should eql 'bar'
           end
+
+          it 'overrides version' do
+            parameters['Version'].should eql 'baz'
+          end
         end
 
         context 'given over 20 items' do

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: vacuum
 version: !ruby/object:Gem::Version
-  version: 0.2.1
+  version: 0.2.2
   prerelease: 
 platform: ruby
 authors:
@@ -9,11 +9,11 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2012-04-04 00:00:00.000000000 Z
+date: 2012-04-05 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rake
-  requirement: &70156392348820 !ruby/object:Gem::Requirement
+  requirement: &70295025564860 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ~>
@@ -21,10 +21,10 @@ dependencies:
         version: '0.9'
   type: :development
   prerelease: false
-  version_requirements: *70156392348820
+  version_requirements: *70295025564860
 - !ruby/object:Gem::Dependency
   name: rspec
-  requirement: &70156392348380 !ruby/object:Gem::Requirement
+  requirement: &70295025564460 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ~>
@@ -32,10 +32,10 @@ dependencies:
         version: '2.9'
   type: :development
   prerelease: false
-  version_requirements: *70156392348380
+  version_requirements: *70295025564460
 - !ruby/object:Gem::Dependency
   name: addressable
-  requirement: &70156392347700 !ruby/object:Gem::Requirement
+  requirement: &70295025563720 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ~>
@@ -43,10 +43,10 @@ dependencies:
         version: '2.2'
   type: :runtime
   prerelease: false
-  version_requirements: *70156392347700
+  version_requirements: *70295025563720
 - !ruby/object:Gem::Dependency
   name: faraday
-  requirement: &70156392347120 !ruby/object:Gem::Requirement
+  requirement: &70295025563140 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ~>
@@ -54,10 +54,10 @@ dependencies:
         version: 0.7.6
   type: :runtime
   prerelease: false
-  version_requirements: *70156392347120
+  version_requirements: *70295025563140
 - !ruby/object:Gem::Dependency
   name: nokogiri
-  requirement: &70156392346540 !ruby/object:Gem::Requirement
+  requirement: &70295025562560 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ~>
@@ -65,7 +65,7 @@ dependencies:
         version: '1.5'
   type: :runtime
   prerelease: false
-  version_requirements: *70156392346540
+  version_requirements: *70295025562560
 description: ! 'Vacuum is a wrapper to various Amazon Web Services (AWS) APIs, including
 
   Product Advertising and Marketplace Web Services (MWS).