rubocop-rspec 2.29.2 → 3.0.5

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: rubocop-rspec
 version: !ruby/object:Gem::Version
-  version: 2.29.2
+  version: 3.0.5
 platform: ruby
 authors:
 - John Backus
@@ -10,7 +10,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2024-05-02 00:00:00.000000000 Z
+date: 2024-09-07 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rubocop
@@ -18,56 +18,14 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '1.40'
+        version: '1.61'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '1.40'
-- !ruby/object:Gem::Dependency
-  name: rubocop-capybara
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: '2.17'
-  type: :runtime
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: '2.17'
-- !ruby/object:Gem::Dependency
-  name: rubocop-factory_bot
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: '2.22'
-  type: :runtime
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: '2.22'
-- !ruby/object:Gem::Dependency
-  name: rubocop-rspec_rails
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: '2.28'
-  type: :runtime
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: '2.28'
+        version: '1.61'
 description: |
   Code style checking for RSpec files.
   A plugin for the RuboCop code style enforcing & linting tool.
@@ -99,14 +57,6 @@ files:
 - lib/rubocop/cop/rspec/be_eql.rb
 - lib/rubocop/cop/rspec/be_nil.rb
 - lib/rubocop/cop/rspec/before_after_all.rb
-- lib/rubocop/cop/rspec/capybara/current_path_expectation.rb
-- lib/rubocop/cop/rspec/capybara/feature_methods.rb
-- lib/rubocop/cop/rspec/capybara/match_style.rb
-- lib/rubocop/cop/rspec/capybara/negation_matcher.rb
-- lib/rubocop/cop/rspec/capybara/specific_actions.rb
-- lib/rubocop/cop/rspec/capybara/specific_finders.rb
-- lib/rubocop/cop/rspec/capybara/specific_matcher.rb
-- lib/rubocop/cop/rspec/capybara/visibility_matcher.rb
 - lib/rubocop/cop/rspec/change_by_zero.rb
 - lib/rubocop/cop/rspec/class_check.rb
 - lib/rubocop/cop/rspec/contain_exactly.rb
@@ -136,14 +86,8 @@ files:
 - lib/rubocop/cop/rspec/expect_actual.rb
 - lib/rubocop/cop/rspec/expect_change.rb
 - lib/rubocop/cop/rspec/expect_in_hook.rb
+- lib/rubocop/cop/rspec/expect_in_let.rb
 - lib/rubocop/cop/rspec/expect_output.rb
-- lib/rubocop/cop/rspec/factory_bot/attribute_defined_statically.rb
-- lib/rubocop/cop/rspec/factory_bot/consistent_parentheses_style.rb
-- lib/rubocop/cop/rspec/factory_bot/create_list.rb
-- lib/rubocop/cop/rspec/factory_bot/factory_class_name.rb
-- lib/rubocop/cop/rspec/factory_bot/factory_name_style.rb
-- lib/rubocop/cop/rspec/factory_bot/syntax_methods.rb
-- lib/rubocop/cop/rspec/file_path.rb
 - lib/rubocop/cop/rspec/focus.rb
 - lib/rubocop/cop/rspec/hook_argument.rb
 - lib/rubocop/cop/rspec/hooks_before_examples.rb
@@ -167,6 +111,7 @@ files:
 - lib/rubocop/cop/rspec/message_spies.rb
 - lib/rubocop/cop/rspec/metadata_style.rb
 - lib/rubocop/cop/rspec/missing_example_group_argument.rb
+- lib/rubocop/cop/rspec/missing_expectation_target_method.rb
 - lib/rubocop/cop/rspec/mixin/comments_help.rb
 - lib/rubocop/cop/rspec/mixin/empty_line_separation.rb
 - lib/rubocop/cop/rspec/mixin/file_help.rb
@@ -190,13 +135,6 @@ files:
 - lib/rubocop/cop/rspec/pending.rb
 - lib/rubocop/cop/rspec/pending_without_reason.rb
 - lib/rubocop/cop/rspec/predicate_matcher.rb
-- lib/rubocop/cop/rspec/rails/avoid_setup_hook.rb
-- lib/rubocop/cop/rspec/rails/have_http_status.rb
-- lib/rubocop/cop/rspec/rails/http_status.rb
-- lib/rubocop/cop/rspec/rails/inferred_spec_type.rb
-- lib/rubocop/cop/rspec/rails/minitest_assertions.rb
-- lib/rubocop/cop/rspec/rails/negation_be_valid.rb
-- lib/rubocop/cop/rspec/rails/travel_around.rb
 - lib/rubocop/cop/rspec/receive_counts.rb
 - lib/rubocop/cop/rspec/receive_messages.rb
 - lib/rubocop/cop/rspec/receive_never.rb
@@ -235,6 +173,7 @@ files:
 - lib/rubocop/rspec/align_let_brace.rb
 - lib/rubocop/rspec/concept.rb
 - lib/rubocop/rspec/config_formatter.rb
+- lib/rubocop/rspec/cop/generator.rb
 - lib/rubocop/rspec/corrector/move_node.rb
 - lib/rubocop/rspec/description_extractor.rb
 - lib/rubocop/rspec/example.rb
@@ -242,7 +181,6 @@ files:
 - lib/rubocop/rspec/hook.rb
 - lib/rubocop/rspec/inject.rb
 - lib/rubocop/rspec/language.rb
-- lib/rubocop/rspec/language/node_pattern.rb
 - lib/rubocop/rspec/node.rb
 - lib/rubocop/rspec/shared_contexts/default_rspec_language_config_context.rb
 - lib/rubocop/rspec/version.rb
@@ -269,7 +207,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.5.9
+rubygems_version: 3.5.16
 signing_key: 
 specification_version: 4
 summary: Code style checking for RSpec files

data/lib/rubocop/cop/rspec/capybara/current_path_expectation.rb

@@ -1,39 +0,0 @@
-# frozen_string_literal: true
-
-module RuboCop
-  module Cop
-    module RSpec
-      module Capybara
-        # @!parse
-        #   # Checks that no expectations are set on Capybara's `current_path`.
-        #   #
-        #   # The
-        #   # https://www.rubydoc.info/github/teamcapybara/capybara/master/Capybara/RSpecMatchers#have_current_path-instance_method[`have_current_path` matcher]
-        #   # should be used on `page` to set expectations on Capybara's
-        #   # current path, since it uses
-        #   # https://github.com/teamcapybara/capybara/blob/master/README.md#asynchronous-javascript-ajax-and-friends[Capybara's waiting functionality]
-        #   # which ensures that preceding actions (like `click_link`) have
-        #   # completed.
-        #   #
-        #   # This cop does not support autocorrection in some cases.
-        #   #
-        #   # @example
-        #   #   # bad
-        #   #   expect(current_path).to eq('/callback')
-        #   #
-        #   #   # good
-        #   #   expect(page).to have_current_path('/callback')
-        #   #
-        #   #   # bad (does not support autocorrection)
-        #   #   expect(page.current_path).to match(variable)
-        #   #
-        #   #   # good
-        #   #   expect(page).to have_current_path('/callback')
-        #   #
-        #   class CurrentPathExpectation < ::RuboCop::Cop::Base; end
-        CurrentPathExpectation =
-          ::RuboCop::Cop::Capybara::CurrentPathExpectation
-      end
-    end
-  end
-end

data/lib/rubocop/cop/rspec/capybara/feature_methods.rb

@@ -1,104 +0,0 @@
-# frozen_string_literal: true
-
-module RuboCop
-  module Cop
-    module RSpec
-      module Capybara
-        # Checks for consistent method usage in feature specs.
-        #
-        # By default, the cop disables all Capybara-specific methods that have
-        # the same native RSpec method (e.g. are just aliases). Some teams
-        # however may prefer using some of the Capybara methods (like `feature`)
-        # to make it obvious that the test uses Capybara, while still disable
-        # the rest of the methods, like `given` (alias for `let`), `background`
-        # (alias for `before`), etc. You can configure which of the methods to
-        # be enabled by using the EnabledMethods configuration option.
-        #
-        # @example
-        #   # bad
-        #   feature 'User logs in' do
-        #     given(:user) { User.new }
-        #
-        #     background do
-        #       visit new_session_path
-        #     end
-        #
-        #     scenario 'with OAuth' do
-        #       # ...
-        #     end
-        #   end
-        #
-        #   # good
-        #   describe 'User logs in' do
-        #     let(:user) { User.new }
-        #
-        #     before do
-        #       visit new_session_path
-        #     end
-        #
-        #     it 'with OAuth' do
-        #       # ...
-        #     end
-        #   end
-        #
-        class FeatureMethods < Base
-          extend AutoCorrector
-          include InsideExampleGroup
-
-          MSG = 'Use `%<replacement>s` instead of `%<method>s`.'
-
-          # https://github.com/teamcapybara/capybara/blob/e283c1aeaa72441f5403963577e16333bf111a81/lib/capybara/rspec/features.rb#L31-L36
-          MAP = {
-            background: :before,
-            scenario:   :it,
-            xscenario:  :xit,
-            given:      :let,
-            given!:     :let!,
-            feature:    :describe
-          }.freeze
-
-          # @!method capybara_speak(node)
-          def_node_matcher :capybara_speak, <<~PATTERN
-            {#{MAP.keys.map(&:inspect).join(' ')}}
-          PATTERN
-
-          # @!method feature_method(node)
-          def_node_matcher :feature_method, <<~PATTERN
-            (block
-              $(send #rspec? $#capybara_speak ...)
-            ...)
-          PATTERN
-
-          def on_block(node) # rubocop:disable InternalAffairs/NumblockHandler
-            return unless inside_example_group?(node)
-
-            feature_method(node) do |send_node, match|
-              next if enabled?(match)
-
-              add_offense(send_node.loc.selector) do |corrector|
-                corrector.replace(send_node.loc.selector, MAP[match].to_s)
-              end
-            end
-          end
-
-          def message(range)
-            name = range.source.to_sym
-            format(MSG, method: name, replacement: MAP[name])
-          end
-
-          private
-
-          def enabled?(method_name)
-            enabled_methods.include?(method_name)
-          end
-
-          def enabled_methods
-            cop_config
-              .fetch('EnabledMethods', [])
-              .map(&:to_sym)
-          end
-        end
-      end
-    end
-  end
-end

data/lib/rubocop/cop/rspec/capybara/match_style.rb

@@ -1,38 +0,0 @@
-# frozen_string_literal: true
-
-module RuboCop
-  module Cop
-    module RSpec
-      module Capybara
-        # @!parse
-        #   # Checks for usage of deprecated style methods.
-        #   #
-        #   # @example when using `assert_style`
-        #   #   # bad
-        #   #   page.find(:css, '#first').assert_style(display: 'block')
-        #   #
-        #   #   # good
-        #   #   page.find(:css, '#first').assert_matches_style(display: 'block')
-        #   #
-        #   # @example when using `has_style?`
-        #   #   # bad
-        #   #   expect(page.find(:css, 'first')
-        #   #     .has_style?(display: 'block')).to be true
-        #   #
-        #   #   # good
-        #   #   expect(page.find(:css, 'first')
-        #   #     .matches_style?(display: 'block')).to be true
-        #   #
-        #   # @example when using `have_style`
-        #   #   # bad
-        #   #   expect(page).to have_style(display: 'block')
-        #   #
-        #   #   # good
-        #   #   expect(page).to match_style(display: 'block')
-        #   #
-        #   class MatchStyle < ::RuboCop::Cop::Base; end
-        MatchStyle = ::RuboCop::Cop::Capybara::MatchStyle
-      end
-    end
-  end
-end

data/lib/rubocop/cop/rspec/capybara/negation_matcher.rb

@@ -1,33 +0,0 @@
-# frozen_string_literal: true
-
-module RuboCop
-  module Cop
-    module RSpec
-      module Capybara
-        # @!parse
-        #   # Enforces use of `have_no_*` or `not_to` for negated expectations.
-        #   #
-        #   # @example EnforcedStyle: not_to (default)
-        #   #   # bad
-        #   #   expect(page).to have_no_selector
-        #   #   expect(page).to have_no_css('a')
-        #   #
-        #   #   # good
-        #   #   expect(page).not_to have_selector
-        #   #   expect(page).not_to have_css('a')
-        #   #
-        #   # @example EnforcedStyle: have_no
-        #   #   # bad
-        #   #   expect(page).not_to have_selector
-        #   #   expect(page).not_to have_css('a')
-        #   #
-        #   #   # good
-        #   #   expect(page).to have_no_selector
-        #   #   expect(page).to have_no_css('a')
-        #   #
-        #   class NegationMatcher < ::RuboCop::Cop::Base; end
-        NegationMatcher = ::RuboCop::Cop::Capybara::NegationMatcher
-      end
-    end
-  end
-end

data/lib/rubocop/cop/rspec/capybara/specific_actions.rb

@@ -1,29 +0,0 @@
-# frozen_string_literal: true
-
-module RuboCop
-  module Cop
-    module RSpec
-      module Capybara
-        # @!parse
-        #   # Checks for there is a more specific actions offered by Capybara.
-        #   #
-        #   # @example
-        #   #
-        #   #   # bad
-        #   #   find('a').click
-        #   #   find('button.cls').click
-        #   #   find('a', exact_text: 'foo').click
-        #   #   find('div button').click
-        #   #
-        #   #   # good
-        #   #   click_link
-        #   #   click_button(class: 'cls')
-        #   #   click_link(exact_text: 'foo')
-        #   #   find('div').click_button
-        #   #
-        #   class SpecificActions < ::RuboCop::Cop::Base; end
-        SpecificActions = ::RuboCop::Cop::Capybara::SpecificActions
-      end
-    end
-  end
-end

data/lib/rubocop/cop/rspec/capybara/specific_finders.rb

@@ -1,24 +0,0 @@
-# frozen_string_literal: true
-
-module RuboCop
-  module Cop
-    module RSpec
-      module Capybara
-        # @!parse
-        #   # Checks if there is a more specific finder offered by Capybara.
-        #   #
-        #   # @example
-        #   #   # bad
-        #   #   find('#some-id')
-        #   #   find('[visible][id=some-id]')
-        #   #
-        #   #   # good
-        #   #   find_by_id('some-id')
-        #   #   find_by_id('some-id', visible: true)
-        #   #
-        #   class SpecificFinders < ::RuboCop::Cop::Base; end
-        SpecificFinders = ::RuboCop::Cop::Capybara::SpecificFinders
-      end
-    end
-  end
-end

data/lib/rubocop/cop/rspec/capybara/specific_matcher.rb

@@ -1,35 +0,0 @@
-# frozen_string_literal: true
-
-module RuboCop
-  module Cop
-    module RSpec
-      module Capybara
-        # @!parse
-        #   # Checks for there is a more specific matcher offered by Capybara.
-        #   #
-        #   # @example
-        #   #
-        #   #   # bad
-        #   #   expect(page).to have_selector('button')
-        #   #   expect(page).to have_no_selector('button.cls')
-        #   #   expect(page).to have_css('button')
-        #   #   expect(page).to have_no_css('a.cls', href: 'http://example.com')
-        #   #   expect(page).to have_css('table.cls')
-        #   #   expect(page).to have_css('select')
-        #   #   expect(page).to have_css('input', exact_text: 'foo')
-        #   #
-        #   #   # good
-        #   #   expect(page).to have_button
-        #   #   expect(page).to have_no_button(class: 'cls')
-        #   #   expect(page).to have_button
-        #   #   expect(page).to have_no_link('foo', class: 'cls', href: 'http://example.com')
-        #   #   expect(page).to have_table(class: 'cls')
-        #   #   expect(page).to have_select
-        #   #   expect(page).to have_field('foo')
-        #   #
-        #   class SpecificMatcher < ::RuboCop::Cop::Base; end
-        SpecificMatcher = ::RuboCop::Cop::Capybara::SpecificMatcher
-      end
-    end
-  end
-end

data/lib/rubocop/cop/rspec/capybara/visibility_matcher.rb

@@ -1,36 +0,0 @@
-# frozen_string_literal: true
-
-module RuboCop
-  module Cop
-    module RSpec
-      module Capybara
-        # @!parse
-        #   # Checks for boolean visibility in Capybara finders.
-        #   #
-        #   # Capybara lets you find elements that match a certain visibility
-        #   # using the `:visible` option. `:visible` accepts both boolean and
-        #   # symbols as values, however using booleans can have unwanted
-        #   # effects. `visible: false` does not find just invisible elements,
-        #   # but both visible and invisible elements. For expressiveness and
-        #   # clarity, use one of the # symbol values, `:all`, `:hidden` or
-        #   # `:visible`.
-        #   # Read more in
-        #   # https://www.rubydoc.info/gems/capybara/Capybara%2FNode%2FFinders:all[the documentation].
-        #   #
-        #   # @example
-        #   #   # bad
-        #   #   expect(page).to have_selector('.foo', visible: false)
-        #   #   expect(page).to have_css('.foo', visible: true)
-        #   #   expect(page).to have_link('my link', visible: false)
-        #   #
-        #   #   # good
-        #   #   expect(page).to have_selector('.foo', visible: :visible)
-        #   #   expect(page).to have_css('.foo', visible: :all)
-        #   #   expect(page).to have_link('my link', visible: :hidden)
-        #   #
-        #   class VisibilityMatcher < ::RuboCop::Cop::Base; end
-        VisibilityMatcher = ::RuboCop::Cop::Capybara::VisibilityMatcher
-      end
-    end
-  end
-end

data/lib/rubocop/cop/rspec/factory_bot/attribute_defined_statically.rb

@@ -1,35 +0,0 @@
-# frozen_string_literal: true
-
-module RuboCop
-  module Cop
-    module RSpec
-      module FactoryBot
-        # @!parse
-        #   # Always declare attribute values as blocks.
-        #   #
-        #   # @example
-        #   #   # bad
-        #   #   kind [:active, :rejected].sample
-        #   #
-        #   #   # good
-        #   #   kind { [:active, :rejected].sample }
-        #   #
-        #   #   # bad
-        #   #   closed_at 1.day.from_now
-        #   #
-        #   #   # good
-        #   #   closed_at { 1.day.from_now }
-        #   #
-        #   #   # bad
-        #   #   count 1
-        #   #
-        #   #   # good
-        #   #   count { 1 }
-        #   #
-        #   class AttributeDefinedStatically < ::RuboCop::Cop::Base; end
-        AttributeDefinedStatically =
-          ::RuboCop::Cop::FactoryBot::AttributeDefinedStatically
-      end
-    end
-  end
-end

data/lib/rubocop/cop/rspec/factory_bot/consistent_parentheses_style.rb

@@ -1,50 +0,0 @@
-# frozen_string_literal: true
-
-module RuboCop
-  module Cop
-    module RSpec
-      module FactoryBot
-        # @!parse
-        #   # Use a consistent style for parentheses in factory bot calls.
-        #   #
-        #   # @example
-        #   #
-        #   #   # bad
-        #   #   create :user
-        #   #   build(:user)
-        #   #   create(:login)
-        #   #   create :login
-        #   #
-        #   # @example `EnforcedStyle: require_parentheses` (default)
-        #   #
-        #   #   # good
-        #   #   create(:user)
-        #   #   create(:user)
-        #   #   create(:login)
-        #   #   build(:login)
-        #   #
-        #   # @example `EnforcedStyle: omit_parentheses`
-        #   #
-        #   #   # good
-        #   #   create :user
-        #   #   build :user
-        #   #   create :login
-        #   #   create :login
-        #   #
-        #   #   # also good
-        #   #   # when method name and first argument are not on same line
-        #   #   create(
-        #   #     :user
-        #   #   )
-        #   #   build(
-        #   #     :user,
-        #   #     name: 'foo'
-        #   #   )
-        #   #
-        #   class ConsistentParenthesesStyle < ::RuboCop::Cop::Base; end
-        ConsistentParenthesesStyle =
-          ::RuboCop::Cop::FactoryBot::ConsistentParenthesesStyle
-      end
-    end
-  end
-end

data/lib/rubocop/cop/rspec/factory_bot/create_list.rb

@@ -1,40 +0,0 @@
-# frozen_string_literal: true
-
-module RuboCop
-  module Cop
-    module RSpec
-      module FactoryBot
-        # @!parse
-        #   # Checks for create_list usage.
-        #   #
-        #   # This cop can be configured using the `EnforcedStyle` option
-        #   #
-        #   # @example `EnforcedStyle: create_list` (default)
-        #   #   # bad
-        #   #   3.times { create :user }
-        #   #
-        #   #   # good
-        #   #   create_list :user, 3
-        #   #
-        #   #   # bad
-        #   #   3.times { create :user, age: 18 }
-        #   #
-        #   #   # good - index is used to alter the created models attributes
-        #   #   3.times { |n| create :user, age: n }
-        #   #
-        #   #   # good - contains a method call, may return different values
-        #   #   3.times { create :user, age: rand }
-        #   #
-        #   # @example `EnforcedStyle: n_times`
-        #   #   # bad
-        #   #   create_list :user, 3
-        #   #
-        #   #   # good
-        #   #   3.times { create :user }
-        #   #
-        #   class CreateList < ::RuboCop::Cop::Base; end
-        CreateList = ::RuboCop::Cop::FactoryBot::CreateList
-      end
-    end
-  end
-end

data/lib/rubocop/cop/rspec/factory_bot/factory_class_name.rb

@@ -1,29 +0,0 @@
-# frozen_string_literal: true
-
-module RuboCop
-  module Cop
-    module RSpec
-      module FactoryBot
-        # @!parse
-        #   # Use string value when setting the class attribute explicitly.
-        #   #
-        #   # This cop would promote faster tests by lazy-loading of
-        #   # application files. Also, this could help you suppress potential
-        #   # bugs in combination with external libraries by avoiding a preload
-        #   # of application files from the factory files.
-        #   #
-        #   # @example
-        #   #   # bad
-        #   #   factory :foo, class: Foo do
-        #   #   end
-        #   #
-        #   #   # good
-        #   #   factory :foo, class: 'Foo' do
-        #   #   end
-        #   #
-        #   class FactoryClassName < ::RuboCop::Cop::Base; end
-        FactoryClassName = ::RuboCop::Cop::FactoryBot::FactoryClassName
-      end
-    end
-  end
-end

data/lib/rubocop/cop/rspec/factory_bot/factory_name_style.rb

@@ -1,33 +0,0 @@
-# frozen_string_literal: true
-
-module RuboCop
-  module Cop
-    module RSpec
-      module FactoryBot
-        # @!parse
-        #   # Checks for name style for argument of FactoryBot::Syntax::Methods.
-        #   #
-        #   # @example EnforcedStyle: symbol (default)
-        #   #   # bad
-        #   #   create('user')
-        #   #   build "user", username: "NAME"
-        #   #
-        #   #   # good
-        #   #   create(:user)
-        #   #   build :user, username: "NAME"
-        #   #
-        #   # @example EnforcedStyle: string
-        #   #   # bad
-        #   #   create(:user)
-        #   #   build :user, username: "NAME"
-        #   #
-        #   #   # good
-        #   #   create('user')
-        #   #   build "user", username: "NAME"
-        #   #
-        #   class FactoryNameStyle < ::RuboCop::Cop::Base; end
-        FactoryNameStyle = ::RuboCop::Cop::FactoryBot::FactoryNameStyle
-      end
-    end
-  end
-end