test-kitchen 2.11.1 → 3.1.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 42e12936f14dc051e00302c3da2d199ec7b17ef5610a51e0cccbc1bd1dc2c9de
-  data.tar.gz: 935202057c42bcd2ee9b729f0d13d99767fa9c280b1f5511b1364d173d0c4719
+  metadata.gz: fc5e05dc50e7052b2002a94e950f5691d953c78d5e408b80a0f7f977ab63eb35
+  data.tar.gz: 43bc1979cc77277d4a3219f82d8fead18e7aa3cf068fb7484f5176ce422c8767
 SHA512:
-  metadata.gz: 2f8ed93cccb5f7cd8558874c5b86bce33aedcfa93abf146e8748dcb7dc0aae3d8f42a0f220c3b9f6753e90843d3c6a974934d6d467840b7a722a133b87d58024
-  data.tar.gz: 4b3977b0657b7832d9f47fcd34c4e27cc5bd6f50596382e87b4e3214442ee13d23f3f6ec0f066776c103d067d28afd8aaa771df27ea98ec7495d8df21ffd6965
+  metadata.gz: 12453b3cbf1ef1dfb49b683667fca52502b10a888e9cf953d58c22589ce99dffc9864d550463c80341049f7073aa3c3fb98eb76c4e432aa266956b6a0a86c332
+  data.tar.gz: bc19f713f60acc0b5437c5b29ab241255a7a82de2b206ba714596bf2a251c0e9a3c21bc152a18e171461ca24d9291098b55baeec474e495f46824782487350b8

data/Gemfile

@@ -11,11 +11,11 @@ group :integration do
 end
 
 group :debug do
-  gem "pry"
+  gem "pry", "~>0.12"
   gem "pry-byebug"
   gem "pry-stack_explorer"
 end
 
 group :chefstyle do
-  gem "chefstyle", "1.6.2"
+  gem "chefstyle", "2.1.0"
 end

data/Rakefile

@@ -11,7 +11,7 @@ begin
   require "cucumber"
   require "cucumber/rake/task"
   Cucumber::Rake::Task.new(:features) do |t|
-    t.cucumber_opts = ["features", "-x", "--format progress", "--no-color", "--tags ~@ignore"]
+    t.cucumber_opts = ["features", "-x", "--format progress", "--no-color", "--tags 'not @ignore'"]
   end
 rescue LoadError
   puts "cucumber is not available. (sudo) gem install cucumber to run tests."

data/lib/kitchen/cli.rb

@@ -285,7 +285,7 @@ module Kitchen
       perform("exec", "exec", args)
     end
 
-    desc "version", "Print Kitchen's version information"
+    desc "version", "Print Test Kitchen's version information"
     def version
       puts "Test Kitchen version #{Kitchen::VERSION}"
     end
@@ -296,7 +296,7 @@ module Kitchen
       perform("sink", "sink")
     end
 
-    desc "console", "Kitchen Console!"
+    desc "console", "Test Kitchen Console!"
     def console
       perform("console", "console")
     end

data/lib/kitchen/command/console.rb

@@ -43,7 +43,7 @@ module Kitchen
       def prompt(char)
         proc do |target_self, nest_level, pry|
           [
-            "[#{pry.input_array.size}] ",
+            "[#{pry.input_ring.size}] ",
             "kc(#{Pry.view_clip(target_self.class)})",
             "#{":#{nest_level}" unless nest_level == 0}#{char} ",
           ].join

data/lib/kitchen/configurable.rb

@@ -149,7 +149,7 @@ module Kitchen
     # @return [String] joined path for instance's os_type
     def remote_path_join(*parts)
       path = File.join(*parts)
-      windows_os? ? path.tr("/", '\\') : path.tr('\\', "/")
+      windows_os? ? path.tr("/", "\\") : path.tr("\\", "/")
     end
 
     # @return [TrueClass,FalseClass] true if `:os_type` is `"unix"` (or

data/lib/kitchen/data_munger.rb

@@ -720,12 +720,14 @@ module Kitchen
         move_data_to!(:provisioner, suite, :attributes)
         move_data_to!(:provisioner, suite, :run_list)
         move_data_to!(:provisioner, suite, :named_run_list)
+        move_data_to!(:provisioner, suite, :policy_group)
       end
 
       data.fetch(:platforms, []).each do |platform|
         move_data_to!(:provisioner, platform, :attributes)
         move_data_to!(:provisioner, platform, :run_list)
         move_data_to!(:provisioner, platform, :named_run_list)
+        move_data_to!(:provisioner, platform, :policy_group)
       end
     end
 

data/lib/kitchen/lifecycle_hook/remote.rb

@@ -18,8 +18,14 @@ module Kitchen
           end
         end
 
-        conn = instance.transport.connection(state_file.read)
-        conn.execute(command)
+        begin
+          conn = instance.transport.connection(state_file.read)
+          conn.execute(command)
+        rescue Kitchen::Transport::SshFailed => e
+          return if hook[:skippable] && e.message.match(/^SSH exited \(\d{1,3}\) for command: \[.+\]$/)
+
+          raise
+        end
       end
 
       private

data/lib/kitchen/logger.rb

@@ -293,11 +293,11 @@ module Kitchen
       logger = StdoutLogger.new(stdout)
       if colorize
         logger.formatter = proc do |_severity, _datetime, _progname, msg|
-          Color.colorize(msg.to_s, color).concat("\n")
+          Color.colorize(msg.dup.to_s, color).concat("\n")
         end
       else
         logger.formatter = proc do |_severity, _datetime, _progname, msg|
-          msg.concat("\n")
+          msg.dup.concat("\n")
         end
       end
       logger

data/lib/kitchen/provisioner/base.rb

@@ -177,11 +177,9 @@ module Kitchen
       # @raise [ClientError] if the sandbox directory has no yet been created
       #   by calling `#create_sandbox`
       def sandbox_path
-        @sandbox_path ||= begin
-          raise ClientError, "Sandbox directory has not yet " \
+        @sandbox_path ||= raise ClientError, "Sandbox directory has not yet " \
           "been created. Please run #{self.class}#create_sandox before " \
           "trying to access the path."
-        end
       end
 
       # Deletes the sandbox path. Without calling this method, the sandbox path

data/lib/kitchen/provisioner/chef/common_sandbox.rb

@@ -73,6 +73,10 @@ module Kitchen
         # @api private
         attr_reader :sandbox_path
 
+        # @return [String] name of the policy_group, nil results in "local"
+        # @api private
+        attr_reader :policy_group
+
         # Generates a list of all files in the cookbooks directory in the
         # sandbox path.
         #
@@ -281,19 +285,17 @@ module Kitchen
         end
 
         def update_dna_for_policyfile
-          if !config[:run_list].nil? && !config[:run_list].empty?
-            warn("You must set your run_list in your Policyfile instead of "\
-                 "kitchen config. The run_list in your config will be ignored.")
-            warn("Ignored run_list: #{config[:run_list].inspect}")
-          end
-          policy = Chef::Policyfile.new(policyfile, sandbox_path,
+          policy = Chef::Policyfile.new(
+            policyfile, sandbox_path,
             logger: logger,
-            always_update: config[:always_update_cookbooks])
+            always_update: config[:always_update_cookbooks],
+            policy_group: policy_group
+          )
           Kitchen.mutex.synchronize do
             policy.compile
           end
           policy_name = JSON.parse(IO.read(policy.lockfile))["name"]
-          policy_group = "local"
+          policy_group = config[:policy_group] || "local"
           config[:attributes].merge(policy_name: policy_name, policy_group: policy_group)
         end
 
@@ -302,9 +304,12 @@ module Kitchen
         # @api private
         def resolve_with_policyfile
           Kitchen.mutex.synchronize do
-            Chef::Policyfile.new(policyfile, sandbox_path,
+            Chef::Policyfile.new(
+              policyfile, sandbox_path,
               logger: logger,
-              always_update: config[:always_update_cookbooks]).resolve
+              always_update: config[:always_update_cookbooks],
+              policy_group: config[:policy_group]
+            ).resolve
           end
         end
 

data/lib/kitchen/provisioner/chef/policyfile.rb

@@ -41,11 +41,12 @@ module Kitchen
         #   cookbooks
         # @param logger [Kitchen::Logger] a logger to use for output, defaults
         #   to `Kitchen.logger`
-        def initialize(policyfile, path, logger: Kitchen.logger, always_update: false)
+        def initialize(policyfile, path, logger: Kitchen.logger, always_update: false, policy_group: nil)
           @policyfile    = policyfile
           @path          = path
           @logger        = logger
           @always_update = always_update
+          @policy_group  = policy_group
         end
 
         # Loads the library code required to use the resolver.
@@ -59,8 +60,13 @@ module Kitchen
         # Performs the cookbook resolution and vendors the resulting cookbooks
         # in the desired path.
         def resolve
-          info("Exporting cookbook dependencies from Policyfile #{path} using `#{cli_path} export`...")
-          run_command("#{cli_path} export #{escape_path(policyfile)} #{escape_path(path)} --force")
+          if policy_group
+            info("Exporting cookbook dependencies from Policyfile #{path} with policy_group #{policy_group} using `#{cli_path} export`...")
+            run_command("#{cli_path} export #{escape_path(policyfile)} #{escape_path(path)} --policy_group #{policy_group} --force")
+          else
+            info("Exporting cookbook dependencies from Policyfile #{path} using `#{cli_path} export`...")
+            run_command("#{cli_path} export #{escape_path(policyfile)} #{escape_path(path)} --force")
+          end
         end
 
         # Runs `chef install` to determine the correct cookbook set and
@@ -104,6 +110,10 @@ module Kitchen
         # @api private
         attr_reader :always_update
 
+        # @return [String] name of the policy_group, nil results in "local"
+        # @api private
+        attr_reader :policy_group
+
         # Escape spaces in a path in way that works with both Sh (Unix) and
         # Windows.
         #
@@ -118,7 +128,7 @@ module Kitchen
             # Windows command line parsing libraries. This covers the 99% case of
             # spaces in the path without breaking other stuff.
             if /[ \t\n\v"]/.match?(path)
-              "\"#{path.gsub(/[ \t\n\v\"\\]/) { |m| '\\' + m[0] }}\""
+              "\"#{path.gsub(/[ \t\n\v\"\\]/) { |m| "\\" + m[0] }}\""
             else
               path
             end

data/lib/kitchen/provisioner/chef_base.rb

@@ -46,6 +46,7 @@ module Kitchen
       default_config :chef_omnibus_install_options, nil
       default_config :chef_license, nil
       default_config :run_list, []
+      default_config :policy_group, nil
       default_config :attributes, {}
       default_config :config_path, nil
       default_config :log_file, nil
@@ -63,7 +64,7 @@ module Kitchen
       default_config :berksfile_path, nil
       # If set to true (which is the default from `chef generate`), try to update
       # backend cookbook downloader on every kitchen run.
-      default_config :always_update_cookbooks, false
+      default_config :always_update_cookbooks, true
       default_config :cookbook_files_glob, %w(
         README.* VERSION metadata.{json,rb} attributes.rb recipe.rb
         attributes/**/* definitions/**/* files/**/* libraries/**/*
@@ -441,7 +442,7 @@ module Kitchen
         if obj.is_a?(String) && obj =~ /^:/
           obj
         elsif obj.is_a?(String)
-          %{"#{obj.gsub(/\\/, '\\\\\\\\')}"}
+          %{"#{obj.gsub(/\\/, "\\\\\\\\")}"}
         elsif obj.is_a?(Array)
           %{[#{obj.map { |i| format_value(i) }.join(", ")}]}
         else

data/lib/kitchen/provisioner/chef_infra.rb

@@ -0,0 +1,167 @@
+#
+# Author:: Fletcher Nichol (<fnichol@nichol.ca>)
+#
+# Copyright (C) 2013, Fletcher Nichol
+#
+# 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.
+
+require_relative "chef_base"
+
+module Kitchen
+  module Provisioner
+    # Chef Zero provisioner.
+    #
+    # @author Fletcher Nichol <fnichol@nichol.ca>
+    class ChefInfra < ChefBase
+      kitchen_provisioner_api_version 2
+
+      plugin_version Kitchen::VERSION
+
+      default_config :client_rb, {}
+      default_config :named_run_list, {}
+      default_config :json_attributes, true
+      default_config :chef_zero_host, nil
+      default_config :chef_zero_port, 8889
+
+      default_config :chef_client_path do |provisioner|
+        provisioner
+          .remote_path_join(%W{#{provisioner[:chef_omnibus_root]} bin chef-client})
+          .tap { |path| path.concat(".bat") if provisioner.windows_os? }
+      end
+
+      default_config :ruby_bindir do |provisioner|
+        provisioner
+          .remote_path_join(%W{#{provisioner[:chef_omnibus_root]} embedded bin})
+      end
+
+      # (see Base#create_sandbox)
+      def create_sandbox
+        super
+        prepare_validation_pem
+        prepare_config_rb
+      end
+
+      def run_command
+        cmd = "#{sudo(config[:chef_client_path])} --local-mode".tap { |str| str.insert(0, "& ") if powershell_shell? }
+
+        chef_cmd(cmd)
+      end
+
+      private
+
+      # Adds optional flags to a chef-client command, depending on
+      # configuration data. Note that this method mutates the incoming Array.
+      #
+      # @param args [Array<String>] array of flags
+      # @api private
+      # rubocop:disable Metrics/CyclomaticComplexity
+      def add_optional_chef_client_args!(args)
+        if config[:json_attributes]
+          json = remote_path_join(config[:root_path], "dna.json")
+          args << "--json-attributes #{json}"
+        end
+
+        args << "--logfile #{config[:log_file]}" if config[:log_file]
+
+        # these flags are chef-client local mode only and will not work
+        # on older versions of chef-client
+        if config[:chef_zero_host]
+          args << "--chef-zero-host #{config[:chef_zero_host]}"
+        end
+
+        if config[:chef_zero_port]
+          args << "--chef-zero-port #{config[:chef_zero_port]}"
+        end
+
+        args << "--profile-ruby" if config[:profile_ruby]
+
+        if config[:slow_resource_report]
+          if config[:slow_resource_report].is_a?(Integer)
+            args << "--slow-report #{config[:slow_resource_report]}"
+          else
+            args << "--slow-report"
+          end
+        end
+      end
+      # rubocop:enable Metrics/CyclomaticComplexity
+
+      # Returns an Array of command line arguments for the chef client.
+      #
+      # @return [Array<String>] an array of command line arguments
+      # @api private
+      def chef_args(client_rb_filename)
+        level = config[:log_level]
+        args = [
+            "--config #{remote_path_join(config[:root_path], client_rb_filename)}",
+            "--log_level #{level}",
+            "--force-formatter",
+            "--no-color",
+        ]
+        add_optional_chef_client_args!(args)
+
+        args
+      end
+
+      # Generates a string of shell environment variables needed for the
+      # chef-client-zero.rb shim script to properly function.
+      #
+      # @return [String] a shell script string
+      # @api private
+      def chef_client_zero_env
+        root = config[:root_path]
+        gem_home = gem_path = remote_path_join(root, "chef-client-zero-gems")
+        gem_cache = remote_path_join(gem_home, "cache")
+
+        [
+            shell_env_var("CHEF_REPO_PATH", root),
+            shell_env_var("GEM_HOME", gem_home),
+            shell_env_var("GEM_PATH", gem_path),
+            shell_env_var("GEM_CACHE", gem_cache),
+        ].join("\n").concat("\n")
+      end
+
+      # Writes a fake (but valid) validation.pem into the sandbox directory.
+      #
+      # @api private
+      def prepare_validation_pem
+        info("Preparing validation.pem")
+        debug("Using a dummy validation.pem")
+
+        source = File.join(File.dirname(__FILE__),
+                           %w{.. .. .. support dummy-validation.pem})
+        FileUtils.cp(source, File.join(sandbox_path, "validation.pem"))
+      end
+
+      # Returns the command that will run a backwards compatible shim script
+      # that approximates local mode in a modern chef-client run.
+      #
+      # @return [String] the command string
+      # @api private
+      def shim_command
+        ruby = remote_path_join(config[:ruby_bindir], "ruby")
+          .tap { |path| path.concat(".exe") if windows_os? }
+        shim = remote_path_join(config[:root_path], "chef-client-zero.rb")
+
+        "#{chef_client_zero_env}\n#{sudo(ruby)} #{shim}"
+      end
+
+      # This provisioner supports policyfiles, so override the default (which
+      # is false)
+      # @return [true] always returns true
+      # @api private
+      def supports_policyfile?
+        true
+      end
+    end
+  end
+end

data/lib/kitchen/provisioner/chef_zero.rb

@@ -1,156 +1,12 @@
-#
-# Author:: Fletcher Nichol (<fnichol@nichol.ca>)
-#
-# Copyright (C) 2013, Fletcher Nichol
-#
-# 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.
-
-require_relative "chef_base"
+# Deprecated AS PER THE PR - https://github.com/test-kitchen/test-kitchen/pull/1730
+require_relative "chef_infra"
 
 module Kitchen
   module Provisioner
     # Chef Zero provisioner.
     #
     # @author Fletcher Nichol <fnichol@nichol.ca>
-    class ChefZero < ChefBase
-      kitchen_provisioner_api_version 2
-
-      plugin_version Kitchen::VERSION
-
-      default_config :client_rb, {}
-      default_config :named_run_list, {}
-      default_config :json_attributes, true
-      default_config :chef_zero_host, nil
-      default_config :chef_zero_port, 8889
-
-      default_config :chef_client_path do |provisioner|
-        provisioner
-          .remote_path_join(%W{#{provisioner[:chef_omnibus_root]} bin chef-client})
-          .tap { |path| path.concat(".bat") if provisioner.windows_os? }
-      end
-
-      default_config :ruby_bindir do |provisioner|
-        provisioner
-          .remote_path_join(%W{#{provisioner[:chef_omnibus_root]} embedded bin})
-      end
-
-      # (see Base#create_sandbox)
-      def create_sandbox
-        super
-        prepare_validation_pem
-        prepare_config_rb
-      end
-
-      def run_command
-        cmd = "#{sudo(config[:chef_client_path])} --local-mode".tap { |str| str.insert(0, "& ") if powershell_shell? }
-
-        chef_cmd(cmd)
-      end
-
-      private
-
-      # Adds optional flags to a chef-client command, depending on
-      # configuration data. Note that this method mutates the incoming Array.
-      #
-      # @param args [Array<String>] array of flags
-      # @api private
-      # rubocop:disable Metrics/CyclomaticComplexity
-      def add_optional_chef_client_args!(args)
-        if config[:json_attributes]
-          json = remote_path_join(config[:root_path], "dna.json")
-          args << "--json-attributes #{json}"
-        end
-        args << "--logfile #{config[:log_file]}" if config[:log_file]
-
-        # these flags are chef-client local mode only and will not work
-        # on older versions of chef-client
-        if config[:chef_zero_host]
-          args << "--chef-zero-host #{config[:chef_zero_host]}"
-        end
-        if config[:chef_zero_port]
-          args << "--chef-zero-port #{config[:chef_zero_port]}"
-        end
-        args << "--profile-ruby" if config[:profile_ruby]
-      end
-      # rubocop:enable Metrics/CyclomaticComplexity
-
-      # Returns an Array of command line arguments for the chef client.
-      #
-      # @return [Array<String>] an array of command line arguments
-      # @api private
-      def chef_args(client_rb_filename)
-        level = config[:log_level]
-        args = [
-          "--config #{remote_path_join(config[:root_path], client_rb_filename)}",
-          "--log_level #{level}",
-          "--force-formatter",
-          "--no-color",
-        ]
-        add_optional_chef_client_args!(args)
-
-        args
-      end
-
-      # Generates a string of shell environment variables needed for the
-      # chef-client-zero.rb shim script to properly function.
-      #
-      # @return [String] a shell script string
-      # @api private
-      def chef_client_zero_env
-        root = config[:root_path]
-        gem_home = gem_path = remote_path_join(root, "chef-client-zero-gems")
-        gem_cache = remote_path_join(gem_home, "cache")
-
-        [
-          shell_env_var("CHEF_REPO_PATH", root),
-          shell_env_var("GEM_HOME", gem_home),
-          shell_env_var("GEM_PATH", gem_path),
-          shell_env_var("GEM_CACHE", gem_cache),
-        ].join("\n").concat("\n")
-      end
-
-      # Writes a fake (but valid) validation.pem into the sandbox directory.
-      #
-      # @api private
-      def prepare_validation_pem
-        info("Preparing validation.pem")
-        debug("Using a dummy validation.pem")
-
-        source = File.join(File.dirname(__FILE__),
-          %w{.. .. .. support dummy-validation.pem})
-        FileUtils.cp(source, File.join(sandbox_path, "validation.pem"))
-      end
-
-      # Returns the command that will run a backwards compatible shim script
-      # that approximates local mode in a modern chef-client run.
-      #
-      # @return [String] the command string
-      # @api private
-      def shim_command
-        ruby = remote_path_join(config[:ruby_bindir], "ruby")
-          .tap { |path| path.concat(".exe") if windows_os? }
-        shim = remote_path_join(config[:root_path], "chef-client-zero.rb")
-
-        "#{chef_client_zero_env}\n#{sudo(ruby)} #{shim}"
-      end
-
-      # This provisioner supports policyfiles, so override the default (which
-      # is false)
-      # @return [true] always returns true
-      # @api private
-      def supports_policyfile?
-        true
-      end
+    class ChefZero < ChefInfra
     end
   end
 end

data/lib/kitchen/provisioner.rb

@@ -2,6 +2,7 @@
 # Author:: Fletcher Nichol (<fnichol@nichol.ca>)
 #
 # Copyright (C) 2013, Fletcher Nichol
+# Copyright (C) Chef Software Inc.
 #
 # Licensed under the Apache License, Version 2.0 (the "License");
 # you may not use this file except in compliance with the License.
@@ -25,7 +26,7 @@ module Kitchen
   # @author Fletcher Nichol <fnichol@nichol.ca>
   module Provisioner
     # Default provisioner to use
-    DEFAULT_PLUGIN = "chef_solo".freeze
+    DEFAULT_PLUGIN = "chef_infra".freeze
 
     # Returns an instance of a provisioner given a plugin type string.
     #
@@ -34,6 +35,7 @@ module Kitchen
     # @return [Provisioner::Base] a provisioner instance
     # @raise [ClientError] if a provisioner instance could not be created
     def self.for_plugin(plugin, config)
+      plugin, config[:name] = "chef_infra", "chef_infra" if plugin == "chef_zero"
       Kitchen::Plugin.load(self, plugin, config)
     end
   end

data/lib/kitchen/transport/winrm.rb

@@ -365,11 +365,9 @@ module Kitchen
         # @return [Winrm::Shells::Elevated] the elevated shell
         # @api private
         def elevated_session(retry_options = {})
-          @elevated_session ||= begin
-            connection(retry_options).shell(:elevated).tap do |shell|
-              shell.username = options[:elevated_username]
-              shell.password = options[:elevated_password]
-            end
+          @elevated_session ||= connection(retry_options).shell(:elevated).tap do |shell|
+            shell.username = options[:elevated_username]
+            shell.password = options[:elevated_password]
           end
         end
 
@@ -441,18 +439,31 @@ module Kitchen
           elevated_username: data[:elevated_username] || data[:username],
           elevated_password: elevated_password,
         }
-        opts.merge!(additional_transport_args(opts[:transport]))
+        opts.merge!(additional_transport_args(data, opts[:transport]))
+        if opts[:transport].to_sym == :ssl && opts.key?(:client_cert) && opts.key?(:client_key)
+          opts.delete(:user)
+          opts.delete(:password)
+        end
         opts
       end
 
-      def additional_transport_args(transport_type)
+      def additional_transport_args(data, transport_type)
+        opts = {
+          disable_sspi: false,
+          basic_auth_only: false,
+        }
+
         case transport_type.to_sym
-        when :ssl, :negotiate
-          {
-            no_ssl_peer_verification: true,
-            disable_sspi: false,
-            basic_auth_only: false,
-          }
+        when :ssl
+          if data.key?(:client_cert) && data.key?(:client_key)
+            opts[:client_cert] = data[:client_cert]
+            opts[:client_key] = data[:client_key]
+          end
+          opts[:no_ssl_peer_verification] = data.key?(:no_ssl_peer_verification) ? data[:no_ssl_peer_verification] : true
+          opts
+        when :negotiate
+          opts[:no_ssl_peer_verification] = true
+          opts
         when :plaintext
           {
             disable_sspi: true,

data/lib/kitchen/verifier/base.rb

@@ -169,11 +169,9 @@ module Kitchen
       # @raise [ClientError] if the sandbox directory has no yet been created
       #   by calling `#create_sandbox`
       def sandbox_path
-        @sandbox_path ||= begin
-           raise ClientError, "Sandbox directory has not yet " \
+        @sandbox_path ||= raise ClientError, "Sandbox directory has not yet " \
            "been created. Please run #{self.class}#create_sandox before " \
            "trying to access the path."
-         end
       end
 
       # Sets the API version for this verifier. If the verifier does not set

data/lib/kitchen/version.rb

@@ -16,5 +16,5 @@
 # limitations under the License.
 
 module Kitchen
-  VERSION = "2.11.1".freeze
+  VERSION = "3.1.0".freeze
 end

data/test-kitchen.gemspec

@@ -20,7 +20,7 @@ Gem::Specification.new do |gem|
   gem.executables   = %w{kitchen}
   gem.require_paths = ["lib"]
 
-  gem.required_ruby_version = ">= 2.5"
+  gem.required_ruby_version = ">= 2.6"
 
   gem.add_dependency "mixlib-shellout",    ">= 1.2", "< 4.0"
   gem.add_dependency "net-scp",            ">= 1.1", "< 4.0" # pinning until we can confirm 4+ works
@@ -44,9 +44,9 @@ Gem::Specification.new do |gem|
 
   gem.add_development_dependency "aruba",     "~> 0.11", "< 1.0"
   gem.add_development_dependency "fakefs",    "~> 1.0"
-  gem.add_development_dependency "minitest",  "~> 5.3", "< 5.12"
+  gem.add_development_dependency "minitest",  "~> 5.3", "< 5.15"
   gem.add_development_dependency "mocha",     "~> 1.1"
-  gem.add_development_dependency "cucumber",  ">= 2.1", "< 4.0" # we just need to validate 4.0 when it comes out
+  gem.add_development_dependency "cucumber",  ">= 2.1", "< 8.0"
   gem.add_development_dependency "countloc",  "~> 0.4"
   gem.add_development_dependency "maruku",    "~> 0.6"
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: test-kitchen
 version: !ruby/object:Gem::Version
-  version: 2.11.1
+  version: 3.1.0
 platform: ruby
 authors:
 - Fletcher Nichol
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2021-03-02 00:00:00.000000000 Z
+date: 2021-10-02 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: mixlib-shellout
@@ -313,7 +313,7 @@ dependencies:
         version: '5.3'
     - - "<"
       - !ruby/object:Gem::Version
-        version: '5.12'
+        version: '5.15'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
@@ -323,7 +323,7 @@ dependencies:
         version: '5.3'
     - - "<"
       - !ruby/object:Gem::Version
-        version: '5.12'
+        version: '5.15'
 - !ruby/object:Gem::Dependency
   name: mocha
   requirement: !ruby/object:Gem::Requirement
@@ -347,7 +347,7 @@ dependencies:
         version: '2.1'
     - - "<"
       - !ruby/object:Gem::Version
-        version: '4.0'
+        version: '8.0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
@@ -357,7 +357,7 @@ dependencies:
         version: '2.1'
     - - "<"
       - !ruby/object:Gem::Version
-        version: '4.0'
+        version: '8.0'
 - !ruby/object:Gem::Dependency
   name: countloc
   requirement: !ruby/object:Gem::Requirement
@@ -449,6 +449,7 @@ files:
 - lib/kitchen/provisioner/chef/policyfile.rb
 - lib/kitchen/provisioner/chef_apply.rb
 - lib/kitchen/provisioner/chef_base.rb
+- lib/kitchen/provisioner/chef_infra.rb
 - lib/kitchen/provisioner/chef_solo.rb
 - lib/kitchen/provisioner/chef_zero.rb
 - lib/kitchen/provisioner/dummy.rb
@@ -512,14 +513,14 @@ required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
-      version: '2.5'
+      version: '2.6'
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.1.4
+rubygems_version: 3.2.22
 signing_key:
 specification_version: 4
 summary: Test Kitchen is an integration tool for developing and testing infrastructure