Formatting - StandardRB updates

CHANGELOG.md

@@ -1,4 +1,8 @@
-## 0.1.7 ( Unreleased )
+## 0.1.9 (Unreleased)
+- Fix bug allowing non-title case Request Parameters
+- Add the required PartnerTag and PartnerType key values to the search body request
+
+## 0.1.7
 - Add gem 'net-http-persistent' for connection persistance
 - Remove the code which would use other http clients if available
 ## 0.1.3

lib/paapi.rb

@@ -1,81 +1,82 @@
-require 'paapi/version'
+require "paapi/version"
 
-require 'paapi/client'
-require 'paapi/item'
-require 'paapi/listing'
-require 'paapi/response'
+require "paapi/client"
+require "paapi/item"
+require "paapi/listing"
+require "paapi/response"
 
 module Paapi
   class Error < StandardError; end
+
   class NotImplemented < StandardError; end
-  SEARCH_PARAMS = %i[keywords actor, artist, author, brand title].freeze
-  DEFAULT_PARTNER_TYPE = 'Associates'
+  SEARCH_PARAMS = %i[Keywords Actor Artist Author Brand Title].freeze
+  DEFAULT_PARTNER_TYPE = "Associates"
   DEFAULT_MARKET = :us
-  DEFAULT_CONDITION  = 'Any'
+  DEFAULT_CONDITION = "Any"
   DEFAULT_RESOURCES = [
-    'Images.Primary.Large',
-    'ItemInfo.ByLineInfo',
-    'ItemInfo.ContentInfo',
-    'ItemInfo.ExternalIds',
-    'ItemInfo.Features',
-    'ItemInfo.ManufactureInfo',
-    'ItemInfo.ProductInfo',
-    'ItemInfo.TechnicalInfo', # Includes format when Kindle
-    'ItemInfo.Title',
-    'ItemInfo.TradeInInfo',
-    'Offers.Listings.Availability.Message',
-    'Offers.Listings.Condition',
-    'Offers.Listings.Condition.SubCondition',
-    'Offers.Listings.DeliveryInfo.IsAmazonFulfilled',
-    'Offers.Listings.DeliveryInfo.IsFreeShippingEligible',
-    'Offers.Listings.DeliveryInfo.IsPrimeEligible',
-    'Offers.Listings.MerchantInfo',
-    'Offers.Listings.Price',
-    'Offers.Listings.SavingBasis'
+    "Images.Primary.Large",
+    "ItemInfo.ByLineInfo",
+    "ItemInfo.ContentInfo",
+    "ItemInfo.ExternalIds",
+    "ItemInfo.Features",
+    "ItemInfo.ManufactureInfo",
+    "ItemInfo.ProductInfo",
+    "ItemInfo.TechnicalInfo", # Includes format when Kindle
+    "ItemInfo.Title",
+    "ItemInfo.TradeInInfo",
+    "Offers.Listings.Availability.Message",
+    "Offers.Listings.Condition",
+    "Offers.Listings.Condition.SubCondition",
+    "Offers.Listings.DeliveryInfo.IsAmazonFulfilled",
+    "Offers.Listings.DeliveryInfo.IsFreeShippingEligible",
+    "Offers.Listings.DeliveryInfo.IsPrimeEligible",
+    "Offers.Listings.MerchantInfo",
+    "Offers.Listings.Price",
+    "Offers.Listings.SavingBasis"
   ].freeze
 
   Locale = Struct.new(:key, :name, :host, :region) do
     def site
-      host.sub('webservices', 'www')
+      host.sub("webservices", "www")
     end
   end
 
   MARKETPLACES = {
-    au: Locale.new(:au, 'Australia',	          'webservices.amazon.com.au', 'us-west-2'),
-    br: Locale.new(:br, 'Brazil',	              'webservices.amazon.com.br', 'us-east-1'),
-    ca: Locale.new(:ca, 'Canada',	              'webservices.amazon.ca',	   'us-east-1'),
-    fr: Locale.new(:fr, 'France',	              'webservices.amazon.fr',	   'eu-west-1'),
-    de: Locale.new(:de, 'Germany',	            'webservices.amazon.de',	   'eu-west-1'),
-    in: Locale.new(:in, 'India',	              'webservices.amazon.in',	   'eu-west-1'),
-    it: Locale.new(:it, 'Italy',	              'webservices.amazon.it',	   'eu-west-1'),
-    jp: Locale.new(:jp, 'Japan',	              'webservices.amazon.co.jp',	 'us-west-2'),
-    mx: Locale.new(:mx, 'Mexico',	              'webservices.amazon.com.mx', 'us-east-1'),
-    es: Locale.new(:es, 'Spain',                'webservices.amazon.es',	   'eu-west-1'),
-    tr: Locale.new(:tk, 'Turkey',	              'webservices.amazon.com.tr', 'eu-west-1'),
-    ae: Locale.new(:ae, 'United Arab Emirates',	'webservices.amazon.ae',	   'eu-west-1'),
-    uk: Locale.new(:uk, 'United Kingdom',	      'webservices.amazon.co.uk',	 'eu-west-1'),
-    us: Locale.new(:us, 'United States',	      'webservices.amazon.com',	   'us-east-1'),
+    au: Locale.new(:au, "Australia", "webservices.amazon.com.au", "us-west-2"),
+    br: Locale.new(:br, "Brazil", "webservices.amazon.com.br", "us-east-1"),
+    ca: Locale.new(:ca, "Canada", "webservices.amazon.ca", "us-east-1"),
+    fr: Locale.new(:fr, "France", "webservices.amazon.fr", "eu-west-1"),
+    de: Locale.new(:de, "Germany", "webservices.amazon.de", "eu-west-1"),
+    in: Locale.new(:in, "India", "webservices.amazon.in", "eu-west-1"),
+    it: Locale.new(:it, "Italy", "webservices.amazon.it", "eu-west-1"),
+    jp: Locale.new(:jp, "Japan", "webservices.amazon.co.jp", "us-west-2"),
+    mx: Locale.new(:mx, "Mexico", "webservices.amazon.com.mx", "us-east-1"),
+    es: Locale.new(:es, "Spain", "webservices.amazon.es", "eu-west-1"),
+    tr: Locale.new(:tk, "Turkey", "webservices.amazon.com.tr", "eu-west-1"),
+    ae: Locale.new(:ae, "United Arab Emirates",	"webservices.amazon.ae", "eu-west-1"),
+    uk: Locale.new(:uk, "United Kingdom", "webservices.amazon.co.uk", "eu-west-1"),
+    us: Locale.new(:us, "United States", "webservices.amazon.com", "us-east-1")
   }.freeze
 
-  Operation = Struct.new(:target_name, :endpoint_suffix, :http_method, :service )
+  Operation = Struct.new(:target_name, :endpoint_suffix, :http_method, :service)
 
   OPERATIONS = {
-    get_browse_nodes: Operation.new( 'GetBrowseNodes', 'getbrowsenodes', 'POST', 'ProductAdvertisingAPI' ),
-    get_items:        Operation.new( 'GetItems',       'getitems',       'POST', 'ProductAdvertisingAPI' ),
-    get_variations:   Operation.new( 'GetVariations',  'getvariations',  'POST', 'ProductAdvertisingAPI' ),
-    search_items:     Operation.new( 'SearchItems',    'searchitems',    'POST', 'ProductAdvertisingAPI' )
+    get_browse_nodes: Operation.new("GetBrowseNodes", "getbrowsenodes", "POST", "ProductAdvertisingAPI"),
+    get_items: Operation.new("GetItems", "getitems", "POST", "ProductAdvertisingAPI"),
+    get_variations: Operation.new("GetVariations", "getvariations", "POST", "ProductAdvertisingAPI"),
+    search_items: Operation.new("SearchItems", "searchitems", "POST", "ProductAdvertisingAPI")
   }.freeze
 
   class << self
     attr_accessor :access_key,
-                  :secret_key,
-                  :partner_tag,
-                  :partner_type,
-                  :market,
-                  :partner_market,
-                  :condition,
-                  :resources,
-                  :test_mode
+      :secret_key,
+      :partner_tag,
+      :partner_type,
+      :market,
+      :partner_market,
+      :condition,
+      :resources,
+      :test_mode
 
     def configure
       yield self
@@ -85,7 +86,6 @@ module Paapi
   end
 
   def symbolize_keys(hash)
-    Hash[hash.map{|k,v| v.is_a?(Hash) ? [k.to_sym, symbolize_keys(v)] : [k.to_sym, v] }]
+    hash.map { |k, v| v.is_a?(Hash) ? [k.to_sym, symbolize_keys(v)] : [k.to_sym, v] }.to_h
   end
 end
-

lib/paapi/client.rb

@@ -3,7 +3,6 @@ require 'aws-sigv4'
 
 module Paapi
   class Client
-
     attr_accessor :partner_tag, :marketplace, :resources, :condition
     attr_reader :partner_type, :access_key, :secret_key, :market, :http
 
@@ -31,13 +30,13 @@ module Paapi
     def market=(a_market)
       @market = a_market
       @marketplace = MARKETPLACES[market.to_sym]
-      if !Paapi.partner_market.nil?
-        @partner_tag = Paapi.partner_market.dig(a_market.to_sym) || @partner_tag
-      end
+      return if Paapi.partner_market.nil?
+
+      @partner_tag = Paapi.partner_market[a_market.to_sym] || @partner_tag
     end
 
     def get_items(item_ids:, **options)
-      payload = { ItemIds: Array(item_ids), Resources:  @resources }.merge(options)
+      payload = { PartnerTag: partner_tag, PartnerType: 'Associates', ItemIds: Array(item_ids), Resources: @resources }.merge(options)
       request(op: :get_items, payload: payload)
     end
 
@@ -47,12 +46,12 @@ module Paapi
     end
 
     # TODO: Currently we assume Keywords, but we need one of the following: [Keywords Actor Artist Author Brand Title ]
-    def search_items(keywords: nil, **options )
-      raise ArgumentError("Missing keywords") unless (options.keys | SEARCH_PARAMS).length.positive?
+    def search_items(**options )
+      raise ArgumentError.new("Missing keywords") unless (options.keys & SEARCH_PARAMS).length.positive?
 
       search_index = options.dig(:SearchIndex) ||'All'
 
-      payload = { Keywords: keywords, Resources:  @resources, ItemCount: 10, ItemPage: 1, SearchIndex: search_index }.merge(options)
+      payload = { PartnerTag: partner_tag, PartnerType: 'Associates', Resources:  @resources, ItemCount: 10, ItemPage: 1, SearchIndex: search_index }.merge(options)
 
       request(op: :search_items, payload: payload)
     end
@@ -115,7 +114,6 @@ module Paapi
       post_request.body = body.to_json
 
       http.request uri, post_request
-
     end
   end
 end

lib/paapi/version.rb

@@ -1,3 +1,3 @@
 module Paapi
-  VERSION = '0.1.8'
+  VERSION = '0.1.9'
 end

paapi.gemspec

@@ -29,7 +29,7 @@ Gem::Specification.new do |spec|
   spec.add_development_dependency 'rake', '>= 12.3.r3'
   spec.add_development_dependency 'minitest', '~> 5.0'
   spec.add_development_dependency 'byebug', '~> 11'
-  spec.add_development_dependency 'awesome_print', '~> 1.8'
+  spec.add_development_dependency 'standard'
 
   spec.add_dependency 'aws-sigv4', '~> 1'
   spec.add_dependency 'net-http-persistent', '~> 4.0', '>= 4.0.1'