Skip to content

Commit 644e1d0

Browse files
committed
Provide more useful feedback when an API call fails unexpectedly.
1 parent f16c5e6 commit 644e1d0

File tree

2 files changed

+6
-1
lines changed

2 files changed

+6
-1
lines changed

spec/api_spec.rb

+1-1
Original file line numberDiff line numberDiff line change
@@ -158,7 +158,7 @@
158158

159159
expect(createProfResp).not_to eq(nil)
160160
unless createProfResp.messages.resultCode == MessageTypeEnum::Ok
161-
puts createProfResp.messages.messages[0].text
161+
puts createProfResp.messages.messages[0].text, createProfReq.transId
162162
end
163163
expect(createProfResp.messages.resultCode).to eq(MessageTypeEnum::Ok)
164164
expect(createProfResp.customerProfileId).not_to eq(nil)

spec/cim_spec.rb

+5
Original file line numberDiff line numberDiff line change
@@ -253,6 +253,7 @@ def create_payment_profile(payment_profile, profile, validation_mode = :none)
253253
transaction = AuthorizeNet::CIM::Transaction.new(api_login, api_key, gateway: :sandbox)
254254
expect(transaction).to respond_to(:create_transaction_auth_only)
255255
response = transaction.create_transaction_auth_only(@amount, profile, payment_profile, AuthorizeNet::Order.new)
256+
expect(response.message_text).to eq 'Successful.'
256257
expect(response.success?).to eq true
257258
direct_response = response.direct_response
258259
expect(direct_response).to be_instance_of(AuthorizeNet::AIM::Response)
@@ -263,6 +264,7 @@ def create_payment_profile(payment_profile, profile, validation_mode = :none)
263264
# create an auth only transaction
264265
transaction = AuthorizeNet::CIM::Transaction.new(api_login, api_key, gateway: :sandbox)
265266
response = transaction.create_transaction_auth_only(@amount + 10, profile, payment_profile, AuthorizeNet::Order.new)
267+
expect(response.message_text).to eq 'Successful.'
266268
expect(response.success?).to eq true
267269
direct_response = response.direct_response
268270
expect(direct_response.success?).to eq true
@@ -281,6 +283,7 @@ def create_payment_profile(payment_profile, profile, validation_mode = :none)
281283
# create a transaction
282284
transaction = AuthorizeNet::CIM::Transaction.new(api_login, api_key, gateway: :sandbox)
283285
response = transaction.create_transaction_auth_capture(@amount, profile, payment_profile, AuthorizeNet::Order.new)
286+
expect(response.message_text).to eq 'Successful.'
284287
expect(response.success?).to eq true
285288
direct_response = response.direct_response
286289
expect(direct_response.success?).to eq true
@@ -326,6 +329,7 @@ def create_payment_profile(payment_profile, profile, validation_mode = :none)
326329
it "should support custom fields" do
327330
transaction = AuthorizeNet::CIM::Transaction.new(api_login, api_key, gateway: :sandbox)
328331
response = transaction.create_transaction_auth_capture(@amount, profile, payment_profile, AuthorizeNet::Order.new, custom_fields: { foo: '123', bar: '456' })
332+
expect(response.message_text).to eq 'Successful.'
329333
expect(response.success?).to eq true
330334
direct_response = response.direct_response
331335
expect(direct_response.success?).to eq true
@@ -341,6 +345,7 @@ def create_payment_profile(payment_profile, profile, validation_mode = :none)
341345
order.description = 'This order includes invoice num'
342346
order.po_num = 'PO_12345'
343347
response = transaction.create_transaction_auth_capture(@amount, profile, payment_profile, order)
348+
expect(response.message_text).to eq 'Successful.'
344349
expect(response.success?).to eq true
345350
direct_response = response.direct_response
346351
expect(direct_response).to be_instance_of(AuthorizeNet::AIM::Response)

0 commit comments

Comments
 (0)