From 35f38b71dfeacbb4f92a2b1bf4d151c2061700d5 Mon Sep 17 00:00:00 2001 From: zires Date: Tue, 26 Feb 2013 13:09:13 +0800 Subject: Make ActionDispatch::Journey::Path::Pattern#new raise more meaningful exception message. --- actionpack/CHANGELOG.md | 2 ++ actionpack/lib/action_dispatch/journey/path/pattern.rb | 2 +- actionpack/test/journey/path/pattern_test.rb | 4 ++++ 3 files changed, 7 insertions(+), 1 deletion(-) (limited to 'actionpack') diff --git a/actionpack/CHANGELOG.md b/actionpack/CHANGELOG.md index 5197d944b1..75a503b12e 100644 --- a/actionpack/CHANGELOG.md +++ b/actionpack/CHANGELOG.md @@ -1,5 +1,7 @@ ## Rails 4.0.0.beta1 (February 25, 2013) ## +* Make ActionDispatch::Journey::Path::Pattern#new raise more meaningful exception message. *Thierry Zires* + * Fix `respond_to` not using formats that have no block if all is present. *Michael Grosser* * New applications use an encrypted session store by default. diff --git a/actionpack/lib/action_dispatch/journey/path/pattern.rb b/actionpack/lib/action_dispatch/journey/path/pattern.rb index 4a571ec546..d37aa1fbe5 100644 --- a/actionpack/lib/action_dispatch/journey/path/pattern.rb +++ b/actionpack/lib/action_dispatch/journey/path/pattern.rb @@ -20,7 +20,7 @@ module ActionDispatch @separators = strexp.separators.join @anchored = strexp.anchor else - raise "wtf bro: #{strexp}" + raise ArgumentError, "Bad expression: #{strexp}" end @names = nil diff --git a/actionpack/test/journey/path/pattern_test.rb b/actionpack/test/journey/path/pattern_test.rb index 2b7227cd0d..ce02104181 100644 --- a/actionpack/test/journey/path/pattern_test.rb +++ b/actionpack/test/journey/path/pattern_test.rb @@ -75,6 +75,10 @@ module ActionDispatch end end + def test_to_raise_exception_with_bad_expression + assert_raise(ArgumentError, "Bad expression: []") { Pattern.new [] } + end + def test_to_regexp_with_extended_group strexp = Router::Strexp.new( '/page/:name', -- cgit v1.2.3