Update totals on summary step

openfoodfoundation · Sep 5, 2022 · 1d04510 · 1d04510
1 parent 6c7a089
commit 1d04510
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 2 deletions.
diff --git a/app/controllers/split_checkout_controller.rb b/app/controllers/split_checkout_controller.rb
@@ -22,7 +22,7 @@ class SplitCheckoutController < ::BaseController
   def edit
     redirect_to_step_based_on_order unless params[:step]
     check_step if params[:step]
-    @order.create_tax_charge! if params[:step] == "summary"
+    recalculate_tax if params[:step] == "summary"
   end
 
   def update
@@ -156,4 +156,9 @@ def check_step
       redirect_to checkout_step_path(:payment) if params[:step] == "summary"
     end
   end
+
+  def recalculate_tax
+    @order.create_tax_charge!
+    @order.update_order!
+  end
 end
diff --git a/spec/system/consumer/split_checkout_tax_not_incl_spec.rb b/spec/system/consumer/split_checkout_tax_not_incl_spec.rb
@@ -223,7 +223,6 @@
           end
 
           it "should re-calculate the tax accordingly" do
-            pending("#9153")
             select "Victoria", from: "order_bill_address_attributes_state_id"
 
             # it should not be necessary to save as new default bill address