diff options
author | schneems <richard.schneeman@gmail.com> | 2012-08-09 23:59:25 -0500 |
---|---|---|
committer | schneems <richard.schneeman@gmail.com> | 2012-08-10 22:02:48 -0500 |
commit | 60b650b18c82a050ab81f924e23f7d308ffd8972 (patch) | |
tree | d95cebb615c2fbacc0ae445da8bf16e5d559f1bc /actionpack/test | |
parent | 581a9277102e66132a98cd1e8df93f904628585e (diff) | |
download | rails-60b650b18c82a050ab81f924e23f7d308ffd8972.tar.gz rails-60b650b18c82a050ab81f924e23f7d308ffd8972.tar.bz2 rails-60b650b18c82a050ab81f924e23f7d308ffd8972.zip |
check for nil or empty record in form_for
if nil or an empty array is passed into form_for you get a horrible error message, this one is much more indicative of what the programmer needs to know to fix the problem.
Diffstat (limited to 'actionpack/test')
-rw-r--r-- | actionpack/test/template/form_helper_test.rb | 14 |
1 files changed, 14 insertions, 0 deletions
diff --git a/actionpack/test/template/form_helper_test.rb b/actionpack/test/template/form_helper_test.rb index 152b35ff0f..5c6cb45530 100644 --- a/actionpack/test/template/form_helper_test.rb +++ b/actionpack/test/template/form_helper_test.rb @@ -1045,6 +1045,20 @@ class FormHelperTest < ActionView::TestCase end end + def test_form_for_requires_arguments + error = assert_raises(ArgumentError) do + form_for(nil, :html => { :id => 'create-post' }) do + end + end + assert_equal "First argument in form cannot contain nil or be empty", error.message + + error = assert_raises(ArgumentError) do + form_for([nil, nil], :html => { :id => 'create-post' }) do + end + end + assert_equal "First argument in form cannot contain nil or be empty", error.message + end + def test_form_for form_for(@post, :html => { :id => 'create-post' }) do |f| concat f.label(:title) { "The Title" } |