summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
-rw-r--r--ChangeLog5
-rw-r--r--lib/minitest/unit.rb6
-rw-r--r--test/minitest/test_mini_spec.rb4
-rw-r--r--test/minitest/test_mini_test.rb34
4 files changed, 24 insertions, 25 deletions
diff --git a/ChangeLog b/ChangeLog
index d6237c738a..4799ea1159 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,8 @@
+Tue Jan 20 17:59:52 2009 Yuki Sonoda (Yugui) <yugui@yugui.jp>
+
+ * Reverts r21598 and r21599 because the original behavior is
+ intentional.
+
Tue Jan 20 09:26:05 2009 NAKAMURA Usaku <usa@ruby-lang.org>
* ext/nkf/nkf-utf8/nkf.c (nkf_buf_push): maybe a bug.
diff --git a/lib/minitest/unit.rb b/lib/minitest/unit.rb
index 9a7239bac8..0f71126b0b 100644
--- a/lib/minitest/unit.rb
+++ b/lib/minitest/unit.rb
@@ -76,7 +76,6 @@ module MiniTest
def assert_empty obj, msg = nil
msg = message(msg) { "Expected #{obj.inspect} to be empty" }
assert_respond_to obj, :empty?
- self._assertions -= 1
assert obj.empty?, msg
end
@@ -98,7 +97,6 @@ module MiniTest
def assert_includes collection, obj, msg = nil
msg = message(msg) { "Expected #{mu_pp(collection)} to include #{mu_pp(obj)}" }
assert_respond_to collection, :include?
- self._assertions -= 1
assert collection.include?(obj), msg
end
@@ -120,7 +118,6 @@ module MiniTest
def assert_match exp, act, msg = nil
msg = message(msg) { "Expected #{mu_pp(exp)} to match #{mu_pp(act)}" }
assert_respond_to act, :"=~"
- self._assertions -= 1
exp = /#{Regexp.escape(exp)}/ if String === exp && String === act
assert exp =~ act, msg
end
@@ -244,7 +241,6 @@ module MiniTest
def refute_empty obj, msg = nil
msg = message(msg) { "Expected #{obj.inspect} to not be empty" }
assert_respond_to obj, :empty?
- self._assertions -= 1
refute obj.empty?, msg
end
@@ -266,7 +262,6 @@ module MiniTest
def refute_includes collection, obj, msg = nil
msg = message(msg) { "Expected #{mu_pp(collection)} to not include #{mu_pp(obj)}" }
assert_respond_to collection, :include?
- self._assertions -= 1
refute collection.include?(obj), msg
end
@@ -287,7 +282,6 @@ module MiniTest
def refute_match exp, act, msg = nil
msg = message(msg) { "Expected #{mu_pp(exp)} to not match #{mu_pp(act)}" }
assert_respond_to act, :"=~"
- self._assertions -= 1
exp = /#{Regexp.escape(exp)}/ if String === exp && String === act
refute exp =~ act, msg
end
diff --git a/test/minitest/test_mini_spec.rb b/test/minitest/test_mini_spec.rb
index 81e3a61928..f7cd60acda 100644
--- a/test/minitest/test_mini_spec.rb
+++ b/test/minitest/test_mini_spec.rb
@@ -73,7 +73,7 @@ describe MiniTest::Spec do
end
it "needs to verify regexp matches" do
- @assertion_count = 4
+ @assertion_count = 6
"blah".must_match(/\w+/).must_equal true
proc { "blah".must_match(/\d+/) }.must_raise MiniTest::Assertion
@@ -140,7 +140,7 @@ describe MiniTest::Spec do
end
it "needs to verify mismatch" do
- @assertion_count = 4
+ @assertion_count = 6
"blah".wont_match(/\d+/).must_equal false
proc { "blah".wont_match(/\w+/) }.must_raise MiniTest::Assertion
end
diff --git a/test/minitest/test_mini_test.rb b/test/minitest/test_mini_test.rb
index e064ec2544..046ae66d88 100644
--- a/test/minitest/test_mini_test.rb
+++ b/test/minitest/test_mini_test.rb
@@ -440,13 +440,13 @@ class TestMiniTestTestCase < MiniTest::Unit::TestCase
end
def test_assert_empty
- @assertion_count = 1
+ @assertion_count = 2
@tc.assert_empty []
end
def test_assert_empty_triggered
- @assertion_count = 1
+ @assertion_count = 2
util_assert_triggered "Expected [1] to be empty." do
@tc.assert_empty [1]
@@ -494,13 +494,13 @@ class TestMiniTestTestCase < MiniTest::Unit::TestCase
end
def test_assert_includes
- @assertion_count = 1
+ @assertion_count = 2
@tc.assert_includes [true], true
end
def test_assert_includes_triggered
- @assertion_count = 2
+ @assertion_count = 3
e = @tc.assert_raises MiniTest::Assertion do
@tc.assert_includes [true], false
@@ -531,12 +531,12 @@ class TestMiniTestTestCase < MiniTest::Unit::TestCase
end
def test_assert_match
- @assertion_count = 1
+ @assertion_count = 2
@tc.assert_match(/\w+/, "blah blah blah")
end
def test_assert_match_object
- @assertion_count = 1
+ @assertion_count = 2
pattern = Object.new
def pattern.=~(other) true end
@@ -545,7 +545,7 @@ class TestMiniTestTestCase < MiniTest::Unit::TestCase
end
def test_assert_match_object_triggered
- @assertion_count = 1
+ @assertion_count = 2
pattern = Object.new
def pattern.=~(other) false end
@@ -557,7 +557,7 @@ class TestMiniTestTestCase < MiniTest::Unit::TestCase
end
def test_assert_match_triggered
- @assertion_count = 1
+ @assertion_count = 2
util_assert_triggered 'Expected /\d+/ to match "blah blah blah".' do
@tc.assert_match(/\d+/, "blah blah blah")
end
@@ -801,13 +801,13 @@ FILE:LINE:in `test_assert_raises_triggered_subclass'
end
def test_refute_empty
- @assertion_count = 1
+ @assertion_count = 2
@tc.refute_empty [1]
end
def test_refute_empty_triggered
- @assertion_count = 1
+ @assertion_count = 2
util_assert_triggered "Expected [] to not be empty." do
@tc.refute_empty []
@@ -846,13 +846,13 @@ FILE:LINE:in `test_assert_raises_triggered_subclass'
end
def test_refute_includes
- @assertion_count = 1
+ @assertion_count = 2
@tc.refute_includes [true], false
end
def test_refute_includes_triggered
- @assertion_count = 2
+ @assertion_count = 3
e = @tc.assert_raises MiniTest::Assertion do
@tc.refute_includes [true], true
@@ -883,17 +883,17 @@ FILE:LINE:in `test_assert_raises_triggered_subclass'
end
def test_refute_match
- @assertion_count = 1
+ @assertion_count = 2
@tc.refute_match(/\d+/, "blah blah blah")
end
def test_refute_match_object
- @assertion_count = 1
+ @assertion_count = 2
@tc.refute_match Object.new, 5 # default #=~ returns false
end
def test_assert_object_triggered
- @assertion_count = 1
+ @assertion_count = 2
pattern = Object.new
def pattern.=~(other) false end
@@ -905,7 +905,7 @@ FILE:LINE:in `test_assert_raises_triggered_subclass'
end
def test_refute_match_object_triggered
- @assertion_count = 1
+ @assertion_count = 2
pattern = Object.new
def pattern.=~(other) true end
@@ -917,7 +917,7 @@ FILE:LINE:in `test_assert_raises_triggered_subclass'
end
def test_refute_match_triggered
- @assertion_count = 1
+ @assertion_count = 2
util_assert_triggered 'Expected /\w+/ to not match "blah blah blah".' do
@tc.refute_match(/\w+/, "blah blah blah")
end