Skip to content

Commit

Permalink
Added unit test to cover changes to RouteSet.url_for
Browse files Browse the repository at this point in the history
ActionDispatch::Routing::RouteSet.url_for now handles passing params through to
ActionDispatch::Http::Url.url_for

Conflicts:

	actionpack/test/controller/base_test.rb
  • Loading branch information
rmm5t committed Feb 7, 2012
1 parent 6efb849 commit 30e5503
Showing 1 changed file with 16 additions and 0 deletions.
16 changes: 16 additions & 0 deletions actionpack/test/controller/base_test.rb
Expand Up @@ -158,6 +158,22 @@ def setup
rescue_action_in_public!
end

def test_url_for_query_params_included
rs = ActionDispatch::Routing::RouteSet.new
rs.draw do
match 'home' => 'pages#home'
end

options = {
:action => "home",
:controller => "pages",
:only_path => true,
:params => { "token" => "secret" }
}

assert_equal '/home?token=secret', rs.url_for(options)
end

def test_url_options_override
with_routing do |set|
set.draw do
Expand Down

0 comments on commit 30e5503

Please sign in to comment.