aboutsummaryrefslogtreecommitdiffstats
path: root/actionview/test/actionpack/abstract
diff options
context:
space:
mode:
authorCarlos Antonio da Silva <carlosantoniodasilva@gmail.com>2013-12-19 09:03:39 -0200
committerCarlos Antonio da Silva <carlosantoniodasilva@gmail.com>2013-12-19 09:20:51 -0200
commitc5b76b5362e4ba28ff3a5649156c50dff9a86de7 (patch)
tree2612d781e96ce592ae609146f303ffda9eb5f104 /actionview/test/actionpack/abstract
parentd799b9c1cf888a528d062cd44903cf2e0ce34a2c (diff)
downloadrails-c5b76b5362e4ba28ff3a5649156c50dff9a86de7.tar.gz
rails-c5b76b5362e4ba28ff3a5649156c50dff9a86de7.tar.bz2
rails-c5b76b5362e4ba28ff3a5649156c50dff9a86de7.zip
Prefer assert_raise instead of flunk + rescue to test for exceptions
Change most tests to make use of assert_raise returning the raised exception rather than relying on a combination of flunk + rescue to check for exception types/messages.
Diffstat (limited to 'actionview/test/actionpack/abstract')
-rw-r--r--actionview/test/actionpack/abstract/helper_test.rb6
1 files changed, 3 insertions, 3 deletions
diff --git a/actionview/test/actionpack/abstract/helper_test.rb b/actionview/test/actionpack/abstract/helper_test.rb
index 89c4567715..7d346e917d 100644
--- a/actionview/test/actionpack/abstract/helper_test.rb
+++ b/actionview/test/actionpack/abstract/helper_test.rb
@@ -78,9 +78,9 @@ module AbstractController
end
def test_declare_missing_helper
- AbstractHelpers.helper :missing
- flunk "should have raised an exception"
- rescue LoadError => e
+ e = assert_raise AbstractController::Helpers::MissingHelperError do
+ AbstractHelpers.helper :missing
+ end
assert_equal "helpers/missing_helper.rb", e.path
end