diff options
author | Tim Smith <tsmith84@gmail.com> | 2021-12-26 20:32:49 -0800 |
---|---|---|
committer | Tim Smith <tsmith84@gmail.com> | 2021-12-26 20:32:49 -0800 |
commit | 0cefcaa319b0d2f07241b78bf7d6f06de03d07ef (patch) | |
tree | 5c1061b328d1769e8ad1c995db78a5521736c33e /spec/support | |
parent | 50a0001bae754c66feb69c84b64cb305a03a34d2 (diff) | |
download | chef-Performance_StringIdentifierArgument.tar.gz |
Resolve Performance/StringIdentifierArgument warningsPerformance_StringIdentifierArgument
Inernally it's all a symbol so skip the conversion
Signed-off-by: Tim Smith <tsmith@chef.io>
Diffstat (limited to 'spec/support')
-rw-r--r-- | spec/support/platform_helpers.rb | 2 | ||||
-rw-r--r-- | spec/support/shared/functional/file_resource.rb | 2 | ||||
-rw-r--r-- | spec/support/shared/unit/provider/useradd_based_user_provider.rb | 2 |
3 files changed, 3 insertions, 3 deletions
diff --git a/spec/support/platform_helpers.rb b/spec/support/platform_helpers.rb index 1538a4eb34..430d9b055f 100644 --- a/spec/support/platform_helpers.rb +++ b/spec/support/platform_helpers.rb @@ -172,7 +172,7 @@ def wpar? end def supports_cloexec? - Fcntl.const_defined?("F_SETFD") && Fcntl.const_defined?("FD_CLOEXEC") + Fcntl.const_defined?(:F_SETFD) && Fcntl.const_defined?(:FD_CLOEXEC) end def selinux_enabled? diff --git a/spec/support/shared/functional/file_resource.rb b/spec/support/shared/functional/file_resource.rb index 1385d3dc30..70379263b6 100644 --- a/spec/support/shared/functional/file_resource.rb +++ b/spec/support/shared/functional/file_resource.rb @@ -157,7 +157,7 @@ shared_examples_for "a file with the wrong content" do context "when running :create action" do let(:provider) { resource.provider_for_action(:create) } - let(:reporter_messages) { provider.instance_variable_get("@converge_actions").actions[0][0] } + let(:reporter_messages) { provider.instance_variable_get(:@converge_actions).actions[0][0] } before do provider.run_action diff --git a/spec/support/shared/unit/provider/useradd_based_user_provider.rb b/spec/support/shared/unit/provider/useradd_based_user_provider.rb index 1f9f87866c..7e7a8ae857 100644 --- a/spec/support/shared/unit/provider/useradd_based_user_provider.rb +++ b/spec/support/shared/unit/provider/useradd_based_user_provider.rb @@ -169,7 +169,7 @@ shared_examples_for "a useradd-based user provider" do |supported_useradd_option before do provider.new_resource.system true # there is no public API to set attribute's value to nil - provider.new_resource.instance_variable_set("@home", nil) + provider.new_resource.instance_variable_set(:@home, nil) end it "should not include -m or -d in the command options" do |