mocha 0.10.5 → 0.11.0

data/test/acceptance/stubba_test_result_test.rb

@@ -3,17 +3,17 @@ require 'mocha'
 require 'execution_point'
 
 class StubbaTestResultTest < Test::Unit::TestCase
-  
+
   include AcceptanceTest
-  
+
   def setup
     setup_acceptance_test
   end
-  
+
   def teardown
     teardown_acceptance_test
   end
-  
+
   def test_should_include_expectation_verification_in_assertion_count
     test_result = run_as_test do
       object = Class.new { def message; end }.new
@@ -22,14 +22,14 @@ class StubbaTestResultTest < Test::Unit::TestCase
     end
     assert_equal 1, test_result.assertion_count
   end
-  
+
   def test_should_include_assertions_in_assertion_count
     test_result = run_as_test do
       assert true
     end
     assert_equal 1, test_result.assertion_count
   end
-  
+
   def test_should_not_include_stubbing_expectation_verification_in_assertion_count
     test_result = run_as_test do
       object = Class.new { def message; end }.new
@@ -38,7 +38,7 @@ class StubbaTestResultTest < Test::Unit::TestCase
     end
     assert_equal 0, test_result.assertion_count
   end
-  
+
   def test_should_include_expectation_verification_failure_in_failure_count
     test_result = run_as_test do
       object = Class.new { def message; end }.new
@@ -46,14 +46,14 @@ class StubbaTestResultTest < Test::Unit::TestCase
     end
     assert_equal 1, test_result.failure_count
   end
-  
+
   def test_should_include_assertion_failure_in_failure_count
     test_result = run_as_test do
       flunk
     end
     assert_equal 1, test_result.failure_count
   end
-  
+
   def test_should_display_backtrace_indicating_line_number_where_failing_assertion_was_called
     execution_point = nil
     test_result = run_as_test do
@@ -62,5 +62,5 @@ class StubbaTestResultTest < Test::Unit::TestCase
     assert_equal 1, test_result.failure_count
     assert_equal execution_point, ExecutionPoint.new(test_result.failures[0].location)
   end
-  
-end
+
+end

data/test/acceptance/stubbing_error_backtrace_test.rb

@@ -3,17 +3,17 @@ require 'mocha'
 require 'execution_point'
 
 class StubbingErrorBacktraceTest < Test::Unit::TestCase
-  
+
   include AcceptanceTest
-  
+
   def setup
     setup_acceptance_test
   end
-  
+
   def teardown
     teardown_acceptance_test
   end
-  
+
   def test_should_display_backtrace_indicating_line_number_where_attempt_to_stub_non_existent_method_was_made
     execution_point = nil
     object = Object.new
@@ -24,7 +24,7 @@ class StubbingErrorBacktraceTest < Test::Unit::TestCase
     assert_equal 1, test_result.error_count
     assert_equal execution_point, ExecutionPoint.new(test_result.errors[0].exception.backtrace)
   end
-  
+
   def test_should_display_backtrace_indicating_line_number_where_attempt_to_stub_non_public_method_was_made
     execution_point = nil
     object = Class.new do
@@ -60,5 +60,5 @@ class StubbingErrorBacktraceTest < Test::Unit::TestCase
     assert_equal 1, test_result.error_count
     assert_equal execution_point, ExecutionPoint.new(test_result.errors[0].exception.backtrace)
   end
-  
+
 end

data/test/acceptance/stubbing_frozen_object_test.rb

@@ -0,0 +1,88 @@
+require File.expand_path('../acceptance_test_helper', __FILE__)
+require 'mocha'
+require 'execution_point'
+
+class StubbingFrozenObjectTest < Test::Unit::TestCase
+
+  include AcceptanceTest
+
+  def setup
+    setup_acceptance_test
+  end
+
+  def teardown
+    teardown_acceptance_test
+  end
+
+  def test_should_fail_fast_if_attempting_to_stub_method_on_frozen_object
+    object = Object.new
+    object.freeze
+    execution_point = nil
+    test_result = run_as_test do
+      execution_point = ExecutionPoint.current; object.stubs(:stubbed_method)
+    end
+    assert_failed(test_result)
+    assert_equal 1, test_result.error_count
+    assert_equal execution_point, ExecutionPoint.new(test_result.errors[0].exception.backtrace)
+  end
+
+  def test_should_fail_fast_if_attempting_to_expect_method_on_frozen_object
+    object = Object.new
+    object.freeze
+    execution_point = nil
+    test_result = run_as_test do
+      execution_point = ExecutionPoint.current; object.expects(:stubbed_method)
+    end
+    assert_failed(test_result)
+    assert_equal 1, test_result.error_count
+    assert_equal execution_point, ExecutionPoint.new(test_result.errors[0].exception.backtrace)
+  end
+
+  def test_should_fail_fast_if_attempting_to_stub_method_on_frozen_class
+    klass = Class.new
+    klass.freeze
+    execution_point = nil
+    test_result = run_as_test do
+      execution_point = ExecutionPoint.current; klass.stubs(:stubbed_method)
+    end
+    assert_failed(test_result)
+    assert_equal 1, test_result.error_count
+    assert_equal execution_point, ExecutionPoint.new(test_result.errors[0].exception.backtrace)
+  end
+
+  def test_should_fail_fast_if_attempting_to_expect_method_on_frozen_class
+    klass = Class.new
+    klass.freeze
+    execution_point = nil
+    test_result = run_as_test do
+      execution_point = ExecutionPoint.current; klass.expects(:stubbed_method)
+    end
+    assert_failed(test_result)
+    assert_equal 1, test_result.error_count
+    assert_equal execution_point, ExecutionPoint.new(test_result.errors[0].exception.backtrace)
+  end
+
+  def test_should_fail_fast_if_attempting_to_stub_method_on_any_instance_of_frozen_class
+    klass = Class.new
+    klass.freeze
+    execution_point = nil
+    test_result = run_as_test do
+      execution_point = ExecutionPoint.current; klass.any_instance.stubs(:stubbed_method)
+    end
+    assert_failed(test_result)
+    assert_equal 1, test_result.error_count
+    assert_equal execution_point, ExecutionPoint.new(test_result.errors[0].exception.backtrace)
+  end
+
+  def test_should_fail_fast_if_attempting_to_expect_method_on_any_instance_of_frozen_class
+    klass = Class.new
+    klass.freeze
+    execution_point = nil
+    test_result = run_as_test do
+      execution_point = ExecutionPoint.current; klass.any_instance.expects(:stubbed_method)
+    end
+    assert_failed(test_result)
+    assert_equal 1, test_result.error_count
+    assert_equal execution_point, ExecutionPoint.new(test_result.errors[0].exception.backtrace)
+  end
+end

data/test/acceptance/stubbing_method_unnecessarily_test.rb

@@ -2,17 +2,17 @@ require File.expand_path('../acceptance_test_helper', __FILE__)
 require 'mocha'
 
 class StubbingMethodUnnecessarilyTest < Test::Unit::TestCase
-  
+
   include AcceptanceTest
-  
+
   def setup
     setup_acceptance_test
   end
-  
+
   def teardown
     teardown_acceptance_test
   end
-  
+
   def test_should_allow_stubbing_method_unnecessarily
     Mocha::Configuration.allow(:stubbing_method_unnecessarily)
     test_result = run_as_test do
@@ -22,7 +22,7 @@ class StubbingMethodUnnecessarilyTest < Test::Unit::TestCase
     assert_passed(test_result)
     assert !@logger.warnings.include?('stubbing method unnecessarily: #<Mock:mock>.public_method(any_parameters)')
   end
-  
+
   def test_should_warn_when_stubbing_method_unnecessarily
     Mocha::Configuration.warn_when(:stubbing_method_unnecessarily)
     test_result = run_as_test do
@@ -32,7 +32,7 @@ class StubbingMethodUnnecessarilyTest < Test::Unit::TestCase
     assert_passed(test_result)
     assert @logger.warnings.include?('stubbing method unnecessarily: #<Mock:mock>.public_method(any_parameters)')
   end
-  
+
   def test_should_prevent_stubbing_method_unnecessarily
     Mocha::Configuration.prevent(:stubbing_method_unnecessarily)
     test_result = run_as_test do
@@ -42,7 +42,7 @@ class StubbingMethodUnnecessarilyTest < Test::Unit::TestCase
     assert_failed(test_result)
     assert test_result.error_messages.include?('Mocha::StubbingError: stubbing method unnecessarily: #<Mock:mock>.public_method(any_parameters)')
   end
-  
+
   def test_should_default_to_allow_stubbing_method_unnecessarily
     test_result = run_as_test do
       mock = mock('mock')
@@ -51,7 +51,7 @@ class StubbingMethodUnnecessarilyTest < Test::Unit::TestCase
     assert_passed(test_result)
     assert !@logger.warnings.include?('stubbing method unnecessarily: #<Mock:mock>.public_method(any_parameters)')
   end
-  
+
   def test_should_allow_stubbing_method_when_stubbed_method_is_invoked
     Mocha::Configuration.prevent(:stubbing_method_unnecessarily)
     test_result = run_as_test do
@@ -62,4 +62,4 @@ class StubbingMethodUnnecessarilyTest < Test::Unit::TestCase
     assert_passed(test_result)
   end
 
-end
+end

data/test/acceptance/stubbing_nil_test.rb

@@ -0,0 +1,59 @@
+require File.expand_path('../acceptance_test_helper', __FILE__)
+require 'mocha'
+
+class StubbingNilTest < Test::Unit::TestCase
+
+  include AcceptanceTest
+
+  def setup
+    setup_acceptance_test
+  end
+
+  def teardown
+    teardown_acceptance_test
+  end
+
+  def test_should_allow_stubbing_method_on_nil
+    Mocha::Configuration.allow(:stubbing_method_on_nil)
+    test_result = run_as_test do
+      nil.stubs(:stubbed_method)
+    end
+    assert_passed(test_result)
+    assert !@logger.warnings.include?("stubbing method on nil: nil.stubbed_method")
+  end
+
+  def test_should_warn_on_stubbing_method_on_nil
+    Mocha::Configuration.warn_when(:stubbing_method_on_nil)
+    test_result = run_as_test do
+      nil.stubs(:stubbed_method)
+    end
+    assert_passed(test_result)
+    assert @logger.warnings.include?("stubbing method on nil: nil.stubbed_method")
+  end
+
+  def test_should_prevent_stubbing_method_on_nil
+    Mocha::Configuration.prevent(:stubbing_method_on_nil)
+    test_result = run_as_test do
+      nil.stubs(:stubbed_method)
+    end
+    assert_failed(test_result)
+    assert test_result.error_messages.include?("Mocha::StubbingError: stubbing method on nil: nil.stubbed_method")
+  end
+
+  def test_should_default_to_prevent_stubbing_method_on_non_mock_object
+    test_result = run_as_test do
+      nil.stubs(:stubbed_method)
+    end
+    assert_failed(test_result)
+    assert test_result.error_messages.include?("Mocha::StubbingError: stubbing method on nil: nil.stubbed_method")
+  end
+
+  def test_should_allow_stubbing_method_on_non_nil_object
+    Mocha::Configuration.prevent(:stubbing_method_on_nil)
+    object = Object.new
+    test_result = run_as_test do
+      object.stubs(:stubbed_method)
+    end
+    assert_passed(test_result)
+  end
+end

data/test/acceptance/stubbing_non_existent_any_instance_method_test.rb

@@ -2,17 +2,17 @@ require File.expand_path('../acceptance_test_helper', __FILE__)
 require 'mocha'
 
 class StubbingNonExistentAnyInstanceMethodTest < Test::Unit::TestCase
-  
+
   include AcceptanceTest
-  
+
   def setup
     setup_acceptance_test
   end
-  
+
   def teardown
     teardown_acceptance_test
   end
-  
+
   def test_should_allow_stubbing_non_existent_any_instance_method
     Mocha::Configuration.allow(:stubbing_non_existent_method)
     klass = Class.new
@@ -22,7 +22,7 @@ class StubbingNonExistentAnyInstanceMethodTest < Test::Unit::TestCase
     assert !@logger.warnings.include?("stubbing non-existent method: #{klass.any_instance.mocha_inspect}.non_existent_method")
     assert_passed(test_result)
   end
-  
+
   def test_should_warn_when_stubbing_non_existent_any_instance_method
     Mocha::Configuration.warn_when(:stubbing_non_existent_method)
     klass = Class.new
@@ -32,7 +32,7 @@ class StubbingNonExistentAnyInstanceMethodTest < Test::Unit::TestCase
     assert_passed(test_result)
     assert @logger.warnings.include?("stubbing non-existent method: #{klass.any_instance.mocha_inspect}.non_existent_method")
   end
-  
+
   def test_should_prevent_stubbing_non_existent_any_instance_method
     Mocha::Configuration.prevent(:stubbing_non_existent_method)
     klass = Class.new
@@ -42,7 +42,7 @@ class StubbingNonExistentAnyInstanceMethodTest < Test::Unit::TestCase
     assert_failed(test_result)
     assert test_result.error_messages.include?("Mocha::StubbingError: stubbing non-existent method: #{klass.any_instance.mocha_inspect}.non_existent_method")
   end
-  
+
   def test_should_default_to_allow_stubbing_non_existent_any_instance_method
     klass = Class.new
     test_result = run_as_test do
@@ -51,7 +51,7 @@ class StubbingNonExistentAnyInstanceMethodTest < Test::Unit::TestCase
     assert !@logger.warnings.include?("stubbing non-existent method: #{klass.any_instance.mocha_inspect}.non_existent_method")
     assert_passed(test_result)
   end
-  
+
   def test_should_allow_stubbing_existing_public_any_instance_method
     Mocha::Configuration.prevent(:stubbing_non_existent_method)
     klass = Class.new do
@@ -75,7 +75,7 @@ class StubbingNonExistentAnyInstanceMethodTest < Test::Unit::TestCase
     end
     assert_passed(test_result)
   end
-  
+
   def test_should_allow_stubbing_existing_private_any_instance_method
     Mocha::Configuration.prevent(:stubbing_non_existent_method)
     klass = Class.new do
@@ -87,7 +87,7 @@ class StubbingNonExistentAnyInstanceMethodTest < Test::Unit::TestCase
     end
     assert_passed(test_result)
   end
-  
+
   def test_should_allow_stubbing_existing_public_any_instance_superclass_method
     Mocha::Configuration.prevent(:stubbing_non_existent_method)
     superklass = Class.new do
@@ -100,7 +100,7 @@ class StubbingNonExistentAnyInstanceMethodTest < Test::Unit::TestCase
     end
     assert_passed(test_result)
   end
-  
+
   def test_should_allow_stubbing_existing_protected_any_instance_superclass_method
     Mocha::Configuration.prevent(:stubbing_non_existent_method)
     superklass = Class.new do
@@ -113,7 +113,7 @@ class StubbingNonExistentAnyInstanceMethodTest < Test::Unit::TestCase
     end
     assert_passed(test_result)
   end
-  
+
   def test_should_allow_stubbing_existing_private_any_instance_superclass_method
     Mocha::Configuration.prevent(:stubbing_non_existent_method)
     superklass = Class.new do
@@ -126,5 +126,5 @@ class StubbingNonExistentAnyInstanceMethodTest < Test::Unit::TestCase
     end
     assert_passed(test_result)
   end
-  
+
 end

data/test/acceptance/stubbing_non_existent_class_method_test.rb

@@ -2,17 +2,17 @@ require File.expand_path('../acceptance_test_helper', __FILE__)
 require 'mocha'
 
 class StubbingNonExistentClassMethodTest < Test::Unit::TestCase
-  
+
   include AcceptanceTest
-  
+
   def setup
     setup_acceptance_test
   end
-  
+
   def teardown
     teardown_acceptance_test
   end
-  
+
   def test_should_allow_stubbing_non_existent_class_method
     Mocha::Configuration.allow(:stubbing_non_existent_method)
     klass = Class.new
@@ -22,7 +22,7 @@ class StubbingNonExistentClassMethodTest < Test::Unit::TestCase
     assert !@logger.warnings.include?("stubbing non-existent method: #{klass.mocha_inspect}.non_existent_method")
     assert_passed(test_result)
   end
-  
+
   def test_should_warn_when_stubbing_non_existent_class_method
     Mocha::Configuration.warn_when(:stubbing_non_existent_method)
     klass = Class.new
@@ -32,7 +32,7 @@ class StubbingNonExistentClassMethodTest < Test::Unit::TestCase
     assert_passed(test_result)
     assert @logger.warnings.include?("stubbing non-existent method: #{klass.mocha_inspect}.non_existent_method")
   end
-  
+
   def test_should_prevent_stubbing_non_existent_class_method
     Mocha::Configuration.prevent(:stubbing_non_existent_method)
     klass = Class.new
@@ -42,7 +42,7 @@ class StubbingNonExistentClassMethodTest < Test::Unit::TestCase
     assert_failed(test_result)
     assert test_result.error_messages.include?("Mocha::StubbingError: stubbing non-existent method: #{klass.mocha_inspect}.non_existent_method")
   end
-  
+
   def test_should_default_to_allow_stubbing_non_existent_class_method
     klass = Class.new
     test_result = run_as_test do
@@ -51,7 +51,7 @@ class StubbingNonExistentClassMethodTest < Test::Unit::TestCase
     assert !@logger.warnings.include?("stubbing non-existent method: #{klass.mocha_inspect}.non_existent_method")
     assert_passed(test_result)
   end
-  
+
   def test_should_allow_stubbing_existing_public_class_method
     Mocha::Configuration.prevent(:stubbing_non_existent_method)
     klass = Class.new do
@@ -65,7 +65,7 @@ class StubbingNonExistentClassMethodTest < Test::Unit::TestCase
     end
     assert_passed(test_result)
   end
-  
+
   def test_should_allow_stubbing_method_to_which_class_responds
     Mocha::Configuration.prevent(:stubbing_non_existent_method)
     klass = Class.new do
@@ -80,7 +80,7 @@ class StubbingNonExistentClassMethodTest < Test::Unit::TestCase
     end
     assert_passed(test_result)
   end
- 
+
   def test_should_allow_stubbing_existing_protected_class_method
     Mocha::Configuration.prevent(:stubbing_non_existent_method)
     klass = Class.new do
@@ -94,7 +94,7 @@ class StubbingNonExistentClassMethodTest < Test::Unit::TestCase
     end
     assert_passed(test_result)
   end
-  
+
   def test_should_allow_stubbing_existing_private_class_method
     Mocha::Configuration.prevent(:stubbing_non_existent_method)
     klass = Class.new do
@@ -108,7 +108,7 @@ class StubbingNonExistentClassMethodTest < Test::Unit::TestCase
     end
     assert_passed(test_result)
   end
-  
+
   def test_should_allow_stubbing_existing_public_superclass_method
     Mocha::Configuration.prevent(:stubbing_non_existent_method)
     superklass = Class.new do
@@ -123,7 +123,7 @@ class StubbingNonExistentClassMethodTest < Test::Unit::TestCase
     end
     assert_passed(test_result)
   end
-  
+
   def test_should_allow_stubbing_existing_protected_superclass_method
     Mocha::Configuration.prevent(:stubbing_non_existent_method)
     superklass = Class.new do
@@ -138,7 +138,7 @@ class StubbingNonExistentClassMethodTest < Test::Unit::TestCase
     end
     assert_passed(test_result)
   end
-  
+
   def test_should_allow_stubbing_existing_private_superclass_method
     Mocha::Configuration.prevent(:stubbing_non_existent_method)
     superklass = Class.new do
@@ -153,5 +153,5 @@ class StubbingNonExistentClassMethodTest < Test::Unit::TestCase
     end
     assert_passed(test_result)
   end
-  
+
 end