diff options
| author | Rafael França <rafaelmfranca@gmail.com> | 2016-02-24 10:10:45 -0300 |
|---|---|---|
| committer | Rafael França <rafaelmfranca@gmail.com> | 2016-02-24 10:10:45 -0300 |
| commit | 709a554d1b08231defd2d7b34760c706e692dd8c (patch) | |
| tree | 6722770ff795c9c1898495030392abe8b74cc636 /actionpack/test | |
| parent | ba004484cd3456fc8cb6391e7927f3ddc8e32dad (diff) | |
| parent | 5f59c10471fee7bca43f65bb2bc776ed047715b6 (diff) | |
| download | rails-709a554d1b08231defd2d7b34760c706e692dd8c.tar.gz rails-709a554d1b08231defd2d7b34760c706e692dd8c.tar.bz2 rails-709a554d1b08231defd2d7b34760c706e692dd8c.zip | |
Merge pull request #23849 from prathamesh-sonpatki/fix-23822
Show permitted flag in the output of AC::Parameters#inspect
Diffstat (limited to 'actionpack/test')
| -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 |