buckaruby 1.0.2 → 1.1.0

data/lib/buckaruby/signature.rb

@@ -1,22 +1,21 @@
+# frozen_string_literal: true
+
 require 'digest'
 
 module Buckaruby
   # Calculate a signature based on the parameters of the payment request or response.
   # -> see BPE 3.0 Gateway NVP, chapter 4 'Digital Signature'
   class Signature
-    def self.generate_signature(params, options)
-      secret = options[:secret]
-      hash_method = options[:hash_method]
-
-      case hash_method
+    def self.generate_signature(params, config)
+      case config.hash_method
       when :sha1
-        Digest::SHA1.hexdigest(generate_signature_string(params, secret))
+        Digest::SHA1.hexdigest(generate_signature_string(params, config.secret))
       when :sha256
-        Digest::SHA256.hexdigest(generate_signature_string(params, secret))
+        Digest::SHA256.hexdigest(generate_signature_string(params, config.secret))
       when :sha512
-        Digest::SHA512.hexdigest(generate_signature_string(params, secret))
+        Digest::SHA512.hexdigest(generate_signature_string(params, config.secret))
       else
-        raise ArgumentError, "Invalid hash method provided: #{hash_method}"
+        raise ArgumentError, "Invalid hash method provided: #{config.hash_method}"
       end
     end
 

data/lib/buckaruby/support/case_insensitive_hash.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Buckaruby
   module Support
     # The case insensitive Hash is a Hash with case insensitive keys that

data/lib/buckaruby/transaction_status.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Buckaruby
   module TransactionStatus
     SUCCESS = 1

data/lib/buckaruby/transaction_type.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Buckaruby
   module TransactionType
     PAYMENT = 1

data/lib/buckaruby/version.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Buckaruby
-  VERSION = "1.0.2"
+  VERSION = "1.1.0"
 end

data/spec/buckaruby/configuration_spec.rb

@@ -0,0 +1,113 @@
+# frozen_string_literal: true
+
+require 'spec_helper'
+
+describe Buckaruby::Configuration do
+  describe '#website' do
+    it 'sets the website from options' do
+      config = Buckaruby::Configuration.new(website: "12345678", secret: "7C222FB2927D828AF22F592134E8932480637C0D")
+      expect(config.website).to eq("12345678")
+    end
+
+    it 'raises an exception when website is missing' do
+      expect {
+        Buckaruby::Configuration.new(secret: "7C222FB2927D828AF22F592134E8932480637C0D")
+      }.to raise_error(ArgumentError)
+    end
+  end
+
+  describe '#secret' do
+    it 'sets the secret from options' do
+      config = Buckaruby::Configuration.new(website: "12345678", secret: "7C222FB2927D828AF22F592134E8932480637C0D")
+      expect(config.secret).to eq("7C222FB2927D828AF22F592134E8932480637C0D")
+    end
+
+    it 'raises an exception when secret is missing' do
+      expect {
+        Buckaruby::Configuration.new(website: "12345678")
+      }.to raise_error(ArgumentError)
+    end
+  end
+
+  describe '#mode' do
+    it 'returns test when no mode is passed' do
+      config = Buckaruby::Configuration.new(website: "12345678", secret: "7C222FB2927D828AF22F592134E8932480637C0D")
+      expect(config.mode).to eq(:test)
+    end
+
+    it 'returns test when mode test is passed' do
+      config = Buckaruby::Configuration.new(website: "12345678", secret: "7C222FB2927D828AF22F592134E8932480637C0D", mode: :test)
+      expect(config.mode).to eq(:test)
+    end
+
+    it 'returns production when mode production is passed' do
+      config = Buckaruby::Configuration.new(website: "12345678", secret: "7C222FB2927D828AF22F592134E8932480637C0D", mode: :production)
+      expect(config.mode).to eq(:production)
+    end
+
+    it 'raises an exception when an invalid mode is passed' do
+      expect {
+        Buckaruby::Configuration.new(website: "12345678", secret: "7C222FB2927D828AF22F592134E8932480637C0D", mode: :invalid)
+      }.to raise_error(ArgumentError)
+    end
+  end
+
+  describe '#hash_method' do
+    it 'raises an exception when an invalid hash method is passed' do
+      expect {
+        Buckaruby::Configuration.new(website: "12345678", secret: "7C222FB2927D828AF22F592134E8932480637C0D", hash_method: :invalid)
+      }.to raise_error(ArgumentError)
+    end
+
+    it 'accepts SHA-1 as hash method' do
+      config = Buckaruby::Configuration.new(website: "12345678", secret: "7C222FB2927D828AF22F592134E8932480637C0D", hash_method: 'SHA1')
+      expect(config.hash_method).to eq(:sha1)
+    end
+
+    it 'accepts SHA-256 as hash method' do
+      config = Buckaruby::Configuration.new(website: "12345678", secret: "7C222FB2927D828AF22F592134E8932480637C0D", hash_method: :SHA256)
+      expect(config.hash_method).to eq(:sha256)
+    end
+
+    it 'accepts SHA-512 as hash method' do
+      config = Buckaruby::Configuration.new(website: "12345678", secret: "7C222FB2927D828AF22F592134E8932480637C0D", hash_method: :sha512)
+      expect(config.hash_method).to eq(:sha512)
+    end
+  end
+
+  describe '#test?' do
+    it 'returns true when mode is test' do
+      config = Buckaruby::Configuration.new(website: "12345678", secret: "7C222FB2927D828AF22F592134E8932480637C0D", mode: :test)
+      expect(config.test?).to be true
+    end
+
+    it 'returns false when mode is production' do
+      config = Buckaruby::Configuration.new(website: "12345678", secret: "7C222FB2927D828AF22F592134E8932480637C0D", mode: :production)
+      expect(config.test?).to be false
+    end
+  end
+
+  describe '#production?' do
+    it 'returns false when mode is test' do
+      config = Buckaruby::Configuration.new(website: "12345678", secret: "7C222FB2927D828AF22F592134E8932480637C0D", mode: :test)
+      expect(config.production?).to be false
+    end
+
+    it 'returns true when mode is production' do
+      config = Buckaruby::Configuration.new(website: "12345678", secret: "7C222FB2927D828AF22F592134E8932480637C0D", mode: :production)
+      expect(config.production?).to be true
+    end
+  end
+
+  describe '#api_url' do
+    it 'returns the test URL when mode is test' do
+      config = Buckaruby::Configuration.new(website: "12345678", secret: "7C222FB2927D828AF22F592134E8932480637C0D", mode: :test)
+      expect(config.api_url).to eq("https://testcheckout.buckaroo.nl/nvp/")
+    end
+
+    it 'returns the production URL when mode is production' do
+      config = Buckaruby::Configuration.new(website: "12345678", secret: "7C222FB2927D828AF22F592134E8932480637C0D", mode: :production)
+      expect(config.api_url).to eq("https://checkout.buckaroo.nl/nvp/")
+    end
+  end
+end

data/spec/buckaruby/gateway_spec.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'spec_helper'
 
 describe Buckaruby::Gateway do
@@ -21,59 +23,13 @@ describe Buckaruby::Gateway do
         }.to raise_error(ArgumentError)
       end
     end
-
-    describe 'set mode' do
-      it 'should be test when no mode is passed' do
-        gateway = Buckaruby::Gateway.new(website: "12345678", secret: "7C222FB2927D828AF22F592134E8932480637C0D")
-        expect(gateway.options[:mode]).to eq(:test)
-      end
-
-      it 'should be test when mode test is passed' do
-        gateway = Buckaruby::Gateway.new(website: "12345678", secret: "7C222FB2927D828AF22F592134E8932480637C0D", mode: :test)
-        expect(gateway.options[:mode]).to eq(:test)
-      end
-
-      it 'should be production when mode production is passed' do
-        gateway = Buckaruby::Gateway.new(website: "12345678", secret: "7C222FB2927D828AF22F592134E8932480637C0D", mode: :production)
-        expect(gateway.options[:mode]).to eq(:production)
-      end
-
-      it 'should raise an exception when an invalid mode is passed' do
-        expect {
-          Buckaruby::Gateway.new(website: "12345678", secret: "7C222FB2927D828AF22F592134E8932480637C0D", mode: :invalid)
-        }.to raise_error(ArgumentError)
-      end
-    end
-
-    describe 'set hash method' do
-      it 'should raise an exception when an invalid hash method is passed' do
-        expect {
-          Buckaruby::Gateway.new(website: "12345678", secret: "7C222FB2927D828AF22F592134E8932480637C0D", hash_method: :invalid)
-        }.to raise_error(ArgumentError)
-      end
-
-      it 'should accept SHA-1 as hash method' do
-        gateway = Buckaruby::Gateway.new(website: "12345678", secret: "7C222FB2927D828AF22F592134E8932480637C0D", hash_method: 'SHA1')
-        expect(gateway.options[:hash_method]).to eq(:sha1)
-      end
-
-      it 'should accept SHA-256 as hash method' do
-        gateway = Buckaruby::Gateway.new(website: "12345678", secret: "7C222FB2927D828AF22F592134E8932480637C0D", hash_method: :SHA256)
-        expect(gateway.options[:hash_method]).to eq(:sha256)
-      end
-
-      it 'should accept SHA-512 as hash method' do
-        gateway = Buckaruby::Gateway.new(website: "12345678", secret: "7C222FB2927D828AF22F592134E8932480637C0D", hash_method: :sha512)
-        expect(gateway.options[:hash_method]).to eq(:sha512)
-      end
-    end
   end
 
   subject { Buckaruby::Gateway.new(website: "12345678", secret: "7C222FB2927D828AF22F592134E8932480637C0D") }
 
   it { expect(subject).to be_an_instance_of(Buckaruby::Gateway) }
 
-  describe 'get issuers' do
+  describe '#issuers' do
     context 'when no or false parameters are passed' do
       it 'should raise an ArgumentError' do
         expect { subject.issuers }.to raise_error(ArgumentError)
@@ -87,20 +43,27 @@ describe Buckaruby::Gateway do
       it { expect(issuers).to include("INGBNL2A" => "ING") }
     end
 
-    context 'when visa, mastercard, sepa direct debit, paypal or is passed' do
+    context 'when ideal processing is passed' do
+      let(:issuers) { subject.issuers(Buckaruby::PaymentMethod::IDEAL_PROCESSING) }
+      it { expect(issuers.length).to be > 0 }
+      it { expect(issuers).to include("RABONL2U" => "Rabobank") }
+    end
+
+    context 'when visa, mastercard, maestro, bankcontact, sepa direct debit or paypal is passed' do
       it 'should raise an ArgumentError' do
         expect { subject.issuers(Buckaruby::PaymentMethod::VISA) }.to raise_error(ArgumentError)
         expect { subject.issuers(Buckaruby::PaymentMethod::MASTER_CARD) }.to raise_error(ArgumentError)
+        expect { subject.issuers(Buckaruby::PaymentMethod::MAESTRO) }.to raise_error(ArgumentError)
         expect { subject.issuers(Buckaruby::PaymentMethod::SEPA_DIRECT_DEBIT) }.to raise_error(ArgumentError)
+        expect { subject.issuers(Buckaruby::PaymentMethod::BANCONTACT_MISTER_CASH) }.to raise_error(ArgumentError)
         expect { subject.issuers(Buckaruby::PaymentMethod::PAYPAL) }.to raise_error(ArgumentError)
       end
     end
   end
 
-  describe 'initiate transaction' do
+  describe '#setup_transaction' do
     before(:each) do
-      stub_request(:post, "https://testcheckout.buckaroo.nl/nvp/?op=TransactionRequest")
-        .to_return(body: "BRQ_ACTIONREQUIRED=redirect&BRQ_AMOUNT=10.00&BRQ_APIRESULT=ActionRequired&BRQ_CURRENCY=EUR&BRQ_DESCRIPTION=Test&BRQ_INVOICENUMBER=12345&BRQ_PAYMENT=12345&BRQ_PAYMENT_METHOD=ideal&BRQ_REDIRECTURL=https%3A%2F%2Ftestcheckout.buckaroo.nl%2Fhtml%2Fredirect.ashx%3Fr%3D41C48B55FA9164E123CC73B1157459E840BE5D24&BRQ_SERVICE_IDEAL_CONSUMERISSUER=Rabobank&BRQ_STATUSCODE=791&BRQ_STATUSCODE_DETAIL=S002&BRQ_STATUSMESSAGE=An+additional+action+is+required%3A+RedirectToIdeal&BRQ_TEST=true&BRQ_TIMESTAMP=2014-11-05+13%3A10%3A40&BRQ_TRANSACTIONS=41C48B55FA9164E123CC73B1157459E840BE5D24&BRQ_WEBSITEKEY=12345678&BRQ_SIGNATURE=3d6ef7e249d9509d120c7b84f27f081adf06074b")
+      stub_request(:post, "https://testcheckout.buckaroo.nl/nvp/?op=TransactionRequest").to_return(body: File.read("spec/fixtures/responses/setup_transaction_success.txt"))
     end
 
     it 'should raise an exception when initiating a transaction with missing parameters' do
@@ -144,8 +107,7 @@ describe Buckaruby::Gateway do
     end
 
     it 'should raise a ConnectionException when connection the Buckaroo fails' do
-      stub_request(:post, "https://testcheckout.buckaroo.nl/nvp/?op=TransactionRequest")
-        .to_raise(Errno::ECONNREFUSED)
+      stub_request(:post, "https://testcheckout.buckaroo.nl/nvp/?op=TransactionRequest").to_raise(Errno::ECONNREFUSED)
 
       expect {
         subject.setup_transaction(amount: 10, payment_method: Buckaruby::PaymentMethod::IDEAL, payment_issuer: Buckaruby::Ideal::ISSUERS.keys.first, invoicenumber: "12345", return_url: "http://www.return.url/")
@@ -153,8 +115,7 @@ describe Buckaruby::Gateway do
     end
 
     it 'should raise an InvalidResponseException when Buckaroo returns an invalid response' do
-      stub_request(:post, "https://testcheckout.buckaroo.nl/nvp/?op=TransactionRequest")
-        .to_return(status: 500)
+      stub_request(:post, "https://testcheckout.buckaroo.nl/nvp/?op=TransactionRequest").to_return(status: 500)
 
       expect {
         subject.setup_transaction(amount: 10, payment_method: Buckaruby::PaymentMethod::IDEAL, payment_issuer: Buckaruby::Ideal::ISSUERS.keys.first, invoicenumber: "12345", return_url: "http://www.return.url/")
@@ -162,8 +123,7 @@ describe Buckaruby::Gateway do
     end
 
     it 'should raise an ApiException when API result Fail is returned' do
-      stub_request(:post, "https://testcheckout.buckaroo.nl/nvp/?op=TransactionRequest")
-        .to_return(body: "BRQ_APIRESULT=Fail&BRQ_APIERRORMESSAGE=Invalid+request")
+      stub_request(:post, "https://testcheckout.buckaroo.nl/nvp/?op=TransactionRequest").to_return(body: "BRQ_APIRESULT=Fail&BRQ_APIERRORMESSAGE=Invalid+request")
 
       expect {
         subject.setup_transaction(amount: 10, payment_method: Buckaruby::PaymentMethod::IDEAL, payment_issuer: Buckaruby::Ideal::ISSUERS.keys.first, invoicenumber: "12345", return_url: "http://www.return.url/")
@@ -243,11 +203,9 @@ describe Buckaruby::Gateway do
     end
   end
 
-  describe 'initiate recurrent transaction' do
+  describe '#recurrent_transaction' do
     before(:each) do
-      stub_request(:post, "https://testcheckout.buckaroo.nl/nvp/?op=TransactionRequest")
-        .to_return(body:
-"BRQ_AMOUNT=10.00&BRQ_APIRESULT=Success&BRQ_CURRENCY=EUR&BRQ_CUSTOMER_NAME=Test&BRQ_DESCRIPTION=Test&BRQ_INVOICENUMBER=12345&BRQ_ISSUING_COUNTRY=IE&BRQ_PAYMENT=8EE820309AA9455C91350DD5D3160362&BRQ_PAYMENT_METHOD=visa&BRQ_RECURRING=True&BRQ_SERVICE_VISA_CARDEXPIRATIONDATE=2016-01&BRQ_SERVICE_VISA_CARDNUMBERENDING=0969&BRQ_SERVICE_VISA_MASKEDCARDNUMBER=491611%2A%2A%2A%2A%2A%2A0969&BRQ_STATUSCODE=190&BRQ_STATUSCODE_DETAIL=S001&BRQ_STATUSMESSAGE=Transaction+successfully+processed&BRQ_TEST=true&BRQ_TIMESTAMP=2016-01-19+15%3A13%3A44&BRQ_TRANSACTIONS=41C48B55FA9164E123CC73B1157459E840BE5D24&BRQ_WEBSITEKEY=12345678&BRQ_SIGNATURE=b6a5a54c7e0d731f211c2080519e7aabc9775f47")
+      stub_request(:post, "https://testcheckout.buckaroo.nl/nvp/?op=TransactionRequest").to_return(body: File.read("spec/fixtures/responses/recurrent_transaction_success.txt"))
     end
 
     it 'should raise an exception when initiating a recurrent transaction with missing parameters' do
@@ -282,14 +240,66 @@ describe Buckaruby::Gateway do
     end
   end
 
-  describe 'get transaction status' do
+  describe '#refundable?' do
+    it 'should raise an exception when required parameters are missing' do
+      expect {
+        subject.refundable?
+      }.to raise_error(ArgumentError)
+    end
+
+    it 'should return true when the transaction is refundable' do
+      stub_request(:post, "https://testcheckout.buckaroo.nl/nvp/?op=RefundInfo").to_return(body: File.read("spec/fixtures/responses/refund_info_success.txt"))
+
+      response = subject.refundable?(transaction_id: "41C48B55FA9164E123CC73B1157459E840BE5D24")
+      expect(response).to be true
+    end
+
+    it 'should return false when the transaction was not found' do
+      stub_request(:post, "https://testcheckout.buckaroo.nl/nvp/?op=RefundInfo").to_return(body: File.read("spec/fixtures/responses/refund_info_error.txt"))
+
+      response = subject.refundable?(transaction_id: "41C48B55FA9164E123CC73B1157459E840BE5D24")
+      expect(response).to be false
+    end
+  end
+
+  describe '#refund_transaction' do
+    it 'should raise an exception when required parameters are missing' do
+      expect {
+        subject.refund_transaction
+      }.to raise_error(ArgumentError)
+    end
+
+    it 'should raise an exception when the transaction is not refundable' do
+      stub_request(:post, "https://testcheckout.buckaroo.nl/nvp/?op=RefundInfo").to_return(body: File.read("spec/fixtures/responses/refund_info_error.txt"))
+
+      expect {
+        subject.refund_transaction(transaction_id: "41C48B55FA9164E123CC73B1157459E840BE5D24")
+      }.to raise_error(Buckaruby::NonRefundableTransactionException)
+    end
+
+    it 'should refund the transaction' do
+      stub_request(:post, "https://testcheckout.buckaroo.nl/nvp/?op=RefundInfo").to_return(body: File.read("spec/fixtures/responses/refund_info_success.txt"))
+      stub_request(:post, "https://testcheckout.buckaroo.nl/nvp/?op=TransactionRequest").to_return(body: File.read("spec/fixtures/responses/refund_transaction_success.txt"))
+
+      response = subject.refund_transaction(transaction_id: "41C48B55FA9164E123CC73B1157459E840BE5D24")
+      expect(response.transaction_status).to eq(Buckaruby::TransactionStatus::SUCCESS)
+      expect(response.transaction_type).to eq(Buckaruby::TransactionType::PAYMENT)
+      expect(response.payment_method).to eq(Buckaruby::PaymentMethod::IDEAL)
+      expect(response.transaction_id).to eq("8CCE4BB06339F28A506E1A328025D7DF13CCAD59")
+      expect(response.payment_id).to eq("E86256B2787EE7FF0C33D0D4C6159CD922227B79")
+      expect(response.refund_transaction_id).to eq("41C48B55FA9164E123CC73B1157459E840BE5D24")
+      expect(response.invoicenumber).to eq("12345")
+      expect(response.timestamp).to be_an_instance_of(Time)
+      expect(response.to_h).to be_an_instance_of(Hash)
+    end
+  end
+
+  describe '#status' do
     before(:each) do
-      stub_request(:post, "https://testcheckout.buckaroo.nl/nvp/?op=TransactionStatus")
-        .to_return(body:
-"BRQ_AMOUNT=10.00&BRQ_APIRESULT=Success&BRQ_CURRENCY=EUR&BRQ_CUSTOMER_NAME=J.+de+Tester&BRQ_DESCRIPTION=Test&BRQ_INVOICENUMBER=12345&BRQ_MUTATIONTYPE=Collecting&BRQ_PAYER_HASH=e02377112efcd30bb7420bb1b9855a3778864572&BRQ_PAYMENT=E86256B2787EE7FF0C33D0D4C6159CD922227B79&BRQ_SERVICE_IDEAL_CONSUMERBIC=RABONL2U&BRQ_SERVICE_IDEAL_CONSUMERIBAN=NL44RABO0123456789&BRQ_SERVICE_IDEAL_CONSUMERISSUER=ING&BRQ_SERVICE_IDEAL_CONSUMERNAME=J.+de+Tester&BRQ_STATUSCODE=190&BRQ_STATUSCODE_DETAIL=S001&BRQ_STATUSMESSAGE=Transaction+successfully+processed&BRQ_TEST=true&BRQ_TIMESTAMP=2015-02-16+13%3A25%3A58&BRQ_TRANSACTION_CANCELABLE=False&BRQ_TRANSACTION_METHOD=ideal&BRQ_TRANSACTION_TYPE=C021&BRQ_TRANSACTIONS=41C48B55FA9164E123CC73B1157459E840BE5D24&BRQ_WEBSITEKEY=12345678&BRQ_SIGNATURE=b92de342cc863acd0c46ed3c8cb6add87668e22f")
+      stub_request(:post, "https://testcheckout.buckaroo.nl/nvp/?op=TransactionStatus").to_return(body: File.read("spec/fixtures/responses/status_success.txt"))
     end
 
-    it 'should raise an exception when initiating a transaction with missing parameters' do
+    it 'should raise an exception when required parameters are missing' do
       expect {
         subject.status
       }.to raise_error(ArgumentError)
@@ -317,15 +327,73 @@ describe Buckaruby::Gateway do
     end
   end
 
-  describe 'verifying payment response' do
-    it 'should raise an exception when parameters are invalid' do
+  describe '#cancellable?' do
+    it 'should raise an exception when required parameters are missing' do
+      expect {
+        subject.cancellable?
+      }.to raise_error(ArgumentError)
+    end
+
+    it 'should return true when the transaction is cancellable' do
+      stub_request(:post, "https://testcheckout.buckaroo.nl/nvp/?op=TransactionStatus").to_return(body: File.read("spec/fixtures/responses/status_cancellable.txt"))
+
+      response = subject.cancellable?(transaction_id: "41C48B55FA9164E123CC73B1157459E840BE5D24")
+      expect(response).to be true
+    end
+
+    it 'should return false when the transaction is not cancellable' do
+      stub_request(:post, "https://testcheckout.buckaroo.nl/nvp/?op=TransactionStatus").to_return(body: File.read("spec/fixtures/responses/status_noncancellable.txt"))
+
+      response = subject.cancellable?(transaction_id: "41C48B55FA9164E123CC73B1157459E840BE5D24")
+      expect(response).to be false
+    end
+  end
+
+  describe '#cancel_transaction' do
+    it 'should raise an exception when required parameters are missing' do
+      expect {
+        subject.cancel_transaction
+      }.to raise_error(ArgumentError)
+    end
+
+    it 'should raise an exception when the transaction is not cancellable' do
+      stub_request(:post, "https://testcheckout.buckaroo.nl/nvp/?op=TransactionStatus").to_return(body: File.read("spec/fixtures/responses/status_noncancellable.txt"))
+
+      expect {
+        subject.cancel_transaction(transaction_id: "41C48B55FA9164E123CC73B1157459E840BE5D24")
+      }.to raise_error(Buckaruby::NonCancellableTransactionException)
+    end
+
+    it 'should cancel the transaction' do
+      stub_request(:post, "https://testcheckout.buckaroo.nl/nvp/?op=TransactionStatus").to_return(body: File.read("spec/fixtures/responses/status_cancellable.txt"))
+      stub_request(:post, "https://testcheckout.buckaroo.nl/nvp/?op=CancelTransaction").to_return(body: File.read("spec/fixtures/responses/cancel_success.txt"))
+
+      response = subject.cancel_transaction(transaction_id: "41C48B55FA9164E123CC73B1157459E840BE5D24")
+      expect(response).to be_an_instance_of(Buckaruby::CancelResponse)
+    end
+  end
+
+  describe '#callback' do
+    it 'should raise an exception when parameters are missing' do
       expect {
         subject.callback
       }.to raise_error(ArgumentError)
     end
 
+    it 'should raise an exception when parameter is an empty Hash' do
+      expect {
+        subject.callback({})
+      }.to raise_error(ArgumentError)
+    end
+
+    it 'should raise an exception when parameter is an empty String' do
+      expect {
+        subject.callback("")
+      }.to raise_error(ArgumentError)
+    end
+
     it 'should raise a SignatureException when the signature is invalid' do
-      params = { "brq_amount" => "10.00", "brq_currency" => "EUR", "brq_customer_name" => "J. de Tester", "brq_description" => "Test", "brq_invoicenumber" => "12345", "brq_mutationtype" => "Collecting", "brq_payer_hash" => "e02377112efcd30bb7420bb1b9855a3778864572", "brq_payment" => "E86256B2787EE7FF0C33D0D4C6159CD922227B79", "brq_service_ideal_consumerbic" => "RABONL2U", "brq_service_ideal_consumeriban" => "NL44RABO0123456789", "brq_service_ideal_consumerissuer" => "Rabobank", "brq_service_ideal_consumername" => "J. de Tester", "brq_statuscode" => "190", "brq_statuscode_detail" => "S001", "brq_statusmessage" => "Transaction successfully processed", "brq_test" => "true", "brq_timestamp" => "2014-11-05 13:10:42", "brq_transaction_method" => "ideal", "brq_transaction_type" => "C021", "brq_transactions" => "41C48B55FA9164E123CC73B1157459E840BE5D24", "brq_websitekey" => "12345678", "brq_signature" => "abcdefgh1234567890abcdefgh1234567890" }
+      params = File.read("spec/fixtures/responses/callback_invalid_signature.txt")
 
       expect {
         subject.callback(params)
@@ -333,6 +401,20 @@ describe Buckaruby::Gateway do
     end
 
     it 'should return the status when the signature is valid' do
+      params = File.read("spec/fixtures/responses/callback_valid_signature.txt")
+
+      response = subject.callback(params)
+      expect(response).to be_an_instance_of(Buckaruby::CallbackResponse)
+      expect(response.transaction_status).to be
+      expect(response.transaction_type).to be
+      expect(response.payment_method).to be
+      expect(response.payment_id).to be
+      expect(response.invoicenumber).to be
+      expect(response.timestamp).to be_an_instance_of(Time)
+      expect(response.to_h).to be_an_instance_of(Hash)
+    end
+
+    it 'should accept a Hash as parameters' do
       params = { "brq_amount" => "10.00", "brq_currency" => "EUR", "brq_customer_name" => "J. de Tester", "brq_description" => "Test", "brq_invoicenumber" => "12345", "brq_mutationtype" => "Collecting", "brq_payer_hash" => "e02377112efcd30bb7420bb1b9855a3778864572", "brq_payment" => "E86256B2787EE7FF0C33D0D4C6159CD922227B79", "brq_service_ideal_consumerbic" => "RABONL2U", "brq_service_ideal_consumeriban" => "NL44RABO0123456789", "brq_service_ideal_consumerissuer" => "Rabobank", "brq_service_ideal_consumername" => "J. de Tester", "brq_statuscode" => "190", "brq_statuscode_detail" => "S001", "brq_statusmessage" => "Transaction successfully processed", "brq_test" => "true", "brq_timestamp" => "2014-11-05 13:10:42", "brq_transaction_method" => "ideal", "brq_transaction_type" => "C021", "brq_transactions" => "41C48B55FA9164E123CC73B1157459E840BE5D24", "brq_websitekey" => "12345678", "brq_signature" => "0a74bba15fccd8094f33678c001b44851643876d" }
 
       response = subject.callback(params)
@@ -348,7 +430,7 @@ describe Buckaruby::Gateway do
 
     context 'payment response' do
       it 'should set the success status when payment status is success' do
-        params = { "brq_amount" => "10.00", "brq_currency" => "EUR", "brq_customer_name" => "J. de Tester", "brq_description" => "Test", "brq_invoicenumber" => "12345", "brq_mutationtype" => "Collecting", "brq_payer_hash" => "e02377112efcd30bb7420bb1b9855a3778864572", "brq_payment" => "E86256B2787EE7FF0C33D0D4C6159CD922227B79", "brq_service_ideal_consumerbic" => "RABONL2U", "brq_service_ideal_consumeriban" => "NL44RABO0123456789", "brq_service_ideal_consumerissuer" => "Rabobank", "brq_service_ideal_consumername" => "J. de Tester", "brq_statuscode" => "190", "brq_statuscode_detail" => "S001", "brq_statusmessage" => "Transaction successfully processed", "brq_test" => "true", "brq_timestamp" => "2014-11-05 13:10:42", "brq_transaction_method" => "ideal", "brq_transaction_type" => "C021", "brq_transactions" => "41C48B55FA9164E123CC73B1157459E840BE5D24", "brq_websitekey" => "12345678", "brq_signature" => "0a74bba15fccd8094f33678c001b44851643876d" }
+        params = File.read("spec/fixtures/responses/callback_payment_success.txt")
 
         response = subject.callback(params)
         expect(response.transaction_status).to eq(Buckaruby::TransactionStatus::SUCCESS)
@@ -362,7 +444,7 @@ describe Buckaruby::Gateway do
       end
 
       it 'should set the failed status when payment status is failed' do
-        params = { "brq_amount" => "10.00", "brq_currency" => "EUR", "brq_customer_name" => "J. de Tester", "brq_description" => "Test", "brq_invoicenumber" => "12345", "brq_mutationtype" => "Collecting", "brq_payer_hash" => "e02377112efcd30bb7420bb1b9855a3778864572", "brq_payment" => "E86256B2787EE7FF0C33D0D4C6159CD922227B79", "brq_service_ideal_consumerbic" => "RABONL2U", "brq_service_ideal_consumeriban" => "NL44RABO0123456789", "brq_service_ideal_consumerissuer" => "Rabobank", "brq_service_ideal_consumername" => "J. de Tester", "brq_statuscode" => "490", "brq_statuscode_detail" => "S001", "brq_statusmessage" => "Transaction successfully processed", "brq_test" => "true", "brq_timestamp" => "2014-11-05 13:10:42", "brq_transaction_method" => "ideal", "brq_transaction_type" => "C021", "brq_transactions" => "41C48B55FA9164E123CC73B1157459E840BE5D24", "brq_websitekey" => "12345678", "brq_signature" => "7e1957cea05cab55aa6e29c36182f9eed6a9984b" }
+        params = File.read("spec/fixtures/responses/callback_payment_failed.txt")
 
         response = subject.callback(params)
         expect(response.transaction_status).to eq(Buckaruby::TransactionStatus::FAILED)
@@ -376,7 +458,7 @@ describe Buckaruby::Gateway do
       end
 
       it 'should set the rejected status when payment status is rejected' do
-        params = { "brq_amount" => "10.00", "brq_currency" => "EUR", "brq_customer_name" => "J. de Tester", "brq_description" => "Test", "brq_invoicenumber" => "12345", "brq_mutationtype" => "Collecting", "brq_payer_hash" => "e02377112efcd30bb7420bb1b9855a3778864572", "brq_payment" => "E86256B2787EE7FF0C33D0D4C6159CD922227B79", "brq_service_ideal_consumerbic" => "RABONL2U", "brq_service_ideal_consumeriban" => "NL44RABO0123456789", "brq_service_ideal_consumerissuer" => "Rabobank", "brq_service_ideal_consumername" => "J. de Tester", "brq_statuscode" => "690", "brq_statuscode_detail" => "S001", "brq_statusmessage" => "Transaction successfully processed", "brq_test" => "true", "brq_timestamp" => "2014-11-05 13:10:42", "brq_transaction_method" => "ideal", "brq_transaction_type" => "C021", "brq_transactions" => "41C48B55FA9164E123CC73B1157459E840BE5D24", "brq_websitekey" => "12345678", "brq_signature" => "ec7b055655f95a23308ed30c085cd65dedccc197" }
+        params = File.read("spec/fixtures/responses/callback_payment_rejected.txt")
 
         response = subject.callback(params)
         expect(response.transaction_status).to eq(Buckaruby::TransactionStatus::REJECTED)
@@ -390,7 +472,7 @@ describe Buckaruby::Gateway do
       end
 
       it 'should set the cancelled status when payment status is cancelled' do
-        params = { "brq_amount" => "10.00", "brq_currency" => "EUR", "brq_customer_name" => "J. de Tester", "brq_description" => "Test", "brq_invoicenumber" => "12345", "brq_mutationtype" => "Collecting", "brq_payer_hash" => "e02377112efcd30bb7420bb1b9855a3778864572", "brq_payment" => "E86256B2787EE7FF0C33D0D4C6159CD922227B79", "brq_service_ideal_consumerbic" => "RABONL2U", "brq_service_ideal_consumeriban" => "NL44RABO0123456789", "brq_service_ideal_consumerissuer" => "Rabobank", "brq_service_ideal_consumername" => "J. de Tester", "brq_statuscode" => "890", "brq_statuscode_detail" => "S001", "brq_statusmessage" => "Transaction successfully processed", "brq_test" => "true", "brq_timestamp" => "2014-11-05 13:10:42", "brq_transaction_method" => "ideal", "brq_transaction_type" => "C021", "brq_transactions" => "41C48B55FA9164E123CC73B1157459E840BE5D24", "brq_websitekey" => "12345678", "brq_signature" => "71cc4d8a49e4062c6d394201f39f4b81b5a39026" }
+        params = File.read("spec/fixtures/responses/callback_payment_cancelled.txt")
 
         response = subject.callback(params)
         expect(response.transaction_status).to eq(Buckaruby::TransactionStatus::CANCELLED)
@@ -404,7 +486,7 @@ describe Buckaruby::Gateway do
       end
 
       it 'should set the pending status when payment status is pending' do
-        params = { "brq_amount" => "10.00", "brq_currency" => "EUR", "brq_customer_name" => "J. de Tester", "brq_description" => "Test", "brq_invoicenumber" => "12345", "brq_mutationtype" => "Collecting", "brq_payer_hash" => "e02377112efcd30bb7420bb1b9855a3778864572", "brq_payment" => "E86256B2787EE7FF0C33D0D4C6159CD922227B79", "brq_service_ideal_consumerbic" => "RABONL2U", "brq_service_ideal_consumeriban" => "NL44RABO0123456789", "brq_service_ideal_consumerissuer" => "Rabobank", "brq_service_ideal_consumername" => "J. de Tester", "brq_statuscode" => "790", "brq_statuscode_detail" => "S001", "brq_statusmessage" => "Transaction successfully processed", "brq_test" => "true", "brq_timestamp" => "2014-11-05 13:10:42", "brq_transaction_method" => "ideal", "brq_transaction_type" => "C021", "brq_transactions" => "41C48B55FA9164E123CC73B1157459E840BE5D24", "brq_websitekey" => "12345678", "brq_signature" => "3e166e22a7ab5880b7424d5614a7fb227d8a7770" }
+        params = File.read("spec/fixtures/responses/callback_payment_pending.txt")
 
         response = subject.callback(params)
         expect(response.transaction_status).to eq(Buckaruby::TransactionStatus::PENDING)
@@ -418,7 +500,7 @@ describe Buckaruby::Gateway do
       end
 
       it 'should include account iban, bic and name for an ideal response' do
-        params = { "brq_amount" => "10.00", "brq_currency" => "EUR", "brq_customer_name" => "J. de Tester", "brq_description" => "Test", "brq_invoicenumber" => "12345", "brq_mutationtype" => "Collecting", "brq_payer_hash" => "e02377112efcd30bb7420bb1b9855a3778864572", "brq_payment" => "E86256B2787EE7FF0C33D0D4C6159CD922227B79", "brq_service_ideal_consumerbic" => "RABONL2U", "brq_service_ideal_consumeriban" => "NL44RABO0123456789", "brq_service_ideal_consumerissuer" => "Rabobank", "brq_service_ideal_consumername" => "J. de Tester", "brq_statuscode" => "190", "brq_statuscode_detail" => "S001", "brq_statusmessage" => "Transaction successfully processed", "brq_test" => "true", "brq_timestamp" => "2014-11-05 13:10:42", "brq_transaction_method" => "ideal", "brq_transaction_type" => "C021", "brq_transactions" => "41C48B55FA9164E123CC73B1157459E840BE5D24", "brq_websitekey" => "12345678", "brq_signature" => "0a74bba15fccd8094f33678c001b44851643876d" }
+        params = File.read("spec/fixtures/responses/callback_payment_success.txt")
 
         response = subject.callback(params)
         expect(response.transaction_status).to eq(Buckaruby::TransactionStatus::SUCCESS)
@@ -432,7 +514,7 @@ describe Buckaruby::Gateway do
       end
 
       it 'should include account iban, name, mandate reference and collect date for a sepa direct debit response' do
-        params = { "brq_amount" => "10.00", "brq_currency" => "EUR", "brq_customer_name" => "J. Tester", "brq_description" => "Test", "brq_invoicenumber" => "12345", "brq_mutationtype" => "Collecting", "brq_payment" => "E86256B2787EE7FF0C33D0D4C6159CD922227B79", "brq_service_sepadirectdebit_collectdate" => "2014-11-13", "brq_service_sepadirectdebit_customeriban" => "NL13TEST0123456789", "brq_service_sepadirectdebit_directdebittype" => "OneOff", "brq_service_sepadirectdebit_mandatedate" => "2014-11-05", "brq_service_sepadirectdebit_mandatereference" => "012345", "brq_statuscode" => "791", "brq_statuscode_detail" => "C620", "brq_statusmessage" => "Awaiting transfer to bank.", "brq_test" => "true", "brq_timestamp" => "2014-11-05 13:45:18", "brq_transaction_method" => "SepaDirectDebit", "brq_transaction_type" => "C004", "brq_transactions" => "41C48B55FA9164E123CC73B1157459E840BE5D24", "brq_websitekey" => "12345678", "brq_signature" => "bcb7924473707d3b7067eb566cd6956dcf354d4c" }
+        params = File.read("spec/fixtures/responses/callback_payment_sepa.txt")
 
         response = subject.callback(params)
         expect(response.transaction_status).to eq(Buckaruby::TransactionStatus::PENDING)
@@ -450,7 +532,7 @@ describe Buckaruby::Gateway do
       end
 
       it 'should return transaction type payment when cancelling a visa or mastercard transaction (empty transaction type)' do
-        params = { "brq_amount" => "10.00", "brq_currency" => "EUR", "brq_description" => "Test cancel", "brq_invoicenumber" => "12345", "brq_mutationtype" => "NotSet", "brq_statuscode" => "890", "brq_statusmessage" => "Cancelled by user", "brq_test" => "false", "brq_timestamp" => "2015-01-27 10:26:42", "brq_transaction_type" => "", "brq_transactions" => "41C48B55FA9164E123CC73B1157459E840BE5D24", "brq_websitekey" => "12345678", "brq_signature" => "9bf412565dfd7399245975a132f649caa7d1d66f" }
+        params = File.read("spec/fixtures/responses/callback_payment_empty_transaction_type.txt")
 
         response = subject.callback(params)
         expect(response.transaction_status).to eq(Buckaruby::TransactionStatus::CANCELLED)
@@ -464,7 +546,7 @@ describe Buckaruby::Gateway do
 
     context 'payment recurrent response' do
       it 'should recognize a visa payment recurrent response' do
-        params = { "brq_amount" => "10.00", "brq_currency" => "EUR", "brq_customer_name" => "Test", "brq_description" => "test", "brq_invoicenumber" => "12345", "brq_issuing_country" => "NL", "brq_mutationtype" => "Collecting", "brq_payment" => "E86256B2787EE7FF0C33D0D4C6159CD922227B79", "brq_recurring" => "True", "brq_SERVICE_visa_CardExpirationDate" => "2017-08", "brq_SERVICE_visa_CardNumberEnding" => "0005", "brq_SERVICE_visa_MaskedCreditcardNumber" => "456355******0005", "brq_statuscode" => "190", "brq_statuscode_detail" => "S001", "brq_statusmessage" => "Transaction successfully processed", "brq_test" => "true", "brq_timestamp" => "2017-08-08 15:01:23", "brq_transaction_method" => "visa", "brq_transaction_type" => "C044", "brq_transactions" => "B51118F58785274E117EFE1BF99D4D50CCB96949", "brq_websitekey" => "12345678", "brq_signature" => "2d445ce39320a10e6637b6ace6896559ce040cb3" }
+        params = File.read("spec/fixtures/responses/callback_recurrent_visa.txt")
 
         response = subject.callback(params)
         expect(response.transaction_status).to eq(Buckaruby::TransactionStatus::SUCCESS)
@@ -478,7 +560,7 @@ describe Buckaruby::Gateway do
       end
 
       it 'should recognize a sepa direct debit payment recurrent response' do
-        params = { "brq_amount" => "1.00", "brq_currency" => "EUR", "brq_customer_name" => "J. Tester", "brq_description" => "Recurrent: Test", "brq_invoicenumber" => "12345", "brq_mutationtype" => "Collecting", "brq_payment" => "E86256B2787EE7FF0C33D0D4C6159CD922227B79", "brq_recurring" => "True", "brq_SERVICE_sepadirectdebit_CollectDate" => "2016-09-16", "brq_SERVICE_sepadirectdebit_CustomerIBAN" => "NL13TEST0123456789", "brq_SERVICE_sepadirectdebit_DirectDebitType" => "First", "brq_SERVICE_sepadirectdebit_MandateDate" => "2016-09-07", "brq_SERVICE_sepadirectdebit_MandateReference" => "e1a31b3e461ab74cdbacf16bccd5290f9a284618", "brq_statuscode" => "190", "brq_statusmessage" => "Success", "brq_test" => "false", "brq_timestamp" => "2016-09-16 00:26:03", "brq_transaction_method" => "SepaDirectDebit", "brq_transaction_type" => "C005", "brq_transactions" => "B51118F58785274E117EFE1BF99D4D50CCB96949", "brq_websitekey" => "12345678", "brq_signature" => "851d0eacc314ceb9070abeb83d1853270459d81a" }
+        params = File.read("spec/fixtures/responses/callback_recurrent_sepa.txt")
 
         response = subject.callback(params)
         expect(response.transaction_status).to eq(Buckaruby::TransactionStatus::SUCCESS)
@@ -494,7 +576,7 @@ describe Buckaruby::Gateway do
 
     context 'refund response' do
       it 'should recognize an ideal refund response' do
-        params = { "brq_amount_credit" => "10.00", "brq_currency" => "EUR", "brq_customer_name" => "J. de Tester", "brq_description" => "Refund: Test", "brq_invoicenumber" => "12345", "brq_mutationtype" => "Collecting", "brq_payment" => "E86256B2787EE7FF0C33D0D4C6159CD922227B79", "brq_relatedtransaction_refund" => "41C48B55FA9164E123CC73B1157459E840BE5D24", "brq_service_ideal_customeraccountname" => "J. de Tester", "brq_service_ideal_customerbic" => "RABONL2U", "brq_service_ideal_customeriban" => "NL44RABO0123456789", "brq_statuscode" => "190", "brq_statuscode_detail" => "S001", "brq_statusmessage" => "Transaction successfully processed", "brq_test" => "true", "brq_timestamp" => "2014-11-06 15:27:31", "brq_transaction_method" => "ideal", "brq_transaction_type" => "C121", "brq_transactions" => "B51118F58785274E117EFE1BF99D4D50CCB96949", "brq_websitekey" => "12345678", "brq_signature" => "395bacf2bb5231501b94b3d22996f4f61c013f4b" }
+        params = File.read("spec/fixtures/responses/callback_refund_ideal.txt")
 
         response = subject.callback(params)
         expect(response.transaction_status).to eq(Buckaruby::TransactionStatus::SUCCESS)
@@ -509,7 +591,7 @@ describe Buckaruby::Gateway do
       end
 
       it 'should recognize a paypal refund response' do
-        params = { "brq_amount_credit" => "10.00", "brq_currency" => "EUR", "brq_customer_name" => "J. Tester", "brq_description" => "Refund: Test", "brq_invoicenumber" => "12345", "brq_mutationtype" => "Processing", "brq_payment" => "E86256B2787EE7FF0C33D0D4C6159CD922227B79", "brq_relatedtransaction_refund" => "41C48B55FA9164E123CC73B1157459E840BE5D24", "brq_statuscode" => "190", "brq_statuscode_detail" => "S001", "brq_statusmessage" => "Transaction successfully processed", "brq_test" => "false", "brq_timestamp" => "2014-11-06 15:04:20", "brq_transaction_method" => "paypal", "brq_transaction_type" => "V110", "brq_transactions" => "B51118F58785274E117EFE1BF99D4D50CCB96949", "brq_websitekey" => "12345678", "brq_signature" => "da55998b9681cc9d10f983e21f44c0510b474c12" }
+        params = File.read("spec/fixtures/responses/callback_refund_paypal.txt")
 
         response = subject.callback(params)
         expect(response.transaction_status).to eq(Buckaruby::TransactionStatus::SUCCESS)
@@ -526,7 +608,7 @@ describe Buckaruby::Gateway do
 
     context 'reversal response' do
       it 'should recognize a sepa direct debit reversal response' do
-        params = { "brq_amount_credit" => "10.00", "brq_currency" => "EUR", "brq_customer_name" => "J. Tester", "brq_description" => "Reversal: Test", "brq_invoicenumber" => "12345", "brq_mutationtype" => "Collecting", "brq_payment" => "E86256B2787EE7FF0C33D0D4C6159CD922227B79", "brq_relatedtransaction_reversal" => "41C48B55FA9164E123CC73B1157459E840BE5D24", "brq_service_sepadirectdebit_reasoncode" => "MD06", "brq_service_sepadirectdebit_reasonexplanation" => "Debtor uses 8 weeks reversal right", "brq_statuscode" => "190", "brq_statusmessage" => "Success", "brq_test" => "false", "brq_timestamp" => "2014-11-28 03:37:48", "brq_transaction_method" => "SepaDirectDebit", "brq_transaction_type" => "C501", "brq_transactions" => "8CCE4BB06339F28A506E1A328025D7DF13CCAD59", "brq_websitekey" => "12345678", "brq_signature" => "4b8ccf699dadca7465510ba94a59fa9d7b5b050a" }
+        params = File.read("spec/fixtures/responses/callback_reversal_sepa.txt")
 
         response = subject.callback(params)
         expect(response.transaction_status).to eq(Buckaruby::TransactionStatus::SUCCESS)
@@ -541,7 +623,7 @@ describe Buckaruby::Gateway do
       end
 
       it 'should recognize a paypal reversal response' do
-        params = { "brq_amount_credit" => "15.40", "brq_currency" => "EUR", "brq_customer_name" => "J. Tester", "brq_description" => "Reversal: Test", "brq_invoicenumber" => "12345", "brq_mutationtype" => "Processing", "brq_payment" => "E86256B2787EE7FF0C33D0D4C6159CD922227B79", "brq_relatedtransaction_reversal" => "41C48B55FA9164E123CC73B1157459E840BE5D24", "brq_statuscode" => "190", "brq_statusmessage" => "Success", "brq_test" => "false", "brq_timestamp" => "2015-12-12 08:26:46", "brq_transaction_method" => "paypal", "brq_transaction_type" => "V111", "brq_transactions" => "8CCE4BB06339F28A506E1A328025D7DF13CCAD59", "brq_websitekey" => "12345678", "brq_signature" => "124f1a2bdfbd952a8d909d45cce421b6734a2c41" }
+        params = File.read("spec/fixtures/responses/callback_reversal_paypal.txt")
 
         response = subject.callback(params)
         expect(response.transaction_status).to eq(Buckaruby::TransactionStatus::SUCCESS)