From e38dd7bfa4360e241eadf0cf44abdf86ea33a393 Mon Sep 17 00:00:00 2001 From: Yves Senn Date: Fri, 1 May 2015 11:10:31 +0200 Subject: add test-case for `link_to_if` behavior with a block. This illustrates the purpose of the block for `link_to_if` and `link_to_unless` helper methods. It should help to prevent further mistakes like #19844. --- actionview/test/template/url_helper_test.rb | 10 ++++++++++ 1 file changed, 10 insertions(+) (limited to 'actionview') diff --git a/actionview/test/template/url_helper_test.rb b/actionview/test/template/url_helper_test.rb index ef4df0407a..9ba837d4e7 100644 --- a/actionview/test/template/url_helper_test.rb +++ b/actionview/test/template/url_helper_test.rb @@ -379,6 +379,11 @@ class UrlHelperTest < ActiveSupport::TestCase assert_dom_equal %{Listing}, link_to_if(true, "Listing", url_hash) end + def test_link_to_if_with_block + assert_equal "Fallback", link_to_if(false, "Showing", url_hash) { "Fallback" } + assert_dom_equal %{Listing}, link_to_if(true, "Listing", url_hash) { "Fallback" } + end + def request_for_url(url, opts = {}) env = Rack::MockRequest.env_for("http://www.example.com#{url}", opts) ActionDispatch::Request.new(env) @@ -479,6 +484,11 @@ class UrlHelperTest < ActiveSupport::TestCase link_to_unless_current("Listing", "http://www.example.com/") end + def test_link_to_unless_with_block + assert_equal %{Showing}, link_to_unless(false, "Showing", url_hash) { "Fallback" } + assert_dom_equal "Fallback", link_to_unless(true, "Listing", url_hash) { "Fallback" } + end + def test_mail_to assert_dom_equal %{david@loudthinking.com}, mail_to("david@loudthinking.com") assert_dom_equal %{David Heinemeier Hansson}, mail_to("david@loudthinking.com", "David Heinemeier Hansson") -- cgit v1.2.3