gecko-ruby 0.9.0 → 0.12.0

data/test/record/invoice_test.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'test_helper'
 
 class Gecko::InvoiceTest < Minitest::Test

data/test/record/location_adapter_test.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'test_helper'
 
 class Gecko::Record::LocationAdapterTest < Minitest::Test

data/test/record/location_test.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'test_helper'
 
 class Gecko::LocationTest < Minitest::Test

data/test/record/note_test.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'test_helper'
 
 class Gecko::NoteTest < Minitest::Test

data/test/record/order_adapter_test.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'test_helper'
 
 class Gecko::Record::OrderAdapterTest < Minitest::Test

data/test/record/order_line_item_adapter_test.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'test_helper'
 
 class Gecko::Record::OrderLineItemAdapterTest < Minitest::Test

data/test/record/order_line_item_test.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'test_helper'
 
 class Gecko::OrderLineItemTest < Minitest::Test

data/test/record/order_test.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'test_helper'
 
 class Gecko::OrderTest < Minitest::Test

data/test/record/payment_term_adapter_test.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'test_helper'
 
 class Gecko::Record::PaymentTermAdapterTest < Minitest::Test

data/test/record/payment_term_test.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'test_helper'
 
 class Gecko::PaymentTermTest < Minitest::Test

data/test/record/product_adapter_test.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'test_helper'
 
 class Gecko::Record::ProductAdapterTest < Minitest::Test

data/test/record/product_test.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'test_helper'
 
 class Gecko::ProductTest < Minitest::Test

data/test/record/purchase_order_adapter_test.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'test_helper'
 
 class Gecko::Record::PurchaseOrderAdapterTest < Minitest::Test

data/test/record/purchase_order_line_item_adapter_test.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'test_helper'
 
 class Gecko::Record::PurchaseOrderLineItemAdapterTest < Minitest::Test

data/test/record/purchase_order_line_item_test.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'test_helper'
 
 class Gecko::PurchaseOrderLineItemTest < Minitest::Test

data/test/record/purchase_order_test.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'test_helper'
 
 class Gecko::PurchaseOrderTest < Minitest::Test

data/test/record/tax_type_adapter_test.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'test_helper'
 
 class Gecko::Record::TaxTypeAdapterTest < Minitest::Test

data/test/record/tax_type_test.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'test_helper'
 
 class Gecko::TaxTypeTest < Minitest::Test

data/test/record/user_adapter_test.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'test_helper'
 
 class Gecko::Record::UserAdapterTest < Minitest::Test

data/test/record/user_test.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'test_helper'
 
 class Gecko::UserTest < Minitest::Test

data/test/record/variant_adapter_test.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'test_helper'
 
 class Gecko::Record::VariantAdapterTest < Minitest::Test

data/test/record/variant_test.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'test_helper'
 
 class Gecko::VariantTest < Minitest::Test
@@ -22,7 +24,7 @@ class Gecko::VariantTest < Minitest::Test
   end
 
   def test_variant_prices
-    json = {variant_prices: [{price_list_id: "buy", value: "12.50"}, {price_list_id: 123, value: "14.00"}]}
+    json = { variant_prices: [{ price_list_id: "buy", value: "12.50" }, { price_list_id: 123, value: "14.00" }] }
     variant_prices = record_class.new(client, json).variant_prices
     assert_instance_of(Gecko::Record::Variant::VariantPrice, variant_prices.first)
     assert_equal(variant_prices[0].price_list_id, "buy")
@@ -31,10 +33,10 @@ class Gecko::VariantTest < Minitest::Test
   end
 
   def test_variant_locations
-    json = {locations: [
+    json = { locations: [
       { location_id: 1, stock_on_hand: "12.50",
-        committed: "0", bin_location: "AB-123" },
-    ]}
+        committed: "0", bin_location: "AB-123" }
+    ] }
 
     locations = record_class.new(client, json).locations
     assert_instance_of(Gecko::Record::Variant::VariantLocation, locations.first)

data/test/record/webhook_test.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'test_helper'
 
 class Gecko::WebhookTest < Minitest::Test

data/test/support/let.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Minitest
   module Let
     def let(name, &block)

data/test/support/shared_adapter_examples.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 # Shared tests for Gecko::Record Adapters
 # requires definition of several variables
 # - adapter
@@ -5,7 +7,8 @@
 # - record_class
 
 require 'json'
-module SharedAdapterExamples
+
+module SharedAdapterExamples # rubocop:disable Metrics/ModuleLength
   def test_adapter_all
     VCR.use_cassette(plural_name) do
       collection = adapter.where(limit: 5)
@@ -49,29 +52,29 @@ module SharedAdapterExamples
 
   def test_has_record_for_id
     mock_record = Object.new
-    adapter.instance_variable_set(:@identity_map, {12 => mock_record})
+    adapter.instance_variable_set(:@identity_map, { 12 => mock_record })
     assert adapter.has_record_for_id?(12)
     assert !adapter.has_record_for_id?(12345)
   end
 
   def test_fetch
-    request_stub = stub_request(:get, /#{plural_name}\/\d+/)
-      .to_return({
-        headers: {"Content-Type" => "application/json"},
-        body:    JSON.dump({plural_name.singularize => {id: 12345}})
-      })
+    request_stub = stub_request(:get, %r{#{plural_name}/\d+})
+                   .to_return({
+                     headers: { "Content-Type" => "application/json" },
+                     body:    JSON.dump({ plural_name.singularize => { id: 12345 } })
+                   })
     record = adapter.fetch(12345)
     assert_equal(12345, record.id)
     assert_requested(request_stub)
   end
 
-  def test_fetch_miss
-    stub_request(:get, /#{plural_name}\/\d+/)
+  def test_fetch_miss # rubocop:disable Metrics/MethodLength
+    stub_request(:get, %r{#{plural_name}/\d+})
       .to_return({
         status:  404,
-        headers: {"Content-Type" => "application/json"},
-        body: JSON.dump({
-          type: "Not Found",
+        headers: { "Content-Type" => "application/json" },
+        body:    JSON.dump({
+          type:    "Not Found",
           message: "Couldn't find #{plural_name.singularize} with id 12345"
         })
       })
@@ -105,9 +108,8 @@ module SharedAdapterExamples
   def test_saving_new_record
     record = adapter.build
     mock_api_request(record,
-      [:post, plural_name],
-      [200, {plural_name.singularize => {id: 123}}]
-    )
+                     [:post, plural_name],
+                     [200, { plural_name.singularize => { id: 123 } }])
     adapter.save(record)
     assert_equal(record.id, 123)
     assert(record.valid?)
@@ -116,9 +118,8 @@ module SharedAdapterExamples
   def test_saving_new_invalid_record
     record = adapter.build
     mock_api_request(record,
-      [:post, plural_name],
-      [422, {"errors" => {title: ["can not be bounced"]}}]
-    )
+                     [:post, plural_name],
+                     [422, { "errors" => { title: ["can not be bounced"] } }])
     adapter.save(record)
     assert_nil(record.id)
     assert(!record.valid?)
@@ -127,9 +128,8 @@ module SharedAdapterExamples
   def test_saving_existing_record
     record = existing_record
     mock_api_request(record,
-      [:put, "#{plural_name}/#{record.id}"],
-      [204, '']
-    )
+                     [:put, "#{plural_name}/#{record.id}"],
+                     [204, ''])
     adapter.save(record)
     assert(record.valid?)
   end
@@ -137,23 +137,22 @@ module SharedAdapterExamples
   def test_saving_existing_invalid_record
     record = existing_record
     mock_api_request(record,
-      [:put, "#{plural_name}/#{record.id}"],
-      [422, {"errors" => {title: ["can not be bounced"]}}]
-    )
+                     [:put, "#{plural_name}/#{record.id}"],
+                     [422, { "errors" => { title: ["can not be bounced"] } }])
     adapter.save(record)
     assert(!record.valid?)
     assert(record.errors[:title].any?)
   end
 
-  def test_saving_record_with_idempotency_key
+  def test_saving_record_with_idempotency_key # rubocop:disable Metrics/MethodLength
     record = adapter.build
     mock_token = mock
-    mock_response = mock(status: 200, parsed: {plural_name.singularize => {id: 123}})
+    mock_response = mock(status: 200, parsed: { plural_name.singularize => { id: 123 } }, error: nil)
     mock_token.expects(:request)
               .with(:post, plural_name, {
-                body: record.as_json.to_json,
+                body:         record.as_json.to_json,
                 raise_errors: false,
-                headers: {
+                headers:      {
                   'Content-Type' => 'application/json',
                   'Idempotency-Key' => 'abcdefghijkl'
                 }
@@ -163,11 +162,16 @@ module SharedAdapterExamples
   end
 
 private
+
   def mock_api_request(record, request, response)
     mock_token = mock
-    mock_response = mock(status: response[0], parsed: response[1])
+    mock_response = mock(status: response[0], parsed: response[1], error: nil)
     mock_token.expects(:request)
-              .with(request[0], request[1], body: record.as_json.to_json, raise_errors: false, headers: {'Content-Type' => 'application/json'})
+              .with(request[0], request[1], {
+                body:         record.as_json.to_json,
+                raise_errors: false,
+                headers:      { 'Content-Type' => 'application/json' }
+              })
               .returns(mock_response)
     adapter.client.access_token = mock_token
   end
@@ -177,6 +181,6 @@ private
   end
 
   def random_attribute
-    @rattr ||= record_class.attribute_set.find { |att| att.type == Axiom::Types::String }.name
+    @random_attribute ||= record_class.attribute_set.find { |att| att.type == Axiom::Types::String }.name
   end
 end

data/test/support/shared_record_examples.rb

@@ -1,6 +1,8 @@
+# frozen_string_literal: true
+
 module SharedRecordExamples
   def test_sets_up_id
-    assert_instance_of(Fixnum, @record.id)
+    assert_instance_of(Integer, @record.id)
     assert_equal(@json['id'], @record.id)
   end
 

data/test/support/shared_sideloaded_data_parsing_examples.rb

@@ -1,7 +1,8 @@
+# frozen_string_literal: true
+
 module SharedSideloadedDataParsingExamples
   def test_adapter_sideloaded_data_parsing
     VCR.use_cassette(plural_name + '_sideloaded') do
-
       collection = adapter.where(limit: 5, _include: children.join(","))
 
       children.each do |child|

data/test/support/testing_adapter.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module TestingAdapter
   require 'dotenv'
   def setup

data/test/support/vcr_support.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module VCRHelper
   def load_vcr_hash(fixture, key)
     yaml = YAML.load_file(VCR.configuration.cassette_library_dir + "/#{fixture}.yml")

data/test/test_helper.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'gecko'
 require 'minitest/autorun'
 require 'minitest/pride'

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: gecko-ruby
 version: !ruby/object:Gem::Version
-  version: 0.9.0
+  version: 0.12.0
 platform: ruby
 authors:
 - Bradley Priest
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2019-07-24 00:00:00.000000000 Z
+date: 2020-07-09 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler
@@ -25,7 +25,7 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
-  name: rake
+  name: dotenv
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
@@ -53,7 +53,7 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
-  name: vcr
+  name: mocha
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
@@ -67,7 +67,7 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
-  name: webmock
+  name: pry
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
@@ -81,7 +81,7 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
-  name: mocha
+  name: rake
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
@@ -95,7 +95,7 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
-  name: timecop
+  name: thor
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
@@ -109,7 +109,7 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
-  name: dotenv
+  name: timecop
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
@@ -123,7 +123,7 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
-  name: thor
+  name: vcr
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
@@ -137,7 +137,7 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
-  name: pry
+  name: webmock
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
@@ -151,35 +151,35 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
-  name: oauth2
+  name: activesupport
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ">"
+    - - ">="
       - !ruby/object:Gem::Version
-        version: 1.1.0
+        version: '0'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ">"
+    - - ">="
       - !ruby/object:Gem::Version
-        version: 1.1.0
+        version: '0'
 - !ruby/object:Gem::Dependency
-  name: virtus
+  name: oauth2
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - ">"
       - !ruby/object:Gem::Version
-        version: '0'
+        version: 1.1.0
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - ">"
       - !ruby/object:Gem::Version
-        version: '0'
+        version: 1.1.0
 - !ruby/object:Gem::Dependency
-  name: activesupport
+  name: virtus
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
@@ -199,10 +199,11 @@ executables: []
 extensions: []
 extra_rdoc_files: []
 files:
+- ".github/config/rubocop_linter_action.yml"
+- ".github/workflows/testing.yml"
 - ".gitignore"
 - ".hound.yml"
 - ".rubocop.yml"
-- ".travis.yml"
 - CHANGELOG.md
 - CONTRIBUTING.md
 - Gemfile
@@ -365,7 +366,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.0.3
+rubygems_version: 3.1.2
 signing_key: 
 specification_version: 4
 summary: A Ruby interface to the TradeGecko API.