Sha256: 32b18d0fbb05214f9df86021a49878cbf70a2a42132e6dec5362d8b5b47b3e17
Contents?: true
Size: 1.96 KB
Versions: 2
Compression:
Stored size: 1.96 KB
Contents
# frozen_string_literal: true module RuboCop module Cop module RSpec # Checks for explicitly referenced test subjects. # # RSpec lets you declare an "implicit subject" using `subject { ... }` # which allows for tests like `it { should be_valid }`. If you need to # reference your test subject you should explicitly name it using # `subject(:your_subject_name) { ... }`. Your test subjects should be # the most important object in your tests so they deserve a descriptive # name. # # @example # # bad # RSpec.describe User do # subject { described_class.new } # # it 'is valid' do # expect(subject.valid?).to be(true) # end # end # # # good # RSpec.describe Foo do # subject(:user) { described_class.new } # # it 'is valid' do # expect(user.valid?).to be(true) # end # end # # # also good # RSpec.describe Foo do # subject(:user) { described_class.new } # # it { should be_valid } # end class NamedSubject < Cop MSG = 'Name your test subject if '\ 'you need to reference it explicitly.'.freeze def_node_matcher :rspec_block?, <<-PATTERN (block (send nil {:it :specify :before :after :around} ...) ...) PATTERN def_node_matcher :unnamed_subject, '$(send nil :subject)' def on_block(node) return unless rspec_block?(node) subject_usage(node) do |subject_node| add_offense(subject_node, :selector) end end private def subject_usage(node, &block) return unless node.instance_of?(Node) unnamed_subject(node, &block) node.children.each do |child| subject_usage(child, &block) end end end end end end
Version data entries
2 entries across 2 versions & 1 rubygems
Version | Path |
---|---|
rubocop-rspec-1.9.1 | lib/rubocop/cop/rspec/named_subject.rb |
rubocop-rspec-1.9.0 | lib/rubocop/cop/rspec/named_subject.rb |