gecko-ruby 0.9.1 → 0.12.1

data/test/helpers/validation_helper_test.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'gecko'
 
 class Gecko::Helpers::ValidationHelperTest < Minitest::Test
@@ -17,7 +19,7 @@ class Gecko::Helpers::ValidationHelperTest < Minitest::Test
 
   def test_from_response
     record = @klass.new(@client, name: "Gecko")
-    record.errors.from_response({name: ["is not shiny"]})
+    record.errors.from_response({ name: ["is not shiny"] })
     assert(!record.valid?)
     assert_equal(record.errors[:name], ["is not shiny"])
   end

data/test/record/account_adapter_test.rb

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

data/test/record/address_adapter_test.rb

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

data/test/record/address_test.rb

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

data/test/record/company_adapter_test.rb

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

data/test/record/company_test.rb

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

data/test/record/contact_adapter_test.rb

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

data/test/record/contact_test.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'test_helper'
 
 class Gecko::ContactTest < Minitest::Test
@@ -15,4 +17,4 @@ class Gecko::ContactTest < Minitest::Test
   def test_initializes_record
     assert_instance_of(Gecko::Record::Contact, @record)
   end
-end
+end

data/test/record/currency_adapter_test.rb

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

data/test/record/currency_test.rb

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

data/test/record/fulfillment_adapter_test.rb

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

data/test/record/fulfillment_line_item_adapter_test.rb

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

data/test/record/fulfillment_line_item_test.rb

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

data/test/record/fulfillment_test.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'test_helper'
 
 class Gecko::FulfillmentTest < Minitest::Test
@@ -17,11 +19,11 @@ class Gecko::FulfillmentTest < Minitest::Test
   end
 
   def test_serialization
-    json_keys = %w(
+    json_keys = %w[
       order_id shipping_address_id billing_address_id status stock_location_id exchange_rate
       delivery_type tracking_number notes tracking_url tracking_company packed_at
       service shipped_at received_at receipt
-    ).map(&:to_sym)
+    ].map(&:to_sym)
     assert_equal json_keys.sort, @record.as_json[:fulfillment].keys.sort
   end
 end

data/test/record/image_adapter_test.rb

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

data/test/record/image_test.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'test_helper'
 
 class Gecko::ImageTest < Minitest::Test
@@ -22,4 +24,20 @@ class Gecko::ImageTest < Minitest::Test
     assert_equal("https://example.com/Gecko.jpg", @record.url(:full))
     assert_equal("https://example.com/thumbnail_Gecko.jpg", @record.url(:thumbnail))
   end
+
+  def test_building_record # rubocop:disable Metrics/MethodLength
+    record = @client.adapter_for(record_class.demodulized_name).build({
+      variant_ids: 1,
+      product_id:  1,
+      url:         "https://example.com/Gecko.jpg"
+    })
+    assert_equal(record.product_id, 1)
+    assert_equal(record.variant_ids, [1])
+    assert_equal(record.serializable_hash, {
+      product_id:  1,
+      variant_ids: [1],
+      name:        nil,
+      url:         "https://example.com/Gecko.jpg"
+    })
+  end
 end

data/test/record/invoice_adapter_test.rb

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

data/test/record/invoice_line_item_adapter_test.rb

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

data/test/record/invoice_line_item_test.rb

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

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_method_adapter_test.rb

@@ -0,0 +1,16 @@
+# frozen_string_literal: true
+
+require 'test_helper'
+
+class Gecko::Record::PaymentMethodAdapterTest < Minitest::Test
+  include TestingAdapter
+  include SharedAdapterExamples
+
+  let(:adapter)       { @client.PaymentMethod }
+  let(:plural_name)   { "payment_methods" }
+  let(:record_class)  { Gecko::Record::PaymentMethod }
+
+  def test_initializes_adapter
+    assert_instance_of(Gecko::Record::PaymentMethodAdapter, @client.PaymentMethod)
+  end
+end

data/test/record/payment_method_test.rb

@@ -0,0 +1,20 @@
+# frozen_string_literal: true
+
+require 'test_helper'
+
+class Gecko::PaymentMethodTest < Minitest::Test
+  include VCRHelper
+  include SharedRecordExamples
+
+  let(:plural_name)   { "payment_methods" }
+  let(:record_class)  { Gecko::Record::PaymentMethod }
+
+  def setup
+    @json   = load_vcr_hash("payment_methods", "payment_methods").first
+    @record = record_class.new(client, @json)
+  end
+
+  def test_initializes_record
+    assert_instance_of(Gecko::Record::PaymentMethod, @record)
+  end
+end

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