Fix the failing CyberSource remote test. The CyberSource gateway resp…

…onds with a 500 for invalid logins so ActiveMerchant::Connection raises a ResponseError instead of returning the response itself [Patrick Joyce]
freuma · May 14, 2010 · 0815ffd · 0815ffd
1 parent 2fa48b1
commit 0815ffd
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 3 deletions.
diff --git a/CHANGELOG b/CHANGELOG
@@ -1,5 +1,6 @@
 = ActiveMerchant CHANGELOG
 
+* Fix failing CyberSource remote test [Patrick Joyce]
 * Support for Garanti Sanal Pos: Turkish bank and billing gateway [Selem Delul]
 * Add deprecation note for Money objects to Bogus gateway [Soleone]
 * Updated test URL for Merchant eSolutions and added valid remote test credentials [Soleone]

diff --git a/test/remote/gateways/remote_cyber_source_test.rb b/test/remote/gateways/remote_cyber_source_test.rb
@@ -126,9 +126,11 @@ def test_failed_capture_bad_auth_info
 
   def test_invalid_login
     gateway = CyberSourceGateway.new( :login => '', :password => '' )
-    assert response = gateway.purchase(@amount, @credit_card, @options)
-    assert_match /wsse:InvalidSecurity/, response.message
-    assert_failure response
+    authentication_exception = assert_raise ActiveMerchant::ResponseError, 'Failed with 500 Internal Server Error' do
+      gateway.purchase(@amount, @credit_card, @options)
+    end
+    assert response = authentication_exception.response
+    assert_match /wsse:InvalidSecurity/, response.body
   end
 
   def test_successful_credit