diff options
author | yuuji.yaginuma <yuuji.yaginuma@gmail.com> | 2016-08-29 09:05:32 +0900 |
---|---|---|
committer | yuuji.yaginuma <yuuji.yaginuma@gmail.com> | 2016-08-29 09:06:10 +0900 |
commit | 6582c7c54a4be7275ea9df5c787d8e686f74b618 (patch) | |
tree | a3fb5302269cf1ebf5767f4e993eaff2c350c858 /activesupport | |
parent | 9aa6a75eef4f6b4c6f840ec626d31f308e7feba6 (diff) | |
download | rails-6582c7c54a4be7275ea9df5c787d8e686f74b618.tar.gz rails-6582c7c54a4be7275ea9df5c787d8e686f74b618.tar.bz2 rails-6582c7c54a4be7275ea9df5c787d8e686f74b618.zip |
use `inspect` for show `from` value
If `from` is nil, in order to avoid the blank is showed.
Diffstat (limited to 'activesupport')
-rw-r--r-- | activesupport/lib/active_support/testing/assertions.rb | 2 | ||||
-rw-r--r-- | activesupport/test/test_case_test.rb | 9 |
2 files changed, 10 insertions, 1 deletions
diff --git a/activesupport/lib/active_support/testing/assertions.rb b/activesupport/lib/active_support/testing/assertions.rb index 6a6941c72a..8c9ea2c0e8 100644 --- a/activesupport/lib/active_support/testing/assertions.rb +++ b/activesupport/lib/active_support/testing/assertions.rb @@ -147,7 +147,7 @@ module ActiveSupport retval = yield unless from == UNTRACKED - error = "#{expression.inspect} isn't #{from}" + error = "#{expression.inspect} isn't #{from.inspect}" error = "#{message}.\n#{error}" if message assert from === before, error end diff --git a/activesupport/test/test_case_test.rb b/activesupport/test/test_case_test.rb index d3d2ed8d7a..dfbb8aa41d 100644 --- a/activesupport/test/test_case_test.rb +++ b/activesupport/test/test_case_test.rb @@ -138,6 +138,15 @@ class AssertDifferenceTest < ActiveSupport::TestCase end end + def test_assert_changes_with_from_option_with_nil + error = assert_raises Minitest::Assertion do + assert_changes "@object.num", from: nil do + @object.increment + end + end + assert_equal "\"@object.num\" isn't nil", error.message + end + def test_assert_changes_with_to_option assert_changes "@object.num", to: 1 do @object.increment |