simplecov 0.3.7 → 0.3.9

data/test/test_1_8_fallbacks.rb

@@ -0,0 +1,33 @@
+require 'helper'
+
+# Tests that verify that on 1.8 versions of ruby, simplecov simply
+# does not launch and does not cause errors on the way
+#
+# TODO: This should be expanded upon all methods that could potentially
+# be called in a test/spec-helper simplecov config block
+#
+class Test18FallBacks < Test::Unit::TestCase
+  on_ruby '1.8' do
+    should "return false when calling SimpleCov.start" do
+      assert_equal false, SimpleCov.start
+    end
+    
+    should "return false when calling SimpleCov.start with a block" do
+      assert_equal false, SimpleCov.start { raise "Shouldn't reach this!?" }
+    end
+    
+    should "return false when calling SimpleCov.configure with a block" do
+      assert_equal false, SimpleCov.configure { raise "Shouldn't reach this!?" }
+    end
+    
+    should "allow to define an adapter" do
+      begin
+        SimpleCov.adapters.define 'testadapter' do
+          add_filter '/config/'
+        end
+      rescue => err
+        assert false, "Adapter definition should have been fine, but raised #{err}"
+      end
+    end
+  end
+end

data/test/test_command_guesser.rb

@@ -1,18 +1,20 @@
 require 'helper'
 
 class TestCommandGuesser < Test::Unit::TestCase
-  def self.should_guess_command_name(expectation, *argv)
-    argv.each do |args|
-      should "return '#{expectation}' for '#{args}'" do
-        assert_equal expectation, SimpleCov::CommandGuesser.guess(args)
+  on_ruby '1.9' do
+    def self.should_guess_command_name(expectation, *argv)
+      argv.each do |args|
+        should "return '#{expectation}' for '#{args}'" do
+          assert_equal expectation, SimpleCov::CommandGuesser.guess(args)
+        end
       end
     end
-  end
   
-  should_guess_command_name "Unit Tests", '/some/path/test/units/foo_bar_test.rb', 'test/units/foo.rb', 'test/foo.rb'
-  should_guess_command_name "Functional Tests", '/some/path/test/functional/foo_bar_controller_test.rb'
-  should_guess_command_name "Integration Tests", '/some/path/test/integration/foo_bar_controller_test.rb'  
-  should_guess_command_name "Cucumber Features", 'features', 'cucumber', 'cucumber features'
-  should_guess_command_name "RSpec", '/some/path/spec/foo.rb'
-  should_guess_command_name "some_arbitrary_command with arguments", 'some_arbitrary_command with arguments'
+    should_guess_command_name "Unit Tests", '/some/path/test/units/foo_bar_test.rb', 'test/units/foo.rb', 'test/foo.rb'
+    should_guess_command_name "Functional Tests", '/some/path/test/functional/foo_bar_controller_test.rb'
+    should_guess_command_name "Integration Tests", '/some/path/test/integration/foo_bar_controller_test.rb'  
+    should_guess_command_name "Cucumber Features", 'features', 'cucumber', 'cucumber features'
+    should_guess_command_name "RSpec", '/some/path/spec/foo.rb'
+    should_guess_command_name "some_arbitrary_command with arguments", 'some_arbitrary_command with arguments'
+  end
 end

data/test/test_filters.rb

@@ -1,73 +1,75 @@
 require 'helper'
 
 class TestFilters < Test::Unit::TestCase
-  context "A source file initialized with some coverage data" do
-    setup do
-      @source_file = SimpleCov::SourceFile.new(source_fixture('sample.rb'), [nil, 1, 1, 1, nil, nil, 1, 0, nil, nil])
-    end
+  on_ruby '1.9' do
+    context "A source file initialized with some coverage data" do
+      setup do
+        @source_file = SimpleCov::SourceFile.new(source_fixture('sample.rb'), [nil, 1, 1, 1, nil, nil, 1, 0, nil, nil])
+      end
     
-    should "pass a new SimpleCov::StringFilter 'foobar'" do
-      assert SimpleCov::StringFilter.new('foobar').passes?(@source_file)
-    end
+      should "pass a new SimpleCov::StringFilter 'foobar'" do
+        assert SimpleCov::StringFilter.new('foobar').passes?(@source_file)
+      end
     
-    should "pass a new SimpleCov::StringFilter 'some/path'" do
-      assert SimpleCov::StringFilter.new('some/path').passes?(@source_file)
-    end
+      should "pass a new SimpleCov::StringFilter 'some/path'" do
+        assert SimpleCov::StringFilter.new('some/path').passes?(@source_file)
+      end
     
-    should "not pass a new SimpleCov::StringFilter 'test/fixtures'" do
-      assert !SimpleCov::StringFilter.new('test/fixtures').passes?(@source_file)
-    end
+      should "not pass a new SimpleCov::StringFilter 'test/fixtures'" do
+        assert !SimpleCov::StringFilter.new('test/fixtures').passes?(@source_file)
+      end
     
-    should "not pass a new SimpleCov::StringFilter 'test/fixtures/sample.rb'" do
-      assert !SimpleCov::StringFilter.new('test/fixtures/sample.rb').passes?(@source_file)
-    end
+      should "not pass a new SimpleCov::StringFilter 'test/fixtures/sample.rb'" do
+        assert !SimpleCov::StringFilter.new('test/fixtures/sample.rb').passes?(@source_file)
+      end
     
-    should "not pass a new SimpleCov::StringFilter 'sample.rb'" do
-      assert !SimpleCov::StringFilter.new('sample.rb').passes?(@source_file)
-    end
+      should "not pass a new SimpleCov::StringFilter 'sample.rb'" do
+        assert !SimpleCov::StringFilter.new('sample.rb').passes?(@source_file)
+      end
     
-    should "pass a new SimpleCov::BlockFilter that is not applicable" do
-      assert SimpleCov::BlockFilter.new(Proc.new {|s| File.basename(s.filename) == 'foo.rb'}).passes?(@source_file)
-    end
+      should "pass a new SimpleCov::BlockFilter that is not applicable" do
+        assert SimpleCov::BlockFilter.new(Proc.new {|s| File.basename(s.filename) == 'foo.rb'}).passes?(@source_file)
+      end
     
-    should "not pass a new SimpleCov::BlockFilter that is applicable" do
-      assert !SimpleCov::BlockFilter.new(Proc.new {|s| File.basename(s.filename) == 'sample.rb'}).passes?(@source_file)
+      should "not pass a new SimpleCov::BlockFilter that is applicable" do
+        assert !SimpleCov::BlockFilter.new(Proc.new {|s| File.basename(s.filename) == 'sample.rb'}).passes?(@source_file)
+      end
     end
-  end
   
-  context "with no filters set up and a basic source file in an array" do
-    setup do
-      SimpleCov.filters = []
-      @files = [SimpleCov::SourceFile.new(source_fixture('sample.rb'), [nil, 1, 1, 1, nil, nil, 1, 0, nil, nil])]
-    end
+    context "with no filters set up and a basic source file in an array" do
+      setup do
+        SimpleCov.filters = []
+        @files = [SimpleCov::SourceFile.new(source_fixture('sample.rb'), [nil, 1, 1, 1, nil, nil, 1, 0, nil, nil])]
+      end
     
-    should "return 0 items after executing SimpleCov.filtered on files when using a 'sample' string filter" do
-      SimpleCov.add_filter "sample"
-      assert_equal 0, SimpleCov.filtered(@files).count
-    end
+      should "return 0 items after executing SimpleCov.filtered on files when using a 'sample' string filter" do
+        SimpleCov.add_filter "sample"
+        assert_equal 0, SimpleCov.filtered(@files).count
+      end
     
-    should "return 0 items after executing SimpleCov.filtered on files when using a 'test/fixtures/' string filter" do
-      SimpleCov.add_filter "test/fixtures"
-      assert_equal 0, SimpleCov.filtered(@files).count
-    end
+      should "return 0 items after executing SimpleCov.filtered on files when using a 'test/fixtures/' string filter" do
+        SimpleCov.add_filter "test/fixtures"
+        assert_equal 0, SimpleCov.filtered(@files).count
+      end
     
-    should "return 1 item after executing SimpleCov.filtered on files when using a 'fooo' string filter" do
-      SimpleCov.add_filter "fooo"
-      assert_equal 1, SimpleCov.filtered(@files).count
-    end
+      should "return 1 item after executing SimpleCov.filtered on files when using a 'fooo' string filter" do
+        SimpleCov.add_filter "fooo"
+        assert_equal 1, SimpleCov.filtered(@files).count
+      end
     
-    should "return 0 items after executing SimpleCov.filtered on files when using a block filter that returns true" do
-      SimpleCov.add_filter do |src_file|
-        true
+      should "return 0 items after executing SimpleCov.filtered on files when using a block filter that returns true" do
+        SimpleCov.add_filter do |src_file|
+          true
+        end
+        assert_equal 0, SimpleCov.filtered(@files).count
       end
-      assert_equal 0, SimpleCov.filtered(@files).count
-    end
     
-    should "return 1 item after executing SimpleCov.filtered on files when using an always-false block filter" do
-      SimpleCov.add_filter do |src_file|
-        false
+      should "return 1 item after executing SimpleCov.filtered on files when using an always-false block filter" do
+        SimpleCov.add_filter do |src_file|
+          false
+        end
+        assert_equal 1, SimpleCov.filtered(@files).count
       end
-      assert_equal 1, SimpleCov.filtered(@files).count
     end
   end
 end

data/test/test_merge_helpers.rb

@@ -1,105 +1,107 @@
 require 'helper'
 
 class TestMergeHelpers < Test::Unit::TestCase
-  context "With two faked coverage resultsets" do
-    setup do
-      SimpleCov.use_merging true
-      @resultset1 = {source_fixture('sample.rb') => [nil, 1, 1, 1, nil, nil, 1, 1, nil, nil],
-          source_fixture('app/models/user.rb') => [nil, 1, 1, 1, nil, nil, 1, 0, nil, nil],
-          source_fixture('app/controllers/sample_controller.rb') => [nil, 1, 1, 1, nil, nil, 1, 0, nil, nil],
-          source_fixture('resultset1.rb') => [1, 1, 1, 1]}
-          
-      @resultset2 = {source_fixture('sample.rb') => [1, nil, 1, 1, nil, nil, 1, 1, nil, nil],
-          source_fixture('app/models/user.rb') => [nil, 1, 5, 1, nil, nil, 1, 0, nil, nil],
-          source_fixture('app/controllers/sample_controller.rb') => [nil, 3, 1, nil, nil, nil, 1, 0, nil, nil],
-          source_fixture('resultset2.rb') => [nil, 1, 1, nil]}
-    end
-    
-    context "a merge" do
+  on_ruby '1.9' do
+    context "With two faked coverage resultsets" do
       setup do
-        assert @merged = @resultset1.merge_resultset(@resultset2)
+        SimpleCov.use_merging true
+        @resultset1 = {source_fixture('sample.rb') => [nil, 1, 1, 1, nil, nil, 1, 1, nil, nil],
+            source_fixture('app/models/user.rb') => [nil, 1, 1, 1, nil, nil, 1, 0, nil, nil],
+            source_fixture('app/controllers/sample_controller.rb') => [nil, 1, 1, 1, nil, nil, 1, 0, nil, nil],
+            source_fixture('resultset1.rb') => [1, 1, 1, 1]}
+          
+        @resultset2 = {source_fixture('sample.rb') => [1, nil, 1, 1, nil, nil, 1, 1, nil, nil],
+            source_fixture('app/models/user.rb') => [nil, 1, 5, 1, nil, nil, 1, 0, nil, nil],
+            source_fixture('app/controllers/sample_controller.rb') => [nil, 3, 1, nil, nil, nil, 1, 0, nil, nil],
+            source_fixture('resultset2.rb') => [nil, 1, 1, nil]}
       end
+    
+      context "a merge" do
+        setup do
+          assert @merged = @resultset1.merge_resultset(@resultset2)
+        end
       
-      should "have proper results for sample.rb" do
-        assert_equal [1, 1, 2, 2, nil, nil, 2, 2, nil, nil], @merged[source_fixture('sample.rb')]
-      end
+        should "have proper results for sample.rb" do
+          assert_equal [1, 1, 2, 2, nil, nil, 2, 2, nil, nil], @merged[source_fixture('sample.rb')]
+        end
       
-      should "have proper results for user.rb" do
-        assert_equal [nil, 2, 6, 2, nil, nil, 2, 0, nil, nil], @merged[source_fixture('app/models/user.rb')]
-      end
+        should "have proper results for user.rb" do
+          assert_equal [nil, 2, 6, 2, nil, nil, 2, 0, nil, nil], @merged[source_fixture('app/models/user.rb')]
+        end
       
-      should "have proper results for sample_controller.rb" do
-        assert_equal [nil, 4, 2, 1, nil, nil, 2, 0, nil, nil], @merged[source_fixture('app/controllers/sample_controller.rb')]
-      end
+        should "have proper results for sample_controller.rb" do
+          assert_equal [nil, 4, 2, 1, nil, nil, 2, 0, nil, nil], @merged[source_fixture('app/controllers/sample_controller.rb')]
+        end
       
-      should "have proper results for resultset1.rb" do
-        assert_equal [1, 1, 1, 1], @merged[source_fixture('resultset1.rb')]
-      end
+        should "have proper results for resultset1.rb" do
+          assert_equal [1, 1, 1, 1], @merged[source_fixture('resultset1.rb')]
+        end
       
-      should "have proper results for resultset2.rb" do
-        assert_equal [nil, 1, 1, nil], @merged[source_fixture('resultset2.rb')]
+        should "have proper results for resultset2.rb" do
+          assert_equal [nil, 1, 1, nil], @merged[source_fixture('resultset2.rb')]
+        end
       end
-    end
     
-    # See Github issue #6
-    should "return an empty hash when the resultset cache file is empty" do
-      File.open(SimpleCov::ResultMerger.resultset_path, "w+") {|f| f.puts ""}
-      assert_equal Hash.new, SimpleCov::ResultMerger.resultset 
-    end
+      # See Github issue #6
+      should "return an empty hash when the resultset cache file is empty" do
+        File.open(SimpleCov::ResultMerger.resultset_path, "w+") {|f| f.puts ""}
+        assert_equal Hash.new, SimpleCov::ResultMerger.resultset 
+      end
 
-    # See Github issue #6    
-    should "return an empty hash when the resultset cache file is not present" do
-      system "rm #{SimpleCov::ResultMerger.resultset_path}" if File.exist?(SimpleCov::ResultMerger.resultset_path)
-      assert_equal Hash.new, SimpleCov::ResultMerger.resultset 
-    end
-    
-    context "and results generated from those" do
-      setup do
+      # See Github issue #6    
+      should "return an empty hash when the resultset cache file is not present" do
         system "rm #{SimpleCov::ResultMerger.resultset_path}" if File.exist?(SimpleCov::ResultMerger.resultset_path)
-        @result1 = SimpleCov::Result.new(@resultset1)
-        @result1.command_name = "result1"
-        @result2 = SimpleCov::Result.new(@resultset2)
-        @result2.command_name = "result2"
+        assert_equal Hash.new, SimpleCov::ResultMerger.resultset 
       end
-      
-      context "with stored results" do
+    
+      context "and results generated from those" do
         setup do
-          assert SimpleCov::ResultMerger.store_result(@result1)
-          assert SimpleCov::ResultMerger.store_result(@result2)
-        end
-        
-        should "have stored data in resultset_path yaml file" do
-          assert File.readlines(SimpleCov::ResultMerger.resultset_path).length > 50
+          system "rm #{SimpleCov::ResultMerger.resultset_path}" if File.exist?(SimpleCov::ResultMerger.resultset_path)
+          @result1 = SimpleCov::Result.new(@resultset1)
+          @result1.command_name = "result1"
+          @result2 = SimpleCov::Result.new(@resultset2)
+          @result2.command_name = "result2"
         end
+      
+        context "with stored results" do
+          setup do
+            assert SimpleCov::ResultMerger.store_result(@result1)
+            assert SimpleCov::ResultMerger.store_result(@result2)
+          end
         
-        should "return a hash containing keys ['result1' and 'result2'] for resultset" do
-          assert_equal ['result1', 'result2'], SimpleCov::ResultMerger.resultset.keys.sort
-        end
+          should "have stored data in resultset_path yaml file" do
+            assert File.readlines(SimpleCov::ResultMerger.resultset_path).length > 50
+          end
         
-        should "return proper values for merged_result" do
-          assert_equal [nil, 2, 6, 2, nil, nil, 2, 0, nil, nil], SimpleCov::ResultMerger.merged_result.source_files.find {|s| s.filename =~ /user/}.lines.map(&:coverage)
-        end
+          should "return a hash containing keys ['result1' and 'result2'] for resultset" do
+            assert_equal ['result1', 'result2'], SimpleCov::ResultMerger.resultset.keys.sort
+          end
         
-        context "with second result way above the merge_timeout" do
-          setup do 
-            @result2.created_at = Time.mktime(Time.now.year, Time.now.month, Time.now.day-2)
-            assert SimpleCov::ResultMerger.store_result(@result2)
+          should "return proper values for merged_result" do
+            assert_equal [nil, 2, 6, 2, nil, nil, 2, 0, nil, nil], SimpleCov::ResultMerger.merged_result.source_files.find {|s| s.filename =~ /user/}.lines.map(&:coverage)
           end
+        
+          context "with second result way above the merge_timeout" do
+            setup do 
+              @result2.created_at = Time.mktime(Time.now.year, Time.now.month, Time.now.day-2)
+              assert SimpleCov::ResultMerger.store_result(@result2)
+            end
           
-          should "have only one result in SimpleCov::ResultMerger.results" do
-            assert_equal 1, SimpleCov::ResultMerger.results.length
+            should "have only one result in SimpleCov::ResultMerger.results" do
+              assert_equal 1, SimpleCov::ResultMerger.results.length
+            end
           end
-        end
         
-        context "with merging disabled" do
-          setup { SimpleCov.use_merging false }
+          context "with merging disabled" do
+            setup { SimpleCov.use_merging false }
           
-          should "return nil for SimpleCov.result" do
-            assert_nil SimpleCov.result
+            should "return nil for SimpleCov.result" do
+              assert_nil SimpleCov.result
+            end
           end
         end
-      end
       
+      end
     end
   end
 end

data/test/test_result.rb

@@ -1,133 +1,135 @@
 require 'helper'
 
 class TestResult < Test::Unit::TestCase
-  context "With a (mocked) Coverage.result" do
-    setup do
-      SimpleCov.filters = []
-      SimpleCov.groups = {}
-      SimpleCov.formatter = nil
-      @original_result = {source_fixture('sample.rb') => [nil, 1, 1, 1, nil, nil, 1, 1, nil, nil],
-          source_fixture('app/models/user.rb') => [nil, 1, 1, 1, nil, nil, 1, 0, nil, nil],
-          source_fixture('app/controllers/sample_controller.rb') => [nil, 1, 1, 1, nil, nil, 1, 0, nil, nil]}
-    end
+  on_ruby "1.9" do
+    context "With a (mocked) Coverage.result" do
+      setup do
+        SimpleCov.filters = []
+        SimpleCov.groups = {}
+        SimpleCov.formatter = nil
+        @original_result = {source_fixture('sample.rb') => [nil, 1, 1, 1, nil, nil, 1, 1, nil, nil],
+            source_fixture('app/models/user.rb') => [nil, 1, 1, 1, nil, nil, 1, 0, nil, nil],
+            source_fixture('app/controllers/sample_controller.rb') => [nil, 1, 1, 1, nil, nil, 1, 0, nil, nil]}
+      end
   
-    context "a simple cov result initialized from that" do
-      setup { @result = SimpleCov::Result.new(@original_result) }
+      context "a simple cov result initialized from that" do
+        setup { @result = SimpleCov::Result.new(@original_result) }
     
-      should "have 3 filenames" do
-        assert_equal 3, @result.filenames.count
-      end
+        should "have 3 filenames" do
+          assert_equal 3, @result.filenames.count
+        end
     
-      should "have 3 source files" do
-        assert_equal 3, @result.source_files.count
-        assert @result.source_files.all? {|s| s.instance_of?(SimpleCov::SourceFile)}, "Not alL instances are of SimpleCov::SourceFile type"
-      end
+        should "have 3 source files" do
+          assert_equal 3, @result.source_files.count
+          assert @result.source_files.all? {|s| s.instance_of?(SimpleCov::SourceFile)}, "Not alL instances are of SimpleCov::SourceFile type"
+        end
     
-      should "have files equal to source_files" do
-        assert_equal @result.files, @result.source_files
-      end
+        should "have files equal to source_files" do
+          assert_equal @result.files, @result.source_files
+        end
       
-      should "have accurate covered percent" do
-        # in our fixture, there are 13 covered line (result in 1) in all 15 relevant line (result in non-nil)
-        assert_equal 100.0*13/15, @result.covered_percent
-      end
+        should "have accurate covered percent" do
+          # in our fixture, there are 13 covered line (result in 1) in all 15 relevant line (result in non-nil)
+          assert_equal 100.0*13/15, @result.covered_percent
+        end
       
-      context "dumped with to_yaml" do
-        setup { @yaml = @result.to_yaml }
-        should("be a string") { assert_equal String, @yaml.class }
+        context "dumped with to_yaml" do
+          setup { @yaml = @result.to_yaml }
+          should("be a string") { assert_equal String, @yaml.class }
         
-        context "loaded back with from_yaml" do
-          setup { @dumped_result = SimpleCov::Result.from_yaml(@yaml) }
+          context "loaded back with from_yaml" do
+            setup { @dumped_result = SimpleCov::Result.from_yaml(@yaml) }
           
-          should "have 3 source files" do
-            assert_equal @result.source_files.count, @dumped_result.source_files.count
-          end
+            should "have 3 source files" do
+              assert_equal @result.source_files.count, @dumped_result.source_files.count
+            end
           
-          should "have the same covered_percent" do
-            assert_equal @result.covered_percent, @dumped_result.covered_percent
-          end
+            should "have the same covered_percent" do
+              assert_equal @result.covered_percent, @dumped_result.covered_percent
+            end
           
-          should "have the same created_at" do
-            assert_equal @result.created_at, @dumped_result.created_at
-          end
+            should "have the same created_at" do
+              assert_equal @result.created_at, @dumped_result.created_at
+            end
           
-          should "have the same command_name" do
-            assert_equal @result.command_name, @dumped_result.command_name
-          end
+            should "have the same command_name" do
+              assert_equal @result.command_name, @dumped_result.command_name
+            end
           
-          should "have the same original_result" do
-            assert_equal @result.original_result, @dumped_result.original_result
+            should "have the same original_result" do
+              assert_equal @result.original_result, @dumped_result.original_result
+            end
           end
         end
       end
-    end
     
-    context "with some filters set up" do
-      setup do
-        SimpleCov.add_filter 'sample.rb'
-      end
+      context "with some filters set up" do
+        setup do
+          SimpleCov.add_filter 'sample.rb'
+        end
       
-      should "have 2 files in a new simple cov result" do
-        assert_equal 2, SimpleCov::Result.new(@original_result).source_files.length
-      end
+        should "have 2 files in a new simple cov result" do
+          assert_equal 2, SimpleCov::Result.new(@original_result).source_files.length
+        end
       
-      should "have 80 covered percent" do
-        assert_equal 80, SimpleCov::Result.new(@original_result).covered_percent
+        should "have 80 covered percent" do
+          assert_equal 80, SimpleCov::Result.new(@original_result).covered_percent
+        end
       end
-    end
     
-    context "with groups set up for all files" do
-      setup do
-        SimpleCov.add_group 'Models', 'app/models'
-        SimpleCov.add_group 'Controllers', 'app/controllers'
-        SimpleCov.add_group 'Other' do |src_file|
-          File.basename(src_file.filename) == 'sample.rb'
+      context "with groups set up for all files" do
+        setup do
+          SimpleCov.add_group 'Models', 'app/models'
+          SimpleCov.add_group 'Controllers', 'app/controllers'
+          SimpleCov.add_group 'Other' do |src_file|
+            File.basename(src_file.filename) == 'sample.rb'
+          end
+          @result = SimpleCov::Result.new(@original_result)
         end
-        @result = SimpleCov::Result.new(@original_result)
-      end
       
-      should "have 3 groups" do
-        assert_equal 3, @result.groups.length
-      end
+        should "have 3 groups" do
+          assert_equal 3, @result.groups.length
+        end
       
-      should "have user.rb in 'Models' group" do
-        assert_equal 'user.rb', File.basename(@result.groups['Models'].first.filename)
-      end
+        should "have user.rb in 'Models' group" do
+          assert_equal 'user.rb', File.basename(@result.groups['Models'].first.filename)
+        end
       
-      should "have sample_controller.rb in 'Controllers' group" do
-        assert_equal 'sample_controller.rb', File.basename(@result.groups['Controllers'].first.filename)
-      end
+        should "have sample_controller.rb in 'Controllers' group" do
+          assert_equal 'sample_controller.rb', File.basename(@result.groups['Controllers'].first.filename)
+        end
       
-      context "and simple formatter being used" do
-        setup {SimpleCov.formatter = SimpleCov::Formatter::SimpleFormatter}
+        context "and simple formatter being used" do
+          setup {SimpleCov.formatter = SimpleCov::Formatter::SimpleFormatter}
         
-        should "return a formatted string with result.format!" do
-          assert_equal String, @result.format!.class
+          should "return a formatted string with result.format!" do
+            assert_equal String, @result.format!.class
+          end
         end
       end
-    end
   
-    context "with groups set up that do not match all files" do
-      setup do
-        SimpleCov.configure do
-          add_group 'Models', 'app/models'
-          add_group 'Controllers', 'app/controllers'
+      context "with groups set up that do not match all files" do
+        setup do
+          SimpleCov.configure do
+            add_group 'Models', 'app/models'
+            add_group 'Controllers', 'app/controllers'
+          end
+          @result = SimpleCov::Result.new(@original_result)
         end
-        @result = SimpleCov::Result.new(@original_result)
-      end
       
-      should "have 3 groups" do
-        assert_equal 3, @result.groups.length
-      end
+        should "have 3 groups" do
+          assert_equal 3, @result.groups.length
+        end
       
-      should "have 1 item per group" do
-        @result.groups.each do |name, files|
-          assert_equal 1, files.length, "Group #{name} should have 1 file"
+        should "have 1 item per group" do
+          @result.groups.each do |name, files|
+            assert_equal 1, files.length, "Group #{name} should have 1 file"
+          end
         end
-      end
 
-      should "have sample.rb in 'Ungrouped' group" do
-        assert_equal 'sample.rb', File.basename(@result.groups['Ungrouped'].first.filename)
+        should "have sample.rb in 'Ungrouped' group" do
+          assert_equal 'sample.rb', File.basename(@result.groups['Ungrouped'].first.filename)
+        end
       end
     end
   end