chef 12.6.0 → 12.7.2

data/spec/integration/recipes/resource_converge_if_changed_spec.rb

@@ -1,4 +1,4 @@
-require 'support/shared/integration/integration_helper'
+require "support/shared/integration/integration_helper"
 
 describe "Resource::ActionClass#converge_if_changed" do
   include IntegrationSupport
@@ -22,11 +22,12 @@ describe "Resource::ActionClass#converge_if_changed" do
     let(:resource_class) {
       result = Class.new(Chef::Resource) do
         def self.to_s; resource_name; end
+
         def self.inspect; resource_name.inspect; end
-        property :identity1, identity: true, default: 'default_identity1'
-        property :control1, desired_state: false, default: 'default_control1'
-        property :state1, default: 'default_state1'
-        property :state2, default: 'default_state2'
+        property :identity1, identity: true, default: "default_identity1"
+        property :control1, desired_state: false, default: "default_control1"
+        property :state1, default: "default_state1"
+        property :state2, default: "default_state2"
         attr_accessor :converged
         def initialize(*args)
           super
@@ -51,8 +52,8 @@ describe "Resource::ActionClass#converge_if_changed" do
       context "and current_resource with state1=current, state2=current" do
         before :each do
           resource_class.load_current_value do
-            state1 'current_state1'
-            state2 'current_state2'
+            state1 "current_state1"
+            state2 "current_state2"
           end
         end
 
@@ -198,8 +199,8 @@ EOM
       context "and current_resource with identity1=current, control1=current" do
         before :each do
           resource_class.load_current_value do
-            identity1 'current_identity1'
-            control1 'current_control1'
+            identity1 "current_identity1"
+            control1 "current_control1"
           end
         end
 
@@ -314,8 +315,8 @@ EOM
       context "and current_resource with state1=current, state2=current" do
         before :each do
           resource_class.load_current_value do
-            state1 'current_state1'
-            state2 'current_state2'
+            state1 "current_state1"
+            state2 "current_state2"
           end
         end
 

data/spec/integration/recipes/resource_load_spec.rb

@@ -1,4 +1,4 @@
-require 'support/shared/integration/integration_helper'
+require "support/shared/integration/integration_helper"
 
 describe "Resource.load_current_value" do
   include IntegrationSupport
@@ -21,11 +21,13 @@ describe "Resource.load_current_value" do
   let(:resource_class) {
     result = Class.new(Chef::Resource) do
       def self.to_s; resource_name; end
+
       def self.inspect; resource_name.inspect; end
       property :x, default: lazy { "default #{Namer.incrementing_value}" }
       def self.created_x=(value)
         @created = value
       end
+
       def self.created_x
         @created
       end
@@ -43,9 +45,9 @@ describe "Resource.load_current_value" do
   context "with a resource with load_current_value" do
     before :each do
       resource_class.load_current_value do
-        x "loaded #{Namer.incrementing_value} (#{self.class.properties.sort_by { |name,p| name }.
-          select { |name,p| p.is_set?(self) }.
-          map { |name,p| "#{name}=#{p.get(self)}" }.
+        x "loaded #{Namer.incrementing_value} (#{self.class.properties.sort_by { |name, p| name }.
+          select { |name, p| p.is_set?(self) }.
+          map { |name, p| "#{name}=#{p.get(self)}" }.
           join(", ") })"
       end
     end
@@ -55,29 +57,29 @@ describe "Resource.load_current_value" do
         e = self
         r = nil
         converge {
-          r = public_send(e.resource_name, 'blah') do
-            x 'desired'
+          r = public_send(e.resource_name, "blah") do
+            x "desired"
           end
         }
         r
       end
 
       it "current_resource is passed name but not x" do
-        expect(resource.current_value.x).to eq 'loaded 2 (name=blah)'
+        expect(resource.current_value.x).to eq "loaded 2 (name=blah)"
       end
 
       it "resource.current_value returns a different resource" do
-        expect(resource.current_value.x).to eq 'loaded 2 (name=blah)'
-        expect(resource.x).to eq 'desired'
+        expect(resource.current_value.x).to eq "loaded 2 (name=blah)"
+        expect(resource.x).to eq "desired"
       end
 
       it "resource.current_value constructs the resource anew each time" do
-        expect(resource.current_value.x).to eq 'loaded 2 (name=blah)'
-        expect(resource.current_value.x).to eq 'loaded 3 (name=blah)'
+        expect(resource.current_value.x).to eq "loaded 2 (name=blah)"
+        expect(resource.current_value.x).to eq "loaded 3 (name=blah)"
       end
 
       it "the provider accesses the current value of x" do
-        expect(resource.class.created_x).to eq 'desired'
+        expect(resource.class.created_x).to eq "desired"
       end
 
       context "and identity: :i and :d with desired_state: false" do
@@ -89,12 +91,12 @@ describe "Resource.load_current_value" do
         }
 
         before {
-          resource.i 'desired_i'
-          resource.d 'desired_d'
+          resource.i "desired_i"
+          resource.d "desired_d"
         }
 
         it "i, name and d are passed to load_current_value, but not x" do
-          expect(resource.current_value.x).to eq 'loaded 2 (d=desired_d, i=desired_i, name=blah)'
+          expect(resource.current_value.x).to eq "loaded 2 (d=desired_d, i=desired_i, name=blah)"
         end
       end
 
@@ -107,12 +109,12 @@ describe "Resource.load_current_value" do
         }
 
         before {
-          resource.i 'desired_i'
-          resource.d 'desired_d'
+          resource.i "desired_i"
+          resource.d "desired_d"
         }
 
         it "i, name and d are passed to load_current_value, but not x" do
-          expect(resource.current_value.x).to eq 'loaded 2 (d=desired_d, i=desired_i, name=blah)'
+          expect(resource.current_value.x).to eq "loaded 2 (d=desired_d, i=desired_i, name=blah)"
         end
       end
     end
@@ -122,14 +124,14 @@ describe "Resource.load_current_value" do
         e = self
         r = nil
         converge {
-          r = public_send(e.resource_name, 'blah') do
+          r = public_send(e.resource_name, "blah") do
           end
         }
         r
       end
 
       it "the provider accesses values from load_current_value" do
-        expect(resource.class.created_x).to eq 'loaded 1 (name=blah)'
+        expect(resource.class.created_x).to eq "loaded 1 (name=blah)"
       end
     end
 
@@ -151,8 +153,8 @@ describe "Resource.load_current_value" do
       e = self
       r = nil
       converge {
-        r = public_send(e.subresource_name, 'blah') do
-          x 'desired'
+        r = public_send(e.subresource_name, "blah") do
+          x "desired"
         end
       }
       r
@@ -160,7 +162,7 @@ describe "Resource.load_current_value" do
 
     context "and a child resource class with no load_current_value" do
       it "the parent load_current_value is used" do
-        expect(subresource.current_value.x).to eq 'loaded 2 (name=blah)'
+        expect(subresource.current_value.x).to eq "loaded 2 (name=blah)"
       end
       it "load_current_value yields a copy of the child class" do
         expect(subresource.current_value).to be_kind_of(subresource_class)
@@ -170,17 +172,17 @@ describe "Resource.load_current_value" do
     context "And a child resource class with load_current_value" do
       before {
         subresource_class.load_current_value do
-          y "loaded_y #{Namer.incrementing_value} (#{self.class.properties.sort_by { |name,p| name }.
-            select { |name,p| p.is_set?(self) }.
-            map { |name,p| "#{name}=#{p.get(self)}" }.
+          y "loaded_y #{Namer.incrementing_value} (#{self.class.properties.sort_by { |name, p| name }.
+            select { |name, p| p.is_set?(self) }.
+            map { |name, p| "#{name}=#{p.get(self)}" }.
             join(", ") })"
         end
       }
 
       it "the overridden load_current_value is used" do
         current_resource = subresource.current_value
-        expect(current_resource.x).to eq 'default 3'
-        expect(current_resource.y).to eq 'loaded_y 2 (name=blah)'
+        expect(current_resource.x).to eq "default 3"
+        expect(current_resource.y).to eq "loaded_y 2 (name=blah)"
       end
     end
 
@@ -188,17 +190,17 @@ describe "Resource.load_current_value" do
       before {
         subresource_class.load_current_value do
           super()
-          y "loaded_y #{Namer.incrementing_value} (#{self.class.properties.sort_by { |name,p| name }.
-            select { |name,p| p.is_set?(self) }.
-            map { |name,p| "#{name}=#{p.get(self)}" }.
+          y "loaded_y #{Namer.incrementing_value} (#{self.class.properties.sort_by { |name, p| name }.
+            select { |name, p| p.is_set?(self) }.
+            map { |name, p| "#{name}=#{p.get(self)}" }.
             join(", ") })"
         end
       }
 
       it "the original load_current_value is called as well as the child one" do
         current_resource = subresource.current_value
-        expect(current_resource.x).to eq 'loaded 3 (name=blah)'
-        expect(current_resource.y).to eq 'loaded_y 4 (name=blah, x=loaded 3 (name=blah))'
+        expect(current_resource.x).to eq "loaded 3 (name=blah)"
+        expect(current_resource.y).to eq "loaded_y 4 (name=blah, x=loaded 3 (name=blah))"
       end
     end
   end

data/spec/integration/solo/solo_spec.rb

@@ -1,9 +1,9 @@
-require 'support/shared/integration/integration_helper'
-require 'chef/mixin/shell_out'
-require 'chef/run_lock'
-require 'chef/config'
-require 'timeout'
-require 'fileutils'
+require "support/shared/integration/integration_helper"
+require "chef/mixin/shell_out"
+require "chef/run_lock"
+require "chef/config"
+require "timeout"
+require "fileutils"
 
 describe "chef-solo" do
   include IntegrationSupport
@@ -19,12 +19,12 @@ describe "chef-solo" do
 
   when_the_repository "has a cookbook with a basic recipe" do
     before do
-      file 'cookbooks/x/metadata.rb', cookbook_x_100_metadata_rb
-      file 'cookbooks/x/recipes/default.rb', 'puts "ITWORKS"'
+      file "cookbooks/x/metadata.rb", cookbook_x_100_metadata_rb
+      file "cookbooks/x/recipes/default.rb", 'puts "ITWORKS"'
     end
 
     it "should complete with success" do
-      file 'config/solo.rb', <<EOM
+      file "config/solo.rb", <<EOM
 cookbook_path "#{path_to('cookbooks')}"
 file_cache_path "#{path_to('config/cache')}"
 EOM
@@ -34,12 +34,12 @@ EOM
     end
 
     it "should evaluate its node.json file" do
-      file 'config/solo.rb', <<EOM
+      file "config/solo.rb", <<EOM
 cookbook_path "#{path_to('cookbooks')}"
 file_cache_path "#{path_to('config/cache')}"
 EOM
 
-      file 'config/node.json',<<-E
+      file "config/node.json", <<-E
 {"run_list":["x::default"]}
 E
 
@@ -52,15 +52,15 @@ E
 
   when_the_repository "has a cookbook with an undeclared dependency" do
     before do
-      file 'cookbooks/x/metadata.rb', cookbook_x_100_metadata_rb
-      file 'cookbooks/x/recipes/default.rb', 'include_recipe "ancient::aliens"'
+      file "cookbooks/x/metadata.rb", cookbook_x_100_metadata_rb
+      file "cookbooks/x/recipes/default.rb", 'include_recipe "ancient::aliens"'
 
-      file 'cookbooks/ancient/metadata.rb', cookbook_ancient_100_metadata_rb
-      file 'cookbooks/ancient/recipes/aliens.rb', 'print "it was aliens"'
+      file "cookbooks/ancient/metadata.rb", cookbook_ancient_100_metadata_rb
+      file "cookbooks/ancient/recipes/aliens.rb", 'print "it was aliens"'
     end
 
     it "should exit with an error" do
-      file 'config/solo.rb', <<EOM
+      file "config/solo.rb", <<EOM
 cookbook_path "#{path_to('cookbooks')}"
 file_cache_path "#{path_to('config/cache')}"
 EOM
@@ -72,9 +72,9 @@ EOM
 
   when_the_repository "has a cookbook with an incompatible chef_version" do
     before do
-      file 'cookbooks/x/metadata.rb', cb_metadata('x', '1.0.0', "\nchef_version '~> 999.0'")
-      file 'cookbooks/x/recipes/default.rb', 'puts "ITWORKS"'
-      file 'config/solo.rb', <<EOM
+      file "cookbooks/x/metadata.rb", cb_metadata("x", "1.0.0", "\nchef_version '~> 999.0'")
+      file "cookbooks/x/recipes/default.rb", 'puts "ITWORKS"'
+      file "config/solo.rb", <<EOM
 cookbook_path "#{path_to('cookbooks')}"
 file_cache_path "#{path_to('config/cache')}"
 EOM
@@ -89,9 +89,9 @@ EOM
 
   when_the_repository "has a cookbook with an incompatible ohai_version" do
     before do
-      file 'cookbooks/x/metadata.rb', cb_metadata('x', '1.0.0', "\nohai_version '~> 999.0'")
-      file 'cookbooks/x/recipes/default.rb', 'puts "ITWORKS"'
-      file 'config/solo.rb', <<EOM
+      file "cookbooks/x/metadata.rb", cb_metadata("x", "1.0.0", "\nohai_version '~> 999.0'")
+      file "cookbooks/x/recipes/default.rb", 'puts "ITWORKS"'
+      file "config/solo.rb", <<EOM
 cookbook_path "#{path_to('cookbooks')}"
 file_cache_path "#{path_to('config/cache')}"
 EOM
@@ -104,13 +104,12 @@ EOM
     end
   end
 
-
   when_the_repository "has a cookbook with a recipe with sleep" do
     before do
-      directory 'logs'
-      file 'logs/runs.log', ''
-      file 'cookbooks/x/metadata.rb', cookbook_x_100_metadata_rb
-      file 'cookbooks/x/recipes/default.rb', <<EOM
+      directory "logs"
+      file "logs/runs.log", ""
+      file "cookbooks/x/metadata.rb", cookbook_x_100_metadata_rb
+      file "cookbooks/x/recipes/default.rb", <<EOM
 ruby_block "sleeping" do
   block do
     sleep 5
@@ -120,7 +119,7 @@ EOM
     end
 
     it "while running solo concurrently" do
-      file 'config/solo.rb', <<EOM
+      file "config/solo.rb", <<EOM
 cookbook_path "#{path_to('cookbooks')}"
 file_cache_path "#{path_to('config/cache')}"
 EOM
@@ -149,15 +148,15 @@ EOM
       # Unfortunately file / directory helpers in integration tests
       # are implemented using before(:each) so we need to do all below
       # checks in one example.
-      run_log = File.read(path_to('logs/runs.log'))
+      run_log = File.read(path_to("logs/runs.log"))
 
       # both of the runs should succeed
-      expect(run_log.lines.reject {|l| !l.include? "INFO: Chef Run complete in"}.length).to eq(2)
+      expect(run_log.lines.reject { |l| !l.include? "INFO: Chef Run complete in" }.length).to eq(2)
 
       # second run should have a message which indicates it's waiting for the first run
-      pid_lines = run_log.lines.reject {|l| !l.include? "Chef-client pid:"}
+      pid_lines = run_log.lines.reject { |l| !l.include? "Chef-client pid:" }
       expect(pid_lines.length).to eq(2)
-      pids = pid_lines.map {|l| l.split(" ").last}
+      pids = pid_lines.map { |l| l.split(" ").last }
       expect(run_log).to include("Chef client #{pids[0]} is running, will wait for it to finish and then run.")
 
       # second run should start after first run ends

data/spec/scripts/ssl-serve.rb

@@ -5,11 +5,10 @@
 # You can use it to test various HTTP behaviors in chef, like chef-client's
 # `-j` and `-c` options and remote_file with https connections.
 #
-require 'pp'
-require 'openssl'
-require 'webrick'
-require 'webrick/https'
-
+require "pp"
+require "openssl"
+require "webrick"
+require "webrick/https"
 
 $ssl = true
 
@@ -21,23 +20,21 @@ key = OpenSSL::PKey::RSA.new(key_text)
 
 server_opts = {}
 if $ssl
-server_opts.merge!( { :SSLEnable => true,
-                :SSLVerifyClient => OpenSSL::SSL::VERIFY_NONE,
-                :SSLCertificate => cert,
-                :SSLPrivateKey => key })
+  server_opts.merge!( { :SSLEnable => true,
+                        :SSLVerifyClient => OpenSSL::SSL::VERIFY_NONE,
+                        :SSLCertificate => cert,
+                        :SSLPrivateKey => key })
 end
 
-
-
 # 5 == debug, 3 == warning
 LOGGER = WEBrick::Log.new(STDOUT, 5)
 DEFAULT_OPTIONS = {
-  :server => 'webrick',
+  :server => "webrick",
   :Port => 9000,
-  :Host => 'localhost',
+  :Host => "localhost",
   :environment => :none,
   :Logger => LOGGER,
-  :DocumentRoot => File.expand_path("/tmp/chef-118-sampledata")
+  :DocumentRoot => File.expand_path("#{Dir.tmpdir}/chef-118-sampledata")
   #:AccessLog => [] # Remove this option to enable the access log when debugging.
 }
 
@@ -45,8 +42,6 @@ webrick_opts = DEFAULT_OPTIONS.merge(server_opts)
 pp :webrick_opts => webrick_opts
 
 server = WEBrick::HTTPServer.new(webrick_opts)
-trap 'INT' do server.shutdown end
+trap "INT" do server.shutdown end
 
 server.start
-
-

data/spec/spec_helper.rb

@@ -1,6 +1,6 @@
 #
-# Author:: Adam Jacob (<adam@opscode.com>)
-# Copyright:: Copyright (c) 2008-2015 Chef Software, Inc.
+# Author:: Adam Jacob (<adam@chef.io>)
+# Copyright:: Copyright 2008-2016, Chef Software, Inc.
 # License:: Apache License, Version 2.0
 #
 # Licensed under the Apache License, Version 2.0 (the "License");
@@ -27,16 +27,15 @@ end
 # Ruby 1.9 Compat
 $:.unshift File.expand_path("../..", __FILE__)
 
-
-require 'rubygems'
-require 'rspec/mocks'
+require "rubygems"
+require "rspec/mocks"
 
 $:.unshift(File.join(File.dirname(__FILE__), "..", "lib"))
 $:.unshift(File.expand_path("../lib", __FILE__))
 $:.unshift(File.dirname(__FILE__))
 
 if ENV["COVERAGE"]
-  require 'simplecov'
+  require "simplecov"
   SimpleCov.start do
     add_filter "/spec/"
     add_group "Remote File", "remote_file"
@@ -46,34 +45,38 @@ if ENV["COVERAGE"]
   end
 end
 
-require 'chef'
-require 'chef/knife'
+require "chef"
+require "chef/knife"
+
+Dir["lib/chef/knife/**/*.rb"].
+  map { |f| f.gsub("lib/", "") }.
+  map { |f| f.gsub(%r{\.rb$}, "") }.
+  each { |f| require f }
 
-Dir['lib/chef/knife/**/*.rb'].
-  map {|f| f.gsub('lib/', '') }.
-  map {|f| f.gsub(%r[\.rb$], '') }.
-  each {|f| require f }
+require "chef/resource_resolver"
+require "chef/provider_resolver"
 
-require 'chef/resource_resolver'
-require 'chef/provider_resolver'
+require "chef/mixins"
+require "chef/dsl"
+require "chef/application"
+require "chef/applications"
 
-require 'chef/mixins'
-require 'chef/dsl'
-require 'chef/application'
-require 'chef/applications'
+require "chef/shell"
+require "chef/util/file_edit"
 
-require 'chef/shell'
-require 'chef/util/file_edit'
+require "chef/config"
 
-require 'chef/config'
+if ENV["CHEF_FIPS"] == "1"
+  Chef::Config.init_openssl
+end
 
 # If you want to load anything into the testing environment
 # without versioning it, add it to spec/support/local_gems.rb
-require 'spec/support/local_gems.rb' if File.exists?(File.join(File.dirname(__FILE__), 'support', 'local_gems.rb'))
+require "spec/support/local_gems.rb" if File.exists?(File.join(File.dirname(__FILE__), "support", "local_gems.rb"))
 
 # Explicitly require spec helpers that need to load first
-require 'spec/support/platform_helpers'
-require 'spec/support/shared/unit/mock_shellout'
+require "spec/support/platform_helpers"
+require "spec/support/shared/unit/mock_shellout"
 
 # Autoloads support files
 # Excludes support/platforms by default
@@ -81,23 +84,23 @@ require 'spec/support/shared/unit/mock_shellout'
 Dir["spec/support/**/*.rb"].
   reject { |f| f =~ %r{^spec/support/platforms} }.
   reject { |f| f =~ %r{^spec/support/pedant} }.
-  map { |f| f.gsub(%r{.rb$}, '') }.
-  map { |f| f.gsub(%r[spec/], '')}.
+  map { |f| f.gsub(%r{.rb$}, "") }.
+  map { |f| f.gsub(%r{spec/}, "") }.
   each { |f| require f }
 
 OHAI_SYSTEM = Ohai::System.new
-OHAI_SYSTEM.all_plugins("platform")
+OHAI_SYSTEM.all_plugins(["platform", "hostname", "languages/powershell"])
 
 test_node = Chef::Node.new
-test_node.automatic['os'] = (OHAI_SYSTEM['os'] || 'unknown_os').dup.freeze
-test_node.automatic['platform_family'] = (OHAI_SYSTEM['platform_family'] || 'unknown_platform_family').dup.freeze
-test_node.automatic['platform'] = (OHAI_SYSTEM['platform'] || 'unknown_platform').dup.freeze
-test_node.automatic['platform_version'] = (OHAI_SYSTEM['platform_version'] || 'unknown_platform_version').dup.freeze
+test_node.automatic["os"] = (OHAI_SYSTEM["os"] || "unknown_os").dup.freeze
+test_node.automatic["platform_family"] = (OHAI_SYSTEM["platform_family"] || "unknown_platform_family").dup.freeze
+test_node.automatic["platform"] = (OHAI_SYSTEM["platform"] || "unknown_platform").dup.freeze
+test_node.automatic["platform_version"] = (OHAI_SYSTEM["platform_version"] || "unknown_platform_version").dup.freeze
 TEST_NODE = test_node.freeze
-TEST_OS = TEST_NODE['os']
-TEST_PLATFORM = TEST_NODE['platform']
-TEST_PLATFORM_VERSION = TEST_NODE['platform_version']
-TEST_PLATFORM_FAMILY = TEST_NODE['platform_family']
+TEST_OS = TEST_NODE["os"]
+TEST_PLATFORM = TEST_NODE["platform"]
+TEST_PLATFORM_VERSION = TEST_NODE["platform_version"]
+TEST_PLATFORM_FAMILY = TEST_NODE["platform_family"]
 
 RSpec.configure do |config|
   config.include(Matchers)
@@ -127,10 +130,11 @@ RSpec.configure do |config|
 
   config.filter_run_excluding :windows_only => true unless windows?
   config.filter_run_excluding :not_supported_on_mac_osx_106 => true if mac_osx_106?
-  config.filter_run_excluding :not_supported_on_mac_osx=> true if mac_osx?
-  config.filter_run_excluding :mac_osx_only=> true if !mac_osx?
+  config.filter_run_excluding :not_supported_on_mac_osx => true if mac_osx?
+  config.filter_run_excluding :mac_osx_only => true if !mac_osx?
   config.filter_run_excluding :not_supported_on_win2k3 => true if windows_win2k3?
   config.filter_run_excluding :not_supported_on_solaris => true if solaris?
+  config.filter_run_excluding :not_supported_on_gce => true if gce?
   config.filter_run_excluding :not_supported_on_nano => true if windows_nano_server?
   config.filter_run_excluding :win2k3_only => true unless windows_win2k3?
   config.filter_run_excluding :windows_2008r2_or_later => true unless windows_2008r2_or_later?
@@ -164,6 +168,7 @@ RSpec.configure do |config|
   config.filter_run_excluding :aes_256_gcm_only => true unless aes_256_gcm?
   config.filter_run_excluding :broken => true
   config.filter_run_excluding :not_wpar => true unless wpar?
+  config.filter_run_excluding :not_supported_under_fips => true if fips?
 
   running_platform_arch = `uname -m`.strip unless windows?
 
@@ -179,7 +184,7 @@ RSpec.configure do |config|
     node = TEST_NODE.dup
     resource_class = Chef::ResourceResolver.resolve(type, node: node)
     if resource_class
-      resource = resource_class.new('test', Chef::RunContext.new(node, nil, nil))
+      resource = resource_class.new("test", Chef::RunContext.new(node, nil, nil))
       begin
         provider = resource.provider_for_action(Array(resource_class.default_action).first)
         provider.class != target_provider
@@ -202,20 +207,20 @@ RSpec.configure do |config|
     Chef::Config.treat_deprecation_warnings_as_errors(true)
 
     # Set environment variable so the setting persists in child processes
-    ENV['CHEF_TREAT_DEPRECATION_WARNINGS_AS_ERRORS'] = "1"
+    ENV["CHEF_TREAT_DEPRECATION_WARNINGS_AS_ERRORS"] = "1"
   end
 
   # raise if anyone commits any test to CI with :focus set on it
   config.before(:example, :focus) do
-    raise 'This example was committed with `:focus` and should not have been'
-  end if ENV['CI']
+    raise "This example was committed with `:focus` and should not have been"
+  end if ENV["CI"]
 
   config.before(:suite) do
     ARGV.clear
   end
 end
 
-require 'webrick/utils'
+require "webrick/utils"
 
 #    Webrick uses a centralized/synchronized timeout manager. It works by
 #    starting a thread to check for timeouts on an interval. The timeout
@@ -241,4 +246,4 @@ module WEBrick
 end
 
 # Enough stuff needs json serialization that I'm just adding it here for equality asserts
-require 'chef/json_compat'
+require "chef/json_compat"