peddler 0.14.0 → 0.15.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 4d37ce78b08881900ca193e7c541a2ce9dd47dd0
-  data.tar.gz: 2ccd80425d28a56835350179392cedbc03248645
+  metadata.gz: 04fe2f05cb29fdcfed397c2a449cee124e59e804
+  data.tar.gz: 703bbfa1b3586e6de667e9588c08160f16e44483
 SHA512:
-  metadata.gz: 6c87a66569da567099b9b8faf59e9fc0161730b1a0b7dfb1eaf779c1931f7e275a7054de2027dabeb0100415f0f2c45c961e0a30189260bcd4904a8b1f27d2ce
-  data.tar.gz: f59e5ca83abbae992b837c280af72641732f2b17956f8cc149a5f2b7496ff140d797511f2a15f27226cbc0db77f8727e80e0845f80e8fba5b75f3ad163ed2b1b
+  metadata.gz: f2bb404705d1ebf31383bfa2175f1f42319594a7b6bd1e1f0aa49959db89da39bc49a22a59ba4baa17d4fb8a288b480ecbf8ba3fb12f9b47e98259eef1c97641
+  data.tar.gz: 1998213239427a3ab9bbbd5d363e707827ef01ad973d8a891426775fbe1e1f4808d29d241638b81fbf0891aed1389ab22837e14b79633035bc4d4cce3fc293ff

data/lib/mws/sellers/client.rb

@@ -11,7 +11,6 @@ module MWS
       # Lists the marketplaces the seller participates in
       #
       # @see http://docs.developer.amazonservices.com/en_US/sellers/Sellers_ListMarketplaceParticipations.html
-      # @param next_token [String]
       # @return [Peddler::XMLParser]
       def list_marketplace_participations
         operation('ListMarketplaceParticipations')

data/lib/peddler/client.rb

@@ -12,22 +12,17 @@ module Peddler
     extend Forwardable
     include Jeff
 
-    # @return [String] the MWSAuthToken used to access another seller's account
+    # The MWSAuthToken used to access another seller's account
+    # @return [String]
     attr_accessor :auth_token
 
-    # @return [String] the merchant's Seller ID
-    attr_writer :merchant_id
+    attr_writer :merchant_id, :marketplace_id, :path
 
-    # @return [String] the merchant's Marketplace ID
-    attr_writer :marketplace_id
-
-    # @return [String] the HTTP path of the API
-    attr_writer :path
-
-    # @return [String] the version of the API
+    # @api private
     attr_writer :version
 
-    # @return [String] the body of the HTTP request
+    # The body of the HTTP request
+    # @return [String]
     attr_reader :body
 
     alias_method :configure, :tap
@@ -41,24 +36,34 @@ module Peddler
     )
 
     class << self
+      # @api private
       attr_reader :error_handler
 
+      # @api private
       def parser
         @parser ||= Parser
       end
 
+      # A custom parser
+      # @!parse attr_writer :parser
+      # @return [Object]
       def parser=(parser)
         @parser = parser
       end
 
+      # @api private
       def path(path = nil)
         path ? @path = path : @path ||= '/'
       end
 
+      # @api private
       def version(version = nil)
         version ? @version = version : @version
       end
 
+      # Sets an error handler
+      # @yieldparam request [Excon::Request]
+      # @yieldparam response [Excon::Response]
       def on_error(&blk)
         @error_handler = blk
       end
@@ -71,59 +76,88 @@ module Peddler
       end
     end
 
+    # Creates a new client instance
+    #
+    # @param opts [Hash]
+    # @option opts [String] :marketplace_id
+    # @option opts [String] :merchant_id
+    # @option opts [String] :aws_access_key_id
+    # @option opts [String] :aws_secret_access_key
+    # @option opts [String] :auth_token
     def initialize(opts = {})
       opts.each { |k, v| send("#{k}=", v) }
     end
 
+    # @api private
     def aws_endpoint
       "https://#{host}#{path}"
     end
 
+    # The merchant's Marketplace ID
+    # @!parse attr_reader :marketplace_id
+    # @return [String]
     def marketplace_id
       @marketplace_id ||= ENV['MWS_MARKETPLACE_ID']
     end
 
+    # The merchant's Seller ID
+    # @!parse attr_reader :merchant_id
+    # @return [String]
     def merchant_id
       @merchant_id ||= ENV['MWS_MERCHANT_ID']
     end
 
+    # @api private
     def marketplace
       @marketplace ||= find_marketplace
     end
 
-    def defaults
-      @defaults ||= { expects: 200 }
-    end
-
-    def headers
-      @headers ||= {}
-    end
-
+    # The HTTP path of the API
+    # @!parse attr_reader :path
+    # @return [String]
     def path
       @path ||= self.class.path
     end
 
+    # @api private
     def version
       @version ||= self.class.version
     end
 
+    # @!parse attr_writer :body
     def body=(str)
       headers['Content-Type'] = content_type(str)
       @body = str
     end
 
+    # @api private
+    def defaults
+      @defaults ||= { expects: 200 }
+    end
+
+    # @api private
+    def headers
+      @headers ||= {}
+    end
+
+    # Sets an error handler
+    # @yieldparam request [Excon::Request]
+    # @yieldparam response [Excon::Response]
     def on_error(&blk)
       @error_handler = blk
     end
 
+    # @api private
     def error_handler
       @error_handler || self.class.error_handler
     end
 
+    # @api private
     def operation(action = nil)
       action ? @operation = Operation.new(action) : @operation
     end
 
+    # @api private
     def run
       opts = defaults.merge(query: operation, headers: headers)
       opts.store(:body, body) if body

data/lib/peddler/version.rb

@@ -1,3 +1,3 @@
 module Peddler
-  VERSION = '0.14.0'
+  VERSION = '0.15.0'
 end

data/test/helper.rb

@@ -58,7 +58,8 @@ VCR.configure do |c|
 
   # HTTP errors are not Peddler's concern, so ignore them to ease development.
   c.before_record do |interaction|
-    interaction.ignore! if interaction.response.status.code >= 400
+    code = interaction.response.status.code
+    interaction.ignore! if code >= 400 && code != 414
   end
 
   # Ignore transient params when building VCR fixtures.

data/test/integration/test_fulfillment_inbound_shipment.rb

@@ -21,4 +21,13 @@ class TestFulfillmentInboundShipment < IntegrationTest
       refute_empty res.parse
     end
   end
+
+  def test_handles_large_requests
+    address = Address.new('John', '1 Main St', 'New York', 'NY', '10001', 'US')
+    items = 100.times.map { |count| Item.new(count, 1) }
+    clients.each do |client|
+      res = client.create_inbound_shipment_plan(address, items)
+      assert_equal 200, res.status
+    end
+  end
 end

data/test/mws.yml.1

@@ -0,0 +1,4 @@
+- marketplace_id: ATVPDKIKX0DER
+  aws_access_key_id: AKIAJOD3NLXHOMANRNQQ
+  aws_secret_access_key: 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
+  merchant_id: A2JYSO6W6KEP83