aboutsummaryrefslogtreecommitdiffstats
path: root/actionpack/test/journey/router_test.rb
diff options
context:
space:
mode:
authorGuo Xiang Tan <tgx_world@hotmail.com>2014-07-04 12:04:52 -0700
committerGuo Xiang Tan <tgx_world@hotmail.com>2014-08-21 16:35:39 +0800
commit92120426317083ae90cb64cff07a7dd3a8acdc65 (patch)
tree3273b0fa45fb07c43cdefb3cd08c840b0edd241d /actionpack/test/journey/router_test.rb
parentd3eb92d95a694905a80668dcde1fb49e2d08f388 (diff)
downloadrails-92120426317083ae90cb64cff07a7dd3a8acdc65.tar.gz
rails-92120426317083ae90cb64cff07a7dd3a8acdc65.tar.bz2
rails-92120426317083ae90cb64cff07a7dd3a8acdc65.zip
Improve router test.
We should assert that routes will not be recognized if the verbs do not match.
Diffstat (limited to 'actionpack/test/journey/router_test.rb')
-rw-r--r--actionpack/test/journey/router_test.rb17
1 files changed, 11 insertions, 6 deletions
diff --git a/actionpack/test/journey/router_test.rb b/actionpack/test/journey/router_test.rb
index 8fc965adf4..8e90d4100f 100644
--- a/actionpack/test/journey/router_test.rb
+++ b/actionpack/test/journey/router_test.rb
@@ -544,19 +544,24 @@ module ActionDispatch
def test_recognize_cares_about_verbs
path = Path::Pattern.from_string "/books(/:action(.:format))"
app = Object.new
- conditions = {
- :request_method => 'GET'
- }
+ conditions = { request_method: 'GET' }
@router.routes.add_route(app, path, conditions, {})
+ env = rails_env 'PATH_INFO' => '/books/list.rss',
+ "REQUEST_METHOD" => "POST"
+
+ called = false
+ @router.recognize(env) do |r, params|
+ called = true
+ end
+
+ assert_not called
+
conditions = conditions.dup
conditions[:request_method] = 'POST'
post = @router.routes.add_route(app, path, conditions, {})
- env = rails_env 'PATH_INFO' => '/books/list.rss',
- "REQUEST_METHOD" => "POST"
-
called = false
@router.recognize(env) do |r, params|
assert_equal post, r