diff options
| author | Rafael França <rafaelmfranca@gmail.com> | 2018-04-04 18:19:35 -0400 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2018-04-04 18:19:35 -0400 |
| commit | fe4e9d4c5d82806fdaf28c87c731132f81f0ac3e (patch) | |
| tree | 7d96907def4d04a69495206f6e806c0590ad835e /actionpack/test/controller/parameters | |
| parent | 826b17cd8a4f0416fd6b8451199fd0f1fa20f417 (diff) | |
| parent | c1ceafc9d128b430f27d43d98369683c59c33021 (diff) | |
| download | rails-fe4e9d4c5d82806fdaf28c87c731132f81f0ac3e.tar.gz rails-fe4e9d4c5d82806fdaf28c87c731132f81f0ac3e.tar.bz2 rails-fe4e9d4c5d82806fdaf28c87c731132f81f0ac3e.zip | |
Merge pull request #32441 from composerinteralia/refute-not
Add custom RuboCop for `assert_not` over `refute`
Diffstat (limited to 'actionpack/test/controller/parameters')
| -rw-r--r-- | actionpack/test/controller/parameters/parameters_permit_test.rb | 10 |
1 files changed, 5 insertions, 5 deletions
diff --git a/actionpack/test/controller/parameters/parameters_permit_test.rb b/actionpack/test/controller/parameters/parameters_permit_test.rb index 295f3a03ef..e60003dc64 100644 --- a/actionpack/test/controller/parameters/parameters_permit_test.rb +++ b/actionpack/test/controller/parameters/parameters_permit_test.rb @@ -309,7 +309,7 @@ class ParametersPermitTest < ActiveSupport::TestCase merged_params = @params.reverse_merge(default_params) assert_equal "1234", merged_params[:id] - refute_predicate merged_params[:person], :empty? + assert_not_predicate merged_params[:person], :empty? end test "#with_defaults is an alias of reverse_merge" do @@ -317,11 +317,11 @@ class ParametersPermitTest < ActiveSupport::TestCase merged_params = @params.with_defaults(default_params) assert_equal "1234", merged_params[:id] - refute_predicate merged_params[:person], :empty? + assert_not_predicate merged_params[:person], :empty? end test "not permitted is sticky beyond reverse_merge" do - refute_predicate @params.reverse_merge(a: "b"), :permitted? + assert_not_predicate @params.reverse_merge(a: "b"), :permitted? end test "permitted is sticky beyond reverse_merge" do @@ -334,7 +334,7 @@ class ParametersPermitTest < ActiveSupport::TestCase @params.reverse_merge!(default_params) assert_equal "1234", @params[:id] - refute_predicate @params[:person], :empty? + assert_not_predicate @params[:person], :empty? end test "#with_defaults! is an alias of reverse_merge!" do @@ -342,7 +342,7 @@ class ParametersPermitTest < ActiveSupport::TestCase @params.with_defaults!(default_params) assert_equal "1234", @params[:id] - refute_predicate @params[:person], :empty? + assert_not_predicate @params[:person], :empty? end test "modifying the parameters" do |