winrm 1.7.1 → 1.7.2

data/spec/transport_spec.rb

@@ -1,124 +1,124 @@
-# encoding: UTF-8
-require 'rubyntlm'
-require 'winrm/http/transport'
-
-describe WinRM::HTTP::HttpNegotiate, unit: true do
-  describe '#init' do
-    let(:endpoint) { 'some_endpoint' }
-    let(:domain) { 'some_domain' }
-    let(:user) { 'some_user' }
-    let(:password) { 'some_password' }
-    let(:options) { {} }
-
-    context 'user is not domain prefixed' do
-      it 'does not pass a domain to the NTLM client' do
-        expect(Net::NTLM::Client).to receive(:new).with(user, password, options)
-        WinRM::HTTP::HttpNegotiate.new(endpoint, user, password, options)
-      end
-    end
-
-    context 'user is domain prefixed' do
-      it 'passes prefixed domain to the NTLM client' do
-        expect(Net::NTLM::Client).to receive(:new) do |passed_user, passed_password, passed_options|
-          expect(passed_user).to eq user
-          expect(passed_password).to eq password
-          expect(passed_options[:domain]).to eq domain
-        end
-        WinRM::HTTP::HttpNegotiate.new(endpoint, "#{domain}\\#{user}", password, options)
-      end
-    end
-
-    context 'option is passed with a domain' do
-      let(:options) { { domain: domain } }
-
-      it 'passes domain option to the NTLM client' do
-        expect(Net::NTLM::Client).to receive(:new) do |passed_user, passed_password, passed_options|
-          expect(passed_user).to eq user
-          expect(passed_password).to eq password
-          expect(passed_options[:domain]).to eq domain
-        end
-        WinRM::HTTP::HttpNegotiate.new(endpoint, user, password, options)
-      end
-    end
-  end
-end
-
-describe 'WinRM connection', integration: true do
-  let(:winrm_connection) do
-    endpoint = config[:endpoint].dup
-    if auth_type == :ssl
-      endpoint.sub!('5985', '5986')
-      endpoint.sub!('http', 'https')
-    end
-    winrm = WinRM::WinRMWebService.new(
-      endpoint, auth_type, options)
-    winrm.logger.level = :error
-    winrm
-  end
-  let(:options) do
-    opts = {}
-    opts[:user] = config[:options][:user]
-    opts[:pass] = config[:options][:pass]
-    opts[:basic_auth_only] = basic_auth_only
-    opts[:no_ssl_peer_verification] = no_ssl_peer_verification
-    opts[:ssl_peer_fingerprint] = ssl_peer_fingerprint
-    opts
-  end
-  let(:basic_auth_only) { false }
-  let(:no_ssl_peer_verification) { false }
-  let(:ssl_peer_fingerprint) { nil }
-
-  subject(:output) do
-    executor = winrm_connection.create_executor
-    executor.run_cmd('ipconfig')
-  end
-
-  shared_examples 'a valid_connection' do
-    it 'has a 0 exit code' do
-      expect(subject).to have_exit_code 0
-    end
-
-    it 'includes command output' do
-      expect(subject).to have_stdout_match(/Windows IP Configuration/)
-    end
-
-    it 'has no errors' do
-      expect(subject).to have_no_stderr
-    end
-  end
-
-  context 'HttpPlaintext' do
-    let(:basic_auth_only) { true }
-    let(:auth_type) { :plaintext }
-
-    it_behaves_like 'a valid_connection'
-  end
-
-  context 'HttpNegotiate' do
-    let(:auth_type) { :negotiate }
-
-    it_behaves_like 'a valid_connection'
-  end
-
-  context 'BasicAuthSSL', skip: ENV['winrm_cert'].nil? do
-    let(:basic_auth_only) { true }
-    let(:auth_type) { :ssl }
-    let(:no_ssl_peer_verification) { true }
-
-    it_behaves_like 'a valid_connection'
-  end
-
-  context 'Negotiate over SSL', skip: ENV['winrm_cert'].nil? do
-    let(:auth_type) { :ssl }
-    let(:no_ssl_peer_verification) { true }
-
-    it_behaves_like 'a valid_connection'
-  end
-
-  context 'SSL fingerprint', skip: ENV['winrm_cert'].nil? do
-    let(:auth_type) { :ssl }
-    let(:ssl_peer_fingerprint) { ENV['winrm_cert'] }
-
-    it_behaves_like 'a valid_connection'
-  end
-end
+# encoding: UTF-8
+require 'rubyntlm'
+require 'winrm/http/transport'
+
+describe WinRM::HTTP::HttpNegotiate, unit: true do
+  describe '#init' do
+    let(:endpoint) { 'some_endpoint' }
+    let(:domain) { 'some_domain' }
+    let(:user) { 'some_user' }
+    let(:password) { 'some_password' }
+    let(:options) { {} }
+
+    context 'user is not domain prefixed' do
+      it 'does not pass a domain to the NTLM client' do
+        expect(Net::NTLM::Client).to receive(:new).with(user, password, options)
+        WinRM::HTTP::HttpNegotiate.new(endpoint, user, password, options)
+      end
+    end
+
+    context 'user is domain prefixed' do
+      it 'passes prefixed domain to the NTLM client' do
+        expect(Net::NTLM::Client).to receive(:new) do |passed_user, passed_password, passed_options|
+          expect(passed_user).to eq user
+          expect(passed_password).to eq password
+          expect(passed_options[:domain]).to eq domain
+        end
+        WinRM::HTTP::HttpNegotiate.new(endpoint, "#{domain}\\#{user}", password, options)
+      end
+    end
+
+    context 'option is passed with a domain' do
+      let(:options) { { domain: domain } }
+
+      it 'passes domain option to the NTLM client' do
+        expect(Net::NTLM::Client).to receive(:new) do |passed_user, passed_password, passed_options|
+          expect(passed_user).to eq user
+          expect(passed_password).to eq password
+          expect(passed_options[:domain]).to eq domain
+        end
+        WinRM::HTTP::HttpNegotiate.new(endpoint, user, password, options)
+      end
+    end
+  end
+end
+
+describe 'WinRM connection', integration: true do
+  let(:winrm_connection) do
+    endpoint = config[:endpoint].dup
+    if auth_type == :ssl
+      endpoint.sub!('5985', '5986')
+      endpoint.sub!('http', 'https')
+    end
+    winrm = WinRM::WinRMWebService.new(
+      endpoint, auth_type, options)
+    winrm.logger.level = :error
+    winrm
+  end
+  let(:options) do
+    opts = {}
+    opts[:user] = config[:options][:user]
+    opts[:pass] = config[:options][:pass]
+    opts[:basic_auth_only] = basic_auth_only
+    opts[:no_ssl_peer_verification] = no_ssl_peer_verification
+    opts[:ssl_peer_fingerprint] = ssl_peer_fingerprint
+    opts
+  end
+  let(:basic_auth_only) { false }
+  let(:no_ssl_peer_verification) { false }
+  let(:ssl_peer_fingerprint) { nil }
+
+  subject(:output) do
+    executor = winrm_connection.create_executor
+    executor.run_cmd('ipconfig')
+  end
+
+  shared_examples 'a valid_connection' do
+    it 'has a 0 exit code' do
+      expect(subject).to have_exit_code 0
+    end
+
+    it 'includes command output' do
+      expect(subject).to have_stdout_match(/Windows IP Configuration/)
+    end
+
+    it 'has no errors' do
+      expect(subject).to have_no_stderr
+    end
+  end
+
+  context 'HttpPlaintext' do
+    let(:basic_auth_only) { true }
+    let(:auth_type) { :plaintext }
+
+    it_behaves_like 'a valid_connection'
+  end
+
+  context 'HttpNegotiate' do
+    let(:auth_type) { :negotiate }
+
+    it_behaves_like 'a valid_connection'
+  end
+
+  context 'BasicAuthSSL', skip: ENV['winrm_cert'].nil? do
+    let(:basic_auth_only) { true }
+    let(:auth_type) { :ssl }
+    let(:no_ssl_peer_verification) { true }
+
+    it_behaves_like 'a valid_connection'
+  end
+
+  context 'Negotiate over SSL', skip: ENV['winrm_cert'].nil? do
+    let(:auth_type) { :ssl }
+    let(:no_ssl_peer_verification) { true }
+
+    it_behaves_like 'a valid_connection'
+  end
+
+  context 'SSL fingerprint', skip: ENV['winrm_cert'].nil? do
+    let(:auth_type) { :ssl }
+    let(:ssl_peer_fingerprint) { ENV['winrm_cert'] }
+
+    it_behaves_like 'a valid_connection'
+  end
+end

data/spec/winrm_options_spec.rb

@@ -1,76 +1,76 @@
-# encoding: UTF-8
-describe 'WinRM options', unit: true do
-  let(:subject) { WinRM::WinRMWebService.new('http://localhost:55985/wsman', :plaintext) }
-
-  context 'when operations timeout is set to 60' do
-    before(:each) { subject.set_timeout(60) }
-    describe '#receive_timeout' do
-      it 'is set to 70s' do
-        transportclass = subject.instance_variable_get(:@xfer)
-        expect(transportclass.receive_timeout).to eql(70)
-      end
-    end
-    describe '#op_timeout' do
-      it 'is set to 60s' do
-        expect(subject.timeout).to eql('PT60S')
-      end
-    end
-  end
-
-  context 'when operations timeout is set to 30 and receive timeout is set to 120' do
-    before(:each) { subject.set_timeout(30, 120) }
-    describe '#receive_timeout' do
-      it 'is set to 120s' do
-        transportclass = subject.instance_variable_get(:@xfer)
-        expect(transportclass.receive_timeout).to eql(120)
-      end
-    end
-    describe '#op_timeout' do
-      it 'is set to 30s' do
-        expect(subject.timeout).to eql('PT30S')
-      end
-    end
-  end
-
-  context 'when max_env_size is set to 614400' do
-    before(:each) { subject.max_env_size(614_400) }
-    describe '@max_env_sz' do
-      it 'is set to 614400' do
-        expect(subject.instance_variable_get('@max_env_sz')).to eq(614_400)
-      end
-    end
-  end
-
-  context 'when locale is set to en-CA' do
-    before(:each) { subject.locale('en-CA') }
-    describe '@locale' do
-      it 'is set to en-CA' do
-        expect(subject.instance_variable_get('@locale')).to eq('en-CA')
-      end
-    end
-  end
-
-  context 'default' do
-    describe '#receive_timeout' do
-      it 'should be 3600ms' do
-        transportclass = subject.instance_variable_get(:@xfer)
-        expect(transportclass.receive_timeout).to eql(3600)
-      end
-    end
-    describe '#timeout' do
-      it 'should be 60s' do
-        expect(subject.timeout).to eql('PT60S')
-      end
-    end
-    describe '@locale' do
-      it 'should be en-US' do
-        expect(subject.instance_variable_get('@locale')).to eq('en-US')
-      end
-    end
-    describe '@max_env_sz' do
-      it 'should be 153600' do
-        expect(subject.instance_variable_get('@max_env_sz')).to eq(153_600)
-      end
-    end
-  end
-end
+# encoding: UTF-8
+describe 'WinRM options', unit: true do
+  let(:subject) { WinRM::WinRMWebService.new('http://localhost:55985/wsman', :plaintext) }
+
+  context 'when operations timeout is set to 60' do
+    before(:each) { subject.set_timeout(60) }
+    describe '#receive_timeout' do
+      it 'is set to 70s' do
+        transportclass = subject.instance_variable_get(:@xfer)
+        expect(transportclass.receive_timeout).to eql(70)
+      end
+    end
+    describe '#op_timeout' do
+      it 'is set to 60s' do
+        expect(subject.timeout).to eql('PT60S')
+      end
+    end
+  end
+
+  context 'when operations timeout is set to 30 and receive timeout is set to 120' do
+    before(:each) { subject.set_timeout(30, 120) }
+    describe '#receive_timeout' do
+      it 'is set to 120s' do
+        transportclass = subject.instance_variable_get(:@xfer)
+        expect(transportclass.receive_timeout).to eql(120)
+      end
+    end
+    describe '#op_timeout' do
+      it 'is set to 30s' do
+        expect(subject.timeout).to eql('PT30S')
+      end
+    end
+  end
+
+  context 'when max_env_size is set to 614400' do
+    before(:each) { subject.max_env_size(614_400) }
+    describe '@max_env_sz' do
+      it 'is set to 614400' do
+        expect(subject.instance_variable_get('@max_env_sz')).to eq(614_400)
+      end
+    end
+  end
+
+  context 'when locale is set to en-CA' do
+    before(:each) { subject.locale('en-CA') }
+    describe '@locale' do
+      it 'is set to en-CA' do
+        expect(subject.instance_variable_get('@locale')).to eq('en-CA')
+      end
+    end
+  end
+
+  context 'default' do
+    describe '#receive_timeout' do
+      it 'should be 3600ms' do
+        transportclass = subject.instance_variable_get(:@xfer)
+        expect(transportclass.receive_timeout).to eql(3600)
+      end
+    end
+    describe '#timeout' do
+      it 'should be 60s' do
+        expect(subject.timeout).to eql('PT60S')
+      end
+    end
+    describe '@locale' do
+      it 'should be en-US' do
+        expect(subject.instance_variable_get('@locale')).to eq('en-US')
+      end
+    end
+    describe '@max_env_sz' do
+      it 'should be 153600' do
+        expect(subject.instance_variable_get('@max_env_sz')).to eq(153_600)
+      end
+    end
+  end
+end

data/spec/winrm_primitives_spec.rb

@@ -1,51 +1,51 @@
-# encoding: UTF-8
-describe 'winrm client primitives' do
-  before(:all) do
-    @winrm = winrm_connection
-  end
-
-  describe 'open and close shell', integration: true do
-    it 'should #open_shell and #close_shell' do
-      sid = @winrm.open_shell
-      expect(sid).to be_a_uid
-      expect(@winrm.close_shell(sid)).to be true
-    end
-
-    it 'should #run_command and #cleanup_command' do
-      sid = @winrm.open_shell
-
-      cmd_id = @winrm.run_command(sid, 'ipconfig', %w(/all))
-      expect(sid).to be_a_uid
-
-      expect(@winrm.cleanup_command(sid, cmd_id)).to be true
-      @winrm.close_shell(sid)
-    end
-
-    it 'should #get_command_output' do
-      sid = @winrm.open_shell
-      cmd_id = @winrm.run_command(sid, 'ipconfig', %w(/all))
-
-      output = @winrm.get_command_output(sid, cmd_id)
-      expect(output).to have_exit_code 0
-      expect(output).to have_stdout_match(/.+/)
-      expect(output).to have_no_stderr
-
-      @winrm.cleanup_command(sid, cmd_id)
-      @winrm.close_shell(sid)
-    end
-
-    it 'should #get_command_output with a block' do
-      sid = @winrm.open_shell
-      cmd_id = @winrm.run_command(sid, 'ipconfig', %w(/all))
-
-      outvar = ''
-      @winrm.get_command_output(sid, cmd_id) do |stdout, _stderr|
-        outvar << stdout
-      end
-      expect(outvar).to match(/Windows IP Configuration/)
-
-      @winrm.cleanup_command(sid, cmd_id)
-      @winrm.close_shell(sid)
-    end
-  end
-end
+# encoding: UTF-8
+describe 'winrm client primitives' do
+  before(:all) do
+    @winrm = winrm_connection
+  end
+
+  describe 'open and close shell', integration: true do
+    it 'should #open_shell and #close_shell' do
+      sid = @winrm.open_shell
+      expect(sid).to be_a_uid
+      expect(@winrm.close_shell(sid)).to be true
+    end
+
+    it 'should #run_command and #cleanup_command' do
+      sid = @winrm.open_shell
+
+      cmd_id = @winrm.run_command(sid, 'ipconfig', %w(/all))
+      expect(sid).to be_a_uid
+
+      expect(@winrm.cleanup_command(sid, cmd_id)).to be true
+      @winrm.close_shell(sid)
+    end
+
+    it 'should #get_command_output' do
+      sid = @winrm.open_shell
+      cmd_id = @winrm.run_command(sid, 'ipconfig', %w(/all))
+
+      output = @winrm.get_command_output(sid, cmd_id)
+      expect(output).to have_exit_code 0
+      expect(output).to have_stdout_match(/.+/)
+      expect(output).to have_no_stderr
+
+      @winrm.cleanup_command(sid, cmd_id)
+      @winrm.close_shell(sid)
+    end
+
+    it 'should #get_command_output with a block' do
+      sid = @winrm.open_shell
+      cmd_id = @winrm.run_command(sid, 'ipconfig', %w(/all))
+
+      outvar = ''
+      @winrm.get_command_output(sid, cmd_id) do |stdout, _stderr|
+        outvar << stdout
+      end
+      expect(outvar).to match(/Windows IP Configuration/)
+
+      @winrm.cleanup_command(sid, cmd_id)
+      @winrm.close_shell(sid)
+    end
+  end
+end