Split spec_helper and rails_helper

tvdeyen · tvdeyen · commit ad1886d3d12c · 2019-08-23T10:08:04.000+02:00
diff --git a/spec/controllers/admin/users_controller_spec.rb b/spec/controllers/admin/users_controller_spec.rb
@@ -1,4 +1,4 @@
-require 'spec_helper'
+require 'rails_helper'
 
 module Alchemy
   describe Admin::UsersController do
diff --git a/spec/controllers/base_controller_spec.rb b/spec/controllers/base_controller_spec.rb
@@ -1,4 +1,4 @@
-require 'spec_helper'
+require 'rails_helper'
 
 describe Alchemy::BaseController do
 
diff --git a/spec/controllers/user_sessions_controller_spec.rb b/spec/controllers/user_sessions_controller_spec.rb
@@ -1,4 +1,4 @@
-require 'spec_helper'
+require 'rails_helper'
 
 describe Alchemy::UserSessionsController do
   routes { Alchemy::Engine.routes }
diff --git a/spec/features/admin/users_feature_spec.rb b/spec/features/admin/users_feature_spec.rb
@@ -1,4 +1,4 @@
-require 'spec_helper'
+require 'rails_helper'
 
 describe "Admin users feature." do
 
diff --git a/spec/features/login_feature_spec.rb b/spec/features/login_feature_spec.rb
@@ -1,4 +1,4 @@
-require 'spec_helper'
+require 'rails_helper'
 
 describe "Login: " do
   context "If user is present" do
diff --git a/spec/features/password_reset_feature_spec.rb b/spec/features/password_reset_feature_spec.rb
@@ -1,4 +1,4 @@
-require 'spec_helper'
+require 'rails_helper'
 
 describe "Password reset feature." do
   let(:user) { create(:alchemy_admin_user) }
diff --git a/spec/features/security_feature_spec.rb b/spec/features/security_feature_spec.rb
@@ -1,4 +1,4 @@
-require 'spec_helper'
+require 'rails_helper'
 
 describe "Security: " do
   context "If user is present" do
diff --git a/spec/libraries/ability_spec.rb b/spec/libraries/ability_spec.rb
@@ -1,4 +1,4 @@
-require 'spec_helper'
+require 'rails_helper'
 require 'cancan/matchers'
 
 describe Alchemy::Permissions do
diff --git a/spec/mailers/notifications_spec.rb b/spec/mailers/notifications_spec.rb
@@ -1,4 +1,4 @@
-require 'spec_helper'
+require 'rails_helper'
 
 module Alchemy
   describe Notifications do
diff --git a/spec/models/user_spec.rb b/spec/models/user_spec.rb
@@ -1,4 +1,4 @@
-require 'spec_helper'
+require 'rails_helper'
 require 'alchemy/test_support/factories/page_factory'
 
 module Alchemy
diff --git a/spec/rails_helper.rb b/spec/rails_helper.rb
@@ -0,0 +1,42 @@
+# frozen_string_literal: true
+
+require_relative './spec_helper'
+
+# Configure Rails Environment
+ENV["RAILS_ENV"] ||= "test"
+
+require File.expand_path("../dummy/config/environment.rb", __FILE__)
+require "rspec/rails"
+require 'rspec/active_model/mocks'
+require "capybara/rails"
+require 'factory_bot'
+require 'alchemy/seeder'
+require 'alchemy/test_support/controller_requests'
+require 'alchemy/test_support/integration_helpers'
+require 'alchemy/devise/test_support/factories'
+
+ActionMailer::Base.delivery_method = :test
+ActionMailer::Base.perform_deliveries = true
+ActionMailer::Base.default_url_options[:host] = "test.com"
+Capybara.default_driver = :rack_test
+Capybara.default_selector = :css
+Capybara.ignore_hidden_elements = false
+ActiveJob::Base.queue_adapter = :test
+
+ActiveSupport::Deprecation.silenced = true
+
+RSpec.configure do |config|
+  config.infer_spec_type_from_file_location!
+  config.use_transactional_fixtures = true
+  config.include Devise::TestHelpers, :type => :controller
+  config.include Alchemy::TestSupport::ControllerRequests, :type => :controller
+  config.include Alchemy::Engine.routes.url_helpers
+  config.include FactoryBot::Syntax::Methods
+  config.include ActiveJob::TestHelper
+  [:controller, :feature, :request].each do |type|
+    config.include Alchemy::TestSupport::IntegrationHelpers, type: type
+  end
+  config.before(:suite) do
+    Alchemy::Seeder.seed!
+  end
+end
diff --git a/spec/requests/admin/users_controller_request_spec.rb b/spec/requests/admin/users_controller_request_spec.rb
@@ -1,4 +1,4 @@
-require 'spec_helper'
+require 'rails_helper'
 
 RSpec.describe Alchemy::Admin::UsersController, type: :request do
   before do
diff --git a/spec/routing/password_routing_spec.rb b/spec/routing/password_routing_spec.rb
@@ -1,4 +1,4 @@
-require 'spec_helper'
+require 'rails_helper'
 
 describe "Password Routing" do
   routes { Alchemy::Engine.routes }
diff --git a/spec/routing/session_routing_spec.rb b/spec/routing/session_routing_spec.rb
@@ -1,4 +1,4 @@
-require 'spec_helper'
+require 'rails_helper'
 
 describe "Session Routing" do
   routes { Alchemy::Engine.routes }
diff --git a/spec/routing/user_routing_spec.rb b/spec/routing/user_routing_spec.rb
@@ -1,4 +1,4 @@
-require 'spec_helper'
+require 'rails_helper'
 
 describe "User Routing" do
   routes { Alchemy::Engine.routes }
diff --git a/spec/spec_helper.rb b/spec/spec_helper.rb
@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'simplecov'
 if ENV['TRAVIS']
   require "codeclimate-test-reporter"
@@ -7,42 +9,10 @@
   add_filter '/lib/generators'
 end
 
-# Configure Rails Environment
-ENV["RAILS_ENV"] ||= "test"
-
-require File.expand_path("../dummy/config/environment.rb", __FILE__)
-require "rspec/rails"
-require 'rspec/active_model/mocks'
-require "capybara/rails"
-require 'factory_bot'
-require 'alchemy/seeder'
-require 'alchemy/test_support/controller_requests'
-require 'alchemy/test_support/integration_helpers'
-require 'alchemy/devise/test_support/factories'
-
-ActionMailer::Base.delivery_method = :test
-ActionMailer::Base.perform_deliveries = true
-ActionMailer::Base.default_url_options[:host] = "test.com"
-Capybara.default_driver = :rack_test
-Capybara.default_selector = :css
-Capybara.ignore_hidden_elements = false
-ActiveJob::Base.queue_adapter = :test
-
-ActiveSupport::Deprecation.silenced = true
+require 'rspec/core'
 
 RSpec.configure do |config|
-  config.infer_spec_type_from_file_location!
   config.raise_errors_for_deprecations!
-  config.use_transactional_fixtures = true
-  config.include Devise::TestHelpers, :type => :controller
-  config.include Alchemy::TestSupport::ControllerRequests, :type => :controller
-  config.include Alchemy::Engine.routes.url_helpers
-  config.include FactoryBot::Syntax::Methods
-  config.include ActiveJob::TestHelper
-  [:controller, :feature, :request].each do |type|
-    config.include Alchemy::TestSupport::IntegrationHelpers, type: type
-  end
-  config.before(:suite) do
-    Alchemy::Seeder.seed!
-  end
+  config.run_all_when_everything_filtered = true
+  config.filter_run :focus
 end

Original file line number	Diff line number	Diff line change
`@@ -1,4 +1,4 @@`
`1`		`-require 'spec_helper'`
	`1`	`+require 'rails_helper'`
`2`	`2`
`3`	`3`	`module Alchemy`
`4`	`4`	`describe Admin::UsersController do`