From cc1fa7f563aa88a58317b76ab9cf487878e66e7f Mon Sep 17 00:00:00 2001 From: David Cook Date: Tue, 9 Jul 2024 13:57:06 +1000 Subject: [PATCH] Always avoid error on redirect_to --- spec/controllers/admin/product_import_controller_spec.rb | 7 +++++-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/spec/controllers/admin/product_import_controller_spec.rb b/spec/controllers/admin/product_import_controller_spec.rb index 404e4ed763..060c951939 100644 --- a/spec/controllers/admin/product_import_controller_spec.rb +++ b/spec/controllers/admin/product_import_controller_spec.rb @@ -4,6 +4,11 @@ require 'spec_helper' RSpec.describe Admin::ProductImportController, type: :controller do describe 'validate_file_path' do + before do + # Avoid error on redirect_to + allow(controller).to receive(:raise_invalid_file_path).and_return(false) + end + context 'file extension' do it 'should authorize csv extension' do path = '/tmp/product_import123/import.csv' @@ -11,7 +16,6 @@ RSpec.describe Admin::ProductImportController, type: :controller do end it 'should reject other extensions' do - allow(controller).to receive(:raise_invalid_file_path).and_return(false) path = '/tmp/product_import123/import.pdf' expect(controller.__send__(:validate_file_path, path)).to be_falsey path1 = '/tmp/product_import123/import.xslx' @@ -30,7 +34,6 @@ RSpec.describe Admin::ProductImportController, type: :controller do end it 'should reject invalid paths' do - allow(controller).to receive(:raise_invalid_file_path).and_return(false) path = '/tmp/product_import123/../etc/import.csv' expect(controller.__send__(:validate_file_path, path)).to be_falsey