wip: Ensure coordinator_options are integration tested

This new, passing spec should be moved. Additional coverage could be
added. Existing specs in the modified spec file can also be moved to an
integration test file.

Co-authored-by: Noah Silvera <[email protected]>

Author: benjaminwil

core/app/models/spree/stock/estimator.rb

@@ -29,6 +29,8 @@ def shipping_rates(package, frontend_only = true)
 
       private
 
+      attr_reader :coordinator_options
+
       def choose_default_shipping_rate(shipping_rates)
         unless shipping_rates.empty?
           default_shipping_rate = Spree::Config.shipping_rate_selector_class.new(shipping_rates).find_default

core/spec/lib/spree/core/stock_configuration_spec.rb

@@ -37,6 +37,42 @@
     ensure
       Object.send(:remove_const, :MyEstimator)
     end
+
+    # FIXME:
+    # Move this to an integration test file after moving existing specs that are
+    # actually integration specs to that file.
+    #
+    it "uses coordinator options passed in to the simple coordinator" do
+      order = create(:order_with_line_items)
+      my_shipping_rate = create(:shipping_method).shipping_rates.new
+
+      MyEstimator = Class.new(Spree::Stock::Estimator) do
+        def shipping_rates(package, _frontend_only = true)
+          raise ShipmentRequired if package.shipment.nil?
+          raise OrderRequired if package.shipment.order.nil?
+
+          if (shipping_rate = coordinator_options[:arbitrary_shipping_rates].first)
+            shipping_rate.selected = true
+            return [shipping_rate]
+          else
+            raise StandardError, "no shipping rate!"
+          end
+        end
+      end
+      Spree::Config.stock.estimator_class = MyEstimator.to_s
+
+      stock_simple_coordinator = stock_configuration.coordinator_class
+      shipments = stock_simple_coordinator
+        .new(
+          order, coordinator_options: {
+            arbitrary_shipping_rates: [my_shipping_rate]
+          }
+        )
+        .shipments
+
+
+      expect(shipments.first.selected_shipping_rate).to eq my_shipping_rate
+    end
   end
 
   describe '#location_filter_class' do

core/spec/models/spree/stock/simple_coordinator_integration_spec.rb

@@ -0,0 +1,11 @@
+require "rails_helper"
+
+
+# FIXME:
+# - Move over existing simple coordinator specs that are actually integration
+#   tests.
+# - Add test coverage for the usage of `coordinator_options` as passed from the
+#   simple coordinator to other collaborating classes.
+#
+RSpec.describe "Integrating with the simple coordinator" do
+end