rubocop-rspec 2.12.0 → 2.13.1

data/lib/rubocop/cop/rspec/predicate_matcher.rb

@@ -276,6 +276,7 @@ module RuboCop
       #
       #   # good - the above code is rewritten to it by this cop
       #   expect(foo.something?).to be_truthy
+      #
       class PredicateMatcher < Base
         extend AutoCorrector
         include ConfigurableEnforcedStyle
@@ -291,7 +292,7 @@ module RuboCop
           end
         end
 
-        def on_block(node)
+        def on_block(node) # rubocop:disable InternalAffairs/NumblockHandler
           check_explicit(node) if style == :explicit
         end
 

data/lib/rubocop/cop/rspec/rails/avoid_setup_hook.rb

@@ -7,7 +7,6 @@ module RuboCop
         # Checks that tests use RSpec `before` hook over Rails `setup` method.
         #
         # @example
-        #
         #   # bad
         #   setup do
         #     allow(foo).to receive(:bar)
@@ -30,7 +29,7 @@ module RuboCop
               (args) _)
           PATTERN
 
-          def on_block(node)
+          def on_block(node) # rubocop:disable InternalAffairs/NumblockHandler
             setup_call(node) do |setup|
               add_offense(node) do |corrector|
                 corrector.replace setup, 'before'

data/lib/rubocop/cop/rspec/receive_counts.rb

@@ -6,22 +6,21 @@ module RuboCop
       # Check for `once` and `twice` receive counts matchers usage.
       #
       # @example
+      #   # bad
+      #   expect(foo).to receive(:bar).exactly(1).times
+      #   expect(foo).to receive(:bar).exactly(2).times
+      #   expect(foo).to receive(:bar).at_least(1).times
+      #   expect(foo).to receive(:bar).at_least(2).times
+      #   expect(foo).to receive(:bar).at_most(1).times
+      #   expect(foo).to receive(:bar).at_most(2).times
       #
-      #     # bad
-      #     expect(foo).to receive(:bar).exactly(1).times
-      #     expect(foo).to receive(:bar).exactly(2).times
-      #     expect(foo).to receive(:bar).at_least(1).times
-      #     expect(foo).to receive(:bar).at_least(2).times
-      #     expect(foo).to receive(:bar).at_most(1).times
-      #     expect(foo).to receive(:bar).at_most(2).times
-      #
-      #     # good
-      #     expect(foo).to receive(:bar).once
-      #     expect(foo).to receive(:bar).twice
-      #     expect(foo).to receive(:bar).at_least(:once)
-      #     expect(foo).to receive(:bar).at_least(:twice)
-      #     expect(foo).to receive(:bar).at_most(:once)
-      #     expect(foo).to receive(:bar).at_most(:twice).times
+      #   # good
+      #   expect(foo).to receive(:bar).once
+      #   expect(foo).to receive(:bar).twice
+      #   expect(foo).to receive(:bar).at_least(:once)
+      #   expect(foo).to receive(:bar).at_least(:twice)
+      #   expect(foo).to receive(:bar).at_most(:once)
+      #   expect(foo).to receive(:bar).at_most(:twice).times
       #
       class ReceiveCounts < Base
         extend AutoCorrector

data/lib/rubocop/cop/rspec/receive_never.rb

@@ -6,12 +6,11 @@ module RuboCop
       # Prefer `not_to receive(...)` over `receive(...).never`.
       #
       # @example
+      #   # bad
+      #   expect(foo).to receive(:bar).never
       #
-      #     # bad
-      #     expect(foo).to receive(:bar).never
-      #
-      #     # good
-      #     expect(foo).not_to receive(:bar)
+      #   # good
+      #   expect(foo).not_to receive(:bar)
       #
       class ReceiveNever < Base
         extend AutoCorrector

data/lib/rubocop/cop/rspec/repeated_description.rb

@@ -6,44 +6,43 @@ module RuboCop
       # Check for repeated description strings in example groups.
       #
       # @example
+      #   # bad
+      #   RSpec.describe User do
+      #     it 'is valid' do
+      #       # ...
+      #     end
       #
-      #     # bad
-      #     RSpec.describe User do
-      #       it 'is valid' do
-      #         # ...
-      #       end
-      #
-      #       it 'is valid' do
-      #         # ...
-      #       end
+      #     it 'is valid' do
+      #       # ...
       #     end
+      #   end
       #
-      #     # good
-      #     RSpec.describe User do
-      #       it 'is valid when first and last name are present' do
-      #         # ...
-      #       end
+      #   # good
+      #   RSpec.describe User do
+      #     it 'is valid when first and last name are present' do
+      #       # ...
+      #     end
       #
-      #       it 'is valid when last name only is present' do
-      #         # ...
-      #       end
+      #     it 'is valid when last name only is present' do
+      #       # ...
       #     end
+      #   end
       #
-      #     # good
-      #     RSpec.describe User do
-      #       it 'is valid' do
-      #         # ...
-      #       end
+      #   # good
+      #   RSpec.describe User do
+      #     it 'is valid' do
+      #       # ...
+      #     end
       #
-      #       it 'is valid', :flag do
-      #         # ...
-      #       end
+      #     it 'is valid', :flag do
+      #       # ...
       #     end
+      #   end
       #
       class RepeatedDescription < Base
         MSG = "Don't repeat descriptions within an example group."
 
-        def on_block(node)
+        def on_block(node) # rubocop:disable InternalAffairs/NumblockHandler
           return unless example_group?(node)
 
           repeated_descriptions(node).each do |repeated_description|

data/lib/rubocop/cop/rspec/repeated_example.rb

@@ -18,7 +18,7 @@ module RuboCop
       class RepeatedExample < Base
         MSG = "Don't repeat examples within an example group."
 
-        def on_block(node)
+        def on_block(node) # rubocop:disable InternalAffairs/NumblockHandler
           return unless example_group?(node)
 
           repeated_examples(node).each do |repeated_example|

data/lib/rubocop/cop/rspec/repeated_example_group_body.rb

@@ -6,42 +6,41 @@ module RuboCop
       # Check for repeated describe and context block body.
       #
       # @example
+      #   # bad
+      #   describe 'cool feature x' do
+      #     it { cool_predicate }
+      #   end
       #
-      #    # bad
-      #    describe 'cool feature x' do
-      #      it { cool_predicate }
-      #    end
+      #   describe 'cool feature y' do
+      #     it { cool_predicate }
+      #   end
       #
-      #    describe 'cool feature y' do
-      #      it { cool_predicate }
-      #    end
+      #   # good
+      #   describe 'cool feature' do
+      #     it { cool_predicate }
+      #   end
       #
-      #    # good
-      #    describe 'cool feature' do
-      #      it { cool_predicate }
-      #    end
+      #   describe 'another cool feature' do
+      #     it { another_predicate }
+      #   end
       #
-      #    describe 'another cool feature' do
-      #      it { another_predicate }
-      #    end
+      #   # good
+      #   context 'when case x', :tag do
+      #     it { cool_predicate }
+      #   end
       #
-      #    # good
-      #    context 'when case x', :tag do
-      #      it { cool_predicate }
-      #    end
+      #   context 'when case y' do
+      #     it { cool_predicate }
+      #   end
       #
-      #    context 'when case y' do
-      #      it { cool_predicate }
-      #    end
+      #   # good
+      #   context Array do
+      #     it { is_expected.to respond_to :each }
+      #   end
       #
-      #    # good
-      #    context Array do
-      #      it { is_expected.to respond_to :each }
-      #    end
-      #
-      #    context Hash do
-      #      it { is_expected.to respond_to :each }
-      #    end
+      #   context Hash do
+      #     it { is_expected.to respond_to :each }
+      #   end
       #
       class RepeatedExampleGroupBody < Base
         MSG = 'Repeated %<group>s block body on line(s) %<loc>s'

data/lib/rubocop/cop/rspec/repeated_example_group_description.rb

@@ -6,42 +6,41 @@ module RuboCop
       # Check for repeated example group descriptions.
       #
       # @example
+      #   # bad
+      #   describe 'cool feature' do
+      #     # example group
+      #   end
       #
-      #    # bad
-      #    describe 'cool feature' do
-      #      # example group
-      #    end
+      #   describe 'cool feature' do
+      #     # example group
+      #   end
       #
-      #    describe 'cool feature' do
-      #      # example group
-      #    end
+      #   # bad
+      #   context 'when case x' do
+      #     # example group
+      #   end
       #
-      #    # bad
-      #    context 'when case x' do
-      #      # example group
-      #    end
+      #   describe 'when case x' do
+      #     # example group
+      #   end
       #
-      #    describe 'when case x' do
-      #      # example group
-      #    end
+      #   # good
+      #   describe 'cool feature' do
+      #     # example group
+      #   end
       #
-      #    # good
-      #    describe 'cool feature' do
-      #      # example group
-      #    end
+      #   describe 'another cool feature' do
+      #     # example group
+      #   end
       #
-      #    describe 'another cool feature' do
-      #      # example group
-      #    end
+      #   # good
+      #   context 'when case x' do
+      #     # example group
+      #   end
       #
-      #    # good
-      #    context 'when case x' do
-      #      # example group
-      #    end
-      #
-      #    context 'when another case' do
-      #      # example group
-      #    end
+      #   context 'when another case' do
+      #     # example group
+      #   end
       #
       class RepeatedExampleGroupDescription < Base
         MSG = 'Repeated %<group>s block description on line(s) %<loc>s'

data/lib/rubocop/cop/rspec/repeated_include_example.rb

@@ -6,45 +6,44 @@ module RuboCop
       # Check for repeated include of shared examples.
       #
       # @example
+      #   # bad
+      #   describe 'foo' do
+      #     include_examples 'cool stuff'
+      #     include_examples 'cool stuff'
+      #   end
       #
-      #    # bad
-      #    describe 'foo' do
-      #      include_examples 'cool stuff'
-      #      include_examples 'cool stuff'
-      #    end
+      #   # bad
+      #   describe 'foo' do
+      #     it_behaves_like 'a cool', 'thing'
+      #     it_behaves_like 'a cool', 'thing'
+      #   end
       #
-      #    # bad
-      #    describe 'foo' do
-      #      it_behaves_like 'a cool', 'thing'
-      #      it_behaves_like 'a cool', 'thing'
-      #    end
+      #   # bad
+      #   context 'foo' do
+      #     it_should_behave_like 'a duck'
+      #     it_should_behave_like 'a duck'
+      #   end
       #
-      #    # bad
-      #    context 'foo' do
-      #      it_should_behave_like 'a duck'
-      #      it_should_behave_like 'a duck'
-      #    end
+      #   # good
+      #   describe 'foo' do
+      #     include_examples 'cool stuff'
+      #   end
       #
-      #    # good
-      #    describe 'foo' do
-      #      include_examples 'cool stuff'
-      #    end
+      #   describe 'bar' do
+      #     include_examples 'cool stuff'
+      #   end
       #
-      #    describe 'bar' do
-      #      include_examples 'cool stuff'
-      #    end
+      #   # good
+      #   describe 'foo' do
+      #     it_behaves_like 'a cool', 'thing'
+      #     it_behaves_like 'a cool', 'person'
+      #   end
       #
-      #    # good
-      #    describe 'foo' do
-      #      it_behaves_like 'a cool', 'thing'
-      #      it_behaves_like 'a cool', 'person'
-      #    end
-      #
-      #    # good
-      #    context 'foo' do
-      #      it_should_behave_like 'a duck'
-      #      it_should_behave_like 'a goose'
-      #    end
+      #   # good
+      #   context 'foo' do
+      #     it_should_behave_like 'a duck'
+      #     it_should_behave_like 'a goose'
+      #   end
       #
       class RepeatedIncludeExample < Base
         MSG = 'Repeated include of shared_examples %<name>s ' \

data/lib/rubocop/cop/rspec/return_from_stub.rb

@@ -59,7 +59,7 @@ module RuboCop
           check_and_return_call(node)
         end
 
-        def on_block(node)
+        def on_block(node) # rubocop:disable InternalAffairs/NumblockHandler
           return unless style == :and_return
           return unless stub_with_block?(node)
 

data/lib/rubocop/cop/rspec/scattered_let.rb

@@ -31,7 +31,7 @@ module RuboCop
 
         MSG = 'Group all let/let! blocks in the example group together.'
 
-        def on_block(node)
+        def on_block(node) # rubocop:disable InternalAffairs/NumblockHandler
           return unless example_group_with_body?(node)
 
           check_let_declarations(node.body)
@@ -53,10 +53,6 @@ module RuboCop
             end
           end
         end
-
-        def find_first_let(node)
-          node.children.find { |child| let?(child) }
-        end
       end
     end
   end

data/lib/rubocop/cop/rspec/scattered_setup.rb

@@ -26,7 +26,7 @@ module RuboCop
         MSG = 'Do not define multiple `%<hook_name>s` hooks in the same ' \
               'example group (also defined on %<lines>s).'
 
-        def on_block(node)
+        def on_block(node) # rubocop:disable InternalAffairs/NumblockHandler
           return unless example_group?(node)
 
           repeated_hooks(node).each do |occurrences|

data/lib/rubocop/cop/rspec/shared_context.rb

@@ -79,7 +79,7 @@ module RuboCop
         def_node_matcher :shared_example,
                          block_pattern('#SharedGroups.examples')
 
-        def on_block(node)
+        def on_block(node) # rubocop:disable InternalAffairs/NumblockHandler
           context_with_only_examples(node) do
             add_offense(node.send_node, message: MSG_EXAMPLES) do |corrector|
               corrector.replace(node.send_node.loc.selector, 'shared_examples')

data/lib/rubocop/cop/rspec/stubbed_mock.rb

@@ -6,7 +6,6 @@ module RuboCop
       # Checks that message expectations do not have a configured response.
       #
       # @example
-      #
       #   # bad
       #   expect(foo).to receive(:bar).with(42).and_return("hello world")
       #

data/lib/rubocop/cop/rspec/subject_declaration.rb

@@ -6,7 +6,6 @@ module RuboCop
       # Ensure that subject is defined using subject helper.
       #
       # @example
-      #
       #   # bad
       #   let(:subject) { foo }
       #   let!(:subject) { foo }

data/lib/rubocop/cop/rspec/subject_stub.rb

@@ -11,7 +11,7 @@ module RuboCop
       # when subject is also defined in parent example groups.
       #
       # @see https://robots.thoughtbot.com/don-t-stub-the-system-under-test
-      # @see https://samphippen.com/introducing-rspec-smells-and-where-to-find-them#smell-1-stubject
+      # @see https://penelope.zone/2015/12/27/introducing-rspec-smells-and-where-to-find-them.html#smell-1-stubjec
       # @see https://github.com/rubocop-hq/rspec-style-guide#dont-stub-subject
       #
       # @example
@@ -72,7 +72,7 @@ module RuboCop
         def_node_matcher :subject?, <<-PATTERN
           (block
             (send nil?
-              {:subject (sym $_) | $:subject}
+              { #Subjects.all (sym $_) | $#Subjects.all }
             ) args ...)
         PATTERN
 

data/lib/rubocop/cop/rspec/unspecified_exception.rb

@@ -10,26 +10,26 @@ module RuboCop
       # to `raise_error`
       #
       # @example
+      #   # bad
+      #   expect {
+      #     raise StandardError.new('error')
+      #   }.to raise_error
       #
-      #     # bad
-      #     expect {
-      #       raise StandardError.new('error')
-      #     }.to raise_error
+      #   # good
+      #   expect {
+      #     raise StandardError.new('error')
+      #   }.to raise_error(StandardError)
       #
-      #     # good
-      #     expect {
-      #       raise StandardError.new('error')
-      #     }.to raise_error(StandardError)
+      #   expect {
+      #     raise StandardError.new('error')
+      #   }.to raise_error('error')
       #
-      #     expect {
-      #       raise StandardError.new('error')
-      #     }.to raise_error('error')
+      #   expect {
+      #     raise StandardError.new('error')
+      #   }.to raise_error(/err/)
       #
-      #     expect {
-      #       raise StandardError.new('error')
-      #     }.to raise_error(/err/)
+      #   expect { do_something }.not_to raise_error
       #
-      #     expect { do_something }.not_to raise_error
       class UnspecifiedException < Base
         MSG = 'Specify the exception being captured'
         RESTRICT_ON_SEND = %i[to].freeze

data/lib/rubocop/cop/rspec/variable_definition.rb

@@ -22,6 +22,7 @@ module RuboCop
       #   # good
       #   subject('user') { create_user }
       #   let('user_name') { 'Adam' }
+      #
       class VariableDefinition < Base
         extend AutoCorrector
         include ConfigurableEnforcedStyle

data/lib/rubocop/cop/rspec/variable_name.rb

@@ -5,7 +5,7 @@ module RuboCop
     module RSpec
       # Checks that memoized helper names use the configured style.
       #
-      # Variables can be excluded from checking using the `IgnoredPatterns`
+      # Variables can be excluded from checking using the `AllowedPatterns`
       # option.
       #
       # @example EnforcedStyle: snake_case (default)
@@ -26,22 +26,21 @@ module RuboCop
       #   subject(:userName1) { 'Adam' }
       #   let(:userName2) { 'Adam' }
       #
-      # @example IgnoredPatterns configuration
-      #
+      # @example AllowedPatterns configuration
       #   # rubocop.yml
       #   # RSpec/VariableName:
       #   #   EnforcedStyle: snake_case
-      #   #   IgnoredPatterns:
+      #   #   AllowedPatterns:
       #   #     - ^userFood
       #
       # @example
-      #   # okay because it matches the `^userFood` regex in `IgnoredPatterns`
+      #   # okay because it matches the `^userFood` regex in `AllowedPatterns`
       #   subject(:userFood_1) { 'spaghetti' }
       #   let(:userFood_2) { 'fettuccine' }
       #
       class VariableName < Base
         include ConfigurableNaming
-        include IgnoredPattern
+        include AllowedPattern
         include Variable
 
         MSG = 'Use %<style>s for variable names.'
@@ -49,7 +48,7 @@ module RuboCop
         def on_send(node)
           variable_definition?(node) do |variable|
             return if variable.dstr_type? || variable.dsym_type?
-            return if matches_ignored_pattern?(variable.value)
+            return if matches_allowed_pattern?(variable.value)
 
             check_name(node, variable.value, variable.loc.expression)
           end

data/lib/rubocop/cop/rspec/verified_doubles.rb

@@ -22,6 +22,7 @@ module RuboCop
       #   let(:foo) do
       #     instance_double("ClassName", method_name: 'returned value')
       #   end
+      #
       class VerifiedDoubles < Base
         MSG = 'Prefer using verifying doubles over normal doubles.'
         RESTRICT_ON_SEND = %i[double spy].freeze

data/lib/rubocop/cop/rspec/void_expect.rb

@@ -11,6 +11,7 @@ module RuboCop
       #
       #   # good
       #   expect(something).to be(1)
+      #
       class VoidExpect < Base
         MSG = 'Do not use `expect()` without `.to` or `.not_to`. ' \
               'Chain the methods or remove it.'
@@ -32,7 +33,7 @@ module RuboCop
           check_expect(node)
         end
 
-        def on_block(node)
+        def on_block(node) # rubocop:disable InternalAffairs/NumblockHandler
           return unless expect_block?(node)
 
           check_expect(node)

data/lib/rubocop/cop/rspec/yield.rb

@@ -11,6 +11,7 @@ module RuboCop
       #
       #   # good
       #   expect(foo).to receive(:bar).and_yield(1)
+      #
       class Yield < Base
         extend AutoCorrector
         include RangeHelp
@@ -26,7 +27,7 @@ module RuboCop
         # @!method block_call?(node)
         def_node_matcher :block_call?, '(send (lvar %) :call ...)'
 
-        def on_block(node)
+        def on_block(node) # rubocop:disable InternalAffairs/NumblockHandler
           return unless method_on_stub?(node.send_node)
 
           block_arg(node.arguments) do |block|

data/lib/rubocop/cop/rspec_cops.rb

@@ -2,6 +2,7 @@
 
 require_relative 'rspec/capybara/current_path_expectation'
 require_relative 'rspec/capybara/feature_methods'
+require_relative 'rspec/capybara/specific_finders'
 require_relative 'rspec/capybara/specific_matcher'
 require_relative 'rspec/capybara/visibility_matcher'
 
@@ -28,6 +29,7 @@ require_relative 'rspec/be_eql'
 require_relative 'rspec/be_nil'
 require_relative 'rspec/before_after_all'
 require_relative 'rspec/change_by_zero'
+require_relative 'rspec/class_check'
 require_relative 'rspec/context_method'
 require_relative 'rspec/context_wording'
 require_relative 'rspec/describe_class'
@@ -77,6 +79,7 @@ require_relative 'rspec/multiple_memoized_helpers'
 require_relative 'rspec/multiple_subjects'
 require_relative 'rspec/named_subject'
 require_relative 'rspec/nested_groups'
+require_relative 'rspec/no_expectation_example'
 require_relative 'rspec/not_to_not'
 require_relative 'rspec/overwriting_setup'
 require_relative 'rspec/pending'