Class: RuboCop::Cop::RSpec::ScatteredSetup
- Defined in:
- lib/rubocop/cop/rspec/scattered_setup.rb
Overview
Checks for setup scattered across multiple hooks in an example group.
Unify `before`, `after`, and `around` hooks when possible.
Constant Summary collapse
- MSG =
'Do not define multiple `%<hook_name>s` hooks in the same ' \ 'example group (also defined on %<lines>s).'
Instance Method Summary collapse
Methods inherited from Base
inherited, #on_new_investigation
Methods included from RSpec::Language::NodePattern
Methods included from RSpec::Language
#example?, #example_group?, #example_group_with_body?, #hook?, #include?, #let?, #rspec?, #shared_group?, #spec_group?, #subject?
Instance Method Details
#lines_msg(numbers) ⇒ Object
57 58 59 60 61 62 63 |
# File 'lib/rubocop/cop/rspec/scattered_setup.rb', line 57 def lines_msg(numbers) if numbers.size == 1 "line #{numbers.first}" else "lines #{numbers.join(', ')}" end end |
#on_block(node) ⇒ Object
29 30 31 32 33 34 35 36 37 38 39 40 41 42 |
# File 'lib/rubocop/cop/rspec/scattered_setup.rb', line 29 def on_block(node) return unless example_group?(node) repeated_hooks(node).each do |occurrences| lines = occurrences.map(&:first_line) occurrences.each do |occurrence| lines_except_current = lines - [occurrence.first_line] = format(MSG, hook_name: occurrences.first.method_name, lines: lines_msg(lines_except_current)) add_offense(occurrence, message: ) end end end |
#repeated_hooks(node) ⇒ Object
44 45 46 47 48 49 50 51 52 53 54 55 |
# File 'lib/rubocop/cop/rspec/scattered_setup.rb', line 44 def repeated_hooks(node) hooks = RuboCop::RSpec::ExampleGroup.new(node) .hooks .select(&:knowable_scope?) .group_by { |hook| [hook.name, hook.scope, hook.] } .values .reject(&:one?) hooks.map do |hook| hook.map(&:to_node) end end |