metric_fu 2.1.1 → 2.1.3.4

data/lib/version.rb

@@ -0,0 +1,3 @@
+module MetricFu
+  VERSION = "2.1.3.4"
+end

data/metric_fu.gemspec

@@ -0,0 +1,65 @@
+# -*- encoding: utf-8 -*-
+require File.expand_path('../lib/version', __FILE__)
+
+Gem::Specification.new do |s|
+  s.name        = "metric_fu"
+  s.platform    = Gem::Platform::RUBY
+  s.version     = MetricFu::VERSION
+  s.summary     = "A fistful of code metrics, with awesome templates and graphs"
+  s.email       = "github@benjaminfleischer.com"
+  s.homepage    = "http://github.com/bf4/metric_fu"
+  s.description = "Code metrics from Flog, Flay, Simplecov-RCov, Saikuro, Churn, Reek, Roodi, Rails' stats task and Rails Best Practices"
+  s.authors     = ["Jake Scruggs", "Sean Soper", "Andre Arko", "Petrik de Heus", "Grant McInnes", "Nick Quaranto", "Édouard Brière", "Carl Youngblood", "Richard Huang", "Dan Mayer", "Benjamin Fleischer"]
+  s.required_ruby_version     = ">= 1.8.7"
+  s.required_rubygems_version = ">= 1.3.6"
+  s.files              = `git ls-files`.split($\)
+  s.test_files         =  s.files.grep(%r{^(test|spec|features)/})
+  s.executables        =  s.files.grep(%r{^bin/}).map{ |f| File.basename(f) }
+  s.require_paths = ["lib"]
+  s.license     = 'MIT'
+
+  {
+    "flay"                  => ["= 1.2.1"],
+    "flog"                  => ["= 2.3.0"],
+    "reek"                  => ["= 1.2.12"],
+    "roodi"                 => ["= 2.1.0"],
+    "rails_best_practices"  => ["~> 0.6"],
+    "churn"                 => ["= 0.0.7"],
+    "sexp_processor"        => ["~> 3.0.3"], # required because of churn.
+    "chronic"               => ["= 0.2.3"], # required by churn
+    "main"                  => ["= 4.7.1"], # required by churn
+    "activesupport"         => [">= 2.0.0"], # ok
+    # "syntax"                => ["= 1.0.0"],
+    "coderay"               => [],
+    "fattr"                 => ["= 2.2.1"],
+    "arrayfields"           => [" =4.7.4"],
+    "map"                   => [" =6.2.0"],
+    "bluff"                 => [],
+    "googlecharts"          => []
+  }.each do |gem, version|
+    if version == []
+      s.add_dependency(gem)
+    else
+      s.add_dependency(gem,version)
+    end
+  end
+  # string comparison ftw
+  if RUBY_VERSION < '1.9'
+    s.add_dependency("ripper",[" =1.0.5"])
+    s.add_dependency("rcov", ["~> 0.8"])
+    s.add_dependency("Saikuro", ["= 1.1.0"])
+  else
+    s.add_dependency("rcov", ["~> 0.8"])
+    s.add_dependency("japgolly-Saikuro", ">= 1.1.1.0")
+    # still using rcov in ruby 1.9 till some errors are fleshed out
+    # s.add_dependency("simplecov", [">= 0.5.4"])
+    # s.add_dependency("simplecov-rcov", [">= 0.2.3"])
+  end
+
+
+  s.add_development_dependency("rake", '<=0.9.2')
+  s.add_development_dependency("rspec", ["= 1.3.0"])
+  s.add_development_dependency("test-construct", [">= 1.2.0"])
+  s.add_development_dependency("googlecharts")
+end
+

data/spec/base/base_template_spec.rb

@@ -57,7 +57,7 @@ describe MetricFu::Template do
     it 'should generate the filename of the template file' do
       section = mock('section')
       section.should_receive(:to_s).and_return('section')
-      @template.should_receive(:this_directory).and_return('dir')
+      @template.should_receive(:template_directory).and_return('dir')
       result = @template.send(:template, section)
       result.should == "dir/section.html.erb"
     end
@@ -75,7 +75,7 @@ describe MetricFu::Template do
   describe "#inline_css" do
     it 'should return the contents of a css file' do
       css = 'mycss.css'
-      @template.should_receive(:this_directory).and_return('dir')
+      @template.should_receive(:template_directory).and_return('dir')
       io = mock('io', :read => "css contents")
       @template.should_receive(:open).and_yield(io)
       result = @template.send(:inline_css, css)
@@ -89,6 +89,7 @@ describe MetricFu::Template do
         config = mock("configuration")
         config.stub!(:platform).and_return('universal-darwin-9.0')
         config.stub!(:darwin_txmt_protocol_no_thanks).and_return(false)
+        config.stub!(:link_prefix).and_return(nil)
         MetricFu.stub!(:configuration).and_return(config)
       end
 
@@ -118,6 +119,7 @@ describe MetricFu::Template do
           config = mock("configuration")
           config.stub!(:platform).and_return('universal-darwin-9.0')
           config.stub!(:darwin_txmt_protocol_no_thanks).and_return(true)
+          config.stub!(:link_prefix).and_return(nil)
           MetricFu.stub!(:configuration).and_return(config)
           File.should_receive(:expand_path).and_return('filename')
         end
@@ -143,6 +145,7 @@ describe MetricFu::Template do
       before(:each) do
         config = mock("configuration")
         config.should_receive(:platform).and_return('other')
+        config.stub!(:link_prefix).and_return(nil)
         config.stub!(:darwin_txmt_protocol_no_thanks).and_return(false)
         MetricFu.stub!(:configuration).and_return(config)
         File.should_receive(:expand_path).and_return('filename')
@@ -154,6 +157,20 @@ describe MetricFu::Template do
         result.should == "<a href='file://filename'>filename</a>"
       end
     end
+    describe "when configured with a link_prefix" do
+      before(:each) do
+        config = mock("configuration")
+        config.should_receive(:link_prefix).and_return('http://example.org/files')
+        MetricFu.stub!(:configuration).and_return(config)
+        File.should_receive(:expand_path).and_return('filename')
+      end
+
+      it 'should return a http protocol link' do
+        name = "filename"
+        result = @template.send(:link_to_filename, name)
+        result.should == "<a href='http://example.org/files/filename'>filename</a>"
+      end
+    end
   end
 
   describe "#cycle" do

data/spec/base/configuration_spec.rb

@@ -39,7 +39,7 @@ describe MetricFu::Configuration do
 
       it 'should return the CC_BUILD_ARTIFACTS environment variable' do
         get_new_config
-        base_directory.should == ENV['CC_BUILD_ARTIFACTS']
+        compare_paths(base_directory, ENV['CC_BUILD_ARTIFACTS'])
       end
     end
 
@@ -71,13 +71,13 @@ describe MetricFu::Configuration do
 
     it 'should set @scratch_directory to scratch relative '+
     'to @base_directory' do
-      scratch_dir = File.join(base_directory, 'scratch')
+      scratch_dir = MetricFu.scratch_dir
       scratch_directory.should == scratch_dir
     end
 
     it 'should set @output_directory to output relative '+
     'to @base_directory' do
-      output_dir = File.join(base_directory, 'output')
+      output_dir = MetricFu.output_dir
       output_directory.should == output_dir
     end
 
@@ -132,6 +132,7 @@ describe MetricFu::Configuration do
       @config.instance_variable_get(:@rcov).
               should ==  { :environment => 'test',
                            :test_files => ['test/**/*_test.rb',
+                                           'spec/spec_helper.rb',
                                            'spec/**/*_spec.rb'],
                            :rcov_opts => ["--sort coverage",
                                          "--no-html",

data/spec/base/generator_spec.rb

@@ -30,8 +30,8 @@ describe MetricFu::Generator do
 
   describe "ConcreteClass#metric_directory" do
     it "should be 'tmp/metric_fu/scratch/concreteclass'" do
-      ConcreteClass.metric_directory.
-                    should == "tmp/metric_fu/scratch/concreteclass"
+      compare_paths(ConcreteClass.metric_directory,
+                    "tmp/metric_fu/scratch/concreteclass")
     end
   end
 

data/spec/base/line_numbers_spec.rb

@@ -37,7 +37,7 @@ describe MetricFu::LineNumbers do
 
     it "should know the name of a class method defined in a 'class << self block at a particular line" do
       ln = MetricFu::LineNumbers.new(File.read(File.dirname(__FILE__) + "/../resources/line_numbers/foo.rb"))
-      ln.method_at_line(23).should == "Foo::neat"
+      ln.method_at_line(22).should == "Foo::neat"
     end
 
     it "should know the name of an instance method at a particular line in a file with two classes" do
@@ -59,4 +59,4 @@ describe MetricFu::LineNumbers do
 
   end
 
-end
+end

data/spec/base/location_spec.rb

@@ -0,0 +1,127 @@
+require File.expand_path(File.dirname(__FILE__) + "/../spec_helper")
+
+describe MetricFu::Location do
+
+  context "with non-standard Reek method names" do
+    # reek reports the method with :: not # on modules like 
+    # module ApplicationHelper \n def signed_in?, convert it so it records correctly
+    # class_or_method_name = class_or_method_name.gsub(/\:\:/,"#") if method_bug_conversion
+
+    before do
+      @location = Location.for("ApplicationHelper::section_link")
+    end
+
+    it "has method name" do
+      @location.method_name.should == 'ApplicationHelper#section_link'
+    end
+
+    it "has nil file path" do
+      @location.file_path.should   == nil
+    end
+
+    it "has class name" do
+      @location.class_name.should  == 'ApplicationHelper'
+    end
+    
+  end
+
+  context "using new" do
+    
+    before do 
+      @location = Location.new("lib/foo.rb", "Foo", "Foo#some_method")
+    end
+
+    it "should return fully qualified method" do
+      @location.method_name.should == 'Foo#some_method'
+    end
+    
+  end
+
+  context "using .for with class" do
+
+    before do 
+      @location = Location.for("Module::Foo")
+    end
+
+    it "has nil method_name" do
+      @location.method_name.should be nil
+    end
+
+    it "has nil file_path" do
+      @location.file_path.should be nil
+    end
+
+    it "has class_name" do
+      @location.class_name.should == 'Foo'
+    end
+
+  end
+
+  context "using .for with method" do
+    
+    before do 
+      @location = Location.for("Module::Foo#some_method")
+    end
+
+    it "strips module from class name" do
+      @location.class_name.should == 'Foo'
+    end
+
+    it "strips module from method name" do
+      @location.method_name.should == 'Foo#some_method'
+    end
+
+    it "has nil file_path" do
+      @location.file_path.should be nil
+    end
+
+  end
+
+  context "with class method" do
+    
+    it "provides non-qualified name" do
+      location = Location.for("Foo.some_class_method")
+      location.simple_method_name.should == '.some_class_method'
+    end
+
+  end
+
+  context "with instance method" do
+    
+    it "provides non-qualified name" do
+      location = Location.for("Foo#some_class_method")
+      location.simple_method_name.should == '#some_class_method'
+    end
+
+  end
+    context "testing equality" do
+    before :each do
+
+      @location1 = MetricFu::Location.get('/some/path','some_class','some_method')
+
+      # ensure that we get a new object
+      @location2 = MetricFu::Location.new('/some/path','some_class','some_method')
+    end
+    it "should match two locations with the same paths as equal" do
+
+      hsh1 = {}
+      hsh1[@location1] = 1
+
+      hsh2 = {}
+      hsh2[@location2] = 1
+
+      hsh1.should == hsh2
+      hsh1.eql?(hsh2).should be_true
+
+      @location1.eql?(@location2).should be_true
+    end
+
+
+    it "should produce the same hash value given the same paths" do
+
+      @location1.hash.should == @location2.hash
+    end
+
+  end
+
+end

data/spec/base/metric_analyzer_spec.rb

@@ -0,0 +1,452 @@
+require File.expand_path(File.dirname(__FILE__) + "/../spec_helper")
+
+describe MetricAnalyzer do
+
+  context "with several types of data" do
+    
+    before do
+      @yaml =<<-__
+--- 
+:reek: 
+  :matches: 
+  - :file_path: lib/client/client.rb
+    :code_smells: 
+    - :type: Large Class
+      :message: has at least 27 methods
+      :method: Devver::Client
+    - :type: Long Method
+      :message: has approx 6 statements
+      :method: Devver::Client#client_requested_sync
+:flog: 
+  :method_containers: 
+  - :highest_score: 61.5870319141946
+    :path: /lib/client/client.rb
+    :methods: 
+      Client#client_requested_sync: 
+        :path: /lib/client/client.rb
+        :score: 37.9270319141946
+        :operators: 
+          :+: 1.70000000000001
+          :/: 1.80000000000001
+          :method_at_line: 1.90000000000001
+          :puts: 1.70000000000001
+          :assignment: 33.0000000000001
+          :in_method?: 1.70000000000001
+          :message: 1.70000000000001
+          :branch: 12.6
+          :<<: 3.40000000000001
+          :each: 1.50000000000001
+          :lit_fixnum: 1.45
+          :raise: 1.80000000000001
+          :each_pair: 1.3
+          :*: 1.60000000000001
+          :to_f: 2.00000000000001
+          :each_with_index: 3.00000000000001
+          :[]: 22.3000000000001
+          :new: 1.60000000000001
+    :average_score: 11.1209009055421
+    :total_score: 1817.6
+    :name: Client#client_requested_sync
+:churn: 
+  :changes: 
+  - :file_path: lib/client/client.rb
+    :times_changed: 54
+  - :file_path: lib/client/foo.rb
+    :times_changed: 52
+__
+    end
+
+    it "gives all files, in order, from worst to best" do 
+      analyzer = MetricAnalyzer.new(@yaml)
+      expected = [
+                  "lib/client/client.rb",
+                  "lib/client/foo.rb"]
+      analyzer.worst_files.should == expected
+    end
+
+    it "gives all issues for a class" do
+      analyzer = MetricAnalyzer.new(@yaml)
+      expected = {
+        :reek => "found 2 code smells",
+        :flog => "complexity is 37.9"
+      }
+      analyzer.problems_with(:class, "Client").should == expected
+    end
+
+    it "gives all issues for a method" do
+      analyzer = MetricAnalyzer.new(@yaml)
+      expected = { 
+        :reek => "found 1 code smells", 
+        :flog => "complexity is 37.9"}
+      analyzer.problems_with(:method, "Client#client_requested_sync").should == expected
+    end
+
+    it "gives all issues for a file" do
+      analyzer = MetricAnalyzer.new(@yaml)
+      expected = { 
+        :reek => "found 2 code smells" ,
+        :flog => "complexity is 37.9",
+        :churn => "detected high level of churn (changed 54 times)"}
+      analyzer.problems_with(:file, "lib/client/client.rb").should == expected
+    end
+
+    it "provide location for a method" do
+      analyzer = MetricAnalyzer.new(@yaml)
+      expected = Location.new("lib/client/client.rb",
+                              "Client",
+                              "Client#client_requested_sync")
+      analyzer.location(:method, "Client#client_requested_sync").should == expected
+    end
+
+    it "provides location for a class" do
+      analyzer = MetricAnalyzer.new(@yaml)
+      expected = Location.new("lib/client/client.rb",
+                              "Client",
+                              nil)
+      analyzer.location(:class, "Client").should == expected
+    end
+
+    it "provides location for a file" do
+      analyzer = MetricAnalyzer.new(@yaml)
+      expected = Location.new("lib/client/client.rb",
+                              nil,
+                              nil)
+      analyzer.location(:file, "lib/client/client.rb").should == expected
+    end
+
+  end
+
+  context "with Reek data" do
+
+    before do
+      @yaml =<<-__
+--- 
+:reek: 
+  :matches: 
+  - :file_path: lib/client/client.rb
+    :code_smells: 
+    - :type: Large Class
+      :message: has at least 27 methods
+      :method: Devver::Client
+    - :type: Long Method
+      :message: has approx 6 statements
+      :method: Devver::Client#client_requested_sync
+    - :type: Large Class
+      :message: has at least 20 methods
+      :method: Devver::Foo
+__
+    end
+
+    it "gives worst method" do
+      analyzer = MetricAnalyzer.new(@yaml)
+      analyzer.worst_methods(1).should == ["Client#client_requested_sync"]
+    end
+
+    it "gives worst class" do
+      analyzer = MetricAnalyzer.new(@yaml)
+      analyzer.worst_classes(1).should == ["Client"]
+    end
+
+    it "gives worst file" do
+      analyzer = MetricAnalyzer.new(@yaml)
+      analyzer.worst_files(1).should == ["lib/client/client.rb"]
+    end
+
+  end
+
+
+  context "with Saikuro data" do
+    
+    before do
+      @yaml =<<-__
+:saikuro: 
+  :files: 
+  - :classes: 
+    - :complexity: 0
+      :methods: []
+      :lines: 3
+      :class_name: Shorty
+    - :complexity: 19
+      :methods: 
+      - :complexity: 9
+        :lines: 6
+        :name: Shorty::Supr#self.handle_full_or_hash_option
+      - :complexity: 1
+        :lines: 9
+        :name: Shorty::Supr#initialize
+      :lines: 92
+      :class_name: Shorty::Supr
+    :filename: supr.rb
+  - :classes: 
+    - :complexity: 12
+      :methods: 
+      - :complexity: 8
+        :lines: 10
+        :name: Shorty::Bitly#info
+      :lines: 104
+      :class_name: Shorty::Bitly
+    :filename: bitly.rb
+__
+    end
+    
+    it "gives worst method" do
+      analyzer = MetricAnalyzer.new(@yaml)
+      analyzer.worst_methods(1).should == ["Supr#self.handle_full_or_hash_option"]
+    end
+
+    it "gives worst class" do
+      analyzer = MetricAnalyzer.new(@yaml)
+      analyzer.worst_classes(1).should == ["Bitly"]
+    end
+
+    it "gives complexity for method" do
+      analyzer = MetricAnalyzer.new(@yaml)
+      expected = {
+        :saikuro => "complexity is 1.0"
+      }
+      analyzer.problems_with(:method, "Supr#initialize").should == expected
+    end
+    
+    it "gives average complexity for class" do
+      analyzer = MetricAnalyzer.new(@yaml)
+      expected = {
+        :saikuro => "average complexity is 5.0"
+      }
+      analyzer.problems_with(:class, "Supr").should == expected
+    end
+
+  end
+
+  context "with Flog data" do
+    
+    before do
+      @yaml =<<-__
+--- 
+:flog: 
+  :method_containers: 
+  - :highest_score: 85.5481735632041
+    :path: ""
+    :methods: 
+      main#none: 
+        :path: 
+        :score: 85.5481735632041
+        :operators: 
+          :+: 9.10000000000001
+          :assignment: 11.6000000000001
+          :require: 38.5000000000002
+          :branch: 8.80000000000009
+          :join: 20.0000000000002
+          :each: 6.60000000000007
+          :[]: 7.80000000000007
+          :task_defined?: 1.10000000000001
+          :load: 1.20000000000001
+    :average_score: 85.5481735632041
+    :total_score: 85.5481735632041
+    :name: main
+  - :highest_score: 61.5870319141946
+    :path: lib/generators/rcov.rb
+    :methods: 
+      Rcov#add_method_data: 
+        :path: lib/generators/rcov.rb:57
+        :score: 61.5870319141946
+        :operators: 
+          :+: 1.70000000000001
+          :/: 1.80000000000001
+          :method_at_line: 1.90000000000001
+          :puts: 1.70000000000001
+          :assignment: 33.0000000000001
+          :in_method?: 1.70000000000001
+          :message: 1.70000000000001
+          :branch: 12.6
+          :<<: 3.40000000000001
+          :each: 1.50000000000001
+          :lit_fixnum: 1.45
+          :raise: 1.80000000000001
+          :each_pair: 1.3
+          :*: 1.60000000000001
+          :to_f: 2.00000000000001
+          :each_with_index: 3.00000000000001
+          :[]: 22.3000000000001
+          :new: 1.60000000000001
+      Rcov#analyze: 
+        :path: lib/generators/rcov.rb:34
+        :score: 19.1504569136092
+        :operators: 
+          :+: 1.40000000000001
+          :metric_directory: 1.6
+          :assignment: 9.10000000000004
+          :assemble_files: 1.3
+          :branch: 1.3
+          :open: 1.5
+          :shift: 1.3
+          :split: 1.3
+          :rcov: 3.10000000000001
+          :add_coverage_percentage: 1.3
+          :read: 1.3
+          :[]: 2.70000000000001
+  :average_score: 27.8909176873585
+  :total_score: 195.23642381151
+  - :highest_score: 60.0573892206447
+    :path: lib/base/metric_analyzer.rb
+    :methods: 
+      MetricAnalyzer#grouping_key: 
+        :path: lib/base/metric_analyzer.rb:117
+        :score: 2.6
+        :operators: 
+          :inspect: 1.3
+          :object_id: 1.3
+      MetricAnalyzer#fix_row_file_path!: 
+        :path: lib/base/metric_analyzer.rb:148
+        :score: 20.2743680542699
+        :operators: 
+          :assignment: 10.0
+          :include?: 3.1
+          :branch: 7.20000000000001
+          :detect: 1.5
+          :file_paths: 1.7
+          :==: 3.1
+          :to_s: 1.3
+          :[]: 5.40000000000001
+__
+    end
+    
+    it "gives worst method" do
+      analyzer = MetricAnalyzer.new(@yaml)
+      analyzer.worst_methods(1).should == ["main#none"]
+    end
+
+    it "gives worst class" do
+      analyzer = MetricAnalyzer.new(@yaml)
+      analyzer.worst_classes(1).should == ["main"]
+    end
+
+    it "gives worst file" do
+      analyzer = MetricAnalyzer.new(@yaml)
+      analyzer.worst_files(1).should == ["lib/generators/rcov.rb:57"]
+    end
+
+  end
+
+  context "with Roodi data" do
+    
+    before do
+      @yaml =<<-__
+:roodi: 
+  :total: 
+  - Found 164 errors.
+  :problems: 
+  - :line: "158"
+    :file: lib/client/client.rb
+    :problem: Method name "process" cyclomatic complexity is 10.  It should be 8 or less.
+  - :line: "232"
+    :file: lib/client/client.rb
+    :problem: Method name "process_ready" cyclomatic complexity is 15.  It should be 8 or less.
+  - :line: "288"
+    :file: lib/client/foobar.rb
+    :problem: Method name "send_tests" cyclomatic complexity is 10.  It should be 8 or less.
+__
+    end
+
+    it "gives worst file" do
+      analyzer = MetricAnalyzer.new(@yaml)
+      analyzer.worst_files(1).should == ["lib/client/client.rb"]
+    end
+
+  end
+
+  context "with Stats data" do
+    
+    before do
+ @yaml =<<-__
+:stats: 
+  :codeLOC: 4222
+  :testLOC: 2111
+  :code_to_test_ratio: 2
+__
+end
+    
+    it "should have codeLOC" do
+      analyzer = MetricAnalyzer.new(@yaml)
+      row = analyzer.table.rows_with('stat_name' => :codeLOC).first
+      row['stat_value'].should == 4222
+    end
+
+    it "should have testLOC" do
+      analyzer = MetricAnalyzer.new(@yaml)
+      row = analyzer.table.rows_with('stat_name' => :testLOC).first
+      row['stat_value'].should == 2111
+    end
+
+    it "should have code_to_test_ration" do
+      analyzer = MetricAnalyzer.new(@yaml)
+      row = analyzer.table.rows_with('stat_name' => :code_to_test_ratio).first
+      row['stat_value'].should == 2
+    end
+
+  end
+
+  context "with three different path representations of file (from Saikuro, Flog, and Reek)" do
+
+        before do
+      @yaml =<<-__
+:saikuro: 
+  :files: 
+  - :classes: 
+    - :complexity: 19
+      :methods: 
+      - :complexity: 1
+        :lines: 9
+        :name: Client#client_requested_sync
+      - :complexity: 1
+        :lines: 9
+        :name: Client#method_that_is_not_mentioned_elsewhere_in_stats
+      :lines: 92
+      :class_name: Devver::Client
+    :filename: client.rb
+:reek: 
+  :matches: 
+  - :file_path: lib/client/client.rb
+    :code_smells: 
+    - :type: Large Class
+      :message: has at least 27 methods
+      :method: Devver::Client
+    - :type: Long Method
+      :message: has approx 6 statements
+      :method: Devver::Client#client_requested_sync
+:flog: 
+  :total: 1817.6
+  :pages: 
+  - :path: /lib/client/client.rb
+    :highest_score: 37.9
+    :average_score: 13.6
+    :scanned_methods: 
+    - :operators: 
+      - :operator: "[]"
+        :score: 11.1
+      :score: 37.9
+      :name: Client#client_requested_sync
+__
+    end
+
+    specify "all records should have full file_path" do
+      analyzer = MetricAnalyzer.new(@yaml)
+      analyzer.table.each do |row|
+        row['file_path'].should == 'lib/client/client.rb'
+      end
+    end
+    
+    specify "all records should have class name" do
+      analyzer = MetricAnalyzer.new(@yaml)
+      analyzer.table.rows_with(:class_name => nil).should have(0).rows
+    end
+
+    specify "one record should not have method name" do
+      analyzer = MetricAnalyzer.new(@yaml)
+      analyzer.table.rows_with(:method_name => nil).should have(1).rows
+    end
+
+  end
+
+end
+
+