Remove unused Flay param 'filetypes'. Not a valid Flay option, either.

Closes #151

Author: bf4

lib/metric_fu/metrics/flay/flay.rb

@@ -7,8 +7,7 @@ def self.metric
     end
 
     def emit
-      minimum_score_parameter = options[:minimum_score] ? "--mass #{options[:minimum_score]} " : ""
-      args =  "#{minimum_score_parameter} #{options[:dirs_to_flay].join(" ")}"
+      args =  "#{minimum_duplication_mass} #{dirs_to_flay}"
       @output = run!(args)
     end
 
@@ -29,5 +28,20 @@ def to_h
       end
       {:flay => {:total_score => total_score, :matches => target.flatten}}
     end
+
+    private
+
+    def minimum_duplication_mass
+      flay_mass = options[:minimum_score]
+      return "" unless flay_mass
+
+
+      "--mass #{flay_mass} "
+    end
+
+    def dirs_to_flay
+      options[:dirs_to_flay].join(" ")
+    end
+
   end
 end

lib/metric_fu/metrics/flay/init.rb

@@ -11,7 +11,7 @@ def default_run_options
       # a long time. This is a really big number, considering
       # the default is 16. Setting it to nil to use the Flay default.
       :minimum_score => nil,
-      :filetypes => ['rb'] }
+      }
     end
 
     def has_graph?

spec/metric_fu/metrics/flay/configuration_spec.rb

@@ -7,7 +7,7 @@
     it 'should set @flay to {:dirs_to_flay => @code_dirs}' do
       load_metric 'flay'
       expect(MetricFu::Metric.get_metric(:flay).run_options).to eq(
-              {:dirs_to_flay => ['lib'], :filetypes=>["rb"], :minimum_score=>nil}
+              {:dirs_to_flay => ['lib'], :minimum_score=>nil}
       )
     end
 

spec/metric_fu/metrics/flay/flay_spec.rb

@@ -4,7 +4,7 @@
 describe MetricFu::FlayGenerator do
   describe "emit method" do
     it "should look at the dirs" do
-      options = { :dirs_to_flay => ['app', 'lib'], :filetypes => ['rb']  }
+      options = { :dirs_to_flay => ['app', 'lib'] }
       allow(File).to receive(:directory?).and_return(true)
       @flay = MetricFu::FlayGenerator.new(options)