Skip to content

Commit

Permalink
Moved pending tests to github issues
Browse files Browse the repository at this point in the history
  • Loading branch information
kevinrutherford committed Jun 13, 2009
1 parent 60d4690 commit ed04b5a
Show file tree
Hide file tree
Showing 3 changed files with 1 addition and 31 deletions.
2 changes: 1 addition & 1 deletion lib/reek/smells/smells.rb
Original file line number Diff line number Diff line change
Expand Up @@ -79,7 +79,7 @@ def all_reekfiles(path)
end

def disable(smell)
@config[smell].adopt!({'enabled' => false})
@config[smell].adopt!({Reek::Smells::SmellDetector::ENABLED_KEY => false})
end
end
end
16 changes: 0 additions & 16 deletions spec/reek/smells/feature_envy_spec.rb
Original file line number Diff line number Diff line change
Expand Up @@ -133,22 +133,6 @@
end'.should_not reek
end

it 'reports the most-used ivar' do
pending('bug')
'def func
@other.a
@other.b
@nother.c
end'.should reek_of(:FeatureEnvy, /@other/)
#
# def other.func(me)
# a
# b
# me.nother_c
# end
#
end

it 'ignores multiple ivars' do
'def func
@other.a
Expand Down
14 changes: 0 additions & 14 deletions spec/reek/smells/utility_function_spec.rb
Original file line number Diff line number Diff line change
Expand Up @@ -39,20 +39,6 @@ def thing(ff); ff; end
end
Son.should_not reek
end

it 'should not report class method' do
pending('bug')
source = <<EOS
class Cache
class << self
def create_unless_known(attributes)
Cache.create(attributes) unless Cache.known?
end
end
end
EOS
source.should_not reek
end

it 'should recognise a deep call' do
src = <<EOS
Expand Down

0 comments on commit ed04b5a

Please sign in to comment.