make package a required, not keyword, argument

openfoodfoundation · Oct 17, 2020 · dc3c2c4 · dc3c2c4
1 parent 81e2f5a
commit dc3c2c4
Showing 1 changed file with 4 additions and 4 deletions.
diff --git a/engines/order_management/spec/services/order_management/stock/estimator_spec.rb b/engines/order_management/spec/services/order_management/stock/estimator_spec.rb
@@ -77,7 +77,7 @@ module Stock
           allow(subject).to receive(:shipping_methods).and_return(shipping_methods)
 
           expected_costs = %w[3.00 4.00 5.00].map(&BigDecimal.method(:new))
-          expect(subject.shipping_rates(package: package).map(&:cost)).to eq expected_costs
+          expect(subject.shipping_rates(package).map(&:cost)).to eq expected_costs
         end
 
         context "general shipping methods" do
@@ -91,7 +91,7 @@ module Stock
 
             allow(subject).to receive(:shipping_methods).and_return(shipping_methods)
 
-            shipping_rates = subject.shipping_rates(package: package)
+            shipping_rates = subject.shipping_rates(package)
             expect(shipping_rates.sort_by(&:cost).map(&:selected)).to eq [true, false]
           end
 
@@ -103,7 +103,7 @@ module Stock
 
             allow(subject).to receive(:shipping_methods).and_return(shipping_methods)
 
-            subject.shipping_rates(package: package)
+            subject.shipping_rates(package)
           end
         end
 
@@ -119,7 +119,7 @@ module Stock
             allow(subject).
               to receive(:shipping_methods).and_return([backend_method, generic_method])
 
-            expect(subject.shipping_rates(package: package).map(&:selected)).to eq [false, true]
+            expect(subject.shipping_rates(package).map(&:selected)).to eq [false, true]
           end
         end
       end