kevinrutherford-reek 1.1.3.10 → 1.1.3.11

data/lib/reek/core_extras.rb

@@ -1,50 +0,0 @@
-require 'reek/source'         # SMELL: should refer to Sniffer
-require 'reek/sniffer'
-
-class File
-  #
-  # Creates a new +Sniffer+ that assumes this File contains Ruby source
-  # code and examines that code for smells.
-  #
-  def sniff
-    result = Reek::Sniffer.new
-    Reek::Source.from_path(self.path, result)
-    result
-  end
-end
-
-class IO
-  #
-  # Creates a new +Sniffer+ that assumes this IO stream contains Ruby source
-  # code and examines that code for smells.
-  #
-  def sniff(description = 'io')
-    code = self.readlines.join
-    result = Reek::Sniffer.new
-    Reek::Source.new(code, description, result)
-    result
-  end
-end
-
-class String
-  #
-  # Creates a new +Sniffer+ that assumes this String contains Ruby source
-  # code and examines that code for smells.
-  #
-  def sniff
-    result = Reek::Sniffer.new
-    Reek::Source.new(self, 'string', result)
-    result
-  end
-end
-
-class Array
-  #
-  # Creates a new +Sniffer+ that assumes this Array contains the names
-  # of Ruby source files and examines those files for smells.
-  #
-  def sniff
-    sniffers = self.map {|path| File.new(path).sniff }
-    Reek::SnifferSet.new(sniffers, 'dir')
-  end
-end

data/lib/reek/options.rb

@@ -1,103 +0,0 @@
-require 'optparse'
-require 'reek'
-require 'reek/source'
-require 'reek/core_extras'       # SMELL
-
-module Reek
-    
-  class Options
-
-    CTX_SORT = '%m%c %w (%s)'
-    SMELL_SORT = '%m[%s] %c %w'
-
-    def self.default_options
-      {
-        :format => CTX_SORT,
-        :show_all => false,
-        :quiet => false
-      }
-    end
-    
-    @@opts = default_options
-
-    def self.[](key)
-      @@opts[key]
-    end
-
-    def self.parse_args(args)
-      result = default_options
-      parser = OptionParser.new { |opts| set_options(opts, result) }
-      parser.parse!(args)
-      result
-    end
-
-    def self.set_options(opts, config)
-      opts.banner = <<EOB
-Usage: #{opts.program_name} [options] files...
-
-If no files are given, Reek reads source code from standard input.
-See http://wiki.github.com/kevinrutherford/reek for detailed help.
-EOB
-      
-      opts.separator "\nOptions:"
-      set_all_options(opts, config)
-    end
-
-    # SMELL: Greedy Module
-    # This creates the command-line parser AND invokes it. And for the
-    # -v and -h options it also executes them. And it holds the config
-    # options for the rest of the application.
-    def self.parse(args)
-      @@opts = parse_args(args)
-      if args.length > 0
-        return args.sniff
-      else
-        return $stdin.sniff('$stdin')
-      end
-    end
-
-  private
-
-    def self.set_all_options(opts, config)
-      set_show_all_option(opts, config)
-      set_help_option(opts)
-      set_sort_option(config, opts)
-      set_version_option(opts)
-    end
-    
-    def self.set_version_option(opts)
-      opts.on("-v", "--version", "Show version") do
-        puts "#{opts.program_name} #{Reek::VERSION}"
-        exit(0)
-      end
-    end
-
-    def self.set_show_all_option(opts, config)
-      opts.on("-a", "--[no-]show-all", "Show all smells, including those masked by config settings") do |opt|
-        config[:show_all] = opt
-      end
-      opts.on("-q", "--quiet", "Suppress headings for smell-free source files") do
-        config[:quiet] = true
-      end
-    end
-
-    def self.set_help_option(opts)
-      opts.on("-h", "--help", "Show this message") do
-        puts opts
-        exit(0)
-      end
-    end
-
-    def self.set_sort_option(config, opts)
-      opts.on('-f', "--format FORMAT", 'Specify the format of smell warnings') do |arg|
-        config[:format] = arg unless arg.nil?
-      end
-      opts.on('-c', '--context-first', "Sort by context; sets the format string to \"#{CTX_SORT}\"") do
-        config[:format] = CTX_SORT
-      end
-      opts.on('-s', '--smell-first', "Sort by smell; sets the format string to \"#{SMELL_SORT}\"") do
-        config[:format] = SMELL_SORT
-      end
-    end
-  end
-end

data/lib/reek/report.rb

@@ -1,115 +0,0 @@
-require 'set'
-require 'reek/sniffer'
-
-module Reek
-  class Report
-    include Enumerable
-
-    def initialize(sniffer)  # :nodoc:
-      @masked_warnings = SortedSet.new
-      @warnings = SortedSet.new
-      @desc = sniffer.desc
-      sniffer.report_on(self)
-    end
-
-    #
-    # Checks this report for instances of +smell_class+, and returns +true+
-    # only if one of them has a report string matching all of the +patterns+.
-    #
-    def has_smell?(smell_class, patterns)
-      @warnings.any? { |smell| smell.matches?(smell_class, patterns) }
-    end
-
-    def <<(smell)  # :nodoc:
-      @warnings << smell
-      true
-    end
-
-    def record_masked_smell(smell)
-      @masked_warnings << smell
-    end
-
-    def num_masked_smells
-      @masked_warnings.length
-    end
-    
-    def empty?
-      @warnings.empty?
-    end
-
-    def length
-      @warnings.length
-    end
-    
-    alias size length
-
-    # Creates a formatted report of all the +Smells::SmellWarning+ objects recorded in
-    # this report, with a heading.
-    def full_report
-      return quiet_report if Options[:quiet]
-      result = header(@warnings.length)
-      result += ":\n#{smell_list}" if should_report
-      result += "\n"
-      result
-    end
-
-    def quiet_report
-      return '' unless should_report
-      "#{header(@warnings.length)}:\n#{smell_list}\n"
-    end
-
-    def should_report
-      @warnings.length > 0 or (Options[:show_all] and @masked_warnings.length > 0)
-    end
-
-    def header(num_smells)
-      result = "#{@desc} -- #{num_smells} warning"
-      result += 's' unless num_smells == 1
-      result += " (+#{@masked_warnings.length} masked)" unless @masked_warnings.empty?
-      result
-    end
-
-    # Creates a formatted report of all the +Smells::SmellWarning+ objects recorded in
-    # this report.
-    def smell_list
-      all = SortedSet.new(@warnings)
-      all.merge(@masked_warnings) if Options[:show_all]
-      all.map {|smell| "  #{smell.report}"}.join("\n")
-    end
-  end
-
-  class ReportList
-    include Enumerable
-
-    def initialize(sniffers)
-      @sniffers = sniffers
-    end
-
-    def empty?
-      length == 0
-    end
-
-    def length
-      @sniffers.inject(0) {|sum, sniffer| sum + sniffer.num_smells }
-    end
-
-    # SMELL: Shotgun Surgery
-    # This method and the next will have to be repeated for every new
-    # kind of report.
-    def full_report
-      @sniffers.map { |sniffer| sniffer.full_report }.join
-    end
-
-    def quiet_report
-      @sniffers.map { |sniffer| sniffer.quiet_report }.join
-    end
-
-    #
-    # Checks this report for instances of +smell_class+, and returns +true+
-    # only if one of them has a report string matching all of the +patterns+.
-    #
-    def has_smell?(smell_class, patterns)
-      @sniffers.any? { |sniffer| sniffer.has_smell?(smell_class, patterns) }
-    end
-  end
-end

data/lib/reek/source.rb

@@ -1,36 +0,0 @@
-require 'reek/sniffer'
-
-module Reek
-
-  #
-  # A +Source+ object represents a chunk of Ruby source code.
-  #
-  class Source
-
-    #
-    # Factory method: creates a +Source+ object by reading Ruby code from
-    # the named file. The source code is not parsed until +report+ is called.
-    #
-    def self.from_path(filename, sniffer)
-      code = IO.readlines(filename).join
-      # SMELL: Greedy Method
-      # The Sniffer should ask this source to configure it.
-      sniffer.configure_along_path(filename)
-      return new(code, filename, sniffer)
-    end
-
-    attr_reader :desc
-    attr_reader :sniffer          # SMELL -- bidirectional link
-
-    def initialize(code, desc, sniffer)     # :nodoc:
-      @source = code
-      @desc = desc
-      @sniffer = sniffer
-      @sniffer.source = self
-    end
-
-    def syntax_tree
-      RubyParser.new.parse(@source, @desc) || s()
-    end
-  end
-end

data/spec/reek/options_spec.rb

@@ -1,13 +0,0 @@
-require File.dirname(__FILE__) + '/../spec_helper.rb'
- 
-require 'reek/options'
-
-include Reek
-
-describe Options, ' when given no arguments' do
-  it "should retain the default sort order" do
-    default_order = Options[:format]
-    Options.parse_args(['nosuchfile.rb'])
-    Options[:format].should == default_order
-  end
-end

data/spec/reek/report_spec.rb

@@ -1,49 +0,0 @@
-require File.dirname(__FILE__) + '/../spec_helper.rb'
-
-require 'reek/smells/smell_detector'
-require 'reek/report'
-require 'reek/source'
-require 'reek/smells/feature_envy'
-
-include Reek
-
-describe Report, " when empty" do
-  before(:each) do
-    @rpt = Report.new(Sniffer.new)
-  end
-
-  it 'should have zero length' do
-    @rpt.length.should == 0
-  end
-
-  it 'should claim to be empty' do
-    @rpt.should be_empty
-  end
-
-  it 'has an empty quiet_report' do
-    @rpt.quiet_report.should == ''
-  end
-end
-
-describe Report, "smell_list" do
-  before(:each) do
-    rpt = 'def simple(a) a[3] end'.sniff.report
-    @report = rpt.smell_list.split("\n")
-  end
-
-  it 'should mention every smell name' do
-    @report.should have_at_least(2).lines
-    @report[0].should match(/[Utility Function]/)
-    @report[1].should match(/[Feature Envy]/)
-  end
-end
-
-describe Report, " as a SortedSet" do
-  it 'should only add a smell once' do
-    rpt = Report.new(Sniffer.new)
-    rpt << SmellWarning.new(Smells::FeatureEnvy.new, "self", 'too many!')
-    rpt.length.should == 1
-    rpt << SmellWarning.new(Smells::FeatureEnvy.new, "self", 'too many!')
-    rpt.length.should == 1
-  end
-end

data/spec/slow/inline_spec.rb

@@ -1,44 +0,0 @@
-require File.dirname(__FILE__) + '/../spec_helper.rb'
-
-require 'reek/report'
-
-include Reek
-
-describe 'sample gem source code' do
-  it "reports the correct smells in inline.rb" do
-    ruby = File.new("#{SAMPLES_DIR}/inline.rb").sniff
-    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, /Inline::C#build/)
-    ruby.should reek_of(:LongMethod, /Inline::C#generate/)
-    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/)
-    Report.new(ruby).length.should == 32
-  end
-end

data/spec/slow/optparse_spec.rb

@@ -1,112 +0,0 @@
-require File.dirname(__FILE__) + '/../spec_helper.rb'
-
-require 'reek/report'
-
-include Reek
-
-describe 'sample gem source code' do
-  it "reports the correct smells in optparse.rb" do
-    ruby = File.new("#{SAMPLES_DIR}/optparse.rb").sniff
-    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#PlacedArgument#parse/)
-    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#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'/)
-    Report.new(ruby).length.should == 117
-  end
-end

data/spec/slow/redcloth_spec.rb

@@ -1,105 +0,0 @@
-require File.dirname(__FILE__) + '/../spec_helper.rb'
-
-require 'reek/report'
-
-include Reek
-
-describe 'sample gem source code' do
-  it "reports the correct smells in redcloth.rb" do
-    ruby = File.new("#{SAMPLES_DIR}/redcloth.rb").sniff
-    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/)
-    Report.new(ruby).length.should == 93
-  end
-end

data/spec/slow/reek_source_spec.rb

@@ -1,13 +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