Skip to content

Commit

Permalink
Deprecated should_render_a_form [thoughtbot#157 state:resolved]
Browse files Browse the repository at this point in the history
  • Loading branch information
jferris committed May 7, 2009
1 parent 8e13aff commit efff1cc
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 1 deletion.
4 changes: 3 additions & 1 deletion lib/shoulda/action_view/macros.rb
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,6 @@ module ActionView # :nodoc:
# get :new
# end
#
# should_render_a_form
# should_render_page_with_metadata :title => /index/
#
# should "do something else really cool" do
Expand All @@ -23,7 +22,10 @@ module ActionView # :nodoc:
module Macros

# Macro that creates a test asserting that the rendered view contains a <form> element.
#
# Deprecated.
def should_render_a_form
warn "[DEPRECATION] should_render_a_form is deprecated."
should "display a form" do
assert_select "form", true, "The template doesn't contain a <form> element"
end
Expand Down
1 change: 1 addition & 0 deletions test/functional/posts_controller_test.rb
Original file line number Diff line number Diff line change
Expand Up @@ -93,6 +93,7 @@ def setup
setup { get :new, :user_id => users(:first) }
should_render_without_layout
should_not_set_the_flash
should_render_a_form
end

context "on POST to #create" do
Expand Down

0 comments on commit efff1cc

Please sign in to comment.