metric_adapter 0.0.1 → 0.0.2

data/examples/annotate.rb

@@ -67,7 +67,7 @@ end
 # Combine metrics from all our sources (See AnalyzerMixin):
 metrics = flay(files, 16) + flog(files) + reek(files)
 
-metrics_by_path = metrics.group_by{|m| m.location.path }
+metrics_by_path = metrics.group_by{|m| m.path }
 
 files.each do |path|
   metrics = metrics_by_path[path]

data/examples/report.rb

@@ -22,7 +22,7 @@ end
 metrics = flay(files) + flog(files) + reek(files)
 
 # Group those metrics by path
-metrics_by_path = metrics.group_by{|m| m.location.path }
+metrics_by_path = metrics.group_by{|m| m.path }
 
 # For each path, print out a little report.
 metrics_by_path.each do |path, metrics|
@@ -31,7 +31,7 @@ metrics_by_path.each do |path, metrics|
   puts divider
   
   puts "  line: message"
-  metrics.sort_by{|m| m.location }.each do |m|
-    puts "%6d: %s" % [m.location.line, m.message]
+  metrics.sort_by{|m| m.line }.each do |m|
+    puts "%6d: %s" % [m.line, m.message]
   end
 end

data/lib/metric.rb

@@ -3,7 +3,7 @@ module MetricAdapter
   # A normalized representation of a code metric.
   class Metric
     
-    # Location where this metric applies
+    # Location where this metric applies (see the Location class)
     attr_accessor :location
     
     # Associated class and method signature
@@ -24,6 +24,17 @@ module MetricAdapter
       @message   = message
       @score     = 0
     end
-  
+    
+    # Returns the path that this metric applies to, for example:
+    #   ./lib/metric.rb
+    def path
+      location.path
+    end
+    
+    # Returns the line number this metric applies to.
+    def line
+      location.line
+    end
+    
   end
 end

data/test/flay_adapter_test.rb

@@ -25,10 +25,9 @@ class FlayAdapterTest < MiniTest::Unit::TestCase
   def test_flay_adapts_locations
     adapter  = flay_self
     metric   = adapter.metrics.first
-    location = metric.location
     
-    assert_equal __FILE__, location.path, "Should reference this file"
-    assert location.line != 0, "Line numbers should be captured (line:0 should not show up in flog)"
+    assert_equal __FILE__, metric.path, "Should reference this file"
+    assert metric.line != 0, "Line numbers should be captured (line:0 should not show up in flog)"
   end
   
   def test_flay_messages

data/test/flog_adapter_test.rb

@@ -16,10 +16,9 @@ class FlogAdapterTest < MiniTest::Unit::TestCase
   def test_flogging_adapts_locations
     adapter  = flog_self
     metric   = adapter.metrics.first
-    location = metric.location
     
-    assert_equal __FILE__, location.path, "Should reference this file"
-    assert location.line != 0, "Line numbers should be captured (line:0 should not show up in flog)"
+    assert_equal __FILE__, metric.path, "Should reference this file"
+    assert metric.line != 0, "Line numbers should be captured (line:0 should not show up in flog)"
   end
   
   def test_flogging_messages

data/test/reek_adapter_test.rb

@@ -18,10 +18,9 @@ class ReekAdapterTest < MiniTest::Unit::TestCase
   def test_examiner_adapts_locations
     adapter  = examine_sample
     metric   = adapter.metrics.first
-    location = metric.location
     
-    assert_equal SAMPLE_PATH, location.path, "Should have included the path"
-    assert location.line > 0, "Should have a non zero line number"
+    assert_equal SAMPLE_PATH, metric.path, "Should have included the path"
+    assert metric.line > 0, "Should have a non zero line number"
   end
   
   def test_examiner_returns_metrics_for_each_line_of_each_smell

metadata

@@ -5,8 +5,8 @@ version: !ruby/object:Gem::Version
   segments: 
   - 0
   - 0
-  - 1
-  version: 0.0.1
+  - 2
+  version: 0.0.2
 platform: ruby
 authors: 
 - Adam Sanderson
@@ -14,7 +14,7 @@ autorequire:
 bindir: bin
 cert_chain: []
 
-date: 2013-06-25 00:00:00 -07:00
+date: 2013-06-26 00:00:00 -07:00
 default_executable: 
 dependencies: []