DRYes card and payment intent let groups

Fixes typo on capture_amount declaration
openfoodfoundation · Dec 12, 2023 · b259185 · b259185
1 parent b45fc0b
commit b259185
Showing 1 changed file with 25 additions and 46 deletions.
diff --git a/spec/models/spree/gateway/stripe_sca_spec.rb b/spec/models/spree/gateway/stripe_sca_spec.rb
@@ -26,33 +26,34 @@
     { order_id: order.number }
   }
 
+  before { Stripe.api_key = secret }
+
+  let(:pm_card) do
+    Stripe::PaymentMethod.create({
+                                   type: 'card',
+                                   card: {
+                                     number: '4242424242424242',
+                                     exp_month: 12,
+                                     exp_year: year_valid,
+                                     cvc: '314',
+                                   },
+                                 })
+  end
+  let(:payment_intent) do
+    Stripe::PaymentIntent.create({
+                                   amount: 10000, # given in AUD cents
+                                   currency: 'aud', # AUD to match order currency
+                                   payment_method: pm_card,
+                                   payment_method_types: ['card'],
+                                   capture_method: 'manual',
+                                 })
+  end
+
   describe "#purchase", :vcr, :stripe_version do
-    before { Stripe.api_key = secret }
-
-    let!(:pm_card) do
-      Stripe::PaymentMethod.create({
-                                     type: 'card',
-                                     card: {
-                                       number: '4242424242424242',
-                                       exp_month: 12,
-                                       exp_year: year_valid,
-                                       cvc: '314',
-                                     },
-                                   })
-    end
-    let!(:payment_intent) do
-      Stripe::PaymentIntent.create({
-                                     amount: 1000, # given in AUD cents
-                                     currency: 'aud', # AUD to match order currency
-                                     payment_method: pm_card,
-                                     payment_method_types: ['card'],
-                                     capture_method: 'manual',
-                                   })
-    end
 
     # Stripe acepts amounts as positive integers representing how much to charge
     # in the smallest currency unit
-    let(:capture_amount) { order.total.to_i * 10 }
+    let(:capture_amount) { order.total.to_i * 100 }
     let(:response) { subject.purchase(capture_amount, credit_card, gateway_options) }
 
     before do
@@ -81,29 +82,7 @@
     end
   end
 
-  context "#error message", :vcr, :stripe_version do
-    before { Stripe.api_key = secret }
-
-    let!(:pm_card) do
-      Stripe::PaymentMethod.create({
-                                     type: 'card',
-                                     card: {
-                                       number: '4242424242424242',
-                                       exp_month: 12,
-                                       exp_year: year_valid,
-                                       cvc: '314',
-                                     },
-                                   })
-    end
-    let!(:payment_intent) do
-      Stripe::PaymentIntent.create({
-                                     amount: 1000, # given in AUD cents
-                                     currency: 'aud', # AUD to match order currency
-                                     payment_method: pm_card,
-                                     payment_method_types: ['card'],
-                                     capture_method: 'manual',
-                                   })
-    end
+  describe "#error message", :vcr, :stripe_version do
 
     context "when payment intent state is not in 'requires_capture' state" do
       before do