Warn about deprecation `%{file}` interpolation in verify command

See chef/chef#3232

1 files changed, 18 insertions, 0 deletions

diff --git a/spec/unit/resource/file/verification_spec.rb b/spec/unit/resource/file/verification_spec.rb
index 0ab4ce28e6..04ae9ad629 100644
--- a/spec/unit/resource/file/verification_spec.rb
+++ b/spec/unit/resource/file/verification_spec.rb
@@ -69,6 +69,10 @@ describe Chef::Resource::File::Verification do
     end
 
     context "with a verification command(String)" do
+      before(:each) do
+        allow(Chef::Log).to receive(:deprecation).and_return(nil)
+      end
+
       def platform_specific_verify_command(variable_name)
         if windows?
           "if \"#{temp_path}\" == \"%{#{variable_name}}\" (exit 0) else (exit 1)"
@@ -83,6 +87,20 @@ describe Chef::Resource::File::Verification do
         expect(v.verify(temp_path)).to eq(true)
       end
 
+      it "warns about deprecation when \%{file} is used" do
+        expect(Chef::Log).to receive(:deprecation).with(/%{file} is deprecated/)
+        test_command = platform_specific_verify_command('file')
+        Chef::Resource::File::Verification.new(parent_resource, test_command, {})
+          .verify(temp_path)
+      end
+
+      it "does not warn about deprecation when \%{file} is not used" do
+        expect(Chef::Log).to_not receive(:deprecation)
+        test_command = platform_specific_verify_command('path')
+        Chef::Resource::File::Verification.new(parent_resource, test_command, {})
+          .verify(temp_path)
+      end
+
       it "substitutes \%{path} with the path" do
         test_command = platform_specific_verify_command('path')
         v = Chef::Resource::File::Verification.new(parent_resource, test_command, {})