vacuum 2.2.0 → 3.4.0

data/test/helper.rb

@@ -0,0 +1,10 @@
+# frozen_string_literal: true
+
+# Keep SimpleCov at top.
+require 'simplecov'
+
+SimpleCov.start do
+  add_filter '/test/'
+end
+
+require 'minitest/autorun'

data/test/integration/test_requests.rb

@@ -0,0 +1,62 @@
+# frozen_string_literal: true
+
+require 'integration_helper'
+
+module Vacuum
+  class TestRequests < IntegrationTest
+    def test_get_browse_nodes
+      requests.each do |request|
+        response = request.get_browse_nodes(browse_node_ids: ['3045'])
+        assert_equal 200, response.status
+      end
+    end
+
+    def test_get_items
+      requests.each do |request|
+        response = request.get_items(item_ids: ['B07212L4G2'])
+        assert_equal 200, response.status
+      end
+    end
+
+    def test_get_items_with_all_resources
+      requests.each do |request|
+        response = request.get_items(item_ids: 'B07212L4G2',
+                                     resources: Resource.all)
+        assert_equal 200, response.status
+        item = response.dig('ItemsResult', 'Items').first
+        assert item.key?('BrowseNodeInfo')
+      end
+    end
+
+    def test_get_variations
+      requests.each do |request|
+        response = request.get_variations(asin: 'B07212L4G2')
+        assert_equal 200, response.status
+      end
+    end
+
+    def test_search_items
+      requests.each do |request|
+        response = request.search_items(keywords: 'Harry Potter')
+        assert_equal 200, response.status
+      end
+    end
+
+    def test_persistent
+      request = requests.sample
+      refute request.client.persistent?
+      request.persistent
+      assert request.client.persistent?
+    end
+
+    def test_logging
+      require 'logger'
+      logdev = StringIO.new
+      logger = Logger.new(logdev)
+      request = requests.sample
+      request.use(logging: { logger: logger })
+      request.search_items(keywords: 'Harry Potter')
+      refute_empty logdev.string
+    end
+  end
+end

data/test/integration_helper.rb

@@ -0,0 +1,51 @@
+# frozen_string_literal: true
+
+require 'helper'
+require 'vcr'
+
+require 'locales'
+require 'vacuum'
+require 'vacuum/matcher'
+
+VCR.configure do |c|
+  c.hook_into :webmock
+  c.cassette_library_dir = 'test/cassettes'
+  c.default_cassette_options = {
+    match_requests_on: [Vacuum::Matcher],
+    record: ENV['RECORD'] ? :new_episodes : :none
+  }
+  c.before_record do |interaction|
+    interaction.ignore! if interaction.response.status.code != 200
+  end
+
+  Locales.each do |record|
+    record.each do |key, val|
+      next if key == :marketplace
+
+      c.filter_sensitive_data(key.upcase) { val }
+    end
+  end
+end
+
+module Vacuum
+  class IntegrationTest < Minitest::Test
+    def setup
+      if ENV['LIVE']
+        VCR.turn_off!
+        WebMock.allow_net_connect!
+      else
+        VCR.insert_cassette('vacuum')
+      end
+    end
+
+    def teardown
+      VCR.eject_cassette if VCR.turned_on?
+    end
+
+    private
+
+    def requests
+      Locales.map { |credentials| Vacuum.new(**credentials) }.shuffle
+    end
+  end
+end

data/test/locales.rb

@@ -0,0 +1,23 @@
+# frozen_string_literal: true
+
+require 'forwardable'
+require 'yaml'
+
+module Locales
+  class << self
+    extend Forwardable
+    include Enumerable
+
+    attr_reader :all
+
+    def_delegators :all, :each
+  end
+
+  %w[locales.yml locales.yml.example].each do |filename|
+    path = File.expand_path(filename, __dir__)
+    if File.exist?(path)
+      @all = YAML.load_file(path)
+      break
+    end
+  end
+end

data/test/locales.yml

@@ -0,0 +1,28 @@
+- :marketplace: CA
+  :access_key: AKIAIWQTVHVWCQA5THXQ
+  :secret_key: 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
+  :partner_tag: adabo00-20
+- :marketplace: ES
+  :access_key: AKIAJJF26BJLZLSJLFBQ
+  :secret_key: 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
+  :partner_tag: adabo00-21
+- :marketplace: DE
+  :access_key: AKIAJGN5UNM43HHV5MZA
+  :secret_key: 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
+  :partner_tag: adabo23-21
+- :marketplace: FR
+  :access_key: AKIAJDNYBNMN23YBFK3A
+  :secret_key: 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
+  :partner_tag: adabo-21
+- :marketplace: IT
+  :access_key: AKIAJVBAH4XJHK6YF46Q
+  :secret_key: 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
+  :partner_tag: adabo0c-21
+- :marketplace: GB
+  :access_key: AKIAJPAXMP45DOQJPHJQ
+  :secret_key: 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
+  :partner_tag: adabo21-21
+- :marketplace: US
+  :access_key: AKIAIWQTVHVWCQA5THXQ
+  :secret_key: VOhSIqIY1wF5selhhJYJGVnIET9k5q0J8QjTX38i
+  :partner_tag:  adabo-20

data/test/locales.yml.example

@@ -0,0 +1,28 @@
+- :marketplace: CA
+  :access_key: access_key
+  :secret_key: secret_key
+  :partner_tag: partner_tag
+- :marketplace: ES
+  :access_key: access_key
+  :secret_key: secret_key
+  :partner_tag: partner_tag
+- :marketplace: DE
+  :access_key: access_key
+  :secret_key: secret_key
+  :partner_tag: partner_tag
+- :marketplace: FR
+  :access_key: access_key
+  :secret_key: secret_key
+  :partner_tag: partner_tag
+- :marketplace: IT
+  :access_key: access_key
+  :secret_key: secret_key
+  :partner_tag: partner_tag
+- :marketplace: GB
+  :access_key: access_key
+  :secret_key: secret_key
+  :partner_tag: partner_tag
+- :marketplace: US
+  :access_key: access_key
+  :secret_key: secret_key
+  :partner_tag: partner_tag

data/test/unit/test_locale.rb

@@ -0,0 +1,43 @@
+# frozen_string_literal: true
+
+require 'helper'
+require 'vacuum/locale'
+
+module Vacuum
+  class TestLocale < Minitest::Test
+    def test_host
+      locale = Locale.new(:us, **credentials)
+      assert locale.host
+    end
+
+    def test_region
+      locale = Locale.new(:us, **credentials)
+      assert locale.region
+    end
+
+    def test_partner_type_default
+      locale = Locale.new(:us, **credentials)
+      assert locale.partner_type
+    end
+
+    def test_upcased_marketplace
+      Locale.new('US', **credentials) # does not raise
+    end
+
+    def test_uk
+      Locale.new(:uk, **credentials) # does not raise
+    end
+
+    def test_not_found
+      assert_raises Locale::NotFound do
+        Locale.new(:foo, **credentials)
+      end
+    end
+
+    private
+
+    def credentials
+      { access_key: '123', secret_key: '123', partner_tag: 'xyz-20' }
+    end
+  end
+end

data/test/unit/test_operation.rb

@@ -0,0 +1,31 @@
+# frozen_string_literal: true
+
+require 'helper'
+require 'vacuum/locale'
+require 'vacuum/operation'
+
+module Vacuum
+  class TestOperation < Minitest::Test
+    def setup
+      @operation = Operation.new('Action', params: { foo: 1 }, locale: locale)
+    end
+
+    def test_body
+      assert @operation.body
+    end
+
+    def test_url
+      assert @operation.url
+    end
+
+    def test_headers
+      assert @operation.headers
+    end
+
+    private
+
+    def locale
+      Locale.new(:us, access_key: '123', secret_key: '123', partner_tag: 'xyz')
+    end
+  end
+end

data/test/unit/test_request.rb

@@ -0,0 +1,61 @@
+# frozen_string_literal: true
+
+require 'helper'
+require 'vacuum/request'
+
+module Vacuum
+  class TestRequest < Minitest::Test
+    def test_search_keywords
+      err = assert_raises(ArgumentError) do
+        api.search_items(keywords: ['Harry Potter'])
+      end
+
+      assert_equal ':keyword argument expects a String', err.message
+    end
+
+    def test_search_resources
+      err = assert_raises(ArgumentError) do
+        api.search_items(
+          keywords: 'Harry Potter',
+          resources: %w[Images.Primary.Large ItemInfo.ExternalIds Offer.Listings.Price]
+        )
+      end
+
+      assert_equal 'There is not such resource: Offer.Listings.Price', err.message
+    end
+
+    def test_get_items_wrong_resource
+      err = assert_raises(ArgumentError) do
+        api.get_items(
+          item_ids: :B07212L4G2,
+          resources: %w[BrowseNodeInfo.BrowseNode.SalesRank]
+        )
+      end
+
+      assert_equal 'There is not such resource: BrowseNodeInfo.BrowseNode.SalesRank', err.message
+    end
+
+    def test_get_variations_wrong_resource
+      err = assert_raises(ArgumentError) do
+        api.get_variations(asin: 'B07212L4G2', resources: %w[BrowseNodeInfo.BrowseNode.SalesRank])
+      end
+
+      assert_equal 'There is not such resource: BrowseNodeInfo.BrowseNode.SalesRank', err.message
+    end
+
+    def test_resource_not_array
+      err = assert_raises(ArgumentError) do
+        api.get_items(item_ids: :B07212L4G2, resources: 'foo')
+      end
+
+      assert_equal ':resources argument expects an Array', err.message
+    end
+
+    def api
+      @api ||= Vacuum::Request.new(marketplace: 'US',
+                                   access_key: 'key',
+                                   secret_key: 'secret',
+                                   partner_tag: 'tag')
+    end
+  end
+end

data/test/unit/test_resource.rb

@@ -0,0 +1,12 @@
+# frozen_string_literal: true
+
+require 'helper'
+require 'vacuum/resource'
+
+module Vacuum
+  class TestResource < Minitest::Test
+    def test_all
+      refute_empty Resource.all
+    end
+  end
+end

data/test/unit/test_response.rb

@@ -0,0 +1,26 @@
+# frozen_string_literal: true
+
+require 'helper'
+require 'vacuum/response'
+
+module Vacuum
+  class TestResponse < Minitest::Test
+    def setup
+      mock = Minitest::Mock.new
+      mock.expect(:body, %({"ItemsResult":{"Items":[{"ASIN":"B07212L4G2"}]}}))
+      @response = Response.new(mock)
+    end
+
+    def test_parse
+      assert @response.parse
+    end
+
+    def test_cast_to_hash
+      assert_kind_of Hash, @response.to_h
+    end
+
+    def test_dig
+      assert @response.dig('ItemsResult', 'Items')
+    end
+  end
+end

metadata

@@ -1,45 +1,46 @@
 --- !ruby/object:Gem::Specification
 name: vacuum
 version: !ruby/object:Gem::Version
-  version: 2.2.0
+  version: 3.4.0
 platform: ruby
 authors:
 - Hakan Ensari
+- Stanislav Katkov
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2019-06-17 00:00:00.000000000 Z
+date: 2020-05-27 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
-  name: jeff
+  name: aws-sigv4
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '2.0'
+        version: '1.0'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '2.0'
+        version: '1.0'
 - !ruby/object:Gem::Dependency
-  name: multi_xml
+  name: http
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.6.0
+        version: '4.0'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.6.0
+        version: '4.0'
 - !ruby/object:Gem::Dependency
-  name: appraisal
+  name: minitest
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
@@ -53,7 +54,7 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
-  name: minitest
+  name: rake
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
@@ -67,7 +68,7 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
-  name: pry
+  name: rubocop
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
@@ -81,7 +82,21 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
-  name: rake
+  name: simplecov
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - '='
+      - !ruby/object:Gem::Version
+        version: 0.17.1
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - '='
+      - !ruby/object:Gem::Version
+        version: 0.17.1
+- !ruby/object:Gem::Dependency
+  name: vcr
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
@@ -95,7 +110,7 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
-  name: rubocop
+  name: webmock
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
@@ -109,7 +124,7 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
-  name: vcr
+  name: yard
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
@@ -125,6 +140,7 @@ dependencies:
 description: A wrapper to the Amazon Product Advertising API
 email:
 - me@hakanensari.com
+- sk@skylup.com
 executables: []
 extensions: []
 extra_rdoc_files: []
@@ -132,12 +148,25 @@ files:
 - LICENSE
 - README.md
 - lib/vacuum.rb
+- lib/vacuum/locale.rb
+- lib/vacuum/matcher.rb
+- lib/vacuum/operation.rb
 - lib/vacuum/request.rb
+- lib/vacuum/resource.rb
 - lib/vacuum/response.rb
 - lib/vacuum/version.rb
 - test/cassettes/vacuum.yml
-- test/test_integration.rb
-- test/test_unit.rb
+- test/helper.rb
+- test/integration/test_requests.rb
+- test/integration_helper.rb
+- test/locales.rb
+- test/locales.yml
+- test/locales.yml.example
+- test/unit/test_locale.rb
+- test/unit/test_operation.rb
+- test/unit/test_request.rb
+- test/unit/test_resource.rb
+- test/unit/test_response.rb
 homepage: https://github.com/hakanensari/vacuum
 licenses:
 - MIT
@@ -150,18 +179,27 @@ required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
-      version: '2.3'
+      version: '2.5'
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.0.3
+rubygems_version: 3.1.2
 signing_key: 
 specification_version: 4
 summary: Amazon Product Advertising in Ruby
 test_files:
-- test/test_integration.rb
+- test/unit/test_operation.rb
+- test/unit/test_request.rb
+- test/unit/test_resource.rb
+- test/unit/test_response.rb
+- test/unit/test_locale.rb
+- test/helper.rb
+- test/integration/test_requests.rb
 - test/cassettes/vacuum.yml
-- test/test_unit.rb
+- test/integration_helper.rb
+- test/locales.rb
+- test/locales.yml
+- test/locales.yml.example