diff options
| author | Michael Grosser <michael@grosser.it> | 2016-09-16 09:44:05 -0700 |
|---|---|---|
| committer | Michael Grosser <michael@grosser.it> | 2016-09-16 12:03:37 -0700 |
| commit | a9aed2ac94d2e0d1a233a3c193985ae78d7e79e9 (patch) | |
| tree | d550056dbf4a70b5c7562f1a715045c5a3968b0c /activemodel/test/cases/errors_test.rb | |
| parent | f62451a50b2c9119adce7acc53ce3dfffc4d41d5 (diff) | |
| download | rails-a9aed2ac94d2e0d1a233a3c193985ae78d7e79e9.tar.gz rails-a9aed2ac94d2e0d1a233a3c193985ae78d7e79e9.tar.bz2 rails-a9aed2ac94d2e0d1a233a3c193985ae78d7e79e9.zip | |
improve error message when include assertions fail
assert [1, 3].includes?(2) fails with unhelpful "Asserting failed" message
assert_includes [1, 3], 2 fails with "Expected [1, 3] to include 2" which makes it easier to debug and more obvious what went wrong
Diffstat (limited to 'activemodel/test/cases/errors_test.rb')
| -rw-r--r-- | activemodel/test/cases/errors_test.rb | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/activemodel/test/cases/errors_test.rb b/activemodel/test/cases/errors_test.rb index da99a0eca0..3288b5543c 100644 --- a/activemodel/test/cases/errors_test.rb +++ b/activemodel/test/cases/errors_test.rb @@ -37,7 +37,7 @@ class ErrorsTest < ActiveModel::TestCase def test_include? errors = ActiveModel::Errors.new(self) errors[:foo] << "omg" - assert errors.include?(:foo), "errors should include :foo" + assert_includes errors, :foo, "errors should include :foo" end def test_dup @@ -125,7 +125,7 @@ class ErrorsTest < ActiveModel::TestCase person.errors[:foo] assert person.errors.empty? assert person.errors.blank? - assert !person.errors.include?(:foo) + assert_not_includes person.errors, :foo end test "include? does not add a key to messages hash" do |