diff --git a/app/models/spree/payment_method/check.rb b/app/models/spree/payment_method/check.rb index 9f4378830c..839698bebc 100644 --- a/app/models/spree/payment_method/check.rb +++ b/app/models/spree/payment_method/check.rb @@ -25,6 +25,10 @@ module Spree ActiveMerchant::Billing::Response.new(true, "", {}, {}) end + def payment_source_class + nil + end + def source_required? false end diff --git a/spec/controllers/checkout_controller_spec.rb b/spec/controllers/checkout_controller_spec.rb index b39ffdcacd..4930eb22d5 100644 --- a/spec/controllers/checkout_controller_spec.rb +++ b/spec/controllers/checkout_controller_spec.rb @@ -338,12 +338,11 @@ RSpec.describe CheckoutController, type: :controller do let!(:payment_method_id) { payment_method.id } it "updates and redirects to summary step" do - expect { put(:update, params:) }.to raise_error(RuntimeError, error_message) + put(:update, params:) - # according to Bugsnag, we should get an error 500 - expect(response.status).to be 200 - expect(response).not_to redirect_to checkout_step_path(:summary) - expect(order.reload.state).to eq "payment" + expect(response.status).to be 302 + expect(response).to redirect_to checkout_step_path(:summary) + expect(order.reload.state).to eq "confirmation" end end