reek 4.7.3 → 4.8.0

data/spec/reek/report/code_climate/code_climate_fingerprint_spec.rb

@@ -5,42 +5,42 @@ RSpec.describe Reek::Report::CodeClimateFingerprint do
   describe '#compute' do
     let(:computed) { described_class.new(warning).compute }
 
-    shared_examples_for 'computes a fingerprint with no parameters' do
+    context 'when fingerprinting a warning with no parameters' do
       let(:expected_fingerprint) { 'e68badd29db51c92363a7c6a2438d722' }
       let(:warning) do
-        FactoryGirl.build(:smell_warning,
-                          smell_detector: Reek::SmellDetectors::UtilityFunction.new,
-                          context:        'alfa',
-                          message:        "doesn't depend on instance state (maybe move it to another class?)",
-                          lines:          lines,
-                          source:         'a/ruby/source/file.rb')
+        build(:smell_warning,
+              smell_detector: Reek::SmellDetectors::UtilityFunction.new,
+              context:        'alfa',
+              message:        "doesn't depend on instance state (maybe move it to another class?)",
+              lines:          lines,
+              source:         'a/ruby/source/file.rb')
       end
 
-      it 'computes the fingerprint' do
-        expect(computed).to eq expected_fingerprint
-      end
-    end
-
-    context 'with code at a specific location' do
-      let(:lines) { [1] }
+      context 'with code at a specific location' do
+        let(:lines) { [1] }
 
-      it_behaves_like 'computes a fingerprint with no parameters'
-    end
+        it 'computes the fingerprint' do
+          expect(computed).to eq expected_fingerprint
+        end
+      end
 
-    context 'with code at a different location' do
-      let(:lines) { [5] }
+      context 'with code at a different location' do
+        let(:lines) { [5] }
 
-      it_behaves_like 'computes a fingerprint with no parameters'
+        it 'computes the same fingerprint' do
+          expect(computed).to eq expected_fingerprint
+        end
+      end
     end
 
     context 'when the fingerprint should not be computed' do
       let(:warning) do
-        FactoryGirl.build(:smell_warning,
-                          smell_detector: Reek::SmellDetectors::ManualDispatch.new,
-                          context:        'Alfa#bravo',
-                          message:        'manually dispatches method call',
-                          lines:          [4],
-                          source:         'a/ruby/source/file.rb')
+        build(:smell_warning,
+              smell_detector: Reek::SmellDetectors::ManualDispatch.new,
+              context:        'Alfa#bravo',
+              message:        'manually dispatches method call',
+              lines:          [4],
+              source:         'a/ruby/source/file.rb')
       end
 
       it 'returns nil' do
@@ -48,77 +48,79 @@ RSpec.describe Reek::Report::CodeClimateFingerprint do
       end
     end
 
-    shared_examples_for 'computes a fingerprint with identifying parameters' do
+    context 'when the smell warning has only identifying parameters' do
       let(:warning) do
-        FactoryGirl.build(:smell_warning,
-                          smell_detector: Reek::SmellDetectors::ClassVariable.new,
-                          context:        'Alfa',
-                          message:        "declares the class variable '@@#{name}'",
-                          lines:          [4],
-                          parameters:     { name: "@@#{name}" },
-                          source:         'a/ruby/source/file.rb')
+        build(:smell_warning,
+              smell_detector: Reek::SmellDetectors::ClassVariable.new,
+              context:        'Alfa',
+              message:        "declares the class variable '@@#{name}'",
+              lines:          [4],
+              parameters:     { name: "@@#{name}" },
+              source:         'a/ruby/source/file.rb')
       end
 
-      it 'computes the fingerprint' do
-        expect(computed).to eq expected_fingerprint
-      end
-    end
+      context 'when the name is one thing' do
+        let(:name) { 'bravo' }
+        let(:expected_fingerprint) { '9c3fd378178118a67e9509f87cae24f9' }
 
-    context 'when the name is one thing it has one fingerprint' do
-      let(:name) { 'bravo' }
-      let(:expected_fingerprint) { '9c3fd378178118a67e9509f87cae24f9' }
-
-      it_behaves_like 'computes a fingerprint with identifying parameters'
-    end
+        it 'computes the fingerprint' do
+          expect(computed).to eq expected_fingerprint
+        end
+      end
 
-    context 'when the name is another thing it has another fingerprint' do
-      let(:name) { 'echo' }
-      let(:expected_fingerprint) { 'd2a6d2703ce04cca65e7300b7de4b89f' }
+      context 'when the name is another thing' do
+        let(:name) { 'echo' }
+        let(:expected_fingerprint) { 'd2a6d2703ce04cca65e7300b7de4b89f' }
 
-      it_behaves_like 'computes a fingerprint with identifying parameters'
+        it 'computes another fingerprint' do
+          expect(computed).to eq expected_fingerprint
+        end
+      end
     end
 
-    shared_examples_for 'computes a fingerprint with identifying and non-identifying parameters' do
+    context 'when the smell warning has identifying and non-identifying parameters' do
       let(:warning) do
-        FactoryGirl.build(:smell_warning,
-                          smell_detector: Reek::SmellDetectors::DuplicateMethodCall.new,
-                          context:        "Alfa##{name}",
-                          message:        "calls '#{name}' #{count} times",
-                          lines:          lines,
-                          parameters:     { name: "@@#{name}", count: count },
-                          source:         'a/ruby/source/file.rb')
+        build(:smell_warning,
+              smell_detector: Reek::SmellDetectors::DuplicateMethodCall.new,
+              context:        "Alfa##{name}",
+              message:        "calls '#{name}' #{count} times",
+              lines:          lines,
+              parameters:     { name: "@@#{name}", count: count },
+              source:         'a/ruby/source/file.rb')
       end
 
-      it 'computes the fingerprint' do
-        expect(computed).to eq expected_fingerprint
-      end
-    end
-
-    context 'when the parameters are provided' do
-      let(:name) { 'bravo' }
-      let(:count) { 5 }
-      let(:lines) { [1, 7, 10, 13, 15] }
-      let(:expected_fingerprint) { '238733f4f51ba5473dcbe94a43ec5400' }
+      context 'when the parameters are provided' do
+        let(:name) { 'bravo' }
+        let(:count) { 5 }
+        let(:lines) { [1, 7, 10, 13, 15] }
+        let(:expected_fingerprint) { '238733f4f51ba5473dcbe94a43ec5400' }
 
-      it_behaves_like 'computes a fingerprint with identifying and non-identifying parameters'
-    end
+        it 'computes the fingerprint' do
+          expect(computed).to eq expected_fingerprint
+        end
+      end
 
-    context 'when the non-identifying parameters change, it computes the same fingerprint' do
-      let(:name) { 'bravo' }
-      let(:count) { 9 }
-      let(:lines) { [1, 7, 10, 13, 15, 17, 19, 20, 25] }
-      let(:expected_fingerprint) { '238733f4f51ba5473dcbe94a43ec5400' }
+      context 'when the non-identifying parameters change' do
+        let(:name) { 'bravo' }
+        let(:count) { 9 }
+        let(:lines) { [1, 7, 10, 13, 15, 17, 19, 20, 25] }
+        let(:expected_fingerprint) { '238733f4f51ba5473dcbe94a43ec5400' }
 
-      it_behaves_like 'computes a fingerprint with identifying and non-identifying parameters'
-    end
+        it 'computes the same fingerprint' do
+          expect(computed).to eq expected_fingerprint
+        end
+      end
 
-    context 'but when the identifying parameters change, it computes a different fingerprint' do
-      let(:name) { 'echo' }
-      let(:count) { 5 }
-      let(:lines) { [1, 7, 10, 13, 15] }
-      let(:expected_fingerprint) { 'e0c35e9223cc19bdb9a04fb3e60573e1' }
+      context 'when the identifying parameters change' do
+        let(:name) { 'echo' }
+        let(:count) { 5 }
+        let(:lines) { [1, 7, 10, 13, 15] }
+        let(:expected_fingerprint) { 'e0c35e9223cc19bdb9a04fb3e60573e1' }
 
-      it_behaves_like 'computes a fingerprint with identifying and non-identifying parameters'
+        it 'computes a different fingerprint' do
+          expect(computed).to eq expected_fingerprint
+        end
+      end
     end
   end
 end

data/spec/reek/report/code_climate/code_climate_formatter_spec.rb

@@ -4,12 +4,12 @@ require_lib 'reek/report/code_climate/code_climate_formatter'
 RSpec.describe Reek::Report::CodeClimateFormatter do
   describe '#render' do
     let(:warning) do
-      FactoryGirl.build(:smell_warning,
-                        smell_detector: Reek::SmellDetectors::UtilityFunction.new,
-                        context:        'context foo',
-                        message:        'message bar',
-                        lines:          [1, 2],
-                        source:         'a/ruby/source/file.rb')
+      build(:smell_warning,
+            smell_detector: Reek::SmellDetectors::UtilityFunction.new,
+            context:        'context foo',
+            message:        'message bar',
+            lines:          [1, 2],
+            source:         'a/ruby/source/file.rb')
     end
     let(:rendered) { described_class.new(warning).render }
     let(:json) { JSON.parse rendered.chop }

data/spec/reek/report/xml_report_spec.rb

@@ -5,7 +5,7 @@ require_lib 'reek/report/xml_report'
 RSpec.describe Reek::Report::XMLReport do
   let(:xml_report) { described_class.new }
 
-  context 'empty source' do
+  context 'with an empty source' do
     it 'prints empty checkstyle XML' do
       xml_report.add_examiner Reek::Examiner.new('')
       xml = "<?xml version='1.0'?>\n<checkstyle/>\n"
@@ -13,7 +13,7 @@ RSpec.describe Reek::Report::XMLReport do
     end
   end
 
-  context 'source with voliations' do
+  context 'with a source with violations' do
     it 'prints non-empty checkstyle XML' do
       xml_report.add_examiner Reek::Examiner.new(SMELLY_FILE)
       xml = SAMPLES_PATH.join('checkstyle.xml').read

data/spec/reek/smell_detectors/boolean_parameter_spec.rb

@@ -27,7 +27,7 @@ RSpec.describe Reek::SmellDetectors::BooleanParameter do
       and reek_of(:BooleanParameter, lines: [1], context: 'alfa', parameter: 'charlie')
   end
 
-  context 'in a method' do
+  context 'when examining an instance method' do
     it 'reports a parameter defaulted to false' do
       src = 'def alfa(bravo = false) end'
       expect(src).to reek_of(:BooleanParameter)
@@ -70,7 +70,7 @@ RSpec.describe Reek::SmellDetectors::BooleanParameter do
     end
   end
 
-  context 'in a singleton method' do
+  context 'when examining a singleton method' do
     it 'reports a parameter defaulted to true' do
       src = 'def self.alfa(bravo = true) end'
       expect(src).to reek_of(:BooleanParameter)

data/spec/reek/smell_detectors/class_variable_spec.rb

@@ -63,49 +63,43 @@ RSpec.describe Reek::SmellDetectors::ClassVariable do
   end
 
   ['class', 'module'].each do |scope|
-    context "Scoped to #{scope}" do
-      context 'set in a method' do
-        it 'reports correctly' do
-          src = <<-EOS
-            #{scope} Alfa
-              def bravo
-                @@charlie = {}
-              end
+    context "when examining a #{scope}" do
+      it 'reports a class variable set in a method' do
+        src = <<-EOS
+          #{scope} Alfa
+            def bravo
+              @@charlie = {}
             end
-          EOS
+          end
+        EOS
 
-          expect(src).to reek_of(:ClassVariable)
-        end
+        expect(src).to reek_of(:ClassVariable, name: '@@charlie')
       end
 
-      context 'used in a method' do
-        it 'reports correctly' do
-          src = <<-EOS
-            #{scope} Alfa
-              def bravo
-                puts @@charlie
-              end
+      it 'reports a class variable used in a method' do
+        src = <<-EOS
+          #{scope} Alfa
+            def bravo
+              puts @@charlie
             end
-          EOS
+          end
+        EOS
 
-          expect(src).to reek_of(:ClassVariable)
-        end
+        expect(src).to reek_of(:ClassVariable, name: '@@charlie')
       end
 
-      context "set in #{scope} and used in a method" do
-        it 'reports correctly' do
-          src = <<-EOS
-            #{scope} Alfa
-              @@bravo = 42
+      it "reports a class variable set in the #{scope} body and used in a method" do
+        src = <<-EOS
+          #{scope} Alfa
+            @@bravo = 42
 
-              def charlie
-                puts @@bravo
-              end
+            def charlie
+              puts @@bravo
             end
-          EOS
+          end
+        EOS
 
-          expect(src).to reek_of(:ClassVariable)
-        end
+        expect(src).to reek_of(:ClassVariable, name: '@@bravo')
       end
     end
   end

data/spec/reek/smell_detectors/control_parameter_spec.rb

@@ -30,7 +30,7 @@ RSpec.describe Reek::SmellDetectors::ControlParameter do
       and reek_of(:ControlParameter, lines: [3], argument: 'charlie')
   end
 
-  context 'parameter not used to determine code path' do
+  context 'when a parameter is not used to determine code path' do
     it 'does not report a ternary check on an ivar' do
       src = 'def alfa(bravo) @charlie ? bravo : false end'
       expect(src).not_to reek_of(:ControlParameter)
@@ -52,7 +52,7 @@ RSpec.describe Reek::SmellDetectors::ControlParameter do
     end
   end
 
-  context 'parameter only used to determine code path' do
+  context 'when a parameter is only used to determine code path' do
     it 'reports a ternary check on a parameter' do
       src = 'def alfa(bravo); bravo ? true : false; end'
       expect(src).to reek_of(:ControlParameter)
@@ -144,7 +144,7 @@ RSpec.describe Reek::SmellDetectors::ControlParameter do
       expect(src).to reek_of(:ControlParameter)
     end
 
-    it 'reports on nested if statements where the inner if is a control parameter' do
+    it 'reports on nested suffix if statements where the inner if is a control parameter' do
       src = <<-EOS
         def nested(bravo)
           if true
@@ -157,6 +157,36 @@ RSpec.describe Reek::SmellDetectors::ControlParameter do
       expect(src).to reek_of(:ControlParameter)
     end
 
+    it 'reports on nested full if statements where the inner if is a control parameter' do
+      src = <<-EOS
+        def alfa(bravo)
+          if true
+            charlie
+          else
+            if bravo
+              delta
+            end
+          end
+        end
+      EOS
+
+      expect(src).to reek_of(:ControlParameter)
+    end
+
+    it 'reports on elsif statements' do
+      src = <<-EOS
+        def alfa(bravo)
+          if true
+            charlie
+          elsif bravo
+            delta
+          end
+        end
+      EOS
+
+      expect(src).to reek_of(:ControlParameter)
+    end
+
     it 'reports on explicit comparison in the condition' do
       src = 'def alfa(bravo); if bravo == charlie then charlie end; end'
       expect(src).to reek_of(:ControlParameter)
@@ -188,7 +218,7 @@ RSpec.describe Reek::SmellDetectors::ControlParameter do
     end
   end
 
-  context 'parameter used besides determining code path' do
+  context 'when a parameter is used besides determining code path' do
     it 'does not report on if conditional expression' do
       src = 'def alfa(bravo); if bravo then charlie(bravo); end end'
       expect(src).not_to reek_of(:ControlParameter)

data/spec/reek/smell_detectors/duplicate_method_call_spec.rb

@@ -150,7 +150,7 @@ RSpec.describe Reek::SmellDetectors::DuplicateMethodCall do
     end
   end
 
-  context 'non-repeated method calls' do
+  context 'with non-repeated method calls' do
     it 'does not report similar calls' do
       src = 'def alfa(bravo) bravo.charlie == self.charlie end'
       expect(src).not_to reek_of(:DuplicateMethodCall)
@@ -162,7 +162,7 @@ RSpec.describe Reek::SmellDetectors::DuplicateMethodCall do
     end
   end
 
-  context 'allowing up to 3 calls' do
+  context 'when allowing up to 3 calls' do
     let(:config) do
       { Reek::SmellDetectors::DuplicateMethodCall::MAX_ALLOWED_CALLS_KEY => 3 }
     end
@@ -189,7 +189,7 @@ RSpec.describe Reek::SmellDetectors::DuplicateMethodCall do
     end
   end
 
-  context 'allowing calls to some methods' do
+  context 'when allowing calls to some methods' do
     it 'does not report calls to some methods' do
       config = { Reek::SmellDetectors::DuplicateMethodCall::ALLOW_CALLS_KEY => ['@bravo.charlie'] }
       src = 'def alfa; @bravo.charlie + @bravo.charlie; end'

data/spec/reek/smell_detectors/module_initialize_spec.rb

@@ -50,6 +50,20 @@ RSpec.describe Reek::SmellDetectors::ModuleInitialize do
     expect(src).not_to reek_of(:ModuleInitialize)
   end
 
+  it 'reports nothing for a method named initialize in a nested dynamic class' do
+    src = <<-EOF
+      module Alfa
+        def self.bravo
+          Class.new do
+            def initialize; end
+          end
+        end
+      end
+    EOF
+
+    expect(src).not_to reek_of(:ModuleInitialize)
+  end
+
   it 'can be disabled via comment' do
     src = <<-EOS
       # :reek:ModuleInitialize