Fix failing test on several methods on Parameter

* `each`
* `each_pair`
* `delete`
* `select!`

2 files changed, 11 insertions, 2 deletions

diff --git a/actionpack/test/controller/parameters/accessors_test.rb b/actionpack/test/controller/parameters/accessors_test.rb
index 7f4e595330..97875c3cbb 100644
--- a/actionpack/test/controller/parameters/accessors_test.rb
+++ b/actionpack/test/controller/parameters/accessors_test.rb
@@ -36,6 +36,15 @@ class ParametersAccessorsTest < ActiveSupport::TestCase
     @params.each { |key, value| assert_not(value.permitted?) if key == "person" }
   end
 
+  test "each_pair carries permitted status" do
+    @params.permit!
+    @params.each_pair { |key, value| assert(value.permitted?) if key == "person" }
+  end
+
+  test "each_pair carries unpermitted status" do
+    @params.each_pair { |key, value| assert_not(value.permitted?) if key == "person" }
+  end
+
   test "except retains permitted status" do
     @params.permit!
     assert @params.except(:person).permitted?
diff --git a/actionpack/test/controller/parameters/mutators_test.rb b/actionpack/test/controller/parameters/mutators_test.rb
index 300dbfa61c..744d8664be 100644
--- a/actionpack/test/controller/parameters/mutators_test.rb
+++ b/actionpack/test/controller/parameters/mutators_test.rb
@@ -63,11 +63,11 @@ class ParametersMutatorsTest < ActiveSupport::TestCase
 
   test "select! retains permitted status" do
     @params.permit!
-    assert @params.select! { |k| k == "person" }.permitted?
+    assert @params.select! { |k| k != "person" }.permitted?
   end
 
   test "select! retains unpermitted status" do
-    assert_not @params.select! { |k| k == "person" }.permitted?
+    assert_not @params.select! { |k| k != "person" }.permitted?
   end
 
   test "slice! retains permitted status" do