diff options
author | Jeff Dean <jeff@zilkey.com> | 2010-06-11 20:25:58 -0400 |
---|---|---|
committer | Michael Koziarski <michael@koziarski.com> | 2010-06-23 16:23:54 +1200 |
commit | ac8d3e3acabf3ece495f591a7195d794d6c611e1 (patch) | |
tree | 6363c272e2c70b87556d2897957c2a9cb3ff425d /actionpack/test | |
parent | e639536ea80e94f5d72493267c8aec21d305cf74 (diff) | |
download | rails-ac8d3e3acabf3ece495f591a7195d794d6c611e1.tar.gz rails-ac8d3e3acabf3ece495f591a7195d794d6c611e1.tar.bz2 rails-ac8d3e3acabf3ece495f591a7195d794d6c611e1.zip |
make text_field and hidden_field omit the value attribute if the developer explicitly passes in :value => nil [#4839 state:resolved]
Signed-off-by: Michael Koziarski <michael@koziarski.com>
Diffstat (limited to 'actionpack/test')
-rw-r--r-- | actionpack/test/template/form_helper_test.rb | 10 |
1 files changed, 10 insertions, 0 deletions
diff --git a/actionpack/test/template/form_helper_test.rb b/actionpack/test/template/form_helper_test.rb index 11a3283092..6ba407e230 100644 --- a/actionpack/test/template/form_helper_test.rb +++ b/actionpack/test/template/form_helper_test.rb @@ -188,6 +188,11 @@ class FormHelperTest < ActionView::TestCase assert_dom_equal expected, text_field("post", "title", :maxlength => 35, :size => nil) end + def test_text_field_with_nil_value + expected = '<input id="post_title" name="post[title]" size="30" type="text" />' + assert_dom_equal expected, text_field("post", "title", :value => nil) + end + def test_text_field_doesnt_change_param_values object_name = 'post[]' expected = '<input id="post_123_title" name="post[123][title]" size="30" type="text" value="Hello World" />' @@ -208,6 +213,11 @@ class FormHelperTest < ActionView::TestCase hidden_field("post", "title") end + def test_hidden_field_with_nil_value + expected = '<input id="post_title" name="post[title]" type="hidden" />' + assert_dom_equal expected, hidden_field("post", "title", :value => nil) + end + def test_text_field_with_options assert_dom_equal '<input id="post_title" name="post[title]" type="hidden" value="Something Else" />', hidden_field("post", "title", :value => "Something Else") |