reek 2.0.4 → 3.0.0

data/spec/reek/smells/duplicate_method_call_spec.rb

@@ -1,10 +1,10 @@
-require 'spec_helper'
-require 'reek/smells/duplicate_method_call'
-require 'reek/core/code_context'
-require 'reek/core/tree_walker'
-require 'reek/smells/smell_detector_shared'
+require_relative '../../spec_helper'
+require_relative '../../../lib/reek/smells/duplicate_method_call'
+require_relative '../../../lib/reek/context/code_context'
+require_relative '../../../lib/reek/tree_walker'
+require_relative 'smell_detector_shared'
 
-describe Reek::Smells::DuplicateMethodCall do
+RSpec.describe Reek::Smells::DuplicateMethodCall do
   context 'when a smell is reported' do
     before :each do
       @source_name = 'dummy_source'
@@ -16,7 +16,7 @@ describe Reek::Smells::DuplicateMethodCall do
           other[@thing]
         end
       EOS
-      ctx = Reek::Core::CodeContext.new(nil, src.to_reek_source.syntax_tree)
+      ctx = Reek::Context::CodeContext.new(nil, Reek::Source::SourceCode.from(src).syntax_tree)
       smells = @detector.examine_context(ctx)
       expect(smells.length).to eq(1)
       @warning = smells[0]

data/spec/reek/smells/feature_envy_spec.rb

@@ -1,8 +1,8 @@
-require 'spec_helper'
-require 'reek/smells/feature_envy'
-require 'reek/smells/smell_detector_shared'
+require_relative '../../spec_helper'
+require_relative '../../../lib/reek/smells/feature_envy'
+require_relative 'smell_detector_shared'
 
-describe Reek::Smells::FeatureEnvy do
+RSpec.describe Reek::Smells::FeatureEnvy do
   context 'with no smell' do
     it 'should not report use of self' do
       expect('def simple() self.to_s + self.to_i end').not_to reek_of(:FeatureEnvy)
@@ -74,7 +74,7 @@ describe Reek::Smells::FeatureEnvy do
         def envy(arga)
           arga.b(arga) + arga.c(@fred)
         end
-      ').to reek_only_of(:FeatureEnvy)
+      ').to reek_of(:FeatureEnvy, name: 'arga')
     end
   end
 
@@ -88,7 +88,8 @@ describe Reek::Smells::FeatureEnvy do
         total *= 1.15
       end
       EOS
-    expect(src).to reek_only_of(:FeatureEnvy)
+    expect(src).to reek_of(:FeatureEnvy, name: 'total')
+    expect(src).not_to reek_of(:FeatureEnvy, name: 'fred')
   end
 
   it 'should report multiple affinities' do
@@ -100,8 +101,8 @@ describe Reek::Smells::FeatureEnvy do
         total += fred.tax
       end
       EOS
-    expect(src).to reek_of(:FeatureEnvy,  name: 'total')
-    expect(src).to reek_of(:FeatureEnvy,  name: 'fred')
+    expect(src).to reek_of(:FeatureEnvy, name: 'total')
+    expect(src).to reek_of(:FeatureEnvy, name: 'fred')
   end
 
   it 'should not be fooled by duplication' do
@@ -179,9 +180,7 @@ describe Reek::Smells::FeatureEnvy do
     EOS
     expect(src).not_to reek_of(:FeatureEnvy)
   end
-end
 
-describe Reek::Smells::FeatureEnvy do
   it 'counts references to self correctly' do
     ruby = <<-EOS
       def report
@@ -212,7 +211,7 @@ describe Reek::Smells::FeatureEnvy do
   end
 end
 
-describe Reek::Smells::FeatureEnvy do
+RSpec.describe Reek::Smells::FeatureEnvy do
   before(:each) do
     @source_name = 'dummy_source'
     @detector = build(:smell_detector, smell_type: :FeatureEnvy, source: @source_name)
@@ -231,8 +230,8 @@ describe Reek::Smells::FeatureEnvy do
           #{@receiver}.fred
         end
       EOS
-      source = src.to_reek_source
-      @mctx = Reek::Core::TreeWalker.new.process_def(source.syntax_tree)
+      source = Reek::Source::SourceCode.from(src)
+      @mctx = Reek::TreeWalker.new.process_def(source.syntax_tree)
       @smells = @detector.examine_context(@mctx)
     end
 

data/spec/reek/smells/irresponsible_module_spec.rb

@@ -1,17 +1,9 @@
-require 'spec_helper'
-require 'reek/core/code_context'
-require 'reek/smells/irresponsible_module'
-require 'reek/smells/smell_detector_shared'
-
-describe Reek::Smells::IrresponsibleModule do
-  before(:each) do
-    @bad_module_name = 'WrongUn'
-    @source_name = 'dummy_source'
-    @detector = build(:smell_detector, smell_type: :IrresponsibleModule, source: @source_name)
-  end
-
-  it_should_behave_like 'SmellDetector'
+require_relative '../../spec_helper'
+require_relative '../../../lib/reek/context/code_context'
+require_relative '../../../lib/reek/smells/irresponsible_module'
+require_relative 'smell_detector_shared'
 
+RSpec.describe Reek::Smells::IrresponsibleModule do
   it 'does not report re-opened modules' do
     src = <<-EOS
       # Abstract base class
@@ -27,19 +19,12 @@ describe Reek::Smells::IrresponsibleModule do
       # test class
       class Responsible; end
     EOS
-    ctx = Reek::Core::CodeContext.new(nil, src.to_reek_source.syntax_tree)
-    expect(@detector.examine_context(ctx)).to be_empty
+    expect(src).not_to reek_of(:IrresponsibleModule)
   end
 
   it 'reports a class without a comment' do
-    src = "class #{@bad_module_name}; end"
-    ctx = Reek::Core::CodeContext.new(nil, src.to_reek_source.syntax_tree)
-    smells = @detector.examine_context(ctx)
-    expect(smells.length).to eq(1)
-    expect(smells[0].smell_category).to eq(Reek::Smells::IrresponsibleModule.smell_category)
-    expect(smells[0].smell_type).to eq(Reek::Smells::IrresponsibleModule.smell_type)
-    expect(smells[0].lines).to eq([1])
-    expect(smells[0].parameters[:name]).to eq(@bad_module_name)
+    src = 'class BadClass; end'
+    expect(src).to reek_of :IrresponsibleModule, name: 'BadClass'
   end
 
   it 'reports a class with an empty comment' do
@@ -47,7 +32,7 @@ describe Reek::Smells::IrresponsibleModule do
       #
       #
       #
-      class #{@bad_module_name}; end
+      class BadClass; end
     EOS
     expect(src).to reek_of :IrresponsibleModule
   end
@@ -63,14 +48,25 @@ describe Reek::Smells::IrresponsibleModule do
     expect(src).to reek_of(:IrresponsibleModule)
   end
 
-  it 'reports a fq module name correctly' do
+  it 'reports a class with a trailing comment' do
+    src = <<-EOS
+      class BadClass
+      end # end BadClass
+    EOS
+    expect(src).to reek_of(:IrresponsibleModule)
+  end
+
+  it 'reports a fully qualified class name correctly' do
     src = 'class Foo::Bar; end'
-    ctx = Reek::Core::CodeContext.new(nil, src.to_reek_source.syntax_tree)
-    smells = @detector.examine_context(ctx)
-    expect(smells.length).to eq(1)
-    expect(smells[0].smell_category).to eq(described_class.smell_category)
-    expect(smells[0].smell_type).to eq(described_class.smell_type)
-    expect(smells[0].parameters[:name]).to eq('Foo::Bar')
-    expect(smells[0].context).to match(/#{smells[0].parameters['name']}/)
+    expect(src).to reek_of :IrresponsibleModule, name: 'Foo::Bar'
+  end
+
+  context 'when a smell is reported' do
+    before do
+      @source_name = 'dummy_source'
+      @detector = build(:smell_detector, smell_type: :IrresponsibleModule, source: @source_name)
+    end
+
+    it_should_behave_like 'SmellDetector'
   end
 end

data/spec/reek/smells/long_parameter_list_spec.rb

@@ -1,9 +1,9 @@
-require 'spec_helper'
-require 'reek/core/code_context'
-require 'reek/smells/long_parameter_list'
-require 'reek/smells/smell_detector_shared'
+require_relative '../../spec_helper'
+require_relative '../../../lib/reek/context/code_context'
+require_relative '../../../lib/reek/smells/long_parameter_list'
+require_relative 'smell_detector_shared'
 
-describe Reek::Smells::LongParameterList do
+RSpec.describe Reek::Smells::LongParameterList do
   context 'for methods with few parameters' do
     it 'should report nothing for no parameters' do
       expect('def simple; f(3);true; end').not_to reek_of(:LongParameterList)
@@ -73,7 +73,7 @@ describe Reek::Smells::LongParameterList do
   end
 end
 
-describe Reek::Smells::LongParameterList do
+RSpec.describe Reek::Smells::LongParameterList do
   before(:each) do
     @source_name = 'dummy_source'
     @detector = build(:smell_detector, smell_type: :LongParameterList, source: @source_name)
@@ -89,7 +89,7 @@ describe Reek::Smells::LongParameterList do
           true
         end
       EOS
-      ctx = Reek::Core::CodeContext.new(nil, src.to_reek_source.syntax_tree)
+      ctx = Reek::Context::CodeContext.new(nil, Reek::Source::SourceCode.from(src).syntax_tree)
       @smells = @detector.examine_context(ctx)
       @warning = @smells[0]
     end

data/spec/reek/smells/long_yield_list_spec.rb

@@ -1,9 +1,9 @@
-require 'spec_helper'
-require 'reek/core/code_context'
-require 'reek/smells/long_yield_list'
-require 'reek/smells/smell_detector_shared'
+require_relative '../../spec_helper'
+require_relative '../../../lib/reek/context/code_context'
+require_relative '../../../lib/reek/smells/long_yield_list'
+require_relative 'smell_detector_shared'
 
-describe Reek::Smells::LongYieldList do
+RSpec.describe Reek::Smells::LongYieldList do
   before(:each) do
     @source_name = 'dummy_source'
     @detector = build(:smell_detector, smell_type: :LongYieldList, source: @source_name)
@@ -38,7 +38,7 @@ describe Reek::Smells::LongYieldList do
           yield(arga,argb,arga,argb)
           end
       EOS
-      ctx = Reek::Core::CodeContext.new(nil, src.to_reek_source.syntax_tree)
+      ctx = Reek::Context::CodeContext.new(nil, Reek::Source::SourceCode.from(src).syntax_tree)
       @smells = @detector.examine_context(ctx)
       @warning = @smells[0]
     end

data/spec/reek/smells/module_initialize_spec.rb

@@ -1,8 +1,8 @@
-require 'spec_helper'
-require 'reek/smells/module_initialize'
-require 'reek/smells/smell_detector_shared'
+require_relative '../../spec_helper'
+require_relative '../../../lib/reek/smells/module_initialize'
+require_relative 'smell_detector_shared'
 
-describe Reek::Smells::ModuleInitialize do
+RSpec.describe Reek::Smells::ModuleInitialize do
   context 'module' do
     context 'with method named initialize' do
       it 'smells' do

data/spec/reek/smells/nested_iterators_spec.rb

@@ -1,8 +1,8 @@
-require 'spec_helper'
-require 'reek/smells/nested_iterators'
-require 'reek/smells/smell_detector_shared'
+require_relative '../../spec_helper'
+require_relative '../../../lib/reek/smells/nested_iterators'
+require_relative 'smell_detector_shared'
 
-describe Reek::Smells::NestedIterators do
+RSpec.describe Reek::Smells::NestedIterators do
   context 'with no iterators' do
     it 'reports no smells' do
       src = 'def fred() nothing = true; end'
@@ -220,7 +220,7 @@ describe Reek::Smells::NestedIterators do
   end
 end
 
-describe Reek::Smells::NestedIterators do
+RSpec.describe Reek::Smells::NestedIterators do
   before(:each) do
     @source_name = 'dummy_source'
     @detector = build(:smell_detector, smell_type: :NestedIterators, source: @source_name)
@@ -237,7 +237,7 @@ describe Reek::Smells::NestedIterators do
           end
         end
       EOS
-      ctx = Reek::Core::CodeContext.new(nil, src.to_reek_source.syntax_tree)
+      ctx = Reek::Context::CodeContext.new(nil, Reek::Source::SourceCode.from(src).syntax_tree)
       @warning = @detector.examine_context(ctx)[0]
     end
 

data/spec/reek/smells/nil_check_spec.rb

@@ -1,9 +1,9 @@
-require 'spec_helper'
-require 'reek/core/code_context'
-require 'reek/smells/nil_check'
-require 'reek/smells/smell_detector_shared'
+require_relative '../../spec_helper'
+require_relative '../../../lib/reek/context/code_context'
+require_relative '../../../lib/reek/smells/nil_check'
+require_relative 'smell_detector_shared'
 
-describe Reek::Smells::NilCheck do
+RSpec.describe Reek::Smells::NilCheck do
   context 'for methods' do
     it 'reports the correct line number' do
       src = <<-EOS
@@ -11,7 +11,7 @@ describe Reek::Smells::NilCheck do
         foo.nil?
       end
       EOS
-      ctx = Reek::Core::CodeContext.new(nil, src.to_reek_source.syntax_tree)
+      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]

data/spec/reek/smells/prima_donna_method_spec.rb

@@ -1,8 +1,8 @@
-require 'spec_helper'
-require 'reek/core/module_context'
-require 'reek/smells/smell_detector_shared'
+require_relative '../../spec_helper'
+require_relative '../../../lib/reek/context/module_context'
+require_relative 'smell_detector_shared'
 
-describe Reek::Smells::PrimaDonnaMethod do
+RSpec.describe Reek::Smells::PrimaDonnaMethod do
   it 'should report nothing when method and bang counterpart exist' do
     expect('class C; def m; end; def m!; end; end').not_to reek_of(:PrimaDonnaMethod)
   end
@@ -14,7 +14,10 @@ describe Reek::Smells::PrimaDonnaMethod do
   describe 'the right smell' do
     let(:detector) { build(:smell_detector, smell_type: :PrimaDonnaMethod, source: 'source_name') }
     let(:src)      { 'class C; def m!; end; end' }
-    let(:ctx)      { Reek::Core::ModuleContext.new(nil, src.to_reek_source.syntax_tree) }
+    let(:ctx)      do
+      Reek::Context::ModuleContext.new(nil,
+                                       Reek::Source::SourceCode.from(src).syntax_tree)
+    end
 
     it 'should be reported' do
       smells = detector.examine_context(ctx)

data/spec/reek/smells/repeated_conditional_spec.rb

@@ -1,9 +1,10 @@
-require 'spec_helper'
-require 'reek/smells/repeated_conditional'
-require 'reek/core/code_context'
-require 'reek/smells/smell_detector_shared'
+require_relative '../../spec_helper'
+require_relative '../../../lib/reek/smells/repeated_conditional'
+require_relative '../../../lib/reek/context/code_context'
+require_relative 'smell_detector_shared'
+require_relative '../../../lib/reek/source/source_code'
 
-describe Reek::Smells::RepeatedConditional do
+RSpec.describe Reek::Smells::RepeatedConditional do
   before(:each) do
     @source_name = 'dummy_source'
     @detector = build(:smell_detector, smell_type: :RepeatedConditional, source: @source_name)
@@ -13,24 +14,24 @@ describe Reek::Smells::RepeatedConditional do
 
   context 'with no conditionals' do
     it 'gathers an empty hash' do
-      ast = 'module Stable; end'.to_reek_source.syntax_tree
-      ctx = Reek::Core::CodeContext.new(nil, ast)
+      ast = Reek::Source::SourceCode.from('module Stable; end').syntax_tree
+      ctx = Reek::Context::CodeContext.new(nil, ast)
       expect(@detector.conditional_counts(ctx).length).to eq(0)
     end
   end
 
   context 'with a test of block_given?' do
     it 'does not record the condition' do
-      ast = 'def fred() yield(3) if block_given?; end'.to_reek_source.syntax_tree
-      ctx = Reek::Core::CodeContext.new(nil, ast)
+      ast = Reek::Source::SourceCode.from('def fred() yield(3) if block_given?; end').syntax_tree
+      ctx = Reek::Context::CodeContext.new(nil, ast)
       expect(@detector.conditional_counts(ctx).length).to eq(0)
     end
   end
 
   context 'with an empty condition' do
     it 'does not record the condition' do
-      ast = 'def fred() case; when 3; end; end'.to_reek_source.syntax_tree
-      ctx = Reek::Core::CodeContext.new(nil, ast)
+      ast = Reek::Source::SourceCode.from('def fred() case; when 3; end; end').syntax_tree
+      ctx = Reek::Context::CodeContext.new(nil, ast)
       expect(@detector.conditional_counts(ctx).length).to eq(0)
     end
   end
@@ -38,7 +39,7 @@ describe Reek::Smells::RepeatedConditional do
   context 'with three identical conditionals' do
     before :each do
       @cond = '@field == :sym'
-      @cond_expr = @cond.to_reek_source.syntax_tree
+      @cond_expr = Reek::Source::SourceCode.from(@cond).syntax_tree
       src = <<-EOS
         class Scrunch
           def first
@@ -56,8 +57,8 @@ describe Reek::Smells::RepeatedConditional do
         end
       EOS
 
-      ast = src.to_reek_source.syntax_tree
-      @ctx = Reek::Core::CodeContext.new(nil, ast)
+      ast = Reek::Source::SourceCode.from(src).syntax_tree
+      @ctx = Reek::Context::CodeContext.new(nil, ast)
       @conds = @detector.conditional_counts(@ctx)
     end
 
@@ -77,7 +78,7 @@ describe Reek::Smells::RepeatedConditional do
   context 'with a matching if and case' do
     before :each do
       cond = '@field == :sym'
-      @cond_expr = cond.to_reek_source.syntax_tree
+      @cond_expr = Reek::Source::SourceCode.from(cond).syntax_tree
       src = <<-EOS
         class Scrunch
           def alpha
@@ -92,8 +93,8 @@ describe Reek::Smells::RepeatedConditional do
         end
       EOS
 
-      ast = src.to_reek_source.syntax_tree
-      ctx = Reek::Core::CodeContext.new(nil, ast)
+      ast = Reek::Source::SourceCode.from(src).syntax_tree
+      ctx = Reek::Context::CodeContext.new(nil, ast)
       @conds = @detector.conditional_counts(ctx)
     end
 

data/spec/reek/{core → smells}/smell_configuration_spec.rb

@@ -1,11 +1,9 @@
-require 'spec_helper'
-require 'reek/core/smell_configuration'
+require_relative '../../spec_helper'
+require_relative '../../../lib/reek/smells/smell_configuration'
 
-include Reek::Core
-
-describe SmellConfiguration do
+RSpec.describe Reek::Smells::SmellConfiguration do
   it 'returns the default value when key not found' do
-    cf = SmellConfiguration.new({})
+    cf = described_class.new({})
     expect(cf.value('fred', nil, 27)).to eq(27)
   end
 
@@ -14,7 +12,7 @@ describe SmellConfiguration do
       @base_config = { 'enabled' => true, 'exclude' => [],
                        'reject' => [/^.$/, /[0-9]$/, /[A-Z]/],
                        'accept' => ['_'] }
-      @smell_config = SmellConfiguration.new(@base_config)
+      @smell_config = described_class.new(@base_config)
     end
 
     it { expect(@smell_config.merge!({})).to eq(@base_config) }

data/spec/reek/smells/smell_detector_shared.rb

@@ -1,9 +1,7 @@
-require 'spec_helper'
-require 'reek/core/smell_configuration'
+require_relative '../../spec_helper'
+require_relative '../../../lib/reek/smells/smell_configuration'
 
-include Reek::Core
-
-shared_examples_for 'SmellDetector' do
+RSpec.shared_examples_for 'SmellDetector' do
   context 'exception matching follows the context' do
     before :each do
       @ctx = double('context')
@@ -23,13 +21,14 @@ shared_examples_for 'SmellDetector' do
 
   context 'configuration' do
     it 'becomes disabled when disabled' do
-      @detector.configure_with(SmellConfiguration::ENABLED_KEY => false)
+      enabled_key = Reek::Smells::SmellConfiguration::ENABLED_KEY
+      @detector.configure_with(enabled_key => false)
       expect(@detector).not_to be_enabled
     end
   end
 end
 
-shared_examples_for 'common fields set correctly' do
+RSpec.shared_examples_for 'common fields set correctly' do
   it 'reports the source' do
     expect(@warning.source).to eq(@source_name)
   end

data/spec/reek/smells/smell_repository_spec.rb

@@ -0,0 +1,29 @@
+require_relative '../../spec_helper'
+require_relative '../../../lib/reek/smells/smell_detector'
+require_relative '../../../lib/reek/smells/smell_repository'
+
+RSpec.describe Reek::Smells::SmellRepository do
+  describe '.smell_types' do
+    let(:smell_types) { described_class.smell_types }
+
+    it 'should include existing smell_types' do
+      expect(smell_types).to include(Reek::Smells::IrresponsibleModule)
+      expect(smell_types).to include(Reek::Smells::TooManyStatements)
+    end
+
+    it 'should exclude certain smell_types' do
+      expect(smell_types).to_not include(Reek::Smells::SmellDetector)
+    end
+
+    it 'should return the smell types in alphabetic order' do
+      expect(smell_types).to eq(smell_types.sort_by(&:name))
+    end
+
+    it "should raise an ArgumentError if smell to configure doesn't exist" do
+      repository = described_class.new
+      expect { repository.configure('SomethingNonExistant', {}) }.
+        to raise_error ArgumentError,
+                       'Unknown smell type SomethingNonExistant found in configuration'
+    end
+  end
+end

data/spec/reek/{smell_warning_spec.rb → smells/smell_warning_spec.rb}

@@ -1,9 +1,7 @@
-require 'spec_helper'
-require 'reek/smell_warning'
+require_relative '../../spec_helper'
+require_relative '../../../lib/reek/smells/smell_warning'
 
-include Reek
-
-describe SmellWarning do
+RSpec.describe Reek::Smells::SmellWarning do
   let(:duplication_detector)  { build(:smell_detector, smell_type: 'DuplicateMethodCall') }
   let(:feature_envy_detector) { build(:smell_detector, smell_type: 'FeatureEnvy') }
   let(:utility_function_detector) { build(:smell_detector, smell_type: 'UtilityFunction') }
@@ -119,10 +117,10 @@ describe SmellWarning do
         @smell_type = 'FeatureEnvy'
         @parameters = { 'one' => 34, 'two' => 'second' }
         @detector = Reek::Smells::FeatureEnvy.new @source
-        @warning = SmellWarning.new(@detector, context: @context_name,
-                                               lines: @lines,
-                                               message: @message,
-                                               parameters: @parameters)
+        @warning = Reek::Smells::SmellWarning.new(@detector, context: @context_name,
+                                                             lines: @lines,
+                                                             message: @message,
+                                                             parameters: @parameters)
         @yaml = @warning.yaml_hash
       end
 

data/spec/reek/smells/too_many_instance_variables_spec.rb

@@ -1,8 +1,8 @@
-require 'spec_helper'
-require 'reek/smells/too_many_instance_variables'
-require 'reek/smells/smell_detector_shared'
+require_relative '../../spec_helper'
+require_relative '../../../lib/reek/smells/too_many_instance_variables'
+require_relative 'smell_detector_shared'
 
-describe Reek::Smells::TooManyInstanceVariables do
+RSpec.describe Reek::Smells::TooManyInstanceVariables do
   before(:each) do
     @source_name = 'dummy_source'
     @detector = build(:smell_detector, smell_type: :TooManyInstanceVariables, source: @source_name)
@@ -72,7 +72,7 @@ describe Reek::Smells::TooManyInstanceVariables do
         end
       end
     EOS
-    ctx = Reek::Core::CodeContext.new(nil, src.to_reek_source.syntax_tree)
+    ctx = Reek::Context::CodeContext.new(nil, Reek::Source::SourceCode.from(src).syntax_tree)
     @warning = @detector.examine_context(ctx)[0]
     expect(@warning.source).to eq(@source_name)
     expect(@warning.smell_category).to eq(Reek::Smells::TooManyInstanceVariables.smell_category)

data/spec/reek/smells/too_many_methods_spec.rb

@@ -1,8 +1,8 @@
-require 'spec_helper'
-require 'reek/smells/too_many_methods'
-require 'reek/smells/smell_detector_shared'
+require_relative '../../spec_helper'
+require_relative '../../../lib/reek/smells/too_many_methods'
+require_relative 'smell_detector_shared'
 
-describe Reek::Smells::TooManyMethods do
+RSpec.describe Reek::Smells::TooManyMethods do
   before(:each) do
     @source_name = 'dummy_source'
     @detector = described_class.new(@source_name)
@@ -19,7 +19,8 @@ describe Reek::Smells::TooManyMethods do
           def m2; end
         end
       EOS
-      ctx = ModuleContext.new(nil, src.to_reek_source.syntax_tree)
+      syntax_tree = Reek::Source::SourceCode.from(src).syntax_tree
+      ctx = Reek::Context::ModuleContext.new(nil, syntax_tree)
       expect(@detector.examine_context(ctx)).to be_empty
     end
 
@@ -31,7 +32,8 @@ describe Reek::Smells::TooManyMethods do
           def m3; end
         end
       EOS
-      ctx = ModuleContext.new(nil, src.to_reek_source.syntax_tree)
+      syntax_tree = Reek::Source::SourceCode.from(src).syntax_tree
+      ctx = Reek::Context::ModuleContext.new(nil, syntax_tree)
       smells = @detector.examine_context(ctx)
       expect(smells.length).to eq(1)
       expect(smells[0].smell_type).to eq(described_class.smell_type)
@@ -53,7 +55,8 @@ describe Reek::Smells::TooManyMethods do
           end
         end
       EOS
-      ctx = ModuleContext.new(nil, src.to_reek_source.syntax_tree)
+      syntax_tree = Reek::Source::SourceCode.from(src).syntax_tree
+      ctx = Reek::Context::ModuleContext.new(nil, syntax_tree)
       expect(@detector.examine_context(ctx)).to be_empty
     end
   end
@@ -67,7 +70,8 @@ describe Reek::Smells::TooManyMethods do
       end
     EOS
 
-    ctx = ModuleContext.new(nil, src.to_reek_source.syntax_tree)
+    syntax_tree = Reek::Source::SourceCode.from(src).syntax_tree
+    ctx = Reek::Context::ModuleContext.new(nil, syntax_tree)
     @warning = @detector.examine_context(ctx)[0]
     expect(@warning.source).to eq(@source_name)
     expect(@warning.smell_category).to eq(described_class.smell_category)