Update HiTrust notification to return a float string for gross

git-svn-id: https://activemerchant.googlecode.com/svn/trunk/active_merchant@651 6513ea26-6c20-0410-8a68-89cd7235086d
activemerchant · Mar 11, 2008 · 66791a7 · 66791a7
1 parent f84c771
commit 66791a7
Show file tree

Hide file tree

Showing 4 changed files with 4 additions and 4 deletions.
diff --git a/CHANGELOG b/CHANGELOG
@@ -1,5 +1,6 @@
 = ActiveMerchant CHANGELOG
 
+* Fix gross in HiTrust notification. Don't use Money object in Verifi gateway [cody]
 * Initial implementation of Payflow Name-Value API [Greg Furmanek]
 * Add support for CyberSource credits [mjuneja]
 

diff --git a/lib/active_merchant/billing/integrations/hi_trust/notification.rb b/lib/active_merchant/billing/integrations/hi_trust/notification.rb
@@ -28,11 +28,11 @@ def currency
           end
 
           def gross
-            params['approveamount']
+            sprintf("%.2f", gross_cents.to_f / 100)
           end
 
           def gross_cents
-            gross.to_i
+            params['approveamount'].to_i
           end
 
           def account

diff --git a/test/unit/gateways/verifi_test.rb b/test/unit/gateways/verifi_test.rb
@@ -38,7 +38,6 @@ def test_unsuccessful_request
   end
 
   def test_amount_style
-    assert_equal '10.34', @gateway.send(:amount, Money.new(1034))
     assert_equal '10.34', @gateway.send(:amount, 1034)
 
     assert_raise(ArgumentError) do

diff --git a/test/unit/integrations/notifications/hi_trust_notification_test.rb b/test/unit/integrations/notifications/hi_trust_notification_test.rb
@@ -13,7 +13,7 @@ def test_accessors
     assert_equal "012345678901", @notification.transaction_id
     assert_equal "1000", @notification.item_id
     assert_equal "101010", @notification.account
-    assert_equal "500", @notification.gross
+    assert_equal "5.00", @notification.gross
     assert_equal "USD", @notification.currency
     assert_equal Time.parse("2007-12-01.12.35.40.123456"), @notification.received_at
     assert @notification.test?