diff options
author | David Chelimsky <dchelimsky@gmail.com> | 2010-05-25 23:46:00 -0500 |
---|---|---|
committer | Mikel Lindsaar <raasdnil@gmail.com> | 2010-06-03 23:32:09 +1000 |
commit | d6fec21d1affe069686d681964d208a4b0df7182 (patch) | |
tree | c58ac9162487bdca68342fb2d30730ccfd926378 /actionpack/test | |
parent | 70dca37cfd04bd029f9684eb41d6c8658200ff78 (diff) | |
download | rails-d6fec21d1affe069686d681964d208a4b0df7182.tar.gz rails-d6fec21d1affe069686d681964d208a4b0df7182.tar.bz2 rails-d6fec21d1affe069686d681964d208a4b0df7182.zip |
In AV::TC, move protect_against_forgery? from the test_case to the
_helper module included in the view.
- ensures that protect_against_forgery? is present when a helper
included in a partial that is rendered by the template under test
calls it (which happens in FormTagHelper#extra_tags_for_form, for
example).
[#4700 state:resolved]
Signed-off-by: José Valim <jose.valim@gmail.com>
Diffstat (limited to 'actionpack/test')
-rw-r--r-- | actionpack/test/template/test_case_test.rb | 15 |
1 files changed, 15 insertions, 0 deletions
diff --git a/actionpack/test/template/test_case_test.rb b/actionpack/test/template/test_case_test.rb index a34bca8145..16e5ee4f72 100644 --- a/actionpack/test/template/test_case_test.rb +++ b/actionpack/test/template/test_case_test.rb @@ -122,6 +122,21 @@ module ActionView helper_method :from_test_case end + class IgnoreProtectAgainstForgeryTest < ActionView::TestCase + module HelperThatInvokesProtectAgainstForgery + def help_me + protect_against_forgery? + end + end + + helper HelperThatInvokesProtectAgainstForgery + + test "protect_from_forgery? in any helpers returns false" do + assert !_view.help_me + end + + end + class ATestHelperTest < ActionView::TestCase include SharedTests test_case = self |