Merge pull request #65 from Lostmyname/fix-refund-lost-cents

Fix for refund amount losing cents
StemboltHQ · Jan 7, 2016 · e1e06db · e1e06db
2 parents 037b218 + da5a35b
commit e1e06db
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 1 deletion.
diff --git a/app/models/spree/adyen/notification_processor.rb b/app/models/spree/adyen/notification_processor.rb
@@ -77,7 +77,7 @@ def handle_modification_event
 
         elsif notification.refund?
           payment.refunds.create!(
-            amount: notification.value / 100, # cents to dollars
+            amount: notification.value / 100.0, # cents to dollars
             transaction_id: notification.psp_reference,
             refund_reason_id: ::Spree::RefundReason.first.id # FIXME
           )

diff --git a/spec/models/spree/adyen/notification_processor_spec.rb b/spec/models/spree/adyen/notification_processor_spec.rb
@@ -200,6 +200,11 @@
             from(0).
             to(1)
         end
+
+        it "creates a refund of the correct value" do
+          subject
+          expect(payment.reload.refunds.last.amount).to eq 23.99
+        end
       end
 
       context "when refunded from Solidus" do