# frozen_string_literal: true # # Copyright:: 2019, Chef Software, Inc. # Author:: Tim Smith () # # Licensed under the Apache License, Version 2.0 (the "License"); # you may not use this file except in compliance with the License. # You may obtain a copy of the License at # # http://www.apache.org/licenses/LICENSE-2.0 # # Unless required by applicable law or agreed to in writing, software # distributed under the License is distributed on an "AS IS" BASIS, # WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. # See the License for the specific language governing permissions and # limitations under the License. # module RuboCop module Cop module Chef module Deprecations # Use the new-style notification syntax which allows you to notify resources defined later in a recipe or resource. # # @example # # ### incorrect # template '/etc/www/configures-apache.conf' do # notifies :restart, resources(service: 'apache') # end # # template '/etc/www/configures-apache.conf' do # notifies :restart, resources(service: 'apache'), :immediately # end # # template '/etc/www/configures-apache.conf' do # notifies :restart, resources(service: service_name_variable), :immediately # end # # template '/etc/www/configures-apache.conf' do # subscribes :restart, resources(service: service_name_variable), :immediately # end # # ### correct # template '/etc/www/configures-apache.conf' do # notifies :restart, 'service[apache]' # end # # template '/etc/www/configures-apache.conf' do # notifies :restart, 'service[apache]', :immediately # end # # template '/etc/www/configures-apache.conf' do # notifies :restart, "service[#{service_name_variable}]", :immediately # end # # template '/etc/www/configures-apache.conf' do # subscribes :restart, "service[#{service_name_variable}]", :immediately # end # class LegacyNotifySyntax < Base extend AutoCorrector MSG = 'Use the new-style notification syntax which allows you to notify resources defined later in a recipe or resource.' RESTRICT_ON_SEND = [:notifies, :subscribes].freeze def_node_matcher :legacy_notify?, <<-PATTERN (send nil? ${:notifies :subscribes} $(sym _) (send nil? :resources (hash (pair $(sym _) $(...) ) ) ) $... ) PATTERN def on_send(node) legacy_notify?(node) do |notify_type, action, type, name, timing| add_offense(node, severity: :warning) do |corrector| service_value = case name.type when :str "'#{type.source}[#{name.value}]'" when :dstr "\"#{type.source}[#{name.value}]\"" else "\"#{type.source}[\#{#{name.source}}]\"" end new_val = +"#{notify_type} #{action.source}, #{service_value}" new_val << ", #{timing.first.source}" unless timing.empty? corrector.replace(node, new_val) end end end end end end end end