knife-windows 3.0.15 → 4.0.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: f5fbbec60197053e26aceab8398496393e38a25d0c6c5f79902a9bd1e394083f
-  data.tar.gz: 4838ed078b2fa860e782b455a56ab873a131ca522aa5a274c6d6e54c9e485a97
+  metadata.gz: 49d47d86c96375be88a4f739bf617cfbc369ce30680144b638e4612233a3e32d
+  data.tar.gz: cb252caa670a3ec82c9d9aaf3a46c5a00d0ab75c0fd8e4c6ddaab722651e4151
 SHA512:
-  metadata.gz: 7793a226c83203460df2b9e9dc9de02be5dc51774bb504402e6a1b1e0264ab7c33ee7c85a7878aaa27e885668f1b02a107d7f7e25b5a98c4995184ef23efd52b
-  data.tar.gz: 9802968c5d5d5053618b50ae74918be38d74e92408f3559b731952113ac763b1754d3e28c960d94f3f259b6892dd726e7146f97f17286f71a924f9fd385e4b37
+  metadata.gz: b60265f9581770bad24a7147b2846b6c844bc4393326008ab728a1c7b6a4bd38edf23bbe7757e9e6f800a765eda84d9921a7aab3103d3e0af2a44f7646de1753
+  data.tar.gz: dc0e8b6a56491263a303654055e258e7bbe64233ee70babf3c59800d9f89c9a805610f91e297de702f2c0876c674d4cbc43431c693f999a6ae30d50a8f689d16

data/lib/chef/knife/bootstrap_windows_ssh.rb

@@ -1,6 +1,6 @@
 #
 # Author:: Seth Chisamore (<schisamo@chef.io>)
-# Copyright:: Copyright (c) 2011-2016 Chef Software, Inc.
+# Copyright:: Copyright (c) Chef Software Inc.
 # License:: Apache License, Version 2.0
 #
 # Licensed under the Apache License, Version 2.0 (the "License");
@@ -17,7 +17,7 @@
 #
 
 require "chef/knife"
-require_relative "bootstrap_windows_base"
+require_relative "helpers/bootstrap_windows_base"
 
 class Chef
   class Knife
@@ -25,16 +25,6 @@ class Chef
 
       include Chef::Knife::BootstrapWindowsBase
 
-      deps do
-        require "chef/json_compat"
-        require "tempfile"
-        require "highline"
-        require "net/ssh"
-        require "net/ssh/multi"
-        Chef::Knife::Ssh.load_deps
-        Chef::Knife::Bootstrap.load_deps
-      end
-
       banner "knife bootstrap windows ssh FQDN (options) DEPRECATED"
 
       option :ssh_user,
@@ -52,13 +42,12 @@ class Chef
         short: "-p PORT",
         long: "--ssh-port PORT",
         description: "The ssh port",
-        proc: Proc.new { |key| Chef::Config[:knife][:ssh_port] = key.strip }
+        proc: Proc.new { |key| key.strip }
 
       option :ssh_gateway,
         short: "-G GATEWAY",
         long: "--ssh-gateway GATEWAY",
-        description: "The ssh gateway",
-        proc: Proc.new { |key| Chef::Config[:knife][:ssh_gateway] = key }
+        description: "The ssh gateway"
 
       option :forward_agent,
         short: "-A",

data/lib/chef/knife/bootstrap_windows_winrm.rb

@@ -1,6 +1,6 @@
 #
 # Author:: Seth Chisamore (<schisamo@chef.io>)
-# Copyright:: Copyright (c) 2011-2016 Chef Software, Inc.
+# Copyright:: Copyright (c) 2011-2020 Chef Software, Inc.
 # License:: Apache License, Version 2.0
 #
 # Licensed under the Apache License, Version 2.0 (the "License");
@@ -17,25 +17,13 @@
 #
 
 require "chef/knife"
-require_relative "bootstrap_windows_base"
-require_relative "winrm_base"
-require_relative "winrm_knife_base" # WinrmCommandSharedFunctions
+require_relative "helpers/bootstrap_windows_base"
 
 class Chef
   class Knife
     class BootstrapWindowsWinrm < Bootstrap
 
       include Chef::Knife::BootstrapWindowsBase
-      include Chef::Knife::WinrmBase
-      include Chef::Knife::WinrmCommandSharedFunctions
-
-      deps do
-        require_relative "winrm"
-        require "chef/json_compat"
-        require "tempfile"
-        Chef::Knife::Winrm.load_deps
-        Chef::Knife::Bootstrap.load_deps
-      end
 
       banner "knife bootstrap windows winrm FQDN (options) DEPRECATED"
 

data/lib/chef/knife/{bootstrap_windows_base.rb → helpers/bootstrap_windows_base.rb}

@@ -1,6 +1,6 @@
 #
 # Author:: Seth Chisamore (<schisamo@chef.io>)
-# Copyright:: Copyright (c) 2011-2016 Chef Software, Inc.
+# Copyright:: Copyright (c) Chef Software Inc.
 # License:: Apache License, Version 2.0
 #
 # Licensed under the Apache License, Version 2.0 (the "License");
@@ -17,28 +17,17 @@
 #
 
 require "chef/knife"
-require_relative "knife_windows_base"
 require "chef/util/path_helper"
 
 class Chef
   class Knife
     module BootstrapWindowsBase
-
-      include Chef::Knife::KnifeWindowsBase
-
       # :nodoc:
       # Would prefer to do this in a rational way, but can't be done b/c of
       # Mixlib::CLI's design :(
       def self.included(includer)
         includer.class_eval do
 
-          deps do
-            require "chef/knife/bootstrap"
-            require "chef/encrypted_data_bag_item"
-            require "readline"
-            require "chef/json_compat"
-          end
-
           option :chef_node_name,
             short: "-N NAME",
             long: "--node-name NAME",
@@ -50,23 +39,19 @@ class Chef
 
           option :bootstrap_version,
             long: "--bootstrap-version VERSION",
-            description: "The version of Chef to install",
-            proc: Proc.new { |v| Chef::Config[:knife][:bootstrap_version] = v }
+            description: "The version of Chef to install"
 
           option :bootstrap_proxy,
             long: "--bootstrap-proxy PROXY_URL",
-            description: "The proxy server for the node being bootstrapped",
-            proc: Proc.new { |p| Chef::Config[:knife][:bootstrap_proxy] = p }
+            description: "The proxy server for the node being bootstrapped"
 
           option :bootstrap_no_proxy,
             long: "--bootstrap-no-proxy [NO_PROXY_URL|NO_PROXY_IP]",
-            description: "Do not proxy locations for the node being bootstrapped; this option is used internally by Opscode",
-            proc: Proc.new { |np| Chef::Config[:knife][:bootstrap_no_proxy] = np }
+            description: "Do not proxy locations for the node being bootstrapped; this option is used internally by Opscode"
 
           option :bootstrap_install_command,
             long: "--bootstrap-install-command COMMANDS",
-            description: "Custom command to install chef-client",
-            proc: Proc.new { |ic| Chef::Config[:knife][:bootstrap_install_command] = ic }
+            description: "Custom command to install chef-client"
 
           option :bootstrap_template,
             short: "-t TEMPLATE",
@@ -83,10 +68,11 @@ class Chef
           option :hint,
             long: "--hint HINT_NAME[=HINT_FILE]",
             description: "Specify Ohai Hint to be set on the bootstrap target. Use multiple --hint options to specify multiple hints.",
-            proc: Proc.new { |h|
-              Chef::Config[:knife][:hints] ||= {}
+            proc: Proc.new { |h, accumulator|
+              accumulator ||= {}
               name, path = h.split("=")
-              Chef::Config[:knife][:hints][name] = path ? Chef::JSONCompat.parse(::File.read(path)) : {}
+              accumulator[name] = path ? Chef::JSONCompat.parse(::File.read(path)) : {}
+              accumulator
             }
 
           option :first_boot_attributes,
@@ -158,12 +144,12 @@ class Chef
           option :bootstrap_vault_item,
             long: "--bootstrap-vault-item VAULT_ITEM",
             description: 'A single vault and item to update as "vault:item"',
-            proc: Proc.new { |i|
+            proc: Proc.new { |i, accumulator|
               (vault, item) = i.split(/:/)
-              Chef::Config[:knife][:bootstrap_vault_item] ||= {}
-              Chef::Config[:knife][:bootstrap_vault_item][vault] ||= []
-              Chef::Config[:knife][:bootstrap_vault_item][vault].push(item)
-              Chef::Config[:knife][:bootstrap_vault_item]
+              accumulator ||= {}
+              accumulator[vault] ||= []
+              accumulator[vault].push(item)
+              accumulator
             }
 
           option :policy_name,

data/lib/chef/knife/{winrm_base.rb → helpers/winrm_base.rb}

@@ -1,6 +1,6 @@
 #
 # Author:: Seth Chisamore (<schisamo@chef.io>)
-# Copyright:: Copyright (c) 2011-2016 Chef Software, Inc.
+# Copyright:: Copyright (c) Chef Software Inc.
 # License:: Apache License, Version 2.0
 #
 # Licensed under the Apache License, Version 2.0 (the "License");
@@ -42,14 +42,12 @@ class Chef
             short: "-x USERNAME",
             long: "--winrm-user USERNAME",
             description: "The WinRM username",
-            default: "Administrator",
-            proc: Proc.new { |key| Chef::Config[:knife][:winrm_user] = key }
+            default: "Administrator"
 
           option :winrm_password,
             short: "-P PASSWORD",
             long: "--winrm-password PASSWORD",
-            description: "The WinRM password",
-            proc: Proc.new { |key| Chef::Config[:knife][:winrm_password] = key }
+            description: "The WinRM password"
 
           option :winrm_shell,
             long: "--winrm-shell SHELL",
@@ -61,42 +59,32 @@ class Chef
             short: "-w TRANSPORT",
             long: "--winrm-transport TRANSPORT",
             description: "The WinRM transport type. Valid choices are [ssl, plaintext]",
-            default: "plaintext",
-            proc: Proc.new { |transport|
-              Chef::Config[:knife][:winrm_port] = "5986" if transport == "ssl"
-              Chef::Config[:knife][:winrm_transport] = transport
-            }
+            default: "plaintext"
 
           option :winrm_port,
             short: "-p PORT",
             long: "--winrm-port PORT",
-            description: "The WinRM port, by default this is '5985' for 'plaintext' and '5986' for 'ssl' winrm transport",
-            default: "5985",
-            proc: Proc.new { |key| Chef::Config[:knife][:winrm_port] = key }
+            description: "The WinRM port, by default this is '5985' for 'plaintext' and '5986' for 'ssl' winrm transport"
 
           option :kerberos_keytab_file,
             short: "-T KEYTAB_FILE",
             long: "--keytab-file KEYTAB_FILE",
-            description: "The Kerberos keytab file used for authentication",
-            proc: Proc.new { |keytab| Chef::Config[:knife][:kerberos_keytab_file] = keytab }
+            description: "The Kerberos keytab file used for authentication"
 
           option :kerberos_realm,
             short: "-R KERBEROS_REALM",
             long: "--kerberos-realm KERBEROS_REALM",
-            description: "The Kerberos realm used for authentication",
-            proc: Proc.new { |realm| Chef::Config[:knife][:kerberos_realm] = realm }
+            description: "The Kerberos realm used for authentication"
 
           option :kerberos_service,
             short: "-S KERBEROS_SERVICE",
             long: "--kerberos-service KERBEROS_SERVICE",
-            description: "The Kerberos service used for authentication",
-            proc: Proc.new { |service| Chef::Config[:knife][:kerberos_service] = service }
+            description: "The Kerberos service used for authentication"
 
           option :ca_trust_file,
             short: "-f CA_TRUST_FILE",
             long: "--ca-trust-file CA_TRUST_FILE",
-            description: "The Certificate Authority (CA) trust file used for SSL transport",
-            proc: Proc.new { |trust| Chef::Config[:knife][:ca_trust_file] = trust }
+            description: "The Certificate Authority (CA) trust file used for SSL transport"
 
           option :winrm_ssl_verify_mode,
             long: "--winrm-ssl-verify-mode SSL_VERIFY_MODE",
@@ -111,8 +99,7 @@ class Chef
           option :winrm_authentication_protocol,
             long: "--winrm-authentication-protocol AUTHENTICATION_PROTOCOL",
             description: "The authentication protocol used during WinRM communication. The supported protocols are #{WINRM_AUTH_PROTOCOL_LIST.join(",")}. Default is 'negotiate'.",
-            default: "negotiate",
-            proc: Proc.new { |protocol| Chef::Config[:knife][:winrm_authentication_protocol] = protocol }
+            default: "negotiate"
 
           option :session_timeout,
             long: "--session-timeout Minutes",

data/lib/chef/knife/{winrm_knife_base.rb → helpers/winrm_knife_base.rb}

@@ -1,6 +1,6 @@
 #
 # Author:: Steven Murawski (<smurawski@chef.io)
-# Copyright:: Copyright (c) 2015-2016 Chef Software, Inc.
+# Copyright:: Copyright (c) Chef Software Inc.
 # License:: Apache License, Version 2.0
 #
 # Licensed under the Apache License, Version 2.0 (the "License");
@@ -19,7 +19,7 @@
 require "chef/knife"
 require_relative "winrm_base"
 require_relative "winrm_shared_options"
-require_relative "knife_windows_base"
+require_relative "winrm_session"
 
 class Chef
   class Knife
@@ -39,10 +39,9 @@ class Chef
 
           include Chef::Knife::WinrmBase
           include Chef::Knife::WinrmSharedOptions
-          include Chef::Knife::KnifeWindowsBase
 
           def validate_winrm_options!
-            winrm_auth_protocol = locate_config_value(:winrm_authentication_protocol)
+            winrm_auth_protocol = config[:winrm_authentication_protocol]
 
             unless Chef::Knife::WinrmBase::WINRM_AUTH_PROTOCOL_LIST.include?(winrm_auth_protocol)
               ui.error "Invalid value '#{winrm_auth_protocol}' for --winrm-authentication-protocol option."
@@ -120,7 +119,7 @@ class Chef
             @session_results = []
             queue = Queue.new
             @winrm_sessions.each { |s| queue << s }
-            num_sessions = locate_config_value(:concurrency)
+            num_sessions = config[:concurrency]
             num_targets = @winrm_sessions.length
             num_sessions = (num_sessions.nil? || num_sessions == 0) ? num_targets : [num_sessions, num_targets].min
 
@@ -146,7 +145,7 @@ class Chef
             if authorization_error?(e)
               unless config[:suppress_auth_failure]
                 # Display errors if the caller hasn't opted to retry
-                ui.error "Failed to authenticate to #{s.host} as #{locate_config_value(:winrm_user)}"
+                ui.error "Failed to authenticate to #{s.host} as #{config[:winrm_user]}"
                 ui.info "Response: #{e.message}"
                 ui.info get_failed_authentication_hint
                 raise e
@@ -202,33 +201,35 @@ class Chef
           end
 
           def resolve_session_options
+            config[:winrm_port] ||= ( config[:winrm_transport] == "ssl" ) ? "5986" : "5985"
+
             @session_opts = {
               user: resolve_winrm_user,
-              password: locate_config_value(:winrm_password),
-              port: locate_config_value(:winrm_port),
+              password: config[:winrm_password],
+              port: config[:winrm_port],
               operation_timeout: resolve_winrm_session_timeout,
               basic_auth_only: resolve_winrm_basic_auth,
               disable_sspi: resolve_winrm_disable_sspi,
               transport: resolve_winrm_transport,
               no_ssl_peer_verification: resolve_no_ssl_peer_verification,
               ssl_peer_fingerprint: resolve_ssl_peer_fingerprint,
-              shell: locate_config_value(:winrm_shell),
-              codepage: locate_config_value(:winrm_codepage),
+              shell: config[:winrm_shell],
+              codepage: config[:winrm_codepage],
             }
 
             if @session_opts[:user] && (not @session_opts[:password])
-              @session_opts[:password] = Chef::Config[:knife][:winrm_password] = config[:winrm_password] = get_password
+              @session_opts[:password] = config[:winrm_password] = get_password
             end
 
             if @session_opts[:transport] == :kerberos
               @session_opts.merge!(resolve_winrm_kerberos_options)
             end
 
-            @session_opts[:ca_trust_path] = locate_config_value(:ca_trust_file) if locate_config_value(:ca_trust_file)
+            @session_opts[:ca_trust_path] = config[:ca_trust_file] if config[:ca_trust_file]
           end
 
           def resolve_winrm_user
-            user = locate_config_value(:winrm_user)
+            user = config[:winrm_user]
 
             # Prefixing with '.\' when using negotiate
             # to auth user against local machine domain
@@ -244,23 +245,23 @@ class Chef
 
           def resolve_winrm_session_timeout
             # 30 min (Default) OperationTimeout for long bootstraps fix for KNIFE_WINDOWS-8
-            locate_config_value(:session_timeout).to_i * 60 if locate_config_value(:session_timeout)
+            config[:session_timeout].to_i * 60 if config[:session_timeout]
           end
 
           def resolve_winrm_basic_auth
-            locate_config_value(:winrm_authentication_protocol) == "basic"
+            config[:winrm_authentication_protocol] == "basic"
           end
 
           def resolve_winrm_kerberos_options
             kerberos_opts = {}
-            kerberos_opts[:keytab] = locate_config_value(:kerberos_keytab_file) if locate_config_value(:kerberos_keytab_file)
-            kerberos_opts[:realm] = locate_config_value(:kerberos_realm) if locate_config_value(:kerberos_realm)
-            kerberos_opts[:service] = locate_config_value(:kerberos_service) if locate_config_value(:kerberos_service)
+            kerberos_opts[:keytab] = config[:kerberos_keytab_file] if config[:kerberos_keytab_file]
+            kerberos_opts[:realm] = config[:kerberos_realm] if config[:kerberos_realm]
+            kerberos_opts[:service] = config[:kerberos_service] if config[:kerberos_service]
             kerberos_opts
           end
 
           def resolve_winrm_transport
-            transport = locate_config_value(:winrm_transport).to_sym
+            transport = config[:winrm_transport].to_sym
             if config.any? { |k, v| k.to_s =~ /kerberos/ && !v.nil? }
               transport = :kerberos
             elsif transport != :ssl && negotiate_auth?
@@ -271,11 +272,11 @@ class Chef
           end
 
           def resolve_no_ssl_peer_verification
-            locate_config_value(:ca_trust_file).nil? && config[:winrm_ssl_verify_mode] == :verify_none && resolve_winrm_transport == :ssl
+            config[:ca_trust_file].nil? && config[:winrm_ssl_verify_mode] == :verify_none && resolve_winrm_transport == :ssl
           end
 
           def resolve_ssl_peer_fingerprint
-            locate_config_value(:ssl_peer_fingerprint)
+            config[:ssl_peer_fingerprint]
           end
 
           def resolve_winrm_disable_sspi
@@ -287,7 +288,7 @@ class Chef
           end
 
           def negotiate_auth?
-            locate_config_value(:winrm_authentication_protocol) == "negotiate"
+            config[:winrm_authentication_protocol] == "negotiate"
           end
 
           def warn_no_ssl_peer_verification

data/lib/chef/knife/windows_cert_generate.rb

@@ -16,7 +16,7 @@
 #
 
 require "chef/knife"
-require_relative "winrm_base"
+require_relative "helpers/winrm_base"
 
 class Chef
   class Knife

data/lib/chef/knife/windows_cert_install.rb

@@ -16,7 +16,7 @@
 #
 
 require "chef/knife"
-require_relative "winrm_base"
+require_relative "helpers/winrm_base"
 
 class Chef
   class Knife

data/lib/chef/knife/windows_listener_create.rb

@@ -16,7 +16,7 @@
 #
 
 require "chef/knife"
-require_relative "winrm_base"
+require_relative "helpers/winrm_base"
 
 class Chef
   class Knife

data/lib/chef/knife/winrm.rb

@@ -1,6 +1,6 @@
 #
 # Author:: Seth Chisamore (<schisamo@chef.io>)
-# Copyright:: Copyright (c) 2011-2016 Chef Software, Inc.
+# Copyright:: Copyright (c) Chef Software Inc.
 # License:: Apache License, Version 2.0
 #
 # Licensed under the Apache License, Version 2.0 (the "License");
@@ -17,21 +17,19 @@
 #
 
 require "chef/knife"
-require_relative "winrm_knife_base" # WinrmCommandSharedFunctions
-require_relative "knife_windows_base"
+require_relative "helpers/winrm_knife_base" # WinrmCommandSharedFunctions
 
 class Chef
   class Knife
     class Winrm < Knife
 
       include Chef::Knife::WinrmCommandSharedFunctions
-      include Chef::Knife::KnifeWindowsBase
 
       deps do
         require_relative "windows_cert_generate"
         require_relative "windows_cert_install"
         require_relative "windows_listener_create"
-        require_relative "winrm_session"
+        require_relative "helpers/winrm_session"
         require "readline"
         require "chef/search/query"
       end

data/lib/chef/knife/wsman_test.rb

@@ -17,8 +17,8 @@
 #
 
 require "chef/knife"
-require_relative "winrm_knife_base"
-require_relative "wsman_endpoint"
+require_relative "helpers/winrm_knife_base"
+require_relative "helpers/wsman_endpoint"
 
 class Chef
   class Knife

data/lib/knife-windows/version.rb

@@ -1,6 +1,6 @@
 module Knife
   module Windows
-    VERSION = "3.0.15".freeze
+    VERSION = "4.0.2".freeze
     MAJOR, MINOR, TINY = VERSION.split(".")
   end
 end

data/spec/spec_helper.rb

@@ -1,6 +1,6 @@
 
 # Author:: Adam Edwards (<adamed@chef.io>)
-# Copyright:: Copyright (c) 2012-2020 Chef Software, Inc.
+# Copyright:: Copyright (c) Chef Software Inc.
 # License:: Apache License, Version 2.0
 #
 # Licensed under the Apache License, Version 2.0 (the "License");
@@ -22,7 +22,22 @@ def sample_data(file_name)
   File.read(file)
 end
 
+class UnexpectedSystemExit < RuntimeError
+  def self.from(system_exit)
+    new(system_exit.message).tap { |e| e.set_backtrace(system_exit.backtrace) }
+  end
+end
+
 RSpec.configure do |config|
+  config.raise_on_warning = true
+  config.raise_errors_for_deprecations!
   config.run_all_when_everything_filtered = true
   config.filter_run focus: true
+  config.around(:example) do |ex|
+    begin
+      ex.run
+    rescue SystemExit => e
+      raise UnexpectedSystemExit.from(e)
+    end
+  end
 end

data/spec/unit/knife/winrm_session_spec.rb

@@ -17,7 +17,7 @@
 #
 
 require "spec_helper"
-require_relative "../../../lib/chef/knife/winrm_session"
+require_relative "../../../lib/chef/knife/helpers/winrm_session"
 require "dummy_winrm_connection"
 
 describe Chef::Knife::WinrmSession do

data/spec/unit/knife/winrm_spec.rb

@@ -1,6 +1,6 @@
 #
 # Author:: Bryan McLellan <btm@chef.io>
-# Copyright:: Copyright (c) 2013-2016 Chef Software, Inc.
+# Copyright:: Copyright (c) Chef Software Inc.
 # License:: Apache License, Version 2.0
 #
 # Licensed under the Apache License, Version 2.0 (the "License");
@@ -250,7 +250,7 @@ describe Chef::Knife::Winrm do
         end
 
         it "sets the user specified winrm port" do
-          Chef::Config[:knife] = { winrm_port: "5988" }
+          winrm_command_http.config[:knife] = { winrm_port: "5988" }
           expect(Chef::Knife::WinrmSession).to receive(:new).with(hash_including(transport: :plaintext)).and_call_original
           expect(WinRM::Connection).to receive(:new).with(hash_including(transport: :plaintext)).and_return(winrm_connection)
           winrm_command_http.configure_chef
@@ -269,7 +269,7 @@ describe Chef::Knife::Winrm do
         let(:winrm_command_https) { Chef::Knife::Winrm.new(["-m", "localhost", "-x", "testuser", "-P", "testpassword", "--winrm-transport", "ssl", "echo helloworld"]) }
 
         it "uses the https uri scheme if the ssl transport is specified" do
-          Chef::Config[:knife] = { winrm_transport: "ssl" }
+          winrm_command_http.config[:winrm_transport] = "ssl"
           expect(Chef::Knife::WinrmSession).to receive(:new).with(hash_including(transport: :ssl)).and_call_original
           expect(WinRM::Connection).to receive(:new).with(hash_including(endpoint: "https://localhost:5986/wsman")).and_return(winrm_connection)
           winrm_command_https.configure_chef
@@ -277,7 +277,7 @@ describe Chef::Knife::Winrm do
         end
 
         it "uses the winrm port '5986' by default for ssl transport" do
-          Chef::Config[:knife] = { winrm_transport: "ssl" }
+          winrm_command_http.config[:winrm_transport] = "ssl"
           expect(Chef::Knife::WinrmSession).to receive(:new).with(hash_including(transport: :ssl)).and_call_original
           expect(WinRM::Connection).to receive(:new).with(hash_including(endpoint: "https://localhost:5986/wsman")).and_return(winrm_connection)
           winrm_command_https.configure_chef
@@ -361,8 +361,8 @@ describe Chef::Knife::Winrm do
 
     before(:each) do
       allow(Chef::Knife::WinrmSession).to receive(:new).and_return(session)
-      Chef::Config[:knife] = { winrm_transport: "plaintext" }
       @winrm = Chef::Knife::Winrm.new(["-m", "localhost", "-x", "testuser", "-P", "testpassword", "--winrm-authentication-protocol", "basic", "echo helloworld"])
+      @winrm.config[:winrm_transport] = "plaintext"
     end
 
     it "returns with 0 if the command succeeds" do
@@ -375,7 +375,6 @@ describe Chef::Knife::Winrm do
       command_status = 510
 
       @winrm.config[:returns] = "0"
-      Chef::Config[:knife][:returns] = [0]
 
       allow(@winrm).to receive(:relay_winrm_command)
       allow(@winrm.ui).to receive(:error)
@@ -386,7 +385,6 @@ describe Chef::Knife::Winrm do
     it "exits with non-zero status if the command fails and returns config is set to 0" do
       command_status = 1
       @winrm.config[:returns] = "0,53"
-      Chef::Config[:knife][:returns] = [0, 53]
       allow(@winrm).to receive(:relay_winrm_command).and_return(command_status)
       allow(@winrm.ui).to receive(:error)
       allow(session).to receive(:exit_code).and_return(command_status)
@@ -395,7 +393,7 @@ describe Chef::Knife::Winrm do
 
     it "exits with a zero status if the command returns an expected non-zero status" do
       command_status = 53
-      Chef::Config[:knife][:returns] = [0, 53]
+      @winrm.config[:returns] = "0,53"
       allow(@winrm).to receive(:relay_winrm_command).and_return(command_status)
       allow(session).to receive(:exit_codes).and_return({ "thishost" => command_status })
       exit_code = @winrm.run
@@ -448,7 +446,7 @@ describe Chef::Knife::Winrm do
 
     context "when winrm_authentication_protocol specified" do
       before do
-        Chef::Config[:knife] = { winrm_transport: "plaintext" }
+        @winrm.config[:winrm_transport] = "plaintext"
         allow(@winrm).to receive(:relay_winrm_command).and_return(0)
       end
 

data/spec/unit/knife/wsman_test_spec.rb

@@ -1,6 +1,6 @@
 #
 # Author:: Steven Murawski (<smurawski@chef.io>)
-# Copyright:: Copyright (c) 2015-2016 Chef Software, Inc.
+# Copyright:: Copyright (c) Chef Software Inc.
 # License:: Apache License, Version 2.0
 #
 # Licensed under the Apache License, Version 2.0 (the "License");
@@ -37,6 +37,7 @@ describe Chef::Knife::WsmanTest do
     allow(Chef::HTTP::DefaultSSLPolicy).to receive(:new)
       .with(http_client_mock.ssl_config)
       .and_return(ssl_policy)
+    subject.merge_configs
   end
 
   subject { Chef::Knife::WsmanTest.new(["-m", "localhost"]) }

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: knife-windows
 version: !ruby/object:Gem::Version
-  version: 3.0.15
+  version: 4.0.2
 platform: ruby
 authors:
 - Seth Chisamore
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2020-02-11 00:00:00.000000000 Z
+date: 2020-05-30 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: chef
@@ -16,14 +16,14 @@ dependencies:
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '15.0'
+        version: '15.11'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '15.0'
+        version: '15.11'
 - !ruby/object:Gem::Dependency
   name: winrm
   requirement: !ruby/object:Gem::Requirement
@@ -75,19 +75,18 @@ extensions: []
 extra_rdoc_files: []
 files:
 - LICENSE
-- lib/chef/knife/bootstrap_windows_base.rb
 - lib/chef/knife/bootstrap_windows_ssh.rb
 - lib/chef/knife/bootstrap_windows_winrm.rb
-- lib/chef/knife/knife_windows_base.rb
+- lib/chef/knife/helpers/bootstrap_windows_base.rb
+- lib/chef/knife/helpers/winrm_base.rb
+- lib/chef/knife/helpers/winrm_knife_base.rb
+- lib/chef/knife/helpers/winrm_session.rb
+- lib/chef/knife/helpers/winrm_shared_options.rb
+- lib/chef/knife/helpers/wsman_endpoint.rb
 - lib/chef/knife/windows_cert_generate.rb
 - lib/chef/knife/windows_cert_install.rb
 - lib/chef/knife/windows_listener_create.rb
 - lib/chef/knife/winrm.rb
-- lib/chef/knife/winrm_base.rb
-- lib/chef/knife/winrm_knife_base.rb
-- lib/chef/knife/winrm_session.rb
-- lib/chef/knife/winrm_shared_options.rb
-- lib/chef/knife/wsman_endpoint.rb
 - lib/chef/knife/wsman_test.rb
 - lib/knife-windows/version.rb
 - spec/assets/fake_trusted_certs/excluded.txt
@@ -123,7 +122,7 @@ required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
-      version: 2.5.0
+      version: 2.6.0
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="

data/lib/chef/knife/knife_windows_base.rb

@@ -1,34 +0,0 @@
-#
-# Author:: Aliasgar Batterywala (<aliasgar.batterywala@clogeny.com>)
-# Copyright:: Copyright (c) 2015-2016 Chef Software, Inc.
-# License:: Apache License, Version 2.0
-#
-# 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 "chef/knife"
-
-class Chef
-  class Knife
-    module KnifeWindowsBase
-
-      def locate_config_value(key)
-        key = key.to_sym
-        value = config[key] || Chef::Config[:knife][key] || default_config[key]
-        Chef::Log.debug("Looking for key #{key} and found value #{value}")
-        value
-      end
-
-    end
-  end
-end