Use alternate method for setting request.referer in controller specs

Apparently stubbing `request.referer` is tricky business in Rails 5... :shrugs:
This commit is contained in:
Matt-Yorkley
2021-01-23 11:25:29 +00:00
parent 12a513c54c
commit d9bd0e8f0e

View File

@@ -174,7 +174,7 @@ describe Spree::Admin::PaymentsController, type: :controller do
let(:params) { { e: 'credit', order_id: order.number, id: payment.id } }
before do
allow(request).to receive(:referer) { 'http://foo.com' }
request.env["HTTP_REFERER"] = "http://foo.com"
allow(Spree::Payment).to receive(:find).with(payment.id.to_s) { payment }
end
@@ -210,7 +210,7 @@ describe Spree::Admin::PaymentsController, type: :controller do
let(:params) { { e: 'refund', order_id: order.number, id: payment.id } }
before do
allow(request).to receive(:referer) { 'http://foo.com' }
request.env["HTTP_REFERER"] = "http://foo.com"
allow(Spree::Payment).to receive(:find).with(payment.id.to_s) { payment }
end
@@ -248,7 +248,7 @@ describe Spree::Admin::PaymentsController, type: :controller do
before do
allow(PaymentMailer).to receive(:authorize_payment) { mail_mock }
allow(request).to receive(:referer) { 'http://foo.com' }
request.env["HTTP_REFERER"] = "http://foo.com"
allow(Spree::Payment).to receive(:find).with(payment.id.to_s) { payment }
allow(payment).to receive(:cvv_response_message).and_return("https://www.stripe.com/authorize")
end
@@ -266,7 +266,7 @@ describe Spree::Admin::PaymentsController, type: :controller do
let(:params) { { e: 'unrecognized_event', order_id: order.number, id: payment.id } }
before do
allow(request).to receive(:referer) { 'http://foo.com' }
request.env["HTTP_REFERER"] = "http://foo.com"
allow(Spree::Payment).to receive(:find).with(payment.id.to_s) { payment }
end