Introduce Client#call_adyen_api_url method

This method allows passing in a full URL for users that do not want to
use the whole concept of services and actions.

Author: Korri

lib/adyen/client.rb

@@ -144,8 +144,18 @@ def service_url(service, action, version)
     def call_adyen_api(service, action, request_data, headers, version, _with_application_info: false)
       # get URL for requested endpoint
       url = service_url(service, action.is_a?(String) ? action : action.fetch(:url), version)
-      
-      auth_type = auth_type(service, request_data)
+
+      # make sure valid authentication has been provided
+      validate_auth_type(service, request_data)
+
+      method = action.is_a?(String) ? 'post' : action.fetch(:method)
+
+      call_adyen_api_url(url, method, request_data, headers, version, _with_application_info: _with_application_info)
+    end
+
+    # send request to adyen API with a given full URL
+    def call_adyen_api_url(url, method, request_data, headers, version, _with_application_info: false)
+      auth_type = auth_type(request_data)
 
       # initialize Faraday connection object
       conn = Faraday.new(url, @connection_options) do |faraday|
@@ -173,49 +183,27 @@ def call_adyen_api(service, action, request_data, headers, version, _with_applic
       # convert to json
       request_data = request_data.to_json
 
-      if action.is_a?(::Hash)
-        if action.fetch(:method) == 'get'
-          begin
-            response = conn.get
-          rescue Faraday::ConnectionFailed => e
-            raise e, "Connection to #{url} failed"
-          end
-        end
-        if action.fetch(:method) == 'delete'
-          begin
-            response = conn.delete
-          rescue Faraday::ConnectionFailed => e
-            raise e, "Connection to #{url} failed"
-          end
-        end
-        if action.fetch(:method) == 'patch'
-          begin
-            response = conn.patch do |req|
-              req.body = request_data
-            end
-          rescue Faraday::ConnectionFailed => e
-            raise e, "Connection to #{url} failed"
-          end
-        end
-        if action.fetch(:method) == 'post'
-          # post request to Adyen
-          begin
-            response = conn.post do |req|
-              req.body = request_data
-            end
-          rescue Faraday::ConnectionFailed => e
-            raise e, "Connection to #{url} failed"
+      begin
+        response = case method
+        when 'get'
+          conn.get
+        when 'delete'
+          conn.delete
+        when 'patch'
+          conn.patch do |req|
+            req.body = request_data
           end
-        end
-      else
-        begin
-          response = conn.post do |req|
+        when 'post'
+          conn.post do |req|
             req.body = request_data
           end
-        rescue Faraday::ConnectionFailed => e
-          raise e, "Connection to #{url} failed"
+        else
+          raise ArgumentError, "Invalid HTTP method: #{method}"
         end
+      rescue Faraday::ConnectionFailed => e
+        raise e, "Connection to #{url} failed"
       end
+
       # check for API errors
       case response.status
         when 400
@@ -326,7 +314,7 @@ def balance_control
       @balance_control ||= Adyen::BalanceControl.new(self)
     end
 
-  
+
     private
 
     def auth_header(auth_type, faraday)
@@ -346,9 +334,7 @@ def auth_header(auth_type, faraday)
       end
     end
 
-    def auth_type(service, request_data)
-      # make sure valid authentication has been provided
-      validate_auth_type(service, request_data)
+    def auth_type(request_data)
       # Will prioritize authentication methods in this order:
       # api-key, oauth, basic
       return "api-key" unless @api_key.nil?

spec/client_spec.rb

@@ -543,4 +543,136 @@
       expect(request_body_sent).to eq(hash_data.to_json)
     end
   end
+
+  describe '#call_adyen_api_url' do
+    it 'successfully makes a POST request with full URL' do
+      client = Adyen::Client.new(api_key: 'test_key', env: :test)
+      mock_faraday_connection = double(Faraday::Connection)
+      response_body = { pspReference: '987654321', resultCode: 'Authorised' }.to_json
+      mock_response = Faraday::Response.new(
+        status: 200,
+        body: response_body,
+        response_headers: { 'content-type' => 'application/json' }
+      )
+
+      expect(Faraday).to receive(:new)
+        .with('https://checkout-test.adyen.com/v71/payments', anything)
+        .and_return(mock_faraday_connection)
+        .and_yield(mock_faraday_connection)
+      allow(mock_faraday_connection).to receive(:adapter)
+      allow(mock_faraday_connection).to receive_message_chain(:headers, :[]=)
+      allow(mock_faraday_connection).to receive(:post).and_return(mock_response)
+
+      result = client.call_adyen_api_url(
+        'https://checkout-test.adyen.com/v71/payments',
+        'post',
+        { amount: { value: 2000 } },
+        {},
+        '71'
+      )
+
+      expect(result).to be_a(Adyen::AdyenResult)
+      expect(result.status).to eq(200)
+      expect(result.response["pspReference"]).to eq('987654321')
+    end
+
+    it 'successfully makes a GET request' do
+      client = Adyen::Client.new(api_key: 'test_key', env: :test)
+      mock_faraday_connection = double(Faraday::Connection)
+      response_body = { data: [{ id: 'comp-1' }] }.to_json
+      mock_response = Faraday::Response.new(status: 200, body: response_body, response_headers: {})
+
+      expect(Faraday).to receive(:new)
+        .with('https://management-test.adyen.com/v1/companies/123', anything)
+        .and_return(mock_faraday_connection)
+        .and_yield(mock_faraday_connection)
+      allow(mock_faraday_connection).to receive(:adapter)
+      allow(mock_faraday_connection).to receive_message_chain(:headers, :[]=)
+      allow(mock_faraday_connection).to receive(:get).and_return(mock_response)
+
+      result = client.call_adyen_api_url(
+        'https://management-test.adyen.com/v1/companies/123',
+        'get',
+        {},
+        {},
+        '1'
+      )
+
+      expect(result).to be_a(Adyen::AdyenResult)
+      expect(result.status).to eq(200)
+    end
+
+    it 'sets correct headers' do
+      client = Adyen::Client.new(api_key: 'test_key', env: :test)
+      mock_faraday_connection = double(Faraday::Connection)
+      mock_response = Faraday::Response.new(status: 200, body: '{}', response_headers: {})
+
+      headers_spy = {}
+      expect(Faraday).to receive(:new)
+        .with('https://custom.adyen.com/api/endpoint', anything)
+        .and_return(mock_faraday_connection)
+        .and_yield(mock_faraday_connection)
+      allow(mock_faraday_connection).to receive(:adapter)
+      allow(mock_faraday_connection).to receive_message_chain(:headers, :[]=) do |key, value|
+        headers_spy[key] = value
+      end
+      allow(mock_faraday_connection).to receive(:post).and_return(mock_response)
+
+      custom_headers = { 'X-Custom-Header' => 'custom-value' }
+      client.call_adyen_api_url('https://custom.adyen.com/api/endpoint', 'post', {}, custom_headers, '1')
+
+      expect(headers_spy['Content-Type']).to eq('application/json')
+      expect(headers_spy['x-api-key']).to eq('test_key')
+      expect(headers_spy['X-Custom-Header']).to eq('custom-value')
+    end
+
+    it 'handles connection failures' do
+      client = Adyen::Client.new(api_key: 'test_key', env: :test)
+      mock_faraday_connection = double(Faraday::Connection)
+
+      expect(Faraday).to receive(:new)
+        .with('https://checkout-test.adyen.com/v71/payments', anything)
+        .and_return(mock_faraday_connection)
+        .and_yield(mock_faraday_connection)
+      allow(mock_faraday_connection).to receive(:adapter)
+      allow(mock_faraday_connection).to receive_message_chain(:headers, :[]=)
+      allow(mock_faraday_connection).to receive(:post).and_raise(Faraday::ConnectionFailed.new('Connection failed'))
+
+      expect {
+        client.call_adyen_api_url('https://checkout-test.adyen.com/v71/payments', 'post', {}, {}, '71')
+      }.to raise_error(Faraday::ConnectionFailed, /Connection to .* failed/)
+    end
+
+    it 'supports different HTTP methods' do
+      client = Adyen::Client.new(api_key: 'test_key', env: :test)
+      mock_faraday_connection = double(Faraday::Connection)
+      mock_response = Faraday::Response.new(status: 200, body: '{"status":"updated"}', response_headers: {})
+
+      expect(Faraday).to receive(:new)
+        .with('https://management-test.adyen.com/v1/companies/123', anything)
+        .and_return(mock_faraday_connection)
+        .and_yield(mock_faraday_connection)
+      allow(mock_faraday_connection).to receive(:adapter)
+      allow(mock_faraday_connection).to receive_message_chain(:headers, :[]=)
+      allow(mock_faraday_connection).to receive(:patch).and_return(mock_response)
+
+      result = client.call_adyen_api_url(
+        'https://management-test.adyen.com/v1/companies/123',
+        'patch',
+        { status: 'active' },
+        {},
+        '1'
+      )
+
+      expect(result).to be_a(Adyen::AdyenResult)
+      expect(result.status).to eq(200)
+    end
+
+    it 'raises for invalid HTTP method' do
+      client = Adyen::Client.new(api_key: 'test_key', env: :test)
+      expect {
+        client.call_adyen_api_url('https://management-test.adyen.com/v1/companies/123', 'invalid', {}, {}, '1')
+      }.to raise_error(ArgumentError, "Invalid HTTP method: invalid")
+    end
+  end
 end