Fix SolidusPromotion OrderRecalculator patch

The method signature of the InMemoryOrderUpdater changed to include the
`persist` flag. This would previously raise an ArgumentError after
configuring the app to use the InMemoryOrderUpdater, and passing an
explicit argument for persist.

Co-authored-by: Adam Mueller <adam@super.gd>

Author: sofiabesenski4

promotions/app/patches/models/solidus_promotions/in_memory_order_updater_patch.rb

@@ -0,0 +1,17 @@
+# frozen_string_literal: true
+
+module SolidusPromotions
+  module InMemoryOrderUpdaterPatch
+    # This is only needed for stores upgrading from the legacy promotion system.
+    # Once we've removed support for the legacy promotion system, we can remove this.
+    def recalculate(persist: true)
+      if SolidusPromotions.config.sync_order_promotions
+        MigrationSupport::OrderPromotionSyncer.new(order: order).call
+      end
+      super
+    end
+
+    Spree::InMemoryOrderUpdater.prepend self
+  end
+end
+

…s_promotions/order_recalculator_patch.rb …olidus_promotions/order_updater_patch.rbpromotions/app/patches/models/solidus_promotions/order_recalculator_patch.rb renamed to promotions/app/patches/models/solidus_promotions/order_updater_patch.rb promotions/app/patches/models/solidus_promotions/order_recalculator_patch.rb renamed to promotions/app/patches/models/solidus_promotions/order_updater_patch.rb

@@ -1,7 +1,7 @@
 # frozen_string_literal: true
 
 module SolidusPromotions
-  module OrderRecalculatorPatch
+  module OrderUpdaterPatch
     # This is only needed for stores upgrading from the legacy promotion system.
     # Once we've removed support for the legacy promotion system, we can remove this.
     def recalculate
@@ -12,6 +12,5 @@ def recalculate
     end
 
     Spree::OrderUpdater.prepend self
-    Spree::InMemoryOrderUpdater.prepend self
   end
 end

promotions/spec/models/promotion/in_memory_integration_spec.rb

@@ -16,5 +16,10 @@
     end
 
     it_behaves_like "a successfully integrated promotion system"
+
+    it "allows in memory order recalculates without persistence" do
+      order = create(:order_with_line_items)
+      expect { Spree::InMemoryOrderUpdater.new(order).recalculate(persist: false) }.not_to raise_error
+    end
   end
 end