rspec-rails 2.14.2 → 3.9.1

data/lib/rspec/rails/example/view_example_group.rb

@@ -1,169 +1,202 @@
 require 'rspec/rails/view_assigns'
-
-module RSpec::Rails
-  module ViewExampleGroup
-    extend ActiveSupport::Concern
-    include RSpec::Rails::RailsExampleGroup
-    include ActionView::TestCase::Behavior
-    include RSpec::Rails::ViewAssigns
-    include RSpec::Rails::Matchers::RenderTemplate
-
-    module ClassMethods
-      def _default_helper
-        base = metadata[:example_group][:description].split('/')[0..-2].join('/')
-        (base.camelize + 'Helper').constantize if base
-      rescue NameError
-        nil
+require 'rspec/rails/view_spec_methods'
+require 'rspec/rails/view_path_builder'
+
+module RSpec
+  module Rails
+    # @api public
+    # Container class for view spec functionality.
+    module ViewExampleGroup
+      extend ActiveSupport::Concern
+      include RSpec::Rails::RailsExampleGroup
+      include ActionView::TestCase::Behavior
+      include RSpec::Rails::ViewAssigns
+      include RSpec::Rails::Matchers::RenderTemplate
+
+      # @private
+      module StubResolverCache
+        def self.resolver_for(hash)
+          @resolvers ||= {}
+          @resolvers[hash] ||= ActionView::FixtureResolver.new(hash)
+        end
       end
 
-      def _default_helpers
-        helpers = [_default_helper].compact
-        helpers << ApplicationHelper if Object.const_defined?('ApplicationHelper')
-        helpers
-      end
-    end
+      # @private
+      module ClassMethods
+        def _default_helper
+          base = metadata[:description].split('/')[0..-2].join('/')
+          (base.camelize + 'Helper').constantize unless base.to_s.empty?
+        rescue NameError
+          nil
+        end
 
-    module ExampleMethods
-      # @overload render
-      # @overload render({:partial => path_to_file})
-      # @overload render({:partial => path_to_file}, {... locals ...})
-      # @overload render({:partial => path_to_file}, {... locals ...}) do ... end
-      #
-      # Delegates to ActionView::Base#render, so see documentation on that
-      # for more info.
-      #
-      # The only addition is that you can call render with no arguments, and RSpec
-      # will pass the top level description to render:
-      #
-      #     describe "widgets/new.html.erb" do
-      #       it "shows all the widgets" do
-      #         render # => view.render(:file => "widgets/new.html.erb")
-      #         # ...
-      #       end
-      #     end
-      def render(options={}, local_assigns={}, &block)
-        options = _default_render_options if Hash === options and options.empty?
-        super(options, local_assigns, &block)
+        def _default_helpers
+          helpers = [_default_helper].compact
+          helpers << ApplicationHelper if Object.const_defined?('ApplicationHelper')
+          helpers
+        end
       end
 
-      # The instance of `ActionView::Base` that is used to render the template.
-      # Use this to stub methods _before_ calling `render`.
-      #
-      #     describe "widgets/new.html.erb" do
-      #       it "shows all the widgets" do
-      #         view.stub(:foo) { "foo" }
-      #         render
-      #         # ...
-      #       end
-      #     end
-      def view
-        _view
-      end
+      # DSL exposed to view specs.
+      module ExampleMethods
+        extend ActiveSupport::Concern
 
-      # Simulates the presence of a template on the file system by adding a
-      # Rails' FixtureResolver to the front of the view_paths list. Designed to
-      # help isolate view examples from partials rendered by the view template
-      # that is the subject of the example.
-      #
-      # @example
-      #
-      #     stub_template("widgets/_widget.html.erb" => "This content.")
-      def stub_template(hash)
-        view.view_paths.unshift(ActionView::FixtureResolver.new(hash))
-      end
+        included do
+          include ::Rails.application.routes.url_helpers
 
-      # Provides access to the params hash that will be available within the
-      # view:
-      #
-      #     params[:foo] = 'bar'
-      def params
-        controller.params
-      end
+          if ::Rails.application.routes.respond_to?(:mounted_helpers)
+            include ::Rails.application.routes.mounted_helpers
+          end
+        end
 
-      # @deprecated Use `view` instead.
-      def template
-        RSpec.deprecate("template", :replacement => "view")
-        view
-      end
+        # @overload render
+        # @overload render({:partial => path_to_file})
+        # @overload render({:partial => path_to_file}, {... locals ...})
+        # @overload render({:partial => path_to_file}, {... locals ...}) do ... end
+        #
+        # Delegates to ActionView::Base#render, so see documentation on that
+        # for more info.
+        #
+        # The only addition is that you can call render with no arguments, and
+        # RSpec will pass the top level description to render:
+        #
+        #     describe "widgets/new.html.erb" do
+        #       it "shows all the widgets" do
+        #         render # => view.render(:file => "widgets/new.html.erb")
+        #         # ...
+        #       end
+        #     end
+        def render(options = {}, local_assigns = {}, &block)
+          options = _default_render_options if Hash === options && options.empty?
+          super(options, local_assigns, &block)
+        end
 
-      # @deprecated Use `rendered` instead.
-      def response
-        # `assert_template` expects `response` to implement a #body method
-        # like an `ActionDispatch::Response` does to force the view to render.
-        # For backwards compatibility, we use #response as an alias for
-        # #rendered, but it needs to implement #body to avoid `assert_template`
-        # raising a `NoMethodError`.
-        unless rendered.respond_to?(:body)
-          def rendered.body; self; end;
+        # The instance of `ActionView::Base` that is used to render the template.
+        # Use this to stub methods _before_ calling `render`.
+        #
+        #     describe "widgets/new.html.erb" do
+        #       it "shows all the widgets" do
+        #         view.stub(:foo) { "foo" }
+        #         render
+        #         # ...
+        #       end
+        #     end
+        def view
+          _view
         end
 
-        rendered
-      end
+        # Simulates the presence of a template on the file system by adding a
+        # Rails' FixtureResolver to the front of the view_paths list. Designed to
+        # help isolate view examples from partials rendered by the view template
+        # that is the subject of the example.
+        #
+        #     stub_template("widgets/_widget.html.erb" => "This content.")
+        def stub_template(hash)
+          view.view_paths.unshift(StubResolverCache.resolver_for(hash))
+        end
 
-    private
+        # Provides access to the params hash that will be available within the
+        # view.
+        #
+        #     params[:foo] = 'bar'
+        def params
+          controller.params
+        end
 
-      def _default_file_to_render
-        example.example_group.top_level_description
-      end
+        # @deprecated Use `view` instead.
+        def template
+          RSpec.deprecate("template", :replacement => "view")
+          view
+        end
 
-      def _default_render_options
-        if ::Rails::VERSION::STRING >= '3.2'
-          # pluck the handler, format, and locale out of, eg, posts/index.de.html.haml
-          template, *components   = _default_file_to_render.split('.')
-          handler, format, locale = *components.reverse
+        # @deprecated Use `rendered` instead.
+        def response
+          # `assert_template` expects `response` to implement a #body method
+          # like an `ActionDispatch::Response` does to force the view to
+          # render. For backwards compatibility, we use #response as an alias
+          # for #rendered, but it needs to implement #body to avoid
+          # `assert_template` raising a `NoMethodError`.
+          unless rendered.respond_to?(:body)
+            def rendered.body
+              self
+            end
+          end
+
+          rendered
+        end
 
-          render_options = {:template => template}
-          render_options[:handlers] = [handler] if handler
-          render_options[:formats] = [format] if format
-          render_options[:locales] = [locale] if locale
+      private
 
-          render_options
-        else
-          {:template => _default_file_to_render}
+        def _default_render_options
+          if ::Rails::VERSION::STRING >= '3.2'
+            # pluck the handler, format, and locale out of, eg, posts/index.de.html.haml
+            template, *components   = _default_file_to_render.split('.')
+            handler, format, locale = *components.reverse
+
+            render_options = { :template => template }
+            render_options[:handlers] = [handler] if handler
+            render_options[:formats] = [format.to_sym] if format
+            render_options[:locales] = [locale] if locale
+
+            render_options
+          else
+            { :template => _default_file_to_render }
+          end
         end
-      end
 
-      def _path_parts
-        _default_file_to_render.split("/")
-      end
+        def _path_parts
+          _default_file_to_render.split("/")
+        end
 
-      def _controller_path
-        _path_parts[0..-2].join("/")
-      end
+        def _controller_path
+          _path_parts[0..-2].join("/")
+        end
 
-      def _inferred_action
-        _path_parts.last.split(".").first
-      end
+        def _inferred_action
+          _path_parts.last.split(".").first
+        end
 
-      def _include_controller_helpers
-        helpers = controller._helpers
-        view.singleton_class.class_eval do
-          include helpers unless included_modules.include?(helpers)
+        def _include_controller_helpers
+          helpers = controller._helpers
+          view.singleton_class.class_exec do
+            include helpers unless included_modules.include?(helpers)
+          end
         end
       end
-    end
 
-    included do
-      include ExampleMethods
+      included do
+        include ExampleMethods
+
+        helper(*_default_helpers)
+
+        before do
+          _include_controller_helpers
+          if view.lookup_context.respond_to?(:prefixes)
+            # rails 3.1
+            view.lookup_context.prefixes << _controller_path
+          end
+
+          controller.controller_path = _controller_path
 
-      metadata[:type] = :view
-      helper(*_default_helpers)
+          path_params_to_merge = {}
+          path_params_to_merge[:controller] = _controller_path
+          path_params_to_merge[:action] = _inferred_action unless _inferred_action =~ /^_/
 
-      before do
-        _include_controller_helpers
-        if view.lookup_context.respond_to?(:prefixes)
-          # rails 3.1
-          view.lookup_context.prefixes << _controller_path
+          path_params = controller.request.path_parameters
+
+          controller.request.path_parameters = path_params.reverse_merge(path_params_to_merge)
+          controller.request.path = ViewPathBuilder.new(::Rails.application.routes).path_for(controller.request.path_parameters)
+          ViewSpecMethods.add_to(::ActionView::TestCase::TestController)
         end
 
-        # fixes bug with differing formats
-        view.lookup_context.view_paths.each(&:clear_cache)
+        after do
+          ViewSpecMethods.remove_from(::ActionView::TestCase::TestController)
+        end
 
-        controller.controller_path = _controller_path
-        controller.request.path_parameters[:controller] = _controller_path
-        controller.request.path_parameters[:action]     = _inferred_action unless _inferred_action =~ /^_/
+        let(:_default_file_to_render) do |example|
+          example.example_group.top_level_description
+        end
       end
     end
   end
 end
-

data/lib/rspec/rails/example.rb

@@ -6,37 +6,6 @@ require 'rspec/rails/example/view_example_group'
 require 'rspec/rails/example/mailer_example_group'
 require 'rspec/rails/example/routing_example_group'
 require 'rspec/rails/example/model_example_group'
+require 'rspec/rails/example/job_example_group'
 require 'rspec/rails/example/feature_example_group'
-
-RSpec::configure do |c|
-  def c.escaped_path(*parts)
-    Regexp.compile(parts.join('[\\\/]') + '[\\\/]')
-  end
-
-  c.include RSpec::Rails::ControllerExampleGroup, :type => :controller, :example_group => {
-    :file_path => c.escaped_path(%w[spec controllers])
-  }
-  c.include RSpec::Rails::HelperExampleGroup, :type => :helper, :example_group => {
-    :file_path => c.escaped_path(%w[spec helpers])
-  }
-  if defined?(RSpec::Rails::MailerExampleGroup)
-    c.include RSpec::Rails::MailerExampleGroup, :type => :mailer, :example_group => {
-      :file_path => c.escaped_path(%w[spec mailers])
-    }
-  end
-  c.include RSpec::Rails::ModelExampleGroup, :type => :model, :example_group => {
-    :file_path => c.escaped_path(%w[spec models])
-  }
-  c.include RSpec::Rails::RequestExampleGroup, :type => :request, :example_group => {
-    :file_path => c.escaped_path(%w[spec (requests|integration|api)])
-  }
-  c.include RSpec::Rails::RoutingExampleGroup, :type => :routing, :example_group => {
-    :file_path => c.escaped_path(%w[spec routing])
-  }
-  c.include RSpec::Rails::ViewExampleGroup, :type => :view, :example_group => {
-    :file_path => c.escaped_path(%w[spec views])
-  }
-  c.include RSpec::Rails::FeatureExampleGroup, :type => :feature, :example_group => {
-    :file_path => c.escaped_path(%w[spec features])
-  }
-end
+require 'rspec/rails/example/system_example_group'

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

@@ -14,4 +14,3 @@ RSpec.configure do |rspec|
     end
   end
 end
-

data/lib/rspec/rails/extensions.rb

@@ -1,2 +1 @@
-require 'rspec/rails/extensions/active_record/base'
 require 'rspec/rails/extensions/active_record/proxy'

data/lib/rspec/rails/feature_check.rb

@@ -0,0 +1,64 @@
+module RSpec
+  module Rails
+    # @private
+    # Disable some cops until https://github.com/bbatsov/rubocop/issues/1310
+    # rubocop:disable Style/IndentationConsistency
+    module FeatureCheck
+    # rubocop:disable Style/IndentationWidth
+    module_function
+      # rubocop:enable Style/IndentationWidth
+
+      def can_check_pending_migrations?
+        has_active_record_migration? &&
+          ::ActiveRecord::Migration.respond_to?(:check_pending!)
+      end
+
+      def can_maintain_test_schema?
+        has_active_record_migration? &&
+          ::ActiveRecord::Migration.respond_to?(:maintain_test_schema!)
+      end
+
+      def has_active_job?
+        defined?(::ActiveJob)
+      end
+
+      def has_active_record?
+        defined?(::ActiveRecord)
+      end
+
+      def has_active_record_migration?
+        has_active_record? && defined?(::ActiveRecord::Migration)
+      end
+
+      def has_action_mailer?
+        defined?(::ActionMailer)
+      end
+
+      def has_action_mailer_preview?
+        has_action_mailer? && defined?(::ActionMailer::Preview)
+      end
+
+      def has_action_mailer_show_preview?
+        has_action_mailer_preview? &&
+          ::ActionMailer::Base.respond_to?(:show_previews=)
+      end
+
+      def has_1_9_hash_syntax?
+        ::Rails::VERSION::STRING > '4.0'
+      end
+
+      def has_file_fixture?
+        ::Rails::VERSION::STRING > '5.0'
+      end
+
+      def type_metatag(type)
+        if has_1_9_hash_syntax?
+          "type: :#{type}"
+        else
+          ":type => :#{type}"
+        end
+      end
+    end
+    # rubocop:enable Style/IndentationConsistency
+  end
+end

data/lib/rspec/rails/file_fixture_support.rb

@@ -0,0 +1,17 @@
+if ::Rails::VERSION::STRING > '5'
+  require 'active_support/testing/file_fixtures'
+
+  module RSpec
+    module Rails
+      # @private
+      module FileFixtureSupport
+        extend ActiveSupport::Concern
+        include ActiveSupport::Testing::FileFixtures
+
+        included do
+          self.file_fixture_path = RSpec.configuration.file_fixture_path
+        end
+      end
+    end
+  end
+end

data/lib/rspec/rails/fixture_file_upload_support.rb

@@ -0,0 +1,40 @@
+module RSpec
+  module Rails
+    # @private
+    module FixtureFileUploadSupport
+      delegate :fixture_file_upload, :to => :rails_fixture_file_wrapper
+
+    private
+
+      def rails_fixture_file_wrapper
+        RailsFixtureFileWrapper.fixture_path = nil
+        resolved_fixture_path = (fixture_path || RSpec.configuration.fixture_path || '').to_s
+        RailsFixtureFileWrapper.fixture_path = File.join(resolved_fixture_path, '') unless resolved_fixture_path.strip.empty?
+        RailsFixtureFileWrapper.instance
+      end
+
+      class RailsFixtureFileWrapper
+        include ActionDispatch::TestProcess if defined?(ActionDispatch::TestProcess)
+
+        class << self
+          attr_reader :fixture_path
+
+          # Get instance of wrapper
+          def instance
+            @instance ||= new
+          end
+
+          # Override fixture_path set
+          # to support Rails 3.0->3.1 using ActionController::TestCase class to resolve fixture_path
+          # see https://apidock.com/rails/v3.0.0/ActionDispatch/TestProcess/fixture_file_upload
+          def fixture_path=(value)
+            if ActionController::TestCase.respond_to?(:fixture_path)
+              ActionController::TestCase.fixture_path = value
+            end
+            @fixture_path = value
+          end
+        end
+      end
+    end
+  end
+end

data/lib/rspec/rails/fixture_support.rb

@@ -1,7 +1,8 @@
 module RSpec
   module Rails
-    if defined?(ActiveRecord::TestFixtures)
-      module FixtureSupport
+    # @private
+    module FixtureSupport
+      if defined?(ActiveRecord::TestFixtures)
         extend ActiveSupport::Concern
         include RSpec::Rails::SetupAndTeardownAdapter
         include RSpec::Rails::MinitestLifecycleAdapter if ::ActiveRecord::VERSION::STRING > '4'
@@ -9,31 +10,49 @@ module RSpec
         include ActiveRecord::TestFixtures
 
         included do
-          # TODO (DC 2011-06-25) this is necessary because fixture_file_upload
+          # TODO: (DC 2011-06-25) this is necessary because fixture_file_upload
           # accesses fixture_path directly on ActiveSupport::TestCase. This is
           # fixed in rails by https://github.com/rails/rails/pull/1861, which
           # should be part of the 3.1 release, at which point we can include
           # these lines for rails < 3.1.
-          ActiveSupport::TestCase.class_eval do
+          ActiveSupport::TestCase.class_exec do
             include ActiveRecord::TestFixtures
             self.fixture_path = RSpec.configuration.fixture_path
           end
           # /TODO
 
           self.fixture_path = RSpec.configuration.fixture_path
-          self.use_transactional_fixtures = RSpec.configuration.use_transactional_fixtures
+          if ::Rails::VERSION::STRING > '5'
+            self.use_transactional_tests = RSpec.configuration.use_transactional_fixtures
+          else
+            self.use_transactional_fixtures = RSpec.configuration.use_transactional_fixtures
+          end
           self.use_instantiated_fixtures  = RSpec.configuration.use_instantiated_fixtures
+
+          def self.fixtures(*args)
+            orig_methods = private_instance_methods
+            super.tap do
+              new_methods = private_instance_methods - orig_methods
+              new_methods.each do |method_name|
+                proxy_method_warning_if_called_in_before_context_scope(method_name)
+              end
+            end
+          end
+
+          def self.proxy_method_warning_if_called_in_before_context_scope(method_name)
+            orig_implementation = instance_method(method_name)
+            define_method(method_name) do |*args, &blk|
+              if inspect.include?("before(:context)")
+                RSpec.warn_with("Calling fixture method in before :context ")
+              else
+                orig_implementation.bind(self).call(*args, &blk)
+              end
+            end
+          end
+
           fixtures RSpec.configuration.global_fixtures if RSpec.configuration.global_fixtures
         end
       end
-
-      RSpec.configure do |c|
-        c.include RSpec::Rails::FixtureSupport
-        c.add_setting :use_transactional_fixtures, :alias_with => :use_transactional_examples
-        c.add_setting :use_instantiated_fixtures
-        c.add_setting :global_fixtures
-        c.add_setting :fixture_path
-      end
     end
   end
 end