Remove shipping_method_id from RestartCheckout, destroying shipments …

…is now enough as order.shipping_method_id is no longer used and will be dropped. See openfoodfoundation#2683 for more details

app/services/restart_checkout.rb

@@ -23,7 +23,6 @@ def reset_state_to_cart
   end
 
   def clear_shipments
-    order.update_attributes!(shipping_method_id: nil)
     order.shipments.with_state(:pending).destroy_all
   end
 

spec/services/restart_checkout_spec.rb

@@ -17,7 +17,6 @@
       let!(:payment_failed) { create(:payment, order: order, state: 'failed') }
 
       before do
-        order.shipping_method_id = shipment_pending.shipping_method_id
         order.update_attribute(:state, "payment")
       end
 
@@ -28,7 +27,6 @@
         RestartCheckout.new(order).call
 
         expect(order.state).to eq 'cart'
-        expect(order.shipping_method_id).to eq nil
         expect(order.shipments.count).to eq 0
         expect(order.adjustments.shipping.count).to eq 0
         expect(order.payments.count).to eq 1