From d9bd0e8f0e0022cee5408a66f27f98a2239e4e9a Mon Sep 17 00:00:00 2001 From: Matt-Yorkley <9029026+Matt-Yorkley@users.noreply.github.com> Date: Sat, 23 Jan 2021 11:25:29 +0000 Subject: [PATCH] Use alternate method for setting request.referer in controller specs Apparently stubbing `request.referer` is tricky business in Rails 5... :shrugs: --- .../admin/orders/payments/payments_controller_spec.rb | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/spec/controllers/spree/admin/orders/payments/payments_controller_spec.rb b/spec/controllers/spree/admin/orders/payments/payments_controller_spec.rb index 1e1437aee2..a6ed887d61 100644 --- a/spec/controllers/spree/admin/orders/payments/payments_controller_spec.rb +++ b/spec/controllers/spree/admin/orders/payments/payments_controller_spec.rb @@ -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