reek 1.1.3 → 1.2.0

data/spec/reek/smells/smell_spec.rb

@@ -1,24 +0,0 @@
-require File.dirname(__FILE__) + '/../../spec_helper.rb'
-
-require 'reek/smells/smells'
-
-include Reek
-
-describe SmellWarning, ' in comparisons' do
-  before :each do
-    @first = SmellWarning.new(Smells::FeatureEnvy.new, "self", "self")
-    @second = SmellWarning.new(Smells::FeatureEnvy.new, "self", "self")
-  end
-
-  it 'should hash equal when the smell is the same' do
-    @first.hash.should == @second.hash
-  end
-
-  it 'should compare equal when the smell is the same' do
-    @first.should == @second
-  end
-
-  it 'should compare equal when using <=>' do
-    (@first <=> @second).should == 0
-  end
-end

data/spec/slow/inline_spec.rb

@@ -1,43 +0,0 @@
-require File.dirname(__FILE__) + '/../spec_helper.rb'
-
-describe 'sample gem source code' do
-  it "reports the correct smells in inline.rb" do
-    ruby = File.new("#{SAMPLES_DIR}/inline.rb").to_source
-    ruby.should reek_of(:ControlCouple, /Inline::C#parse_signature/, /raw/)
-    ruby.should reek_of(:ControlCouple, /Module#inline/, /options/)
-    ruby.should reek_of(:Duplication, /Inline::C#build/, /\(\$\?\ == 0\)/)
-    ruby.should reek_of(:Duplication, /Inline::C#build/, /Inline.directory/)
-    ruby.should reek_of(:Duplication, /Inline::C#build/, /io.puts/)
-    ruby.should reek_of(:Duplication, /Inline::C#build/, /io.puts\("#endif"\)/)
-    ruby.should reek_of(:Duplication, /Inline::C#build/, /io.puts\("#ifdef __cplusplus"\)/)
-    ruby.should reek_of(:Duplication, /Inline::C#build/, /module_name/)
-    ruby.should reek_of(:Duplication, /Inline::C#build/, /warn\("Output:\\n\#\{result\}"\)/)
-    ruby.should reek_of(:Duplication, /Inline::C#crap_for_windoze/, /Config::CONFIG\["libdir"\]/)
-    ruby.should reek_of(:Duplication, /Inline::C#generate/, /result.sub!\(\/\\A\\n\/, ""\)/)
-    ruby.should reek_of(:Duplication, /Inline::C#generate/, /signature\["args"\]/)
-    ruby.should reek_of(:Duplication, /Inline::C#generate/, /signature\["args"\].map/)
-    ruby.should reek_of(:Duplication, /Inline::C#initialize/, /stack.empty?/)
-    ruby.should reek_of(:Duplication, /Inline::C#load/, /so_name/)
-    ruby.should reek_of(:Duplication, /Inline::self.rootdir/, /env.nil?/)
-    ruby.should reek_of(:Duplication, /Module#inline/, /Inline.const_get\(lang\)/)
-    ruby.should reek_of(:FeatureEnvy, /Inline::C#strip_comments/, /src/)
-    ruby.should reek_of(:LargeClass, /Inline::C/, /instance variables/)
-    ruby.should reek_of(:LongMethod, /File#self.write_with_backup/)
-    ruby.should reek_of(:LongMethod, /Inline::C#build/)
-    ruby.should reek_of(:LongMethod, /Inline::C#generate/)
-    ruby.should reek_of(:LongMethod, /Inline::C#load_cache/)
-    ruby.should reek_of(:LongMethod, /Inline::C#module_name/)
-    ruby.should reek_of(:LongMethod, /Inline::C#parse_signature/)
-    ruby.should reek_of(:LongMethod, /Inline::self.rootdir/)
-    ruby.should reek_of(:LongMethod, /Module#inline/)
-    ruby.should reek_of(:NestedIterators, /Inline::C#build/)
-    ruby.should reek_of(:UncommunicativeName, /Inline::C#build/, /'t'/)
-    ruby.should reek_of(:UncommunicativeName, /Inline::C#build/, /'n'/)
-    ruby.should reek_of(:UncommunicativeName, /Inline::C#c/, /'c'/)
-    ruby.should reek_of(:UncommunicativeName, /Inline::C#module_name/, /'m'/)
-    ruby.should reek_of(:UncommunicativeName, /Inline::C#module_name/, /'x'/)
-    ruby.should reek_of(:UncommunicativeName, /Inline::C#parse_signature/, /'x'/)
-    ruby.should reek_of(:UtilityFunction, /Inline::C#strip_comments/)
-    ruby.report.should have_at_most(35).smells
-  end
-end

data/spec/slow/optparse_spec.rb

@@ -1,108 +0,0 @@
-require File.dirname(__FILE__) + '/../spec_helper.rb'
-
-describe 'sample gem source code' do
-  it "reports the correct smells in optparse.rb" do
-    ruby = File.new("#{SAMPLES_DIR}/optparse.rb").to_source
-    ruby.should reek_of(:ControlCouple, /OptionParser#List#accept/, /pat/)
-    ruby.should reek_of(:ControlCouple, /OptionParser#List#update/, /lopts/)
-    ruby.should reek_of(:ControlCouple, /OptionParser#List#update/, /sopts/)
-    ruby.should reek_of(:ControlCouple, /OptionParser#ParseError#set_option/, /eq/)
-    ruby.should reek_of(:ControlCouple, /OptionParser#Switch#NoArgument#parse/, /arg/)
-    ruby.should reek_of(:ControlCouple, /OptionParser#Switch#OptionalArgument#parse/, /arg/)
-    ruby.should reek_of(:ControlCouple, /OptionParser#Switch#RequiredArgument#parse/, /arg/)
-    ruby.should reek_of(:ControlCouple, /OptionParser#block/, /o/)
-    ruby.should reek_of(:ControlCouple, /OptionParser#block/, /s/)
-    ruby.should reek_of(:ControlCouple, /OptionParser#block\/block/, /pkg/)
-    ruby.should reek_of(:ControlCouple, /OptionParser#getopts\/block/, /val/)
-    ruby.should reek_of(:ControlCouple, /OptionParser#parse_in_order/, /setter/)
-    ruby.should reek_of(:Duplication, /OptionParser#Completion::complete/, /candidates.size/)
-    ruby.should reek_of(:Duplication, /OptionParser#Completion::complete/, /k.id2name/)
-    ruby.should reek_of(:Duplication, /OptionParser#Switch#parse_arg/, /s.length/)
-    ruby.should reek_of(:Duplication, /OptionParser#Switch#summarize/, /left.collect \{ \|s\| s\.length \}\.max/)
-    ruby.should reek_of(:Duplication, /OptionParser#Switch#summarize/, /left.collect \{ \|s\| s\.length \}\.max\.to_i/)
-    ruby.should reek_of(:Duplication, /OptionParser#Switch#summarize/, /\(indent \+ l\)/)
-    ruby.should reek_of(:Duplication, /OptionParser#Switch#summarize/, /left.collect/)
-    ruby.should reek_of(:Duplication, /OptionParser#Switch#summarize/, /left.shift/)
-    ruby.should reek_of(:Duplication, /OptionParser#Switch#summarize/, /left\[-1\]/)
-    ruby.should reek_of(:Duplication, /OptionParser#Switch#summarize/, /s.length/)
-    ruby.should reek_of(:Duplication, /OptionParser#getopts/, /result\[opt\] = false/)
-    ruby.should reek_of(:Duplication, /OptionParser#make_switch/, /default_style.guess\(arg = a\)/)
-    ruby.should reek_of(:Duplication, /OptionParser#make_switch/, /\(long << o = q.downcase\)/)
-    ruby.should reek_of(:Duplication, /OptionParser#make_switch/, /notwice\(a \? \(Object\) : \(TrueClass\), klass, "type"\)/)
-    ruby.should reek_of(:Duplication, /OptionParser#make_switch/, /notwice\(NilClass, klass, "type"\)/)
-    ruby.should reek_of(:Duplication, /OptionParser#make_switch/, /pattern.method\(:convert\)/)
-    ruby.should reek_of(:Duplication, /OptionParser#make_switch/, /pattern.method\(:convert\).to_proc/)
-    ruby.should reek_of(:Duplication, /OptionParser#make_switch/, /pattern.respond_to\?\(:convert\)/)
-    ruby.should reek_of(:Duplication, /OptionParser#make_switch/, /q.downcase/)
-    ruby.should reek_of(:Duplication, /OptionParser#make_switch/, /search\(:atype, o\)/)
-    ruby.should reek_of(:Duplication, /OptionParser#make_switch/, /search\(:atype, FalseClass\)/)
-    ruby.should reek_of(:Duplication, /OptionParser#make_switch/, /\(sdesc << "-\#\{q\}"\)/)
-    ruby.should reek_of(:Duplication, /OptionParser#order/, /argv\[0\]/)
-    ruby.should reek_of(:Duplication, /OptionParser#parse/, /argv\[0\]/)
-    ruby.should reek_of(:Duplication, /OptionParser#parse_in_order/, /\$\!.set_option\(arg, true\)/)
-    ruby.should reek_of(:Duplication, /OptionParser#parse_in_order/, /cb.call\(val\)/)
-    ruby.should reek_of(:Duplication, /OptionParser#parse_in_order/, /setter.call\(sw.switch_name, val\)/)
-    ruby.should reek_of(:Duplication, /OptionParser#parse_in_order/, /sw.block/)
-    ruby.should reek_of(:Duplication, /OptionParser#parse_in_order/, /sw.switch_name/)
-    ruby.should reek_of(:Duplication, /OptionParser#permute/, /argv\[0\]/)
-    ruby.should reek_of(:FeatureEnvy, /OptionParser#Completion::complete/, /candidates/)
-    ruby.should reek_of(:FeatureEnvy, /OptionParser#List#accept/, /pat/)
-    ruby.should reek_of(:FeatureEnvy, /OptionParser#Switch#summarize/, /left/)
-    ruby.should reek_of(:FeatureEnvy, /OptionParser#order/, /argv/)
-    ruby.should reek_of(:FeatureEnvy, /OptionParser#parse/, /argv/)
-    ruby.should reek_of(:FeatureEnvy, /OptionParser#permute/, /argv/)
-    ruby.should reek_of(:LargeClass, /OptionParser/)
-    ruby.should reek_of(:LongMethod, /OptionParser#Completion::complete/)
-    ruby.should reek_of(:LongMethod, /OptionParser#List#update/)
-    ruby.should reek_of(:LongMethod, /OptionParser#Switch#parse_arg/)
-    ruby.should reek_of(:LongMethod, /OptionParser#Switch#summarize/)
-    ruby.should reek_of(:LongMethod, /OptionParser#getopts/)
-    ruby.should reek_of(:LongMethod, /OptionParser#make_switch/)
-    ruby.should reek_of(:LongMethod, /OptionParser#parse_in_order/)
-    ruby.should reek_of(:LongParameterList, /OptionParser#List#complete/)
-    ruby.should reek_of(:LongParameterList, /OptionParser#List#update/)
-    ruby.should reek_of(:LongParameterList, /OptionParser#Switch#initialize/)
-    ruby.should reek_of(:LongParameterList, /OptionParser#Switch#summarize/)
-    ruby.should reek_of(:LongParameterList, /OptionParser#complete/)
-    ruby.should reek_of(:LongParameterList, /OptionParser#summarize/)
-    ruby.should reek_of(:NestedIterators, /OptionParser#CompletingHash#match/)
-    ruby.should reek_of(:NestedIterators, /OptionParser#Switch#summarize/)
-    ruby.should reek_of(:NestedIterators, /OptionParser#block/)
-    ruby.should reek_of(:NestedIterators, /OptionParser#complete/)
-    ruby.should reek_of(:NestedIterators, /OptionParser#make_switch/)
-    ruby.should reek_of(:NestedIterators, /OptionParser#make_switch/)
-    ruby.should reek_of(:NestedIterators, /OptionParser#parse_in_order/)
-    ruby.should reek_of(:NestedIterators, /OptionParser#parse_in_order/)
-    ruby.should reek_of(:UncommunicativeName, /OptionParser#Completion::complete/, /'k'/)
-    ruby.should reek_of(:UncommunicativeName, /OptionParser#Completion::complete/, /'v'/)
-    ruby.should reek_of(:UncommunicativeName, /OptionParser#List#accept/, /'t'/)
-    ruby.should reek_of(:UncommunicativeName, /OptionParser#List#reject/, /'t'/)
-    ruby.should reek_of(:UncommunicativeName, /OptionParser#List#update/, /'o'/)
-    ruby.should reek_of(:UncommunicativeName, /OptionParser#Switch#add_banner/, /'s'/)
-    ruby.should reek_of(:UncommunicativeName, /OptionParser#Switch#parse_arg/, /'m'/)
-    ruby.should reek_of(:UncommunicativeName, /OptionParser#Switch#parse_arg/, /'s'/)
-    ruby.should reek_of(:UncommunicativeName, /OptionParser#Switch#self.guess/, /'t'/)
-    ruby.should reek_of(:UncommunicativeName, /OptionParser#Switch#self.incompatible_argument_styles/, /'t'/)
-    ruby.should reek_of(:UncommunicativeName, /OptionParser#Switch#summarize/, /'l'/)
-    ruby.should reek_of(:UncommunicativeName, /OptionParser#Switch#summarize/, /'r'/)
-    ruby.should reek_of(:UncommunicativeName, /OptionParser#Switch#summarize/, /'s'/)
-    ruby.should reek_of(:UncommunicativeName, /OptionParser#block/, /'f'/)
-    ruby.should reek_of(:UncommunicativeName, /OptionParser#block/, /'k'/)
-    ruby.should reek_of(:UncommunicativeName, /OptionParser#block/, /'o'/)
-    ruby.should reek_of(:UncommunicativeName, /OptionParser#block/, /'s'/)
-    ruby.should reek_of(:UncommunicativeName, /OptionParser#block/, /'v'/)
-    ruby.should reek_of(:UncommunicativeName, /OptionParser#load/, /'s'/)
-    ruby.should reek_of(:UncommunicativeName, /OptionParser#make_switch/, /'a'/)
-    ruby.should reek_of(:UncommunicativeName, /OptionParser#make_switch/, /'n'/)
-    ruby.should reek_of(:UncommunicativeName, /OptionParser#make_switch/, /'o'/)
-    ruby.should reek_of(:UncommunicativeName, /OptionParser#make_switch/, /'q'/)
-    ruby.should reek_of(:UncommunicativeName, /OptionParser#make_switch/, /'s'/)
-    ruby.should reek_of(:UncommunicativeName, /OptionParser#make_switch/, /'c'/)
-    ruby.should reek_of(:UncommunicativeName, /OptionParser#make_switch/, /'v'/)
-    ruby.should reek_of(:UncommunicativeName, /OptionParser#search/, /'k'/)
-    ruby.should reek_of(:UncommunicativeName, /OptionParser#summarize/, /'l'/)
-    ruby.should reek_of(:UncommunicativeName, /OptionParser#ver/, /'v'/)
-    ruby.should reek_of(:UncommunicativeName, /block/, /'q'/)
-    ruby.report.should have_at_most(116).smells
-  end
-end

data/spec/slow/redcloth_spec.rb

@@ -1,101 +0,0 @@
-require File.dirname(__FILE__) + '/../spec_helper.rb'
-
-describe 'sample gem source code' do
-  it "reports the correct smells in redcloth.rb" do
-    ruby = File.new("#{SAMPLES_DIR}/redcloth.rb").to_source
-    ruby.should reek_of(:ControlCouple, /RedCloth#blocks\/block/, /deep_code/)
-    ruby.should reek_of(:ControlCouple, /RedCloth#check_refs/, /text/)
-    ruby.should reek_of(:ControlCouple, /RedCloth#pba/, /text_in/)
-    ruby.should reek_of(:ControlCouple, /RedCloth#textile_bq/, /atts/)
-    ruby.should reek_of(:ControlCouple, /RedCloth#textile_bq/, /cite/)
-    ruby.should reek_of(:ControlCouple, /RedCloth#textile_fn_/, /atts/)
-    ruby.should reek_of(:ControlCouple, /RedCloth#textile_p/, /atts/)
-    ruby.should reek_of(:Duplication, /RedCloth#block_textile_lists/, /depth.last/)
-    ruby.should reek_of(:Duplication, /RedCloth#block_textile_lists/, /depth.last.length/)
-    ruby.should reek_of(:Duplication, /RedCloth#block_textile_lists/, /depth\[i\]/)
-    ruby.should reek_of(:Duplication, /RedCloth#block_textile_lists/, /\(line_id - 1\)/)
-    ruby.should reek_of(:Duplication, /RedCloth#block_textile_lists/, /lines\[\(line_id - 1\)\]/)
-    ruby.should reek_of(:Duplication, /RedCloth#block_textile_lists/, /tl.length/)
-    ruby.should reek_of(:Duplication, /RedCloth#clean_html/, /tags\[tag\]/)
-    ruby.should reek_of(:Duplication, /RedCloth#pba/, /\$1.length/)
-    ruby.should reek_of(:Duplication, /RedCloth#rip_offtags/, /@pre_list.last/)
-    ruby.should reek_of(:Duplication, /RedCloth#rip_offtags/, /\(@pre_list.last << line\)/)
-    ruby.should reek_of(:Duplication, /RedCloth#rip_offtags/, /\(codepre - used_offtags.length\)/)
-    ruby.should reek_of(:Duplication, /RedCloth#rip_offtags/, /\(\(codepre - used_offtags.length\) > 0\)/)
-    ruby.should reek_of(:Duplication, /RedCloth#rip_offtags/, /codepre.zero?/)
-    ruby.should reek_of(:Duplication, /RedCloth#rip_offtags/, /htmlesc\(line, :NoQuotes\)/)
-    ruby.should reek_of(:Duplication, /RedCloth#rip_offtags/, /used_offtags.length/)
-    ruby.should reek_of(:Duplication, /RedCloth#rip_offtags/, /used_offtags\["notextile"\]/)
-    ruby.should reek_of(:FeatureEnvy, /RedCloth#block_markdown_atx/, /text/)
-    ruby.should reek_of(:FeatureEnvy, /RedCloth#block_markdown_rule/, /text/)
-    ruby.should reek_of(:FeatureEnvy, /RedCloth#block_markdown_setext/, /text/)
-    ruby.should reek_of(:FeatureEnvy, /RedCloth#block_textile_lists/, /depth/)
-    ruby.should reek_of(:FeatureEnvy, /RedCloth#clean_html/, /raw/)
-    ruby.should reek_of(:FeatureEnvy, /RedCloth#clean_html/, /tags/)
-    ruby.should reek_of(:FeatureEnvy, /RedCloth#clean_white_space/, /text/)
-    ruby.should reek_of(:FeatureEnvy, /RedCloth#flush_left/, /indt/)
-    ruby.should reek_of(:FeatureEnvy, /RedCloth#flush_left/, /text/)
-    ruby.should reek_of(:FeatureEnvy, /RedCloth#footnote_ref/, /text/)
-    ruby.should reek_of(:FeatureEnvy, /RedCloth#htmlesc/, /str/)
-    ruby.should reek_of(:FeatureEnvy, /RedCloth#incoming_entities/, /text/)
-    ruby.should reek_of(:FeatureEnvy, /RedCloth#no_textile/, /text/)
-    ruby.should reek_of(:FeatureEnvy, /RedCloth#pba/, /style/)
-    ruby.should reek_of(:FeatureEnvy, /RedCloth#pba/, /text/)
-    ruby.should reek_of(:LargeClass, /RedCloth/)
-    ruby.should reek_of(:LongMethod, /RedCloth#block_markdown_bq/)
-    ruby.should reek_of(:LongMethod, /RedCloth#block_textile_lists/)
-    ruby.should reek_of(:LongMethod, /RedCloth#block_textile_table/)
-    ruby.should reek_of(:LongMethod, /RedCloth#blocks/)
-    ruby.should reek_of(:LongMethod, /RedCloth#clean_html/)
-    ruby.should reek_of(:LongMethod, /RedCloth#clean_white_space/)
-    ruby.should reek_of(:LongMethod, /RedCloth#glyphs_textile/)
-    ruby.should reek_of(:LongMethod, /RedCloth#inline_markdown_link/)
-    ruby.should reek_of(:LongMethod, /RedCloth#inline_markdown_reflink/)
-    ruby.should reek_of(:LongMethod, /RedCloth#inline_textile_image/)
-    ruby.should reek_of(:LongMethod, /RedCloth#inline_textile_link/)
-    ruby.should reek_of(:LongMethod, /RedCloth#inline_textile_span/)
-    ruby.should reek_of(:LongMethod, /RedCloth#pba/)
-    ruby.should reek_of(:LongMethod, /RedCloth#rip_offtags/)
-    ruby.should reek_of(:LongMethod, /RedCloth#to_html/)
-    ruby.should reek_of(:LongParameterList, /RedCloth#textile_bq/)
-    ruby.should reek_of(:LongParameterList, /RedCloth#textile_fn_/)
-    ruby.should reek_of(:LongParameterList, /RedCloth#textile_p/)
-    ruby.should reek_of(:NestedIterators, /RedCloth#block_textile_lists/)
-    ruby.should reek_of(:NestedIterators, /RedCloth#block_textile_lists/)
-    ruby.should reek_of(:NestedIterators, /RedCloth#block_textile_table/)
-    ruby.should reek_of(:NestedIterators, /RedCloth#block_textile_table/)
-    ruby.should reek_of(:NestedIterators, /RedCloth#blocks/)
-    ruby.should reek_of(:NestedIterators, /RedCloth#clean_html/)
-    ruby.should reek_of(:NestedIterators, /RedCloth#clean_html/)
-    ruby.should reek_of(:NestedIterators, /RedCloth#inline/)
-    ruby.should reek_of(:NestedIterators, /RedCloth#inline_textile_span/)
-    ruby.should reek_of(:UncommunicativeName, /RedCloth#block/, /'a'/)
-    ruby.should reek_of(:UncommunicativeName, /RedCloth#block/, /'b'/)
-    ruby.should reek_of(:UncommunicativeName, /RedCloth#block_textile_lists/, /'i'/)
-    ruby.should reek_of(:UncommunicativeName, /RedCloth#block_textile_lists/, /'v'/)
-    ruby.should reek_of(:UncommunicativeName, /RedCloth#block_textile_table/, /'x'/)
-    ruby.should reek_of(:UncommunicativeName, /RedCloth#clean_html/, /'q'/)
-    ruby.should reek_of(:UncommunicativeName, /RedCloth#clean_html/, /'q2'/)
-    ruby.should reek_of(:UncommunicativeName, /RedCloth#initialize/, /'r'/)
-    ruby.should reek_of(:UncommunicativeName, /RedCloth#inline_markdown_link/, /'m'/)
-    ruby.should reek_of(:UncommunicativeName, /RedCloth#inline_markdown_reflink/, /'m'/)
-    ruby.should reek_of(:UncommunicativeName, /RedCloth#inline_textile_code/, /'m'/)
-    ruby.should reek_of(:UncommunicativeName, /RedCloth#inline_textile_image/, /'m'/)
-    ruby.should reek_of(:UncommunicativeName, /RedCloth#inline_textile_link/, /'m'/)
-    ruby.should reek_of(:UncommunicativeName, /RedCloth#inline_textile_span/, /'m'/)
-    ruby.should reek_of(:UncommunicativeName, /RedCloth#refs_markdown/, /'m'/)
-    ruby.should reek_of(:UncommunicativeName, /RedCloth#refs_textile/, /'m'/)
-    ruby.should reek_of(:UncommunicativeName, /RedCloth#retrieve/, /'i'/)
-    ruby.should reek_of(:UncommunicativeName, /RedCloth#retrieve/, /'r'/)
-    ruby.should reek_of(:UtilityFunction, /RedCloth#block_markdown_rule/)
-    ruby.should reek_of(:UtilityFunction, /RedCloth#clean_html/)
-    ruby.should reek_of(:UtilityFunction, /RedCloth#flush_left/)
-    ruby.should reek_of(:UtilityFunction, /RedCloth#footnote_ref/)
-    ruby.should reek_of(:UtilityFunction, /RedCloth#h_align/)
-    ruby.should reek_of(:UtilityFunction, /RedCloth#htmlesc/)
-    ruby.should reek_of(:UtilityFunction, /RedCloth#incoming_entities/)
-    ruby.should reek_of(:UtilityFunction, /RedCloth#no_textile/)
-    ruby.should reek_of(:UtilityFunction, /RedCloth#v_align/)
-    ruby.report.should have_at_most(93).smells
-  end
-end

data/spec/slow/reek_source_spec.rb

@@ -1,20 +0,0 @@
-require File.dirname(__FILE__) + '/../spec_helper.rb'
-
-describe 'Reek source code:' do
-  Dir['lib/**/*.rb'].each do |path|
-    it "reports no smells in #{path}" do
-      File.new(path).should_not reek
-    end
-  end
-
-  it 'reports no smells via the Dir matcher' do
-    Dir['lib/**/*.rb'].should_not reek
-  end
-end
-
-describe 'RakeTask' do
-  it 'should report no duplication' do
-    report = `rake reek`.split("\n")
-    report.length.should == 1
-  end
-end

data/spec/slow/script_spec.rb

@@ -1,55 +0,0 @@
-require 'reek'
-
-describe 'version number' do
-  it 'should report the correct value' do
-    actual = `ruby -Ilib bin/reek --version`.split
-    $?.exitstatus.should == 0
-    actual[0].should == 'reek'
-    actual[1].should == Reek::VERSION
-  end
-end
-
-describe 'exit status', 'when reek is used incorrectly' do
-  it 'should return non-zero status on bad option' do
-    `ruby -Ilib bin/reek --no-such-option`
-    $?.exitstatus.should == 1
-  end
-
-  it 'should complain about missing file' do
-    `ruby -Ilib bin/reek nosuchfile.rb`
-    $?.exitstatus.should == 1
-  end
-
-  it 'should return non-zero status on missing argument' do
-    `ruby -Ilib bin/reek -f 2>/dev/null`
-    $?.exitstatus.should == 1
-  end
-end
-
-describe 'exit status', 'when reek is used correctly' do
-  it 'should return non-zero status when smells are reported' do
-    `echo "def x() 3; end" | ruby -Ilib bin/reek`
-    $?.exitstatus.should == 2
-  end
-
-  it 'should return zero status with no smells' do
-    `echo "def simple() @fred = 3 end" | ruby -Ilib bin/reek`
-    $?.exitstatus.should == 0
-  end
-end
-
-describe 'report format', 'with one source' do
-  it 'should output nothing with empty source' do
-    `echo "" | ruby -Ilib bin/reek`.should be_empty
-  end
-
-  it 'should output nothing when no smells' do
-    `echo "def simple() @fred = 3; end" | ruby -Ilib bin/reek`.should be_empty
-  end
-
-  it 'should not adorn the list of warnings' do
-    report = `echo  "class Turn; def y() @x = 3; end end" | ruby -Ilib bin/reek`
-    report.split(/\n/).length.should == 2
-    report.should_not match(/\n\n/)
-  end
-end

data/spec/slow/source_list_spec.rb

@@ -1,40 +0,0 @@
-require File.dirname(__FILE__) + '/../spec_helper.rb'
-
-require 'reek/source'
-
-include Reek
-
-describe SourceList, 'from_pathlist' do
-
-  describe 'with no smells in any source' do
-    before :each do
-      @src = Dir['lib/reek/*.rb'].to_source
-    end
-
-    it 'reports no smells' do
-      @src.report.length.should == 0
-    end
-
-    it 'is empty' do
-      @src.report.should be_empty
-    end
-  end
-
-  describe 'with smells in one source' do
-    before :each do
-      @src = Source.from_pathlist(["#{SAMPLES_DIR}/inline.rb", 'lib/reek.rb'])
-    end
-
-    it 'reports some smells in the samples' do
-      @src.report.should have_at_least(30).smells
-    end
-
-    it 'is smelly' do
-      @src.should be_smelly
-    end
-
-    it 'reports an UncommunicativeName' do
-      @src.report.any? {|warning| warning.report =~ /Uncommunicative Name/}.should be_true
-    end
-  end
-end

data/tasks/rspec.rake

@@ -1,21 +0,0 @@
-require 'spec'
-require 'spec/rake/spectask'
-
-namespace 'rspec' do
-  FAST = FileList['spec/reek/**/*_spec.rb']
-  SLOW = FileList['spec/slow/**/*_spec.rb']
-
-  Spec::Rake::SpecTask.new('fast') do |t|
-    t.spec_files = FAST
-    t.ruby_opts = ['-Ilib']
-    t.rcov = false
-  end
-
-  Spec::Rake::SpecTask.new('all') do |t|
-    t.spec_files = FAST + SLOW
-    t.rcov = false
-  end
-end
-
-desc 'runs the unit tests'
-task 'spec' => 'rspec:fast'