chef 12.6.0 → 12.7.2

data/spec/functional/resource/cookbook_file_spec.rb

@@ -1,6 +1,6 @@
 #
-# Author:: Tim Hinderliter (<tim@opscode.com>)
-# Copyright:: Copyright (c) 2012 Opscode, Inc.
+# Author:: Tim Hinderliter (<tim@chef.io>)
+# Copyright:: Copyright 2012-2016, Chef Software Inc.
 # License:: Apache License, Version 2.0
 #
 # Licensed under the Apache License, Version 2.0 (the "License");
@@ -16,16 +16,16 @@
 # limitations under the License.
 #
 
-require 'spec_helper'
+require "spec_helper"
 
 describe Chef::Resource::CookbookFile do
   include_context Chef::Resource::File
 
-  let(:file_base) { 'cookbook_file_spec' }
-  let(:source) { 'java.response' }
-  let(:cookbook_name) { 'java' }
+  let(:file_base) { "cookbook_file_spec" }
+  let(:source) { "java.response" }
+  let(:cookbook_name) { "java" }
   let(:expected_content) do
-    content = File.open(File.join(CHEF_SPEC_DATA, 'cookbooks', 'java', 'files', 'default', 'java.response'), "rb") do |f|
+    content = File.open(File.join(CHEF_SPEC_DATA, "cookbooks", "java", "files", "default", "java.response"), "rb") do |f|
       f.read
     end
     content.force_encoding(Encoding::BINARY) if content.respond_to?(:force_encoding)
@@ -39,7 +39,7 @@ describe Chef::Resource::CookbookFile do
   def create_resource
     # set up cookbook collection for this run to use, based on our
     # spec data.
-    cookbook_repo = File.expand_path(File.join(CHEF_SPEC_DATA, 'cookbooks'))
+    cookbook_repo = File.expand_path(File.join(CHEF_SPEC_DATA, "cookbooks"))
     Chef::Cookbook::FileVendor.fetch_from_disk(cookbook_repo)
     loader = Chef::CookbookLoader.new(cookbook_repo)
     loader.load_cookbooks
@@ -66,7 +66,7 @@ describe Chef::Resource::CookbookFile do
   # implementation
   # stages files in temp.
   context "targets a file outside of the system temp directory" do
-    let(:windows_non_temp_dir) { File.join(ENV['systemdrive'], make_tmpname(file_base, "non-temp")) }
+    let(:windows_non_temp_dir) { File.join(ENV["systemdrive"], make_tmpname(file_base, "non-temp")) }
     let(:path) { File.join(windows_non_temp_dir, make_tmpname(file_base)) }
 
     before do

data/spec/functional/resource/cron_spec.rb

@@ -1,7 +1,7 @@
 # encoding: UTF-8
 #
 # Author:: Kaustubh Deorukhkar (<kaustubh@clogeny.com>)
-# Copyright:: Copyright (c) 2013 Opscode, Inc.
+# Copyright:: Copyright 2013-2016, Chef Software Inc.
 # License:: Apache License, Version 2.0
 #
 # Licensed under the Apache License, Version 2.0 (the "License");
@@ -17,9 +17,9 @@
 # limitations under the License.
 #
 
-require 'spec_helper'
-require 'functional/resource/base'
-require 'chef/mixin/shell_out'
+require "spec_helper"
+require "functional/resource/base"
+require "chef/mixin/shell_out"
 
 describe Chef::Resource::Cron, :requires_root, :unix_only do
 
@@ -55,17 +55,17 @@ describe Chef::Resource::Cron, :requires_root, :unix_only do
   # Actual tests
   let(:new_resource) do
     new_resource = Chef::Resource::Cron.new("Chef functional test cron", run_context)
-    new_resource.user  'root'
+    new_resource.user  "root"
     # @hourly is not supported on solaris, aix
     if ohai[:platform] == "solaris" || ohai[:platform] == "solaris2" || ohai[:platform] == "aix"
       new_resource.minute "0 * * * *"
     else
-      new_resource.minute '@hourly'
+      new_resource.minute "@hourly"
     end
-    new_resource.hour ''
-    new_resource.day ''
-    new_resource.month ''
-    new_resource.weekday ''
+    new_resource.hour ""
+    new_resource.day ""
+    new_resource.month ""
+    new_resource.weekday ""
     new_resource.command "/bin/true"
     new_resource
   end
@@ -107,8 +107,8 @@ describe Chef::Resource::Cron, :requires_root, :unix_only do
   exclude_solaris = ["solaris", "opensolaris", "solaris2", "omnios"].include?(ohai[:platform])
   describe "create action with various attributes", :external => exclude_solaris do
     def create_and_validate_with_attribute(resource, attribute, value)
-      if ohai[:platform] == 'aix'
-         expect {resource.run_action(:create)}.to raise_error(Chef::Exceptions::Cron, /Aix cron entry does not support environment variables. Please set them in script and use script in cron./)
+      if ohai[:platform] == "aix"
+        expect { resource.run_action(:create) }.to raise_error(Chef::Exceptions::Cron, /Aix cron entry does not support environment variables. Please set them in script and use script in cron./)
       else
         resource.run_action(:create)
         # Verify if the cron is created successfully
@@ -117,7 +117,7 @@ describe Chef::Resource::Cron, :requires_root, :unix_only do
     end
 
     def cron_attribute_should_exists(cron_name, attribute, value)
-      return if ['aix', 'solaris'].include?(ohai[:platform])
+      return if ["aix", "solaris"].include?(ohai[:platform])
       # Test if the attribute exists on newly created cron
       cron_should_exists(cron_name, "")
       expect(shell_out("crontab -l -u #{new_resource.user} | grep \"#{attribute.upcase}=#{value}\"").exitstatus).to eq(0)

data/spec/functional/resource/deploy_revision_spec.rb

@@ -1,6 +1,6 @@
 #
-# Author:: Daniel DeLeo (<dan@opscode.com>)
-# Copyright:: Copyright (c) 2012 Opscode, Inc.
+# Author:: Daniel DeLeo (<dan@chef.io>)
+# Copyright:: Copyright 2012-2016, Chef Software Inc.
 # License:: Apache License, Version 2.0
 #
 # Licensed under the Apache License, Version 2.0 (the "License");
@@ -16,8 +16,8 @@
 # limitations under the License.
 #
 
-require 'spec_helper'
-require 'tmpdir'
+require "spec_helper"
+require "tmpdir"
 
 # Deploy relies heavily on symlinks, so it doesn't work on windows.
 describe Chef::Resource::DeployRevision, :unix_only => true do
@@ -193,7 +193,7 @@ describe Chef::Resource::DeployRevision, :unix_only => true do
 
       it "restarts the application" do
         expect(File).to exist(rel_path("current/restart.txt"))
-        expect(actual_operations_order).to eq(%w[deploy_to_latest_rev])
+        expect(actual_operations_order).to eq(%w{deploy_to_latest_rev})
       end
 
       it "is marked as updated" do
@@ -214,7 +214,7 @@ describe Chef::Resource::DeployRevision, :unix_only => true do
       the_app_is_deployed_at_revision(:latest_rev)
 
       it "restarts the application after rolling back" do
-        expect(actual_operations_order).to eq(%w[deploy_to_latest_rev deploy_to_previous_rev deploy_to_latest_rev_again])
+        expect(actual_operations_order).to eq(%w{deploy_to_latest_rev deploy_to_previous_rev deploy_to_latest_rev_again})
       end
 
       it "is marked updated" do
@@ -248,7 +248,7 @@ describe Chef::Resource::DeployRevision, :unix_only => true do
 
       it "restarts the application" do
         expect(File).to exist(rel_path("current/restart.txt"))
-        expect(actual_operations_order).to eq(%w[deploy_to_latest_rev])
+        expect(actual_operations_order).to eq(%w{deploy_to_latest_rev})
       end
 
       it "is marked as updated" do
@@ -265,7 +265,7 @@ describe Chef::Resource::DeployRevision, :unix_only => true do
       the_app_is_deployed_at_revision(:latest_rev)
 
       it "does not restart the app" do
-        expect(actual_operations_order).to eq(%w[deploy_to_latest_rev])
+        expect(actual_operations_order).to eq(%w{deploy_to_latest_rev})
       end
 
       it "is not marked updated" do
@@ -283,7 +283,7 @@ describe Chef::Resource::DeployRevision, :unix_only => true do
       the_app_is_deployed_at_revision(:latest_rev)
 
       it "restarts the app" do
-        expect(actual_operations_order).to eq(%w[deploy_to_latest_rev deploy_to_latest_rev_again])
+        expect(actual_operations_order).to eq(%w{deploy_to_latest_rev deploy_to_latest_rev_again})
       end
 
       it "is marked updated" do
@@ -301,7 +301,7 @@ describe Chef::Resource::DeployRevision, :unix_only => true do
       the_app_is_deployed_at_revision(:latest_rev)
 
       it "restarts the application after the new deploy" do
-        expect(actual_operations_order).to eq(%w[deploy_to_previous_rev deploy_to_latest_rev])
+        expect(actual_operations_order).to eq(%w{deploy_to_previous_rev deploy_to_latest_rev})
       end
 
       it "is marked updated" do
@@ -324,7 +324,7 @@ describe Chef::Resource::DeployRevision, :unix_only => true do
       the_app_is_deployed_at_revision(:latest_rev)
 
       it "restarts the application after rolling back" do
-        expect(actual_operations_order).to eq(%w[deploy_to_latest_rev deploy_to_previous_rev deploy_to_latest_rev_again])
+        expect(actual_operations_order).to eq(%w{deploy_to_latest_rev deploy_to_previous_rev deploy_to_latest_rev_again})
       end
 
       it "is marked updated" do
@@ -349,7 +349,7 @@ describe Chef::Resource::DeployRevision, :unix_only => true do
       the_app_is_deployed_at_revision(:previous_rev)
 
       it "restarts the application after rolling back" do
-        expect(actual_operations_order).to eq(%w[deploy_to_previous_rev deploy_to_latest_rev deploy_to_latest_rev_again])
+        expect(actual_operations_order).to eq(%w{deploy_to_previous_rev deploy_to_latest_rev deploy_to_latest_rev_again})
       end
 
       it "is marked updated" do
@@ -392,7 +392,7 @@ describe Chef::Resource::DeployRevision, :unix_only => true do
       the_app_is_deployed_at_revision(:previous_rev)
 
       it "restarts the application after rolling back" do
-        expect(actual_operations_order).to eq(%w[deploy_to_previous_rev deploy_to_latest_rev deploy_to_previous_rev_again])
+        expect(actual_operations_order).to eq(%w{deploy_to_previous_rev deploy_to_latest_rev deploy_to_previous_rev_again})
       end
 
       it "is marked updated" do
@@ -437,7 +437,7 @@ describe Chef::Resource::DeployRevision, :unix_only => true do
       the_app_is_deployed_at_revision(:second_rev)
 
       it "restarts the application after rolling back" do
-        expect(actual_operations_order).to eq(%w[deploy_to_second_rev deploy_to_previous_rev deploy_to_previous_rev_again deploy_to_latest_rev deploy_to_latest_rev_again])
+        expect(actual_operations_order).to eq(%w{deploy_to_second_rev deploy_to_previous_rev deploy_to_previous_rev_again deploy_to_latest_rev deploy_to_latest_rev_again})
       end
 
       it "is marked updated" do
@@ -474,7 +474,7 @@ describe Chef::Resource::DeployRevision, :unix_only => true do
       the_app_is_deployed_at_revision(:second_rev)
 
       it "restarts the application after rolling back" do
-        expect(actual_operations_order).to eq(%w[deploy_to_second_rev deploy_to_previous_rev deploy_to_second_rev_again deploy_to_latest_rev deploy_to_second_rev_again_again])
+        expect(actual_operations_order).to eq(%w{deploy_to_second_rev deploy_to_previous_rev deploy_to_second_rev_again deploy_to_latest_rev deploy_to_second_rev_again_again})
       end
 
       it "is marked updated" do
@@ -689,7 +689,7 @@ describe Chef::Resource::DeployRevision, :unix_only => true do
     end
 
     it "runs migrations in between the before_migrate and before_symlink steps" do
-      expect(actual_operations_order).to eq(%w[before_migrate migration before_symlink before_restart after_restart])
+      expect(actual_operations_order).to eq(%w{before_migrate migration before_symlink before_restart after_restart})
     end
   end
 
@@ -799,7 +799,7 @@ describe Chef::Resource::DeployRevision, :unix_only => true do
 
         let(:deploy_that_fails) do
           resource = deploy_to_latest_rev.dup
-          errant_callback = lambda {|x| raise Exception, "I am a failed deploy" }
+          errant_callback = lambda { |x| raise Exception, "I am a failed deploy" }
           resource.send(callback, &errant_callback)
           resource
         end
@@ -862,7 +862,7 @@ describe Chef::Resource::DeployRevision, :unix_only => true do
 
       let(:deploy_that_fails) do
         resource = deploy_to_previous_rev.dup
-        resource.after_restart {|x| raise Exception, "I am a failed deploy" }
+        resource.after_restart { |x| raise Exception, "I am a failed deploy" }
         resource
       end
 

data/spec/functional/resource/directory_spec.rb

@@ -1,6 +1,6 @@
 #
-# Author:: Seth Chisamore (<schisamo@opscode.com>)
-# Copyright:: Copyright (c) 2011 Opscode, Inc.
+# Author:: Seth Chisamore (<schisamo@chef.io>)
+# Copyright:: Copyright 2011-2016, Chef Software Inc.
 # License:: Apache License, Version 2.0
 #
 # Licensed under the Apache License, Version 2.0 (the "License");
@@ -16,7 +16,7 @@
 # limitations under the License.
 #
 
-require 'spec_helper'
+require "spec_helper"
 
 describe Chef::Resource::Directory do
   include_context Chef::Resource::Directory
@@ -33,7 +33,7 @@ describe Chef::Resource::Directory do
   end
 
   let(:resource) do
-  	create_resource
+    create_resource
   end
 
   it_behaves_like "a directory resource"

data/spec/functional/resource/dpkg_package_spec.rb

@@ -1,5 +1,5 @@
 #
-# Copyright:: Copyright (c) 2014-2015 Chef Software, Inc.
+# Copyright:: Copyright 2014-2016, Chef Software, Inc.
 # License:: Apache License, Version 2.0
 #
 # Licensed under the Apache License, Version 2.0 (the "License");
@@ -15,8 +15,8 @@
 # limitations under the License.
 #
 
-require 'spec_helper'
-require 'chef/mixin/shell_out'
+require "spec_helper"
+require "chef/mixin/shell_out"
 
 describe Chef::Resource::DpkgPackage, :requires_root, :debian_family_only, arch: "x86_64" do
   include Chef::Mixin::ShellOut
@@ -33,7 +33,7 @@ describe Chef::Resource::DpkgPackage, :requires_root, :debian_family_only, arch:
     Chef::RunContext.new(node, {}, events)
   }
 
-  let(:dpkg_package) { Chef::Resource::DpkgPackage.new(test1_0, run_context)}
+  let(:dpkg_package) { Chef::Resource::DpkgPackage.new(test1_0, run_context) }
 
   before(:each) do
     shell_out("dpkg -P chef-integration-test chef-integration-test2")
@@ -45,7 +45,7 @@ describe Chef::Resource::DpkgPackage, :requires_root, :debian_family_only, arch:
     dpkg_package.package_name name
   end
 
-  def should_be_purged_or_removed(package, action=nil)
+  def should_be_purged_or_removed(package, action = nil)
     status = shell_out("dpkg -s #{package}")
     output = status.stdout + status.stderr
     if action.nil? || action == :purge
@@ -61,7 +61,7 @@ describe Chef::Resource::DpkgPackage, :requires_root, :debian_family_only, arch:
     it "installs a package when given only the filename as a name argument (no source)" do
       dpkg_package.run_action(action)
       expect(dpkg_package).to be_updated_by_last_action
-      shell_out!('dpkg -s chef-integration-test')
+      shell_out!("dpkg -s chef-integration-test")
     end
 
     it "installs a package when given the name and a source argument" do
@@ -69,7 +69,7 @@ describe Chef::Resource::DpkgPackage, :requires_root, :debian_family_only, arch:
       dpkg_package.source test1_0
       dpkg_package.run_action(action)
       expect(dpkg_package).to be_updated_by_last_action
-      shell_out!('dpkg -s chef-integration-test')
+      shell_out!("dpkg -s chef-integration-test")
     end
 
     it "installs a package when given a different name and a source argument" do
@@ -77,7 +77,7 @@ describe Chef::Resource::DpkgPackage, :requires_root, :debian_family_only, arch:
       dpkg_package.source test1_0
       dpkg_package.run_action(action)
       expect(dpkg_package).to be_updated_by_last_action
-      shell_out!('dpkg -s chef-integration-test')
+      shell_out!("dpkg -s chef-integration-test")
     end
 
     it "installs a package when given a path as a package_name and no source" do
@@ -85,7 +85,7 @@ describe Chef::Resource::DpkgPackage, :requires_root, :debian_family_only, arch:
       dpkg_package.package_name test1_0
       dpkg_package.run_action(action)
       expect(dpkg_package).to be_updated_by_last_action
-      shell_out!('dpkg -s chef-integration-test')
+      shell_out!("dpkg -s chef-integration-test")
     end
 
     it "raises an error when the name is not a path and the source is not given" do
@@ -109,15 +109,15 @@ describe Chef::Resource::DpkgPackage, :requires_root, :debian_family_only, arch:
       shell_out!("dpkg -i #{test1_0}")
       dpkg_package.run_action(action)
       expect(dpkg_package).not_to be_updated_by_last_action
-      shell_out!('dpkg -s chef-integration-test')
+      shell_out!("dpkg -s chef-integration-test")
     end
 
     it "should handle a multipackage install" do
       set_dpkg_package_name [ test1_0, test2_0 ]
       dpkg_package.run_action(action)
       expect(dpkg_package).to be_updated_by_last_action
-      shell_out!('dpkg -s chef-integration-test')
-      shell_out!('dpkg -s chef-integration-test2')
+      shell_out!("dpkg -s chef-integration-test")
+      shell_out!("dpkg -s chef-integration-test2")
     end
 
     it "should not update multipackages that are up-to-date" do
@@ -125,8 +125,8 @@ describe Chef::Resource::DpkgPackage, :requires_root, :debian_family_only, arch:
       set_dpkg_package_name [ test1_0, test2_0 ]
       dpkg_package.run_action(action)
       expect(dpkg_package).not_to be_updated_by_last_action
-      shell_out!('dpkg -s chef-integration-test')
-      shell_out!('dpkg -s chef-integration-test2')
+      shell_out!("dpkg -s chef-integration-test")
+      shell_out!("dpkg -s chef-integration-test2")
     end
 
     it "should install the second if the first is installed" do
@@ -134,8 +134,8 @@ describe Chef::Resource::DpkgPackage, :requires_root, :debian_family_only, arch:
       set_dpkg_package_name [ test1_0, test2_0 ]
       dpkg_package.run_action(action)
       expect(dpkg_package).to be_updated_by_last_action
-      shell_out!('dpkg -s chef-integration-test')
-      shell_out!('dpkg -s chef-integration-test2')
+      shell_out!("dpkg -s chef-integration-test")
+      shell_out!("dpkg -s chef-integration-test2")
     end
 
     it "should install the first if the second is installed" do
@@ -143,8 +143,8 @@ describe Chef::Resource::DpkgPackage, :requires_root, :debian_family_only, arch:
       set_dpkg_package_name [ test1_0, test2_0 ]
       dpkg_package.run_action(action)
       expect(dpkg_package).to be_updated_by_last_action
-      shell_out!('dpkg -s chef-integration-test')
-      shell_out!('dpkg -s chef-integration-test2')
+      shell_out!("dpkg -s chef-integration-test")
+      shell_out!("dpkg -s chef-integration-test2")
     end
   end
 
@@ -191,13 +191,13 @@ describe Chef::Resource::DpkgPackage, :requires_root, :debian_family_only, arch:
       shell_out!("dpkg -i #{test1_0}")
       dpkg_package.run_action(action)
       expect(dpkg_package).to be_updated_by_last_action
-      should_be_purged_or_removed('chef-integration-test')
+      should_be_purged_or_removed("chef-integration-test")
     end
 
     it "should do nothing if the package is not installed when the name is a source" do
       dpkg_package.run_action(action)
       expect(dpkg_package).not_to be_updated_by_last_action
-      should_be_purged_or_removed('chef-integration-test')
+      should_be_purged_or_removed("chef-integration-test")
     end
 
     it "should remove a package that is installed when the name is the package name and source is nil" do
@@ -205,14 +205,14 @@ describe Chef::Resource::DpkgPackage, :requires_root, :debian_family_only, arch:
       set_dpkg_package_name "chef-integration-test"
       dpkg_package.run_action(action)
       expect(dpkg_package).to be_updated_by_last_action
-      should_be_purged_or_removed('chef-integration-test')
+      should_be_purged_or_removed("chef-integration-test")
     end
 
     it "should do nothing if the package is not installed when the name is the package name and the source is nil" do
       set_dpkg_package_name "chef-integration-test"
       dpkg_package.run_action(action)
       expect(dpkg_package).not_to be_updated_by_last_action
-      should_be_purged_or_removed('chef-integration-test')
+      should_be_purged_or_removed("chef-integration-test")
     end
 
     it "should remove a package that is installed when the name is changed but the source is a package" do
@@ -221,7 +221,7 @@ describe Chef::Resource::DpkgPackage, :requires_root, :debian_family_only, arch:
       dpkg_package.source test1_0
       dpkg_package.run_action(action)
       expect(dpkg_package).to be_updated_by_last_action
-      should_be_purged_or_removed('chef-integration-test')
+      should_be_purged_or_removed("chef-integration-test")
     end
 
     it "should do nothing if the package is not installed when the name is changed but the source is a package" do
@@ -229,7 +229,7 @@ describe Chef::Resource::DpkgPackage, :requires_root, :debian_family_only, arch:
       dpkg_package.source test1_0
       dpkg_package.run_action(action)
       expect(dpkg_package).not_to be_updated_by_last_action
-      should_be_purged_or_removed('chef-integration-test')
+      should_be_purged_or_removed("chef-integration-test")
     end
 
     it "should remove a package if the name is a file that does not exist, but the source exists" do
@@ -239,7 +239,7 @@ describe Chef::Resource::DpkgPackage, :requires_root, :debian_family_only, arch:
       dpkg_package.source test1_0
       dpkg_package.run_action(action)
       expect(dpkg_package).to be_updated_by_last_action
-      should_be_purged_or_removed('chef-integration-test')
+      should_be_purged_or_removed("chef-integration-test")
     end
 
     it "should do nothing if the package is not installed when the name is a file that does not exist, but the source exists" do
@@ -249,7 +249,7 @@ describe Chef::Resource::DpkgPackage, :requires_root, :debian_family_only, arch:
       dpkg_package.source test1_0
       dpkg_package.run_action(action)
       expect(dpkg_package).not_to be_updated_by_last_action
-      should_be_purged_or_removed('chef-integration-test')
+      should_be_purged_or_removed("chef-integration-test")
     end
 
     it "should remove a package if the package_name is correct, but the source does not exist" do
@@ -259,7 +259,7 @@ describe Chef::Resource::DpkgPackage, :requires_root, :debian_family_only, arch:
       dpkg_package.source File.join(test1_0, "make.it.fail")
       dpkg_package.run_action(action)
       expect(dpkg_package).to be_updated_by_last_action
-      should_be_purged_or_removed('chef-integration-test')
+      should_be_purged_or_removed("chef-integration-test")
     end
 
     it "should do nothing if the package_name is correct, but the source does not exist, and the package is not installed" do
@@ -268,7 +268,7 @@ describe Chef::Resource::DpkgPackage, :requires_root, :debian_family_only, arch:
       dpkg_package.source File.join(test1_0, "make.it.fail")
       dpkg_package.run_action(action)
       expect(dpkg_package).not_to be_updated_by_last_action
-      should_be_purged_or_removed('chef-integration-test')
+      should_be_purged_or_removed("chef-integration-test")
     end
 
     it "should remove both packages when called with two" do
@@ -276,8 +276,8 @@ describe Chef::Resource::DpkgPackage, :requires_root, :debian_family_only, arch:
       set_dpkg_package_name [ "chef-integration-test", "chef-integration-test2" ]
       dpkg_package.run_action(action)
       expect(dpkg_package).to be_updated_by_last_action
-      should_be_purged_or_removed('chef-integration-test')
-      should_be_purged_or_removed('chef-integration-test2', action)
+      should_be_purged_or_removed("chef-integration-test")
+      should_be_purged_or_removed("chef-integration-test2", action)
     end
 
     it "should remove a package when only the first one is installed" do
@@ -285,8 +285,8 @@ describe Chef::Resource::DpkgPackage, :requires_root, :debian_family_only, arch:
       set_dpkg_package_name [ "chef-integration-test", "chef-integration-test2" ]
       dpkg_package.run_action(action)
       expect(dpkg_package).to be_updated_by_last_action
-      should_be_purged_or_removed('chef-integration-test')
-      should_be_purged_or_removed('chef-integration-test2')
+      should_be_purged_or_removed("chef-integration-test")
+      should_be_purged_or_removed("chef-integration-test2")
     end
 
     it "should remove a package when only the second one is installed" do
@@ -294,16 +294,16 @@ describe Chef::Resource::DpkgPackage, :requires_root, :debian_family_only, arch:
       set_dpkg_package_name [ "chef-integration-test", "chef-integration-test2" ]
       dpkg_package.run_action(action)
       expect(dpkg_package).to be_updated_by_last_action
-      should_be_purged_or_removed('chef-integration-test')
-      should_be_purged_or_removed('chef-integration-test2', action)
+      should_be_purged_or_removed("chef-integration-test")
+      should_be_purged_or_removed("chef-integration-test2", action)
     end
 
     it "should do nothing when both packages are not installed" do
       set_dpkg_package_name [ "chef-integration-test", "chef-integration-test2" ]
       dpkg_package.run_action(action)
       expect(dpkg_package).not_to be_updated_by_last_action
-      should_be_purged_or_removed('chef-integration-test')
-      should_be_purged_or_removed('chef-integration-test2')
+      should_be_purged_or_removed("chef-integration-test")
+      should_be_purged_or_removed("chef-integration-test2")
     end
   end
 
@@ -318,7 +318,7 @@ describe Chef::Resource::DpkgPackage, :requires_root, :debian_family_only, arch:
       set_dpkg_package_name "chef-integration-test2"
       dpkg_package.run_action(action)
       expect(dpkg_package).not_to be_updated_by_last_action
-      shell_out!('dpkg -s chef-integration-test2')  # its still 'installed'
+      shell_out!("dpkg -s chef-integration-test2") # its still 'installed'
     end
   end
 
@@ -333,7 +333,7 @@ describe Chef::Resource::DpkgPackage, :requires_root, :debian_family_only, arch:
       set_dpkg_package_name "chef-integration-test2"
       dpkg_package.run_action(action)
       expect(dpkg_package).to be_updated_by_last_action
-      should_be_purged_or_removed('chef-integration-test2', action)
+      should_be_purged_or_removed("chef-integration-test2", action)
     end
   end
 end