aboutsummaryrefslogtreecommitdiffstats
path: root/actionview
diff options
context:
space:
mode:
authorJustin Coyne <jcoyne@justincoyne.com>2015-12-29 09:37:07 -0600
committerJustin Coyne <jcoyne@justincoyne.com>2015-12-29 14:40:17 -0600
commitd8e98897b5703ac49bf0764da71a06d64ecda9b0 (patch)
treebce59f460d660e2c9fb22942d66a23b9b1f2993d /actionview
parent929c61573e289e432a4e571ae157248745ae2eae (diff)
downloadrails-d8e98897b5703ac49bf0764da71a06d64ecda9b0.tar.gz
rails-d8e98897b5703ac49bf0764da71a06d64ecda9b0.tar.bz2
rails-d8e98897b5703ac49bf0764da71a06d64ecda9b0.zip
TestController#parameters returns AC::Parameters
Fixes #22827 ActionView::TestCase::TestController#parameters should return an instance of ActionController::Parameters rather than a hash. This enables helper methods to use the correct interface.
Diffstat (limited to 'actionview')
-rw-r--r--actionview/CHANGELOG.md5
-rw-r--r--actionview/lib/action_view/test_case.rb2
-rw-r--r--actionview/test/template/test_case_test.rb4
3 files changed, 10 insertions, 1 deletions
diff --git a/actionview/CHANGELOG.md b/actionview/CHANGELOG.md
index 85d91825ac..91d47cda11 100644
--- a/actionview/CHANGELOG.md
+++ b/actionview/CHANGELOG.md
@@ -1,3 +1,8 @@
+* `ActionView::TestCase::Controller#params` returns an instance of
+ `ActionController::Parameters`.
+
+ *Justin Coyne*
+
* Fix regression in `submit_tag` when a symbol is used as label argument.
*Yuuji Yaginuma*
diff --git a/actionview/lib/action_view/test_case.rb b/actionview/lib/action_view/test_case.rb
index f6b5696a13..120962b5aa 100644
--- a/actionview/lib/action_view/test_case.rb
+++ b/actionview/lib/action_view/test_case.rb
@@ -28,7 +28,7 @@ module ActionView
@response = ActionDispatch::TestResponse.new
@request.env.delete('PATH_INFO')
- @params = {}
+ @params = ActionController::Parameters.new
end
end
diff --git a/actionview/test/template/test_case_test.rb b/actionview/test/template/test_case_test.rb
index b057d43ee0..d69d5819b6 100644
--- a/actionview/test/template/test_case_test.rb
+++ b/actionview/test/template/test_case_test.rb
@@ -42,6 +42,10 @@ module ActionView
assert_same view, view
end
+ test "exposes params" do
+ assert params.is_a? ActionController::Parameters
+ end
+
test "exposes view as _view for backwards compatibility" do
assert_same _view, view
end