adyen-ruby-api-library 3.0.2 → 4.0.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
-SHA256:
-  metadata.gz: b38d6b5e903949dd5f611270b52d7d232b7446053e4af782809ec0ee80c2072c
-  data.tar.gz: ddb1fa60f2b6dd3d1b177c651ae59372d8065a13193f7b2ef1cbe5c3744b0312
+SHA1:
+  metadata.gz: 57203cec793176b8f389062f66cf734dcc43e183
+  data.tar.gz: 6680ea0897e40cad89569ed46c841dcf45b900a4
 SHA512:
-  metadata.gz: be7130f49c9247fd8f8edb0c0ff8f11d6ed95ac19e1ffc3b15db0d9e758c092ed1850d22c64b1ec17c9b4f63fa22a0149fd3ba9dca4d5b6b7a56baca5c6b98f4
-  data.tar.gz: 9bb005f7548edea11ba546ee08ed045a7ecd743c1691a25aaa9c38b5940a29fbd9d8454cc894553920f20f0f4cf2174dc7e3c4fbe53c2bdf2bc606d1c5a720af
+  metadata.gz: 16075771ada5ca4afb71c798e7823fcd1c2b162a6d0d21919b0a3ea8db301ec5379525c726ebc3d71b0b39d87ada719b539b7457fe927a5aa6ef4754b7dca3be
+  data.tar.gz: 968d9f0cb9a426ce89d8bc17a00cdf50766f1f48dacbb58a1966e3d29ae5fe99a3e784e6d0f40fa86f4f0ef5bfad27576124ad1504746e121af9998abd06f8a1

data/CONTRIBUTING.md

@@ -0,0 +1,20 @@
+# Contribution guidelines
+
+## How to contribute step-by-step
+
+1. Fork the `Adyen/adyen-ruby-api-library` repository.
+2. Create a new branch from `develop` in your fork. This makes it easier for you to keep track of your changes.
+3. Make the desired changes to the code.
+    * If you are adding new functionality or fixing a bug, we recommend you add unit tests that cover it.
+4. Push the changes to your fork.
+5. Create a pull request to the `Adyen/adyen-ruby-api-library` repository.
+6. In your pull request, please describe in detail:
+    * What problem you’re solving
+    * Your approach to fixing the problem
+    * Any tests you wrote
+7. Check Allow edits from maintainers.
+8. Create the pull request.
+9. Ensure that all checks have passed.
+
+After you create your pull request, one of the code owners will review your code.
+We aim to review your request within 2-3 business days.

data/README.md

@@ -66,7 +66,7 @@ response = adyen.checkout.payments({
 
 ### Change API Version
 ```ruby
-adyen.checkout.version = 49
+adyen.checkout.version = 50
 ```
 
 ## List of supported methods
@@ -139,7 +139,15 @@ adyen.checkout.version = 49
 ## Support
 
 If you have any problems, questions or suggestions, create an issue here or send your inquiry to support@adyen.com.
-  
+
+## Contributing
+We strongly encourage you to join us in contributing to this repository so everyone can benefit from:
+* New features and functionality
+* Resolved bug fixes and issues
+* Any general improvements
+
+Read our [**contribution guidelines**](CONTRIBUTING.md) to find out how.
+
 ## Licence
 
 MIT license. For more information, see the LICENSE file.

data/lib/adyen/client.rb

@@ -1,19 +1,21 @@
 require "faraday"
 require "json"
 require_relative "./errors"
+require_relative "./result"
+require_relative "./util"
 
 module Adyen
   class Client
     attr_accessor :ws_user, :ws_password, :api_key, :client, :adapter, :live_url_prefix
     attr_reader :env
 
-    def initialize(ws_user: nil, ws_password: nil, api_key: nil, env: :live, adapter: nil, mock_port: 3001, live_url_prefix: nil)
+    def initialize(ws_user: nil, ws_password: nil, api_key: nil, env: :live, adapter: nil, mock_port: 3001, live_url_prefix: nil, mock_service_url_base: nil)
       @ws_user = ws_user
       @ws_password = ws_password
       @api_key = api_key
       @env = env
       @adapter = adapter || Faraday.default_adapter
-      @mock_port = mock_port
+      @mock_service_url_base = mock_service_url_base || "http://localhost:#{mock_port}"
       @live_url_prefix = live_url_prefix
     end
 
@@ -35,7 +37,7 @@ module Adyen
     def service_url_base(service)
       raise ArgumentError, "Please set Client.live_url_prefix to the portion of your merchant-specific URL prior to '-[service]-live.adyenpayments.com'" if @live_url_prefix.nil? and @env == :live
       if @env == :mock
-        "http://localhost:#{@mock_port}"
+        @mock_service_url_base
       else
         case service
         when "Checkout"
@@ -111,8 +113,6 @@ module Adyen
           faraday.headers[key] = value
         end
       end
-
-
       # if json string convert to hash
       # needed to add applicationInfo
       if request_data.is_a?(String)
@@ -144,7 +144,9 @@ module Adyen
         raise Adyen::PermissionError.new("Missing user permissions; https://docs.adyen.com/user-management/user-roles", request_data)
       end
 
-      response
+      formatted_response = AdyenResult.new(response.body, response.headers, response.status)
+
+      formatted_response
     end
 
     # add application_info for analytics

data/lib/adyen/result.rb

@@ -0,0 +1,16 @@
+require 'json'
+
+module Adyen
+  class AdyenResult
+    attr_reader :response, :header, :status
+
+    def initialize(response, header, status)
+      @response = JSON.parse(response)
+
+      # `header` in Faraday response is not a JSON string, but rather a
+      # Faraday `Headers` object. Convert first before parsing
+      @header = JSON.parse(header.to_json)
+      @status = status
+    end
+  end
+end

data/lib/adyen/services/checkout.rb

@@ -2,7 +2,7 @@ require_relative 'service'
 
 module Adyen
   class Checkout < Service
-    DEFAULT_VERSION = 49
+    DEFAULT_VERSION = 50
 
     def initialize(client, version = DEFAULT_VERSION)
       service = 'Checkout'

data/lib/adyen/services/payments.rb

@@ -3,7 +3,7 @@ require_relative 'service'
 module Adyen
   class Payments < Service
     attr_accessor :version
-    DEFAULT_VERSION = 49
+    DEFAULT_VERSION = 50
 
     def initialize(client, version = DEFAULT_VERSION)
       service = 'Payment'

data/lib/adyen/util.rb

@@ -0,0 +1,21 @@
+# This utility method monkey-patches Ruby's Hash class to allow keys to be read
+# and updated with dot notation. Usage is entirely optional (i.e.,  hash values
+# can still be accessed via symbol and string keys).
+#
+# Credit: https://gist.github.com/winfred/2185384#file-ruby-dot-hash-access-rb
+
+class Hash
+  class NoKeyOrMethodError < NoMethodError; end
+  def method_missing(method,*args)
+    m = method.to_s
+    string_key = m.gsub(/=$/,'')
+    sym_key = string_key.to_sym
+    if self.has_key? string_key
+      m.match(/=$/) ? self.send("[#{string_key}]=", *args) : self[string_key]
+    elsif self.has_key? sym_key
+      m.match(/=$/) ? self.send("[#{sym_key}]=", *args) : self[sym_key]
+    else
+      raise NoKeyOrMethodError
+    end
+  end
+end

data/lib/adyen/version.rb

@@ -1,4 +1,4 @@
-module Adyen
+  module Adyen
   NAME = "adyen-ruby-api-library"
-  VERSION = "3.0.2".freeze
+  VERSION = "4.0.0".freeze
 end

data/spec/checkout_spec.rb

@@ -42,7 +42,10 @@ RSpec.describe Adyen::Checkout, service: "checkout" do
       to_return(
         body: response_body
       )
-    response = @shared_values[:client].checkout.payments.details(request_body)
+
+    result = @shared_values[:client].checkout.payments.details(request_body)
+    # result.response is already a Ruby hash (rather than an unparsed JSON string)
+    response_hash = result.response
 
     expect(request_body[:applicationInfo][:adyenLibrary][:name]).
       to eq(Adyen::NAME)
@@ -50,13 +53,13 @@ RSpec.describe Adyen::Checkout, service: "checkout" do
       to eq(Adyen::VERSION)
     expect(request_body[:applicationInfo][:adyenPaymentSource][:name]).
       to eq("adyen-test")
-    expect(response.status).
+    expect(result.status).
       to eq(200)
-    expect(response.body).
-      to eq(response_body)
-    expect((parsed_body = JSON.parse(response.body)).class).
+    expect(response_hash).
+      to eq(JSON.parse(response_body))
+    expect(response_hash.class).
       to be Hash
-    expect(parsed_body["resultCode"]).
+    expect(response_hash["resultCode"]).
       to eq("RedirectShopper")
   end
 
@@ -78,15 +81,17 @@ RSpec.describe Adyen::Checkout, service: "checkout" do
       to_return(
         body: response_body
       )
-    response = @shared_values[:client].checkout.payments.result(request_body)
 
-    expect(response.status).
+    result = @shared_values[:client].checkout.payments.result(request_body)
+    response_hash = result.response
+
+    expect(result.status).
       to eq(200)
-    expect(response.body).
-      to eq(response_body)
-    expect((parsed_body = JSON.parse(response.body)).class).
+    expect(response_hash).
+      to eq(JSON.parse(response_body))
+    expect(response_hash.class).
       to be Hash
-    expect(parsed_body["resultCode"]).
+    expect(response_hash["resultCode"]).
       to eq("Authorised")
   end
 

data/spec/client_spec.rb

@@ -38,6 +38,24 @@ RSpec.describe Adyen do
     @shared_values[:client].api_key = "api_key"
   end
 
+  it "uses the specified mock service URL" do
+    client = Adyen::Client.new(env: :mock, mock_service_url_base: "https://mock.test")
+    expect(client.service_url_base("Account")).
+      to eq("https://mock.test")
+  end
+
+  it "generates localhost service URL when a mock port is specified" do
+    client = Adyen::Client.new(env: :mock, mock_port: 3005)
+    expect(client.service_url_base("Account")).
+      to eq("http://localhost:3005")
+  end
+
+  it "prefers the mock service URL when both mock service URL and port are specified" do
+    client = Adyen::Client.new(env: :mock, mock_port: 3005, mock_service_url_base: "https://this-url-wins.test")
+    expect(client.service_url_base("Account")).
+      to eq("https://this-url-wins.test")
+  end
+
   it "generates the correct service URL base for CAL TEST" do
     client = Adyen::Client.new(env: :test)
     client.live_url_prefix = "abcdef1234567890-TestCompany"

data/spec/spec_helper.rb

@@ -48,18 +48,20 @@ def create_test(client, service, method_name, parent_object)
     to_return(
       body: response_body
     )
-  response = parent_object.public_send(method_name, request_body)
+  result = parent_object.public_send(method_name, request_body)
+
+  # result.response is already a Ruby hash (rather than an unparsed JSON string)
+  response_hash = result.response
 
   # boilerplate error checks
-  expect(response.status).
+  expect(result.status).
     to eq(200)
-  expect(response.body).
-    to eq(response_body)
-  expect((parsed_body = JSON.parse(response.body)).class).
+  expect(response_hash).
+    to eq(JSON.parse(response_body))
+  expect(response_hash.class).
     to be Hash
 
-  # return hash of response
-  parsed_body
+  response_hash
 end
 
 # creates tests from an array of arrays

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: adyen-ruby-api-library
 version: !ruby/object:Gem::Version
-  version: 3.0.2
+  version: 4.0.0
 platform: ruby
 authors:
 - Adyen
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2019-11-07 00:00:00.000000000 Z
+date: 2019-11-12 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: faraday
@@ -78,6 +78,7 @@ extra_rdoc_files: []
 files:
 - ".gitignore"
 - ".travis.yml"
+- CONTRIBUTING.md
 - Gemfile
 - LICENSE
 - README.md
@@ -102,6 +103,7 @@ files:
 - lib/adyen-ruby-api-library.rb
 - lib/adyen/client.rb
 - lib/adyen/errors.rb
+- lib/adyen/result.rb
 - lib/adyen/services/checkout.rb
 - lib/adyen/services/checkout_utility.rb
 - lib/adyen/services/marketpay.rb
@@ -109,6 +111,7 @@ files:
 - lib/adyen/services/payouts.rb
 - lib/adyen/services/recurring.rb
 - lib/adyen/services/service.rb
+- lib/adyen/util.rb
 - lib/adyen/version.rb
 - spec/account_spec.rb
 - spec/checkout_spec.rb
@@ -242,7 +245,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.7.8
+rubygems_version: 2.2.0
 signing_key: 
 specification_version: 4
 summary: Official Adyen  Ruby API Library