RubyGems - nielsm-metric_fu - Versions diffs - 0.9.1 → 1.1.1 - Mend

nielsm-metric_fu 0.9.1 → 1.1.1

Files changed (94) hide show

data/HISTORY +38 -0
data/README +1 -154
data/Rakefile +9 -2
data/TODO +3 -3
data/lib/base/base_template.rb +146 -0
data/lib/base/configuration.rb +207 -0
data/lib/base/generator.rb +160 -0
data/lib/base/graph.rb +37 -0
data/lib/{metric_fu → base}/md5_tracker.rb +0 -0
data/lib/base/report.rb +100 -0
data/lib/{metric_fu → generators}/churn.rb +26 -23
data/lib/generators/flay.rb +34 -0
data/lib/generators/flog.rb +140 -0
data/lib/generators/rcov.rb +87 -0
data/lib/generators/reek.rb +37 -0
data/lib/generators/roodi.rb +31 -0
data/lib/generators/saikuro.rb +208 -0
data/lib/generators/stats.rb +43 -0
data/lib/graphs/flay_grapher.rb +34 -0
data/lib/graphs/flog_grapher.rb +37 -0
data/lib/graphs/rcov_grapher.rb +34 -0
data/lib/graphs/reek_grapher.rb +44 -0
data/lib/graphs/roodi_grapher.rb +34 -0
data/lib/metric_fu.rb +24 -3
data/lib/templates/awesome/awesome_template.rb +30 -0
data/lib/templates/awesome/churn.html.erb +19 -0
data/lib/templates/awesome/default.css +75 -0
data/lib/templates/awesome/flay.html.erb +27 -0
data/lib/templates/awesome/flog.html.erb +46 -0
data/lib/templates/awesome/index.html.erb +28 -0
data/lib/templates/awesome/layout.html.erb +27 -0
data/lib/templates/awesome/rcov.html.erb +36 -0
data/lib/templates/awesome/reek.html.erb +34 -0
data/lib/templates/awesome/roodi.html.erb +21 -0
data/lib/templates/awesome/saikuro.html.erb +71 -0
data/lib/templates/awesome/stats.html.erb +41 -0
data/lib/templates/{churn.html.erb → standard/churn.html.erb} +13 -4
data/lib/templates/{default.css → standard/default.css} +20 -1
data/lib/templates/{flay.html.erb → standard/flay.html.erb} +13 -9
data/lib/templates/standard/flog.html.erb +53 -0
data/lib/templates/standard/index.html.erb +38 -0
data/lib/templates/standard/rcov.html.erb +43 -0
data/lib/templates/standard/reek.html.erb +42 -0
data/lib/templates/{roodi.html.erb → standard/roodi.html.erb} +11 -8
data/lib/templates/standard/saikuro.html.erb +84 -0
data/lib/templates/standard/standard_template.rb +26 -0
data/lib/templates/standard/stats.html.erb +55 -0
data/spec/base/base_template_spec.rb +161 -0
data/spec/base/configuration_spec.rb +303 -0
data/spec/base/generator_spec.rb +181 -0
data/spec/{md5_tracker_spec.rb → base/md5_tracker_spec.rb} +1 -1
data/spec/base/report_spec.rb +139 -0
data/spec/generators/churn_spec.rb +152 -0
data/spec/generators/flay_spec.rb +104 -0
data/spec/generators/flog_spec.rb +219 -0
data/spec/generators/reek_spec.rb +60 -0
data/spec/generators/saikuro_spec.rb +58 -0
data/spec/generators/stats_spec.rb +74 -0
data/spec/graphs/flog_grapher_spec.rb +15 -0
data/spec/resources/saikuro/app/controllers/sessions_controller.rb_cyclo.html +10 -0
data/spec/resources/saikuro/app/controllers/users_controller.rb_cyclo.html +16 -0
data/spec/resources/saikuro/index_cyclo.html +155 -0
data/spec/resources/saikuro_sfiles/thing.rb_cyclo.html +10 -0
data/spec/spec.opts +8 -0
data/spec/spec_helper.rb +24 -7
data/tasks/metric_fu.rake +22 -0
data/vendor/_fonts/monaco.ttf +0 -0
data/{lib/metric_fu → vendor}/saikuro/saikuro.rb +0 -0
metadata +84 -66
data/lib/metric_fu/base.rb +0 -160
data/lib/metric_fu/flay.rb +0 -17
data/lib/metric_fu/flog.rb +0 -129
data/lib/metric_fu/reek.rb +0 -17
data/lib/metric_fu/roodi.rb +0 -17
data/lib/tasks/churn.rake +0 -9
data/lib/tasks/coverage.rake +0 -54
data/lib/tasks/flay.rake +0 -6
data/lib/tasks/flog.rake +0 -69
data/lib/tasks/metric_fu.rake +0 -24
data/lib/tasks/metric_fu.rb +0 -6
data/lib/tasks/railroad.rake +0 -39
data/lib/tasks/reek.rake +0 -6
data/lib/tasks/roodi.rake +0 -7
data/lib/tasks/saikuro.rake +0 -35
data/lib/tasks/stats.rake +0 -14
data/lib/templates/flog.html.erb +0 -38
data/lib/templates/flog_page.html.erb +0 -25
data/lib/templates/reek.html.erb +0 -30
data/spec/base_spec.rb +0 -57
data/spec/churn_spec.rb +0 -117
data/spec/config_spec.rb +0 -110
data/spec/flay_spec.rb +0 -19
data/spec/flog_spec.rb +0 -208
data/spec/reek_spec.rb +0 -26

data/lib/templates/{roodi.html.erb → standard/roodi.html.erb} RENAMED Viewed

@@ -8,19 +8,22 @@
 <body>
 	<h1>Roodi Results</h1>
+  <a href="index.html">back to menu</a>
   <p><a href="http://roodi.rubyforge.org/">Roodi</a> parses your Ruby code and warns you about design issues you have based on the checks that is has configured.</p>
 	<table>
 		<tr>
 		  <th>File Path</th>
 		  <th>Warning</th>
-		</tr>
-		<% @matches.each_with_index do |match, count| %>
-  		<tr class='<%= cycle("light", "dark", count) %>'>
-  		  <td><%= link_to_filename(match.first.split(':').first) if match.first %></td>
-  		  <td><%= match[1] %></td>
-  		</tr>
-  		<% end %>
+    </tr>
+    <% count = 0 %>
+    <% @roodi[:problems].each do |problem| %>
+      <tr class='<%= cycle("light", "dark", count) %>'>
+        <td><%= problem[:file] %>:<%= problem[:line] %></td>
+        <td><%= problem[:problem] %></td>
+      </tr>
+      <% count += 1 %>
+  	<% end %>
 	</table>
 	<p>Generated on <%= Time.now.localtime %></p>
 </body>
-</html>
+</html>

data/lib/templates/standard/saikuro.html.erb ADDED Viewed

@@ -0,0 +1,84 @@
+<html>
+<head>
+	<title>Saikuro Results</title>
+	<style>
+	<%= inline_css("default.css") %>
+	</style>
+</head>
+<body>
+  <h1>Saikuro Results</h1>
+  <a href="index.html">back to menu</a>
+  <p><a href='http://saikuro.rubyforge.org/'>Saikuro</a> analyzes ruby code for cyclomatic complexity.</p>
+  <h2>Analyzed Methods</h2>
+    <table>
+      <tr>
+        <th>Method Name</th>
+        <th>Complexity</th>
+        <th># Lines</th>
+      </tr>
+      <% @saikuro[:methods].each do |method| %>
+      <tr>
+        <td><%= method[:name] %></td>
+        <td><%= method[:complexity] %></td>
+        <td><%= method[:lines] %></td>
+      </tr>
+      <% end %>
+    </table>
+  <h2>Analyzed Classes</h2>
+    <table>
+      <tr>
+        <th>Class Name</th>
+        <th>Complexity</th>
+        <th># Lines</th>
+      </tr>
+      <% @saikuro[:classes].each do |klass| %>
+      <tr>
+        <td><%= klass[:name] %></td>
+        <td><%= klass[:complexity] %></td>
+        <td><%= klass[:lines] %></td>
+      </tr>
+      <% end %>
+    </table>
+  <h2>Analyzed Files</h2>
+  <% @saikuro[:files].each do |file| %>
+    <% file[:classes].each do |klass| %>
+        <% if !klass[:methods].empty? %>
+          <h3><%= file[:filename] %></h3>
+          <h4>Class : <%= klass[:class_name] %></h4>
+          <h5>Total complexity : <%= klass[:complexity] %></h5>
+          <h5>Total lines : <%= klass[:lines] %></h5>
+          <table>
+              <tr>
+                <th>Method</th>
+                <th>Complexity</th>
+                <th># Lines</th>
+              </tr>
+            <% klass[:methods].each do |method| %>
+              <tr>
+                <td>
+                  <%= method[:name] %>
+                </td>
+                <td>
+                  <%= method[:complexity] %>
+                </td>
+                <td>
+                  <%= method[:lines]  %>
+                </td>
+              </tr>
+            <% end %>
+          </table>
+        <% end %>
+    <% end %>
+ <% end %>
+	<p>Generated on <%= Time.now.localtime %></p>
+</body>
+</html>

data/lib/templates/standard/standard_template.rb ADDED Viewed

@@ -0,0 +1,26 @@
+class StandardTemplate < MetricFu::Template
+  def write
+    report.each_pair do |section, contents|
+      if template_exists?(section)
+        create_instance_var(section, contents)
+        html = erbify(section)
+        fn = output_filename(section)
+        MetricFu.report.save_output(html, MetricFu.output_directory, fn)
+      end
+    end
+    # Instance variables we need should already be created from above
+    if template_exists?('index')
+      html = erbify('index')
+      fn = output_filename('index')
+      MetricFu.report.save_output(html, MetricFu.output_directory, fn)
+    end
+  end
+  def this_directory
+    File.dirname(__FILE__)
+  end
+end

data/lib/templates/standard/stats.html.erb ADDED Viewed

@@ -0,0 +1,55 @@
+<html>
+<head>
+	<title>Rake Stats Results</title>
+	<style>
+	<%= inline_css("default.css") %>
+	</style>
+</head>
+<body>
+	<h1>Rake Stats  Results</h1>
+  <a href="index.html">back to menu</a>
+	<p>Rails rake stats results.</p>
+	<table>
+    <tr>
+      <th>Lines of Code</th>
+      <th>Lines of Test</th>
+      <th>Code to test ratio</th>
+    </tr>
+    <tr>
+      <td><%= @stats[:codeLOC] %></td>
+      <td><%= @stats[:testLOC] %></td>
+      <td>1:<%= @stats[:code_to_test_ratio] %></td>
+    </tr>
+  </table>
+	<table>
+    <tr>
+      <th>Name</th>
+      <th>Lines</th>
+      <th>LOC</th>
+      <th>Classes</th>
+      <th>Methods</th>
+      <th>Methods per class</th>
+      <th>LOC per method</th>
+    </tr>
+    <% count = 0 %>
+    <% @stats[:lines].each do |line| %>
+      <tr>
+        <td><%= line[:name] %></td>
+        <td><%= line[:lines] %></td>
+        <td><%= line[:loc] %></td>
+        <td><%= line[:classes] %></td>
+        <td><%= line[:methods] %></td>
+        <td><%= line[:methods_per_class] %></td>
+        <td><%= line[:loc_per_method] %></td>
+      </tr>
+      <% count += 1 %>
+		<% end %>
+  </table>
+	<p>Generated on <%= Time.now.localtime %></p>
+</body>
+</html>

data/spec/base/base_template_spec.rb ADDED Viewed

@@ -0,0 +1,161 @@
+require File.expand_path(File.dirname(__FILE__) + "/../spec_helper")
+require 'erb'
+describe MetricFu::Template do
+  before(:each) do
+    @template =  Template.new
+  end
+  describe "#erbify" do
+    it 'should evaluate a erb doc' do
+      section = 'section'
+      File.stub!(:read).and_return('foo')
+      erb = mock('erb')
+      erb.should_receive(:result)
+      ERB.should_receive(:new).with('foo').and_return(erb)
+      @template.should_receive(:template).and_return('foo')
+      @template.send(:erbify, section)
+    end
+  end
+  describe "#template_exists? " do
+    before(:each) do
+      @section = mock('section')
+      @template.should_receive(:template).
+                with(@section).and_return(@section)
+    end
+    describe 'if the template exists' do
+      it 'should return true' do
+        File.should_receive(:exist?).with(@section).and_return(true)
+        result = @template.send(:template_exists?, @section)
+        result.should be_true
+      end
+    end
+    describe 'if the template does not exist' do
+      it 'should return false' do
+        File.should_receive(:exist?).with(@section).and_return(false)
+        result = @template.send(:template_exists?, @section)
+        result.should be_false
+      end
+    end
+  end
+  describe "#create_instance_var" do
+    it 'should set an instance variable with the passed contents' do
+      section = 'section'
+      contents = 'contents'
+      @template.send(:create_instance_var, section, contents)
+      @template.instance_variable_get(:@section).should == contents
+    end
+  end
+  describe "#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')
+      result = @template.send(:template, section)
+      result.should == "dir/section.html.erb"
+    end
+  end
+  describe "#output_filename" do
+    it 'should generate the filename of the output file' do
+      section = mock('section')
+      section.should_receive(:to_s).and_return('section')
+      result = @template.send(:output_filename, section)
+      result.should == "section.html"
+    end
+  end
+  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')
+      io = mock('io', :read => "css contents")
+      @template.should_receive(:open).and_yield(io)
+      result = @template.send(:inline_css, css)
+      result.should == 'css contents'
+    end
+  end
+  describe "#link_to_filename " do
+    describe "when on OS X" do
+      before(:each) do
+        config = mock("configuration")
+        config.stub!(:platform).and_return('universal-darwin-9.0')
+        MetricFu.stub!(:configuration).and_return(config)
+      end
+      it 'should return a textmate protocol link' do
+        File.stub!(:expand_path).with('filename').and_return('/expanded/filename')
+        result = @template.send(:link_to_filename, 'filename')
+        result.should eql("<a href='txmt://open/?url=file://" \
+                         + "/expanded/filename'>filename</a>")
+      end
+      it "should do the right thing with a filename that starts with a slash" do
+        File.stub!(:expand_path).with('filename').and_return('/expanded/filename')
+        result = @template.send(:link_to_filename, '/filename')
+        result.should eql("<a href='txmt://open/?url=file://" \
+                         + "/expanded/filename'>/filename</a>")
+      end
+      it "should include a line number" do
+        File.stub!(:expand_path).with('filename').and_return('/expanded/filename')
+        result = @template.send(:link_to_filename, 'filename', 6)
+        result.should eql("<a href='txmt://open/?url=file://" \
+                         + "/expanded/filename&line=6'>filename:6</a>")
+      end
+      describe "and given link text" do
+        it "should use the submitted link text" do
+          File.stub!(:expand_path).with('filename').and_return('/expanded/filename')
+          result = @template.send(:link_to_filename, 'filename', 6, 'link content')
+          result.should eql("<a href='txmt://open/?url=file://" \
+                           + "/expanded/filename&line=6'>link content</a>")
+        end
+      end
+    end
+    describe "when on other platforms"  do
+      before(:each) do
+        config = mock("configuration")
+        config.should_receive(:platform).and_return('other')
+        MetricFu.stub!(:configuration).and_return(config)
+        File.should_receive(:expand_path).and_return('filename')
+      end
+      it 'should return a file protocol link' do
+        name = "filename"
+        result = @template.send(:link_to_filename, name)
+        result.should == "<a href='file://filename'>filename</a>"
+      end
+    end
+  end
+  describe "#cycle" do
+    it 'should return the first_value passed if iteration passed is even' do
+      first_val = "first"
+      second_val = "second"
+      iter = 2
+      result = @template.send(:cycle, first_val, second_val, iter)
+      result.should == first_val
+    end
+    it 'should return the second_value passed if iteration passed is odd' do
+      first_val = "first"
+      second_val = "second"
+      iter = 1
+      result = @template.send(:cycle, first_val, second_val, iter)
+      result.should == second_val
+    end
+  end
+end

data/spec/base/configuration_spec.rb ADDED Viewed

@@ -0,0 +1,303 @@
+require File.expand_path(File.dirname(__FILE__) + "/../spec_helper")
+describe MetricFu::Configuration do
+  def get_new_config
+    @config = Configuration.new
+  end
+  def base_directory
+    @config.instance_variable_get(:@base_directory)
+  end
+  def output_directory
+    @config.instance_variable_get(:@output_directory)
+  end
+  def scratch_directory
+    @config.instance_variable_get(:@scratch_directory)
+  end
+  def template_directory
+    @config.instance_variable_get(:@template_directory)
+  end
+  def template_class
+    @config.instance_variable_get(:@template_class)
+  end
+  def metric_fu_root
+    @config.instance_variable_get(:@metric_fu_root_directory)
+  end
+  describe '#warn_about_deprecated_config_options' do
+    def get_new_config_and_raise_runtime_error
+      lambda { get_new_config  }.should raise_error
+    end
+    describe 'when ::MetricFu::CHURN_OPTIONS is present' do
+      before(:each) { ::MetricFu::CHURN_OPTIONS = 'option' }
+      after(:each)  { ::MetricFu.send(:remove_const, 'CHURN_OPTIONS') }
+      it 'should raise a RuntimeError with "Use config.churn '+
+      'instead of MetricFu::CHURN_OPTIONS"' do
+        get_new_config_and_raise_runtime_error
+      end
+    end
+    describe 'when ::MetricFu::DIRECTORIES_TO_FLOG is present' do
+      before(:each) { ::MetricFu::DIRECTORIES_TO_FLOG = 'option' }
+      after(:each)  { ::MetricFu.send(:remove_const,'DIRECTORIES_TO_FLOG')}
+      it 'should raise a RuntimeError with "Use config.flog '+
+      '[:dirs_to_flog] instead of MetricFu::DIRECTORIES_TO_FLOG' do
+        get_new_config_and_raise_runtime_error
+      end
+    end
+    describe 'when ::MetricFu::SAIKURO_OPTIONS is present' do
+      before(:each) { ::MetricFu::SAIKURO_OPTIONS = 'option' }
+      after(:each)  { ::MetricFu.send(:remove_const,'SAIKURO_OPTIONS')}
+      it 'should raise a RuntimeError with "Use config.saikuro '+
+      'instead of MetricFu::SAIKURO_OPTIONS' do
+        get_new_config_and_raise_runtime_error
+      end
+    end
+    describe 'when SAIKURO_OPTIONS is present' do
+      before(:each) { SAIKURO_OPTIONS = 'option' }
+      after(:each)  { Object.send(:remove_const,'SAIKURO_OPTIONS')}
+      it 'should raise a RuntimeError with "Use config.saikuro '+
+      'instead of SAIKURO_OPTIONS' do
+        get_new_config_and_raise_runtime_error
+      end
+    end
+  end
+  describe "#reset" do
+    before(:each) { get_new_config }
+    describe 'when there is a CC_BUILD_ARTIFACTS environment variable' do
+      before(:each) { ENV['CC_BUILD_ARTIFACTS'] = 'foo' }
+      it 'should return the CC_BUILD_ARTIFACTS environment variable' do
+        get_new_config
+        base_directory.should == ENV['CC_BUILD_ARTIFACTS']
+      end
+    end
+    describe 'when there is no CC_BUILD_ARTIFACTS environment variable' do
+      before(:each) { ENV['CC_BUILD_ARTIFACTS'] = nil  }
+      it 'should return "tmp/metric_fu"' do
+        get_new_config
+        base_directory.should == "tmp/metric_fu"
+      end
+    end
+    it 'should set @metric_fu_root_directory to the base of the '+
+    'metric_fu application' do
+      app_root = File.join(File.dirname(__FILE__), '..', '..')
+      app_root_absolute_path = File.expand_path(app_root)
+      metric_fu_absolute_path = File.expand_path(metric_fu_root)
+      metric_fu_absolute_path.should == app_root_absolute_path
+    end
+    it 'should set @template_directory to the lib/templates relative '+
+    'to @metric_fu_root_directory' do
+      template_dir = File.join(File.dirname(__FILE__),
+                               '..', '..', 'lib','templates')
+      template_dir_abs_path = File.expand_path(template_dir)
+      calc_template_dir_abs_path = File.expand_path(template_directory)
+      calc_template_dir_abs_path.should == template_dir_abs_path
+    end
+    it 'should set @scratch_directory to scratch relative '+
+    'to @base_directory' do
+      scratch_dir = File.join(base_directory, 'scratch')
+      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_directory.should == output_dir
+    end
+    it 'should set @template_class to StandardTemplate' do
+      template_class.should == AwesomeTemplate
+    end
+    it 'should set @flay to {:dirs_to_flay => @code_dirs}' do
+      @config.instance_variable_get(:@flay).
+              should == {:dirs_to_flay => ['lib']}
+    end
+    it 'should set @flog to {:dirs_to_flog => @code_dirs}' do
+      @config.instance_variable_get(:@flog).
+              should == {:dirs_to_flog => ['lib']}
+    end
+    it 'should set @reek to {:dirs_to_reek => @code_dirs}' do
+      @config.instance_variable_get(:@reek).
+              should == {:dirs_to_reek => ['lib']}
+    end
+    it 'should set @roodi to {:dirs_to_roodi => @code_dirs}' do
+      @config.instance_variable_get(:@roodi).
+              should == {:dirs_to_roodi => ['lib']}
+    end
+    it 'should set @churn to {}' do
+      @config.instance_variable_get(:@churn).
+              should == {}
+    end
+    it 'should set @stats to {}' do
+      @config.instance_variable_get(:@stats).
+              should == {}
+    end
+    it 'should set @rcov to { :test_files => ["test/**/*_test.rb",
+                                              "spec/**/*_spec.rb"]
+                              :rcov_opts  => ["--sort coverage",
+                                              "--no-html",
+                                              "--text-coverage",
+                                              "--no-color",
+                                              "--profile",
+                                              "--rails",
+                                              "--exclude /gems/,/Library/,/usr/,spec"]}' do
+      @config.instance_variable_get(:@rcov).
+              should ==  { :test_files => ['test/**/*_test.rb',
+                                           'spec/**/*_spec.rb'],
+                           :rcov_opts => ["--sort coverage",
+                                         "--no-html",
+                                         "--text-coverage",
+                                         "--no-color",
+                                         "--profile",
+                                         "--rails",
+                                         "--exclude /gems/,/Library/,/usr/,spec"]}
+    end
+    it 'should set @saikuro to { :output_directory => @scratch_directory + "/saikuro",
+                                 :input_directory => @code_dirs,
+                                 :cyclo => "",
+                                 :filter_cyclo => "0",
+                                 :warn_cyclo => "5",
+                                 :error_cyclo => "7",
+                                 :formater => "text" }' do
+      @config.instance_variable_get(:@saikuro).
+              should ==  { :output_directory => 'tmp/metric_fu/scratch/saikuro',
+                    :input_directory => ['lib'],
+                    :cyclo => "",
+                    :filter_cyclo => "0",
+                    :warn_cyclo => "5",
+                    :error_cyclo => "7",
+                    :formater => "text"}
+    end
+    describe 'if #rails? is true ' do
+      before(:each) do
+        @config.stub!(:rails?).and_return(true)
+      end
+      describe '#set_metrics ' do
+        it 'should set the @metrics instance var to AVAILABLE_METRICS + '\
+           +'[:stats]' do
+          @config.instance_variable_get(:@metrics).
+                  should == MetricFu::AVAILABLE_METRICS << [:stats]
+        end
+      end
+      describe '#set_code_dirs ' do
+        it 'should set the @code_dirs instance var to ["app", "lib"]' do
+          # This is hard to spec properly because the @code_dirs variable
+          # is set during the reset process.
+          #@config.instance_variable_get(:@code_dirs).
+          #        should == ['app','lib']
+        end
+      end
+    end
+    describe 'if #rails? is false ' do
+      before(:each) do
+        @config.stub!(:rails?).and_return(false)
+      end
+      describe '#set_metrics ' do
+        it 'should set the @metrics instance var to AVAILABLE_METRICS' do
+          @config.instance_variable_get(:@metrics).
+                  should == MetricFu::AVAILABLE_METRICS
+        end
+      end
+      describe '#set_code_dirs ' do
+        it 'should set the @code_dirs instance var to ["lib"]' do
+          @config.instance_variable_get(:@code_dirs).should == ['lib']
+        end
+      end
+    end
+  end
+  describe '#add_attr_accessors_to_self' do
+    before(:each) { get_new_config }
+    MetricFu::AVAILABLE_METRICS.each do |metric|
+      it "should have a reader for #{metric}" do
+        @config.respond_to?(metric).should be_true
+      end
+      it "should have a writer for #{metric}=" do
+        @config.respond_to?((metric.to_s + '=').to_sym).should be_true
+      end
+    end
+  end
+  describe '#add_class_methods_to_metric_fu' do
+    before(:each) { get_new_config }
+    MetricFu::AVAILABLE_METRICS.each do |metric|
+      it "should add a #{metric} class method to the MetricFu module " do
+        MetricFu.respond_to?(metric).should be_true
+      end
+    end
+  end
+  describe '#platform' do
+    before(:each) { get_new_config }
+    it 'should return the value of the PLATFORM constant' do
+      this_platform = PLATFORM
+      @config.platform.should == this_platform
+    end
+  end
+  describe '#is_cruise_control_rb? ' do
+    before(:each) { get_new_config }
+    describe "when the CC_BUILD_ARTIFACTS env var is not nil" do
+      before(:each) { ENV['CC_BUILD_ARTIFACTS'] = 'is set' }
+      it 'should return true'  do
+        @config.is_cruise_control_rb?.should be_true
+      end
+    end
+    describe "when the CC_BUILD_ARTIFACTS env var is nil" do
+      before(:each) { ENV['CC_BUILD_ARTIFACTS'] = nil }
+      it 'should return false' do
+        @config.is_cruise_control_rb?.should be_false
+      end
+    end
+  end
+end