chef 15.8.23-universal-mingw32 → 15.9.17-universal-mingw32

data/spec/unit/data_collector_spec.rb

@@ -626,7 +626,7 @@ describe Chef::DataCollector do
         it "collects deprecation messages" do
           location = Chef::Log.caller_location
           events.deprecation(Chef::Deprecated.create(:internal_api, "deprecation warning", location))
-          expect_converge_message("deprecations" => [{ location: location, message: "deprecation warning", url: "https://docs.chef.io/deprecations_internal_api.html" }])
+          expect_converge_message("deprecations" => [{ location: location, message: "deprecation warning", url: "https://docs.chef.io/deprecations_internal_api/" }])
           send_run_failed_or_completed_event
         end
       end

data/spec/unit/deprecated_spec.rb

@@ -20,7 +20,7 @@ require "chef/deprecated"
 
 describe Chef::Deprecated do
   class TestDeprecation < Chef::Deprecated::Base
-    target 999, "test.html"
+    target 999, "test"
   end
 
   context "loading a deprecation class" do
@@ -44,11 +44,11 @@ describe Chef::Deprecated do
     let(:location) { "the location" }
 
     it "displays the full URL" do
-      expect(TestDeprecation.new.url).to eql("https://docs.chef.io/deprecations_test.html")
+      expect(TestDeprecation.new.url).to eql("https://docs.chef.io/deprecations_test/")
     end
 
     it "formats a complete deprecation message" do
-      expect(TestDeprecation.new(message, location).to_s).to eql("Deprecation CHEF-999 from the location\n\n  A test message\n\nPlease see https://docs.chef.io/deprecations_test.html for further details and information on how to correct this problem.")
+      expect(TestDeprecation.new(message, location).to_s).to eql("Deprecation CHEF-999 from the location\n\n  A test message\n\nPlease see https://docs.chef.io/deprecations_test/ for further details and information on how to correct this problem.")
     end
   end
 

data/spec/unit/encrypted_data_bag_item/check_encrypted_spec.rb

@@ -85,7 +85,7 @@ describe Chef::EncryptedDataBagItem::CheckEncrypted do
       end
     end
 
-    context "when encryption version is 3", :aes_256_gcm_only, ruby: "~> 2.0.0" do
+    context "when encryption version is 3", :aes_256_gcm_only do
       include_examples "encryption detected" do
         let(:version) { 3 }
         let(:encryptor) { Chef::EncryptedDataBagItem::Encryptor::Version3Encryptor }

data/spec/unit/encrypted_data_bag_item_spec.rb

@@ -97,7 +97,7 @@ describe Chef::EncryptedDataBagItem::Encryptor do
       Chef::Config[:data_bag_encrypt_version] = 3
     end
 
-    context "on supported platforms", :aes_256_gcm_only, ruby: "~> 2.0.0" do
+    context "on supported platforms", :aes_256_gcm_only do
 
       it "creates a version 3 encryptor" do
         expect(encryptor).to be_a_instance_of(Chef::EncryptedDataBagItem::Encryptor::Version3Encryptor)
@@ -166,7 +166,7 @@ describe Chef::EncryptedDataBagItem::Decryptor do
 
   context "when decrypting a version 3 (JSON+aes-256-gcm+random iv+auth tag) encrypted value" do
 
-    context "on supported platforms", :aes_256_gcm_only, ruby: "~> 2.0.0" do
+    context "on supported platforms", :aes_256_gcm_only do
 
       let(:encrypted_value) do
         Chef::EncryptedDataBagItem::Encryptor::Version3Encryptor.new(plaintext_data, encryption_key).for_encrypted_item

data/spec/unit/knife/core/windows_bootstrap_context_spec.rb

@@ -164,9 +164,9 @@ describe Chef::Knife::Core::WindowsBootstrapContext do
       expected = <<~EXPECTED
         echo.chef_server_url  "http://chef.example.com:4444"
         echo.validation_client_name "chef-validator-testing"
-        echo.file_cache_path   "c:/chef/cache"
-        echo.file_backup_path  "c:/chef/backup"
-        echo.cache_options     ^({:path =^> "c:/chef/cache/checksums", :skip_expires =^> true}^)
+        echo.file_cache_path   "C:/chef/cache"
+        echo.file_backup_path  "C:/chef/backup"
+        echo.cache_options     ^({:path =^> "C:/chef/cache/checksums", :skip_expires =^> true}^)
         echo.# Using default node name ^(fqdn^)
         echo.log_level :info
         echo.log_location       STDOUT

data/spec/unit/knife_spec.rb

@@ -418,13 +418,10 @@ describe Chef::Knife do
 
   describe "when first created" do
 
-    let(:knife) { KnifeSpecs::TestYourself.new(%w{with some args -s scrogramming}) }
-
-    before do
-      unless KnifeSpecs.const_defined?(:TestYourself)
-        Kernel.load(File.join(CHEF_SPEC_DATA, "knife_subcommand", "test_yourself.rb"))
-      end
-    end
+    let(:knife) {
+      Kernel.load "spec/data/knife_subcommand/test_yourself.rb"
+      KnifeSpecs::TestYourself.new(%w{with some args -s scrogramming})
+    }
 
     it "it parses the options passed to it" do
       expect(knife.config[:scro]).to eq("scrogramming")
@@ -435,6 +432,8 @@ describe Chef::Knife do
     end
 
     it "does not have lazy dependencies loaded" do
+      skip "unstable with randomization... prolly needs more isolation"
+
       expect(knife.class.test_deps_loaded).not_to be_truthy
     end
   end

data/spec/unit/mixin/shell_out_spec.rb

@@ -246,35 +246,33 @@ describe Chef::Mixin::ShellOut do
       let(:new_resource) { CustomResource.new("foo") }
       let(:provider) { new_resource.provider_for_action(:install) }
 
-      describe "on Chef-15", chef: ">= 15" do
-        %i{shell_out shell_out!}.each do |method|
-          stubbed_method = (method == :shell_out) ? :shell_out_compacted : :shell_out_compacted!
-          it "#{method} defaults to 900 seconds" do
-            expect(provider).to receive(stubbed_method).with("foo", timeout: 900)
-            provider.send(method, "foo")
-          end
-          it "#{method} overrides the default timeout with its options" do
-            expect(provider).to receive(stubbed_method).with("foo", timeout: 1)
-            provider.send(method, "foo", timeout: 1)
-          end
-          it "#{method} overrides the new_resource.timeout with the timeout option" do
-            new_resource.timeout(99)
-            expect(provider).to receive(stubbed_method).with("foo", timeout: 1)
-            provider.send(method, "foo", timeout: 1)
-          end
-          it "#{method} defaults to 900 seconds and preserves options" do
-            expect(provider).to receive(stubbed_method).with("foo", env: nil, timeout: 900)
-            provider.send(method, "foo", env: nil)
-          end
-          it "#{method} overrides the default timeout with its options and preserves options" do
-            expect(provider).to receive(stubbed_method).with("foo", timeout: 1, env: nil)
-            provider.send(method, "foo", timeout: 1, env: nil)
-          end
-          it "#{method} overrides the new_resource.timeout with the timeout option and preseves options" do
-            new_resource.timeout(99)
-            expect(provider).to receive(stubbed_method).with("foo", timeout: 1, env: nil)
-            provider.send(method, "foo", timeout: 1, env: nil)
-          end
+      %i{shell_out shell_out!}.each do |method|
+        stubbed_method = (method == :shell_out) ? :shell_out_compacted : :shell_out_compacted!
+        it "#{method} defaults to 900 seconds" do
+          expect(provider).to receive(stubbed_method).with("foo", timeout: 900)
+          provider.send(method, "foo")
+        end
+        it "#{method} overrides the default timeout with its options" do
+          expect(provider).to receive(stubbed_method).with("foo", timeout: 1)
+          provider.send(method, "foo", timeout: 1)
+        end
+        it "#{method} overrides the new_resource.timeout with the timeout option" do
+          new_resource.timeout(99)
+          expect(provider).to receive(stubbed_method).with("foo", timeout: 1)
+          provider.send(method, "foo", timeout: 1)
+        end
+        it "#{method} defaults to 900 seconds and preserves options" do
+          expect(provider).to receive(stubbed_method).with("foo", env: nil, timeout: 900)
+          provider.send(method, "foo", env: nil)
+        end
+        it "#{method} overrides the default timeout with its options and preserves options" do
+          expect(provider).to receive(stubbed_method).with("foo", timeout: 1, env: nil)
+          provider.send(method, "foo", timeout: 1, env: nil)
+        end
+        it "#{method} overrides the new_resource.timeout with the timeout option and preseves options" do
+          new_resource.timeout(99)
+          expect(provider).to receive(stubbed_method).with("foo", timeout: 1, env: nil)
+          provider.send(method, "foo", timeout: 1, env: nil)
         end
       end
     end

data/spec/unit/provider/file_spec.rb

@@ -17,6 +17,7 @@
 # limitations under the License.
 #
 
+require "spec_helper"
 require "support/shared/unit/provider/file"
 
 describe Chef::Provider::File do

data/spec/unit/resource/build_essential_spec.rb

@@ -1,5 +1,5 @@
 #
-# Copyright:: Copyright 2018, Chef Software, Inc.
+# Copyright:: Copyright 2018-2020, Chef Software, Inc.
 # License:: Apache License, Version 2.0
 #
 # Licensed under the Apache License, Version 2.0 (the "License");
@@ -19,7 +19,23 @@ require "spec_helper"
 
 describe Chef::Resource::BuildEssential do
 
-  let(:resource) { Chef::Resource::BuildEssential.new("foo") }
+  let(:node) { Chef::Node.new }
+  let(:events) { Chef::EventDispatch::Dispatcher.new }
+  let(:run_context) { Chef::RunContext.new(node, {}, events) }
+  let(:resource) { Chef::Resource::BuildEssential.new("foo", run_context) }
+  let(:provider) { resource.provider_for_action(:install) }
+
+  let(:softwareupdate_catalina_and_later) do
+    double("shell_out", exitstatus: 0, error!: nil, stdout: "Software Update Tool\n\nFinding available software\nSoftware Update found the following new or updated software:\n* Label: Command Line Tools for Xcode-11.0\n\tTitle: Command Line Tools for Xcode, Version: 11.0, Size: 224868K, Recommended: YES, \n")
+  end
+
+  let(:softwareupdate_catalina_and_later_no_cli) do
+    double("shell_out", exitstatus: 0, error!: nil, stdout: "Software Update Tool\n\nFinding available software\nSoftware Update found the following new or updated software:\n* Label: Chef Infra Client\n\tTitle: Chef Infra Client, Version: 17.0.208, Size: 224868K, Recommended: YES, \n")
+  end
+
+  let(:softwareupdate_pre_catalina) do
+    double("shell_out", exitstatus: 0, error!: nil, stdout: "Software Update Tool\n\nFinding available software\nSoftware Update found the following new or updated software:\n   * Command Line Tools (macOS High Sierra version 10.13) for Xcode-10.0\n")
+  end
 
   it "has a resource name of :build_essential" do
     expect(resource.resource_name).to eql(:build_essential)
@@ -40,4 +56,34 @@ describe Chef::Resource::BuildEssential do
       expect(resource.name).to eql("")
     end
   end
+
+  describe "#xcode_cli_installed?" do
+    it "returns true if the CLI is in the InstallHistory plist" do
+      allow(::File).to receive(:open).with("/Library/Receipts/InstallHistory.plist", "r").and_return(::File.join(::File.dirname(__FILE__), "data/InstallHistory_with_CLT.plist"))
+      expect(provider.xcode_cli_installed?).to eql(true)
+    end
+
+    it "returns false if the pkgutil doesn't list the package" do
+      allow(::File).to receive(:open).with("/Library/Receipts/InstallHistory.plist", "r").and_return(::File.join(::File.dirname(__FILE__), "data/InstallHistory_without_CLT.plist"))
+      expect(provider.xcode_cli_installed?).to eql(false)
+    end
+  end
+
+  describe "#xcode_cli_package_label" do
+    it "returns a package name on macOS < 10.15" do
+      allow(provider).to receive(:shell_out).with("softwareupdate", "--list").and_return(softwareupdate_pre_catalina)
+      expect(provider.xcode_cli_package_label).to eql("Command Line Tools (macOS High Sierra version 10.13) for Xcode-10.0")
+    end
+
+    it "returns a package name on macOS 10.15+" do
+      allow(provider).to receive(:shell_out).with("softwareupdate", "--list").and_return(softwareupdate_catalina_and_later)
+      expect(provider.xcode_cli_package_label).to eql("Command Line Tools for Xcode-11.0")
+    end
+
+    it "returns nil if no update is listed" do
+      allow(provider).to receive(:shell_out).with("softwareupdate", "--list").and_return(softwareupdate_catalina_and_later_no_cli)
+      expect(provider.xcode_cli_package_label).to be_nil
+    end
+
+  end
 end

data/spec/unit/resource/data/InstallHistory_with_CLT.plist

@@ -0,0 +1,92 @@
+<?xml version="1.0" encoding="UTF-8"?>
+<!DOCTYPE plist PUBLIC "-//Apple//DTD PLIST 1.0//EN" "http://www.apple.com/DTDs/PropertyList-1.0.dtd">
+<plist version="1.0">
+<array>
+	<dict>
+		<key>contentType</key>
+		<string>config-data</string>
+		<key>date</key>
+		<date>2019-10-07T20:09:37Z</date>
+		<key>displayName</key>
+		<string>XProtectPlistConfigData</string>
+		<key>displayVersion</key>
+		<string>2106</string>
+		<key>packageIdentifiers</key>
+		<array>
+			<string>com.apple.pkg.XProtectPlistConfigData_10_15.16U4081</string>
+		</array>
+		<key>processName</key>
+		<string>softwareupdated</string>
+	</dict>
+	<dict>
+		<key>contentType</key>
+		<string>config-data</string>
+		<key>date</key>
+		<date>2019-10-07T20:09:37Z</date>
+		<key>displayName</key>
+		<string>Gatekeeper Configuration Data</string>
+		<key>displayVersion</key>
+		<string>181</string>
+		<key>packageIdentifiers</key>
+		<array>
+			<string>com.apple.pkg.GatekeeperConfigData.16U1873</string>
+		</array>
+		<key>processName</key>
+		<string>softwareupdated</string>
+	</dict>
+	<dict>
+		<key>contentType</key>
+		<string>config-data</string>
+		<key>date</key>
+		<date>2019-10-07T20:09:37Z</date>
+		<key>displayName</key>
+		<string>MRTConfigData</string>
+		<key>displayVersion</key>
+		<string>1.50</string>
+		<key>packageIdentifiers</key>
+		<array>
+			<string>com.apple.pkg.MRTConfigData_10_15.16U4082</string>
+		</array>
+		<key>processName</key>
+		<string>softwareupdated</string>
+	</dict>
+	<dict>
+		<key>date</key>
+		<date>2019-10-09T02:37:33Z</date>
+		<key>displayName</key>
+		<string>Chef Infra Client</string>
+		<key>displayVersion</key>
+		<string></string>
+		<key>packageIdentifiers</key>
+		<array>
+			<string>com.getchef.pkg.chef</string>
+		</array>
+		<key>processName</key>
+		<string>installer</string>
+	</dict>
+	<dict>
+		<key>date</key>
+		<date>2019-10-09T02:47:02Z</date>
+		<key>displayName</key>
+		<string>Command Line Tools for Xcode</string>
+		<key>displayVersion</key>
+		<string>11.0</string>
+		<key>packageIdentifiers</key>
+		<array>
+			<string>com.apple.pkg.CLTools_Executables</string>
+			<string>com.apple.pkg.CLTools_SDK_macOS1015</string>
+			<string>com.apple.pkg.CLTools_SDK_macOS1014</string>
+			<string>com.apple.pkg.CLTools_macOS_SDK</string>
+			<string>com.apple.pkg.DevSDK</string>
+			<string>com.apple.pkg.DevSDK_OSX109</string>
+			<string>com.apple.pkg.DevSDK_OSX1010</string>
+			<string>com.apple.pkg.DevSDK_OSX1011</string>
+			<string>com.apple.pkg.DevSDK_OSX1012</string>
+			<string>com.apple.pkg.DevSDK_macOS1013_Public</string>
+			<string>com.apple.pkg.macOS_SDK_headers_for_macOS_10.14</string>
+		</array>
+		<key>processName</key>
+		<string>softwareupdated</string>
+	</dict>
+</array>
+</plist>

data/spec/unit/resource/data/InstallHistory_without_CLT.plist

@@ -0,0 +1,38 @@
+<?xml version="1.0" encoding="UTF-8"?>
+<!DOCTYPE plist PUBLIC "-//Apple//DTD PLIST 1.0//EN" "http://www.apple.com/DTDs/PropertyList-1.0.dtd">
+<plist version="1.0">
+<array>
+	<dict>
+		<key>contentType</key>
+		<string>config-data</string>
+		<key>date</key>
+		<date>2019-09-30T20:36:29Z</date>
+		<key>displayName</key>
+		<string>Gatekeeper Configuration Data</string>
+		<key>displayVersion</key>
+		<string>181</string>
+		<key>packageIdentifiers</key>
+		<array>
+			<string>com.apple.pkg.GatekeeperConfigData.16U1873</string>
+		</array>
+		<key>processName</key>
+		<string>softwareupdated</string>
+	</dict>
+	<dict>
+		<key>contentType</key>
+		<string>config-data</string>
+		<key>date</key>
+		<date>2019-09-30T20:36:29Z</date>
+		<key>displayName</key>
+		<string>MRTConfigData</string>
+		<key>displayVersion</key>
+		<string>1.49</string>
+		<key>packageIdentifiers</key>
+		<array>
+			<string>com.apple.pkg.MRTConfigData_10_15.16U4080</string>
+		</array>
+		<key>processName</key>
+		<string>softwareupdated</string>
+	</dict>
+</array>
+</plist>

data/spec/unit/resource/execute_spec.rb

@@ -109,7 +109,7 @@ describe Chef::Resource::Execute do
   shared_examples_for "a consumer of the Execute resource" do
     context "when running on Windows" do
       before do
-        allow(resource).to receive(:node).and_return({ platform_family: "windows" })
+        allow(resource).to receive(:windows?).and_return(true)
       end
 
       context "when no user, domain, or password is specified" do

data/spec/unit/resource/rhsm_register_spec.rb

@@ -179,6 +179,30 @@ describe Chef::Resource::RhsmRegister do
     end
   end
 
+  describe "#ca_consumer_package_source" do
+    let(:satellite_host) { "sat-host" }
+
+    before do
+      resource.satellite_host = satellite_host
+    end
+
+    context "when https_for_ca_consumer is true" do
+      before { resource.https_for_ca_consumer true }
+
+      it "returns url with https" do
+        expect(provider.ca_consumer_package_source).to eq("https://#{satellite_host}/pub/katello-ca-consumer-latest.noarch.rpm")
+      end
+    end
+
+    context "when https_for_ca_consumer is false" do
+      before { resource.https_for_ca_consumer false }
+
+      it "returns url with http" do
+        expect(provider.ca_consumer_package_source).to eq("http://#{satellite_host}/pub/katello-ca-consumer-latest.noarch.rpm")
+      end
+    end
+  end
+
   describe "#registered_with_rhsm?" do
     let(:cmd) { double("cmd") }
 

data/spec/unit/resource/windows_share_spec.rb

@@ -38,9 +38,11 @@ describe Chef::Resource::WindowsShare do
   end
 
   it "coerces path to a single path separator format" do
-    resource.path("C:/chef")
+    resource.path("C:/chef".freeze)
     expect(resource.path).to eql("C:\\chef")
     resource.path("C:\\chef")
     expect(resource.path).to eql("C:\\chef")
+    resource.path("C:/chef".dup)
+    expect(resource.path).to eql("C:\\chef")
   end
 end

data/tasks/rspec.rb

@@ -82,6 +82,7 @@ begin
     %i{unit functional integration stress}.each do |sub|
       desc "Run the specs under spec/#{sub}"
       RSpec::Core::RakeTask.new(sub) do |t|
+        t.verbose = false
         t.rspec_opts = %w{--profile}
         t.pattern = FileList["spec/#{sub}/**/*_spec.rb"]
       end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: chef
 version: !ruby/object:Gem::Version
-  version: 15.8.23
+  version: 15.9.17
 platform: universal-mingw32
 authors:
 - Adam Jacob
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2020-02-07 00:00:00.000000000 Z
+date: 2020-04-03 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: chef-config
@@ -16,28 +16,28 @@ dependencies:
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 15.8.23
+        version: 15.9.17
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 15.8.23
+        version: 15.9.17
 - !ruby/object:Gem::Dependency
   name: chef-utils
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 15.8.23
+        version: 15.9.17
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 15.8.23
+        version: 15.9.17
 - !ruby/object:Gem::Dependency
   name: train-core
   requirement: !ruby/object:Gem::Requirement
@@ -714,6 +714,7 @@ files:
 - distro/ruby_bin_folder/Newtonsoft.Json.dll
 - distro/ruby_bin_folder/msvcp140.dll
 - distro/ruby_bin_folder/vcruntime140.dll
+- distro/templates/powershell/chef/chef.psm1.erb
 - ext/win32-eventlog/Rakefile
 - ext/win32-eventlog/chef-log.man.erb
 - lib/chef.rb
@@ -2623,6 +2624,8 @@ files:
 - spec/unit/resource/cron_d_spec.rb
 - spec/unit/resource/cron_spec.rb
 - spec/unit/resource/csh_spec.rb
+- spec/unit/resource/data/InstallHistory_with_CLT.plist
+- spec/unit/resource/data/InstallHistory_without_CLT.plist
 - spec/unit/resource/directory_spec.rb
 - spec/unit/resource/dmg_package_spec.rb
 - spec/unit/resource/dnf_package_spec.rb