call actual reek for validation of parsing

Author: bergholdt

lib/metric_fu/metrics/reek/reek_examiner.rb

@@ -19,13 +19,15 @@ def self.get
     end
 
     def self.reek_version
-      Reek.const_defined?(:VERSION) ? Reek::VERSION :
-                                      Reek::Version::STRING
+      Reek::Version::STRING
+    rescue NameError
+      Reek::VERSION
     end
 
     def self.reek_examinor_klass
-      Reek.const_defined?(:Examiner) ? Reek.const_get(:Examiner) :
-                                       Reek.const_get(:Core).const_get(:Examiner)
+      Reek::Examiner
+    rescue NameError
+      Reek::Core::Examiner
     end
 
     class ReekExaminerV1
@@ -38,6 +40,7 @@ def run!(files, config_files)
       end
 
       def analyze
+        return [] if @output.nil?
         @output.smells.group_by(&:source).collect do |file_path, smells|
           { file_path: file_path,
             code_smells: analyze_smells(smells) }
@@ -72,6 +75,7 @@ def run!(files, config_files)
       end
 
       def analyze
+        return [] if @output.nil?
         @output.map(&:smells).flatten.group_by(&:source).collect do |file_path, smells|
           { file_path: file_path,
             code_smells: analyze_smells(smells) }

spec/dummy/lib/bad_code.rb

@@ -0,0 +1,16 @@
+class BadCode
+  def nonsense(some_input, options)
+    x = Array(some_input).map(&:to_s)
+    BadCode.configure do |config|
+      config.something.each do |y|
+        name = y.name.to_s
+        if x.include?(name)
+          feature.enabled = true
+        else
+          feature.enabled = false
+        end
+      end
+    end
+    do_it(options)
+  end
+end