class RuboCop::Cop::Chef::Modernize::ConditionalUsingTest
Use ::File.exist?('/foo/bar') instead of the slower 'test -f /foo/bar' which requires shelling out
@example
#### incorrect only_if 'test -f /bin/foo' #### correct only_if { ::File.exist?('bin/foo') }
Constants
- MSG
- RESTRICT_ON_SEND
Public Instance Methods
on_send(node)
click to toggle source
# File lib/rubocop/cop/chef/modernize/conditional_using_test.rb, line 42 def on_send(node) resource_conditional?(node) do |conditional| return unless conditional.value.match?(/^test -[ef] \S*$/) add_offense(node, message: MSG, severity: :refactor) do |corrector| new_string = "{ ::File.exist?('#{conditional.value.match(/^test -[ef] (\S*)$/)[1]}') }" corrector.replace(conditional, new_string) end end end