fixup! Remove hidden Rails < 5 support code

Author: pirj

example_app_generator/generate_action_mailer_specs.rb

@@ -1,12 +1,6 @@
 require 'active_support'
 require 'active_support/core_ext/module'
 
-# We need to copy this method from Thor for older Rails versions
-def comment_lines(path, flag, *args)
-  flag = flag.respond_to?(:source) ? flag.source : flag
-  gsub_file(path, /^(\s*)([^#|\n]*#{flag})/, '\1# \2', *args)
-end
-
 using_source_path(File.expand_path('..', __FILE__)) do
   # Comment out the default mailer stuff
   comment_lines 'config/environments/development.rb', /action_mailer/

example_app_generator/spec/support/default_preview_path

@@ -47,7 +47,7 @@ require_file_stub 'config/environment' do
       end
     end
 
-    I18n.enforce_available_locales = true if I18n.respond_to?(:enforce_available_locales)
+    I18n.enforce_available_locales = true
   end
 
   # Initialize the Rails application.

lib/rspec/rails/adapters.rb

@@ -146,12 +146,11 @@ module ClassMethods
         # examples without exposing non-assertion methods in Test::Unit or
         # Minitest.
         def assertion_method_names
-          methods = ::RSpec::Rails::Assertions.
+          ::RSpec::Rails::Assertions.
             public_instance_methods.
             select do |m|
               m.to_s =~ /^(assert|flunk|refute)/
             end
-          methods + test_unit_specific_methods
         end
 
         def define_assertion_delegators
@@ -161,12 +160,6 @@ def define_assertion_delegators
             end
           end
         end
-
-        # Starting on Rails 4, Minitest is the default testing framework so no
-        # need to add TestUnit specific methods.
-        def test_unit_specific_methods
-          []
-        end
       end
 
       class AssertionDelegator

lib/rspec/rails/example/feature_example_group.rb

@@ -15,9 +15,7 @@ module FeatureExampleGroup
           include app.routes.url_helpers     if app.routes.respond_to?(:url_helpers)
           include app.routes.mounted_helpers if app.routes.respond_to?(:mounted_helpers)
 
-          if respond_to?(:default_url_options)
-            default_url_options[:host] ||= ::RSpec::Rails::FeatureExampleGroup::DEFAULT_HOST
-          end
+          default_url_options[:host] ||= ::RSpec::Rails::FeatureExampleGroup::DEFAULT_HOST
         end
       end
 

lib/rspec/rails/example/view_example_group.rb

@@ -43,10 +43,7 @@ module ExampleMethods
 
         included do
           include ::Rails.application.routes.url_helpers
-
-          if ::Rails.application.routes.respond_to?(:mounted_helpers)
-            include ::Rails.application.routes.mounted_helpers
-          end
+          include ::Rails.application.routes.mounted_helpers
         end
 
         # @overload render
@@ -188,10 +185,7 @@ def _include_controller_helpers
 
         before do
           _include_controller_helpers
-          if view.lookup_context.respond_to?(:prefixes)
-            # rails 3.1
-            view.lookup_context.prefixes << _controller_path
-          end
+          view.lookup_context.prefixes << _controller_path
 
           controller.controller_path = _controller_path
 

lib/rspec/rails/extensions/active_record/proxy.rb

@@ -2,15 +2,7 @@
   # Delay this in order to give users a chance to configure `expect_with`...
   rspec.before(:suite) do
     if defined?(RSpec::Matchers) && RSpec::Matchers.configuration.syntax.include?(:should) && defined?(ActiveRecord::Associations)
-      # In Rails 3.0, it was AssociationProxy.
-      # In 3.1+, it's CollectionProxy.
-      const_name = [:CollectionProxy, :AssociationProxy].find do |const|
-        ActiveRecord::Associations.const_defined?(const)
-      end
-
-      proxy_class = ActiveRecord::Associations.const_get(const_name)
-
-      RSpec::Matchers.configuration.add_should_and_should_not_to proxy_class
+      RSpec::Matchers.configuration.add_should_and_should_not_to ActiveRecord::Associations::CollectionProxy
     end
   end
 end

rspec-rails.gemspec

@@ -11,7 +11,7 @@ Gem::Specification.new do |s|
   s.email       = "[email protected]"
   s.homepage    = "https://github.com/rspec/rspec-rails"
   s.summary     = "RSpec for Rails"
-  s.description = "rspec-rails is a testing framework for Rails 4+."
+  s.description = "rspec-rails is a testing framework for Rails 5+."
 
   s.metadata = {
     'bug_tracker_uri'   => 'https://github.com/rspec/rspec-rails/issues',

spec/generators/rspec/scaffold/scaffold_generator_spec.rb

@@ -126,7 +126,7 @@
           it { is_expected.to contain(/assert_select "input\[name=\?\]", "post\[title\]/) }
         end
       end
-    when Rails.version.to_f >= 4.0
+    when Rails.version.to_f >= 5.0
       describe 'with reference attribute' do
         before { run_generator %w(posts title:string author:references) }
         describe 'edit' do

spec/spec_helper.rb

@@ -11,7 +11,7 @@ class Application < ::Rails::Application
     end
   end
 end
-I18n.enforce_available_locales = true if I18n.respond_to?(:enforce_available_locales)
+I18n.enforce_available_locales = true
 
 require 'rspec/support/spec'
 require 'rspec/rails'