From bba41177b48a193cd80d74eebee96dddef7cdcef Mon Sep 17 00:00:00 2001 From: Maikel Linke Date: Fri, 24 Feb 2023 15:04:03 +1100 Subject: [PATCH] Remove old test setup code of feature specs The `js: true` tag was only used in feature specs. Some system specs still contain that tag but they don't require this file. So it's dead code. --- spec/spec_helper.rb | 7 ------- 1 file changed, 7 deletions(-) diff --git a/spec/spec_helper.rb b/spec/spec_helper.rb index a51e5d0361..e6813ee07b 100644 --- a/spec/spec_helper.rb +++ b/spec/spec_helper.rb @@ -15,19 +15,12 @@ RSpec.configure do |config| DatabaseCleaner.clean_with :deletion, except: ['spree_countries', 'spree_states'] } config.before(:each) { DatabaseCleaner.strategy = :transaction } - config.before(:each, js: true) { - DatabaseCleaner.strategy = :deletion, { except: ['spree_countries', 'spree_states'] } - } config.before(:each, concurrency: true) { DatabaseCleaner.strategy = :deletion, { except: ['spree_countries', 'spree_states'] } } config.before(:each) { DatabaseCleaner.start } config.after(:each) { DatabaseCleaner.clean } - config.after(:each, js: true) do - Capybara.reset_sessions! - end - # Precompile Webpacker assets (once) when starting the suite. The default setup can result # in the assets getting compiled many times throughout the build, slowing it down. config.before :suite do