mirror of
https://github.com/rails/rails.git
synced 2022-11-09 12:12:34 -05:00
Change OrderedHash with array options to simple hash usage
This commit is contained in:
parent
c421870c7a
commit
f7746e9be7
1 changed files with 8 additions and 8 deletions
|
@ -31,13 +31,13 @@ class UrlHelperTest < ActiveSupport::TestCase
|
|||
|
||||
setup :_prepare_context
|
||||
|
||||
def hash_for(opts = [])
|
||||
ActiveSupport::OrderedHash[*([:controller, "foo", :action, "bar"].concat(opts))]
|
||||
def hash_for(options = {})
|
||||
{ :controller => "foo", :action => "bar" }.merge!(options)
|
||||
end
|
||||
alias url_hash hash_for
|
||||
|
||||
def test_url_for_does_not_escape_urls
|
||||
assert_equal "/?a=b&c=d", url_for(hash_for([:a, :b, :c, :d]))
|
||||
assert_equal "/?a=b&c=d", url_for(hash_for(:a => :b, :c => :d))
|
||||
end
|
||||
|
||||
def test_url_for_with_back
|
||||
|
@ -168,7 +168,7 @@ class UrlHelperTest < ActiveSupport::TestCase
|
|||
end
|
||||
|
||||
def test_link_tag_with_host_option
|
||||
hash = hash_for([:host, "www.example.com"])
|
||||
hash = hash_for(:host => "www.example.com")
|
||||
expected = %q{<a href="http://www.example.com/">Test Link</a>}
|
||||
assert_dom_equal(expected, link_to('Test Link', hash))
|
||||
end
|
||||
|
@ -343,7 +343,7 @@ class UrlHelperTest < ActiveSupport::TestCase
|
|||
def test_current_page_with_params_that_match
|
||||
@request = request_for_url("/?order=desc&page=1")
|
||||
|
||||
assert current_page?(hash_for([:order, "desc", :page, "1"]))
|
||||
assert current_page?(hash_for(:order => "desc", :page => "1"))
|
||||
assert current_page?("http://www.example.com/?order=desc&page=1")
|
||||
end
|
||||
|
||||
|
@ -371,20 +371,20 @@ class UrlHelperTest < ActiveSupport::TestCase
|
|||
@request = request_for_url("/?order=desc&page=1")
|
||||
|
||||
assert_equal "Showing",
|
||||
link_to_unless_current("Showing", hash_for([:order, 'desc', :page, '1']))
|
||||
link_to_unless_current("Showing", hash_for(:order => 'desc', :page => '1'))
|
||||
assert_equal "Showing",
|
||||
link_to_unless_current("Showing", "http://www.example.com/?order=desc&page=1")
|
||||
|
||||
@request = request_for_url("/?order=desc")
|
||||
|
||||
assert_equal %{<a href="/?order=asc">Showing</a>},
|
||||
link_to_unless_current("Showing", hash_for([:order, :asc]))
|
||||
link_to_unless_current("Showing", hash_for(:order => :asc))
|
||||
assert_equal %{<a href="http://www.example.com/?order=asc">Showing</a>},
|
||||
link_to_unless_current("Showing", "http://www.example.com/?order=asc")
|
||||
|
||||
@request = request_for_url("/?order=desc")
|
||||
assert_equal %{<a href="/?order=desc&page=2\">Showing</a>},
|
||||
link_to_unless_current("Showing", hash_for([:order, "desc", :page, 2]))
|
||||
link_to_unless_current("Showing", hash_for(:order => "desc", :page => 2))
|
||||
assert_equal %{<a href="http://www.example.com/?order=desc&page=2">Showing</a>},
|
||||
link_to_unless_current("Showing", "http://www.example.com/?order=desc&page=2")
|
||||
|
||||
|
|
Loading…
Reference in a new issue