Sha256: 03ef0ceb75639809012185058d716a7fefed6f26a64387bf36f4a42072c19df6
Contents?: true
Size: 1.95 KB
Versions: 11
Compression:
Stored size: 1.95 KB
Contents
require_relative '../../spec_helper'
require_relative '../../../lib/reek/context/code_context'
require_relative '../../../lib/reek/smells/nil_check'
require_relative 'smell_detector_shared'
RSpec.describe Reek::Smells::NilCheck do
context 'for methods' do
it 'reports the correct line number' do
src = <<-EOS
def nilcheck foo
foo.nil?
end
EOS
ctx = Reek::Context::CodeContext.new(nil, Reek::Source::SourceCode.from(src).syntax_tree)
detector = build(:smell_detector, smell_type: :NilCheck, source: 'source_name')
smells = detector.examine_context(ctx)
expect(smells[0].lines).to eq [2]
end
it 'reports nothing when scope includes no nil checks' do
expect('def no_nils; end').not_to reek_of(:NilCheck)
end
it 'reports when scope uses multiple nil? methods' do
src = <<-EOS
def chk_multi_nil(para)
para.nil?
puts "Hello"
\"\".nil?
end
EOS
expect(src).to reek_of(:NilCheck)
end
it 'reports twice when scope uses == nil and === nil' do
src = <<-EOS
def chk_eq_nil(para)
para == nil
para === nil
end
EOS
expect(src).to reek_of(:NilCheck)
end
it 'reports when scope uses nil ==' do
expect('def chk_eq_nil_rev(para); nil == para; end').to reek_of(:NilCheck)
end
it 'reports when scope uses multiple case-clauses checking nil' do
src = <<-EOS
def case_nil
case @inst_var
when nil then puts "Nil"
end
puts "Hello"
case @inst_var2
when 1 then puts 1
when nil then puts nil.inspect
end
end
EOS
expect(src).to reek_of(:NilCheck)
end
it 'reports a when clause that checks nil and other values' do
src = <<-EOS
def case_nil
case @inst_var
when nil, false then puts "Hello"
end
end
EOS
expect(src).to reek_of(:NilCheck)
end
end
end
Version data entries
11 entries across 11 versions & 1 rubygems