only try to load Test::Unit if Test::Unit is defined

History.txt

@@ -11,9 +11,10 @@
   * should throw_symbol accepts an optional argument: should throw_symbol(:sym, arg)
   * fixed --line for jruby (Zach Moazeni)
 
-* 1 bug fix
+* 2 bug fixex
 
   * fixed bug where {:a => 1, :b => 2}.should include(:a, :b) failed (Yossef Mendelssohn)
+  * only try to load Test::Unit if Test::Unit is defined
 
 === Version 1.1.11 / 2008-10-24
 

lib/spec.rb

@@ -7,7 +7,11 @@
 require 'spec/version'
 require 'spec/dsl'
 
-if Object.const_defined?(:Test)
+def test_unit_defined?
+  Object.const_defined?(:Test) && Test.const_defined?(:Unit)
+end
+
+if test_unit_defined?
   require 'spec/interop/test'
 end
 
@@ -23,7 +27,7 @@ def run
     end
 
     def exit?
-      !Object.const_defined?(:Test) || Test::Unit.run?
+      !test_unit_defined? || Test::Unit.run?
     end
 
     def spec_command?

spec/spec/interop/test/unit/spec_spec.rb

@@ -2,11 +2,7 @@
 
 describe "ExampleGroup with test/unit/interop" do
   include TestUnitSpecHelper
-
-  def resources
-    File.dirname(__FILE__) + "/resources"
-  end
-
+
   describe "with passing examples" do
     it "should output 0 failures" do
       output = ruby("#{resources}/spec_that_passes.rb")

spec/spec/interop/test/unit/test_unit_spec_helper.rb

@@ -3,6 +3,10 @@
 
 module TestUnitSpecHelper
   include RubyForker
+
+  def resources
+    File.dirname(__FILE__) + "/resources"
+  end
 
   def run_script(file_name)
     output = ruby(file_name)

stories/interop/test_but_not_test_unit

@@ -0,0 +1,14 @@
+Story: Test is defined, but not Test::Unit
+
+  As an RSpec user who has my own library named Test (but not Test::Unit)
+  I want to run examples without getting Test::Unit NameErrors
+
+  Scenario: Run with ruby
+    Given the file test/spec_including_test_but_not_unit.rb
+    When I run it with the ruby interpreter
+    Then the stderr should not match "Test::Unit"
+
+  Scenario: Run with spec
+    Given the file test/spec_including_test_but_not_unit.rb
+    When I run it with the spec script
+    Then the stderr should not match "Test::Unit"

stories/resources/test/spec_including_test_but_not_unit.rb

@@ -0,0 +1,11 @@
+$:.push File.join(File.dirname(__FILE__), *%w[.. .. .. lib])
+require 'spec'
+
+module Test
+end
+
+describe "description" do
+  it "should description" do
+    1.should == 1
+  end
+end