diff options
author | Prathamesh Sonpatki <csonpatki@gmail.com> | 2016-02-24 12:48:39 +0530 |
---|---|---|
committer | Prathamesh Sonpatki <csonpatki@gmail.com> | 2016-02-24 13:55:37 +0530 |
commit | 5f59c10471fee7bca43f65bb2bc776ed047715b6 (patch) | |
tree | a396b7393481625ad9f50ed504a8455854e02988 /actionpack/test/controller | |
parent | 22db455dbe9c26fe6d723cac0758705d9943ea4b (diff) | |
download | rails-5f59c10471fee7bca43f65bb2bc776ed047715b6.tar.gz rails-5f59c10471fee7bca43f65bb2bc776ed047715b6.tar.bz2 rails-5f59c10471fee7bca43f65bb2bc776ed047715b6.zip |
Show permitted flag in the output of AC::Parameters#inspect
- Fixes #23822.
Diffstat (limited to 'actionpack/test/controller')
-rw-r--r-- | actionpack/test/controller/parameters/accessors_test.rb | 16 |
1 files changed, 13 insertions, 3 deletions
diff --git a/actionpack/test/controller/parameters/accessors_test.rb b/actionpack/test/controller/parameters/accessors_test.rb index 4ef5bed30d..cea265f9ab 100644 --- a/actionpack/test/controller/parameters/accessors_test.rb +++ b/actionpack/test/controller/parameters/accessors_test.rb @@ -4,6 +4,8 @@ require 'active_support/core_ext/hash/transform_values' class ParametersAccessorsTest < ActiveSupport::TestCase setup do + ActionController::Parameters.permit_all_parameters = false + @params = ActionController::Parameters.new( person: { age: '32', @@ -176,12 +178,20 @@ class ParametersAccessorsTest < ActiveSupport::TestCase assert(@params != false) end - test "inspect shows both class name and parameters" do + test "inspect shows both class name, parameters and permitted flag" do assert_equal( '<ActionController::Parameters {"person"=>{"age"=>"32", '\ - '"name"=>{"first"=>"David", "last"=>"Heinemeier Hansson"}, ' \ - '"addresses"=>[{"city"=>"Chicago", "state"=>"Illinois"}]}}>', + '"name"=>{"first"=>"David", "last"=>"Heinemeier Hansson"}, ' \ + '"addresses"=>[{"city"=>"Chicago", "state"=>"Illinois"}]}} permitted: false>', @params.inspect ) end + + test "inspect prints updated permitted flag in the output" do + assert_match(/permitted: false/, @params.inspect) + + @params.permit! + + assert_match(/permitted: true/, @params.inspect) + end end |