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