From 954125b7f6862f2bcfa701d0a37867795bde59bb Mon Sep 17 00:00:00 2001 From: Kelly Date: Tue, 16 Apr 2024 15:30:21 -0400 Subject: [PATCH] style improvements after running rubocop --- spec/controllers/payments_controller_spec.rb | 74 ++++++++++---------- 1 file changed, 37 insertions(+), 37 deletions(-) diff --git a/spec/controllers/payments_controller_spec.rb b/spec/controllers/payments_controller_spec.rb index b68478eb4c..2af4e7a481 100644 --- a/spec/controllers/payments_controller_spec.rb +++ b/spec/controllers/payments_controller_spec.rb @@ -3,42 +3,42 @@ require 'spec_helper' describe PaymentsController, type: :controller do - let!(:user) { create(:user) } - let!(:order) { create(:order, user: user) } - let!(:payment) { create(:payment, order: order) } - - describe "testing redirect_to_authorize" do - context "when user isn't logged in" do - it "redirects to the login page and set error flash msg" do - get :redirect_to_authorize, params: { id: payment.id } - expect(response).to redirect_to(root_path(anchor: "/login", after_login: request.original_fullpath)) - expect(flash[:error]).to eq I18n.t("spree.orders.edit.login_to_view_order") - end - end - - context "when user is logged in" do - before do - allow(controller).to receive(:spree_current_user).and_return(user) - end + let!(:user) { create(:user) } + let!(:order) { create(:order, user:) } + let!(:payment) { create(:payment, order:) } - context "has cvv response message" do - before do - allow_any_instance_of(Spree::Payment).to receive(:cvv_response_message).and_return('http://example.com') - end - - it "redirects to the CVV response URL" do - get :redirect_to_authorize, params: { id: payment.id } - expect(response).to redirect_to('http://example.com') - end - - end - - context "doesn't have cvv response message" do - it "redirect to order URL" do - get :redirect_to_authorize, params: { id: payment.id } - expect(response).to redirect_to(order_url(order)) - end - end - end + describe "testing redirect_to_authorize" do + context "when user isn't logged in" do + it "redirects to the login page and set error flash msg" do + get :redirect_to_authorize, params: { id: payment.id } + expect(response).to redirect_to(root_path(anchor: "/login", + after_login: request.original_fullpath)) + expect(flash[:error]).to eq I18n.t("spree.orders.edit.login_to_view_order") + end end -end \ No newline at end of file + + context "when user is logged in" do + before do + allow(controller).to receive(:spree_current_user).and_return(user) + end + + context "has cvv response message" do + before do + allow_any_instance_of(Spree::Payment).to receive(:cvv_response_message).and_return('http://example.com') + end + + it "redirects to the CVV response URL" do + get :redirect_to_authorize, params: { id: payment.id } + expect(response).to redirect_to('http://example.com') + end + end + + context "doesn't have cvv response message" do + it "redirect to order URL" do + get :redirect_to_authorize, params: { id: payment.id } + expect(response).to redirect_to(order_url(order)) + end + end + end + end +end