diff options
author | Xavier Noria <fxn@hashref.com> | 2010-08-17 03:29:57 +0200 |
---|---|---|
committer | Xavier Noria <fxn@hashref.com> | 2010-08-17 03:32:11 +0200 |
commit | fb6b80562041e8d2378cad1b51f8c234fe76fd5e (patch) | |
tree | e407af937ad119b1a5a6c977b13319f24994cde2 /activesupport/lib | |
parent | d14e2987b583a3dcb6836d310f77ce367c7396f8 (diff) | |
download | rails-fb6b80562041e8d2378cad1b51f8c234fe76fd5e.tar.gz rails-fb6b80562041e8d2378cad1b51f8c234fe76fd5e.tar.bz2 rails-fb6b80562041e8d2378cad1b51f8c234fe76fd5e.zip |
code gardening: we have assert_(nil|blank|present), more concise, with better default failure messages - let's use them
Diffstat (limited to 'activesupport/lib')
-rw-r--r-- | activesupport/lib/active_support/testing/assertions.rb | 10 |
1 files changed, 6 insertions, 4 deletions
diff --git a/activesupport/lib/active_support/testing/assertions.rb b/activesupport/lib/active_support/testing/assertions.rb index 8c68f42781..a4e0361a32 100644 --- a/activesupport/lib/active_support/testing/assertions.rb +++ b/activesupport/lib/active_support/testing/assertions.rb @@ -67,15 +67,17 @@ module ActiveSupport # Test if an expression is blank. Passes if object.blank? is true. # # assert_blank [] # => true - def assert_blank(object) - assert object.blank?, "#{object.inspect} is not blank" + def assert_blank(object, message=nil) + message ||= "#{object.inspect} is not blank" + assert object.blank?, message end # Test if an expression is not blank. Passes if object.present? is true. # # assert_present {:data => 'x' } # => true - def assert_present(object) - assert object.present?, "#{object.inspect} is blank" + def assert_present(object, message=nil) + message ||= "#{object.inspect} is blank" + assert object.present?, message end end end |