winrm 1.7.2 → 1.7.3

data/spec/powershell_spec.rb

@@ -1,97 +1,97 @@
-# encoding: UTF-8
-describe 'winrm client powershell', integration: true do
-  before(:all) do
-    @winrm = winrm_connection
-  end
-
-  describe 'ipconfig' do
-    subject(:output) { @winrm.powershell('ipconfig') }
-    it { should have_exit_code 0 }
-    it { should have_stdout_match(/Windows IP Configuration/) }
-    it { should have_no_stderr }
-  end
-
-  describe 'echo \'hello world\' using apostrophes' do
-    subject(:output) { @winrm.powershell("echo 'hello world'") }
-    it { should have_exit_code 0 }
-    it { should have_stdout_match(/hello world/) }
-    it { should have_no_stderr }
-  end
-
-  describe 'dir with incorrect argument /z' do
-    subject(:output) { @winrm.powershell('dir /z') }
-    it { should have_exit_code 1 }
-    it { should have_no_stdout }
-  end
-
-  describe 'Math area calculation' do
-    subject(:output) do
-      @winrm.powershell(<<-EOH
-        $diameter = 4.5
-        $area = [Math]::pow([Math]::PI * ($diameter/2), 2)
-        Write-Host $area
-      EOH
-      )
-    end
-    it { should have_exit_code 0 }
-    it { should have_stdout_match(/49.9648722805149/) }
-    it { should have_no_stderr }
-  end
-
-  describe 'ipconfig with a block' do
-    subject(:stdout) do
-      outvar = ''
-      @winrm.powershell('ipconfig') do |stdout, _stderr|
-        outvar << stdout
-      end
-      outvar
-    end
-    it { should match(/Windows IP Configuration/) }
-  end
-
-  describe 'capturing output from Write-Host and Write-Error' do
-    subject(:output) do
-      script = <<-eos
-      Write-Host 'Hello'
-      $host.ui.WriteErrorLine(', world!')
-      eos
-
-      @captured_stdout = ''
-      @captured_stderr = ''
-      @winrm.powershell(script) do |stdout, stderr|
-        @captured_stdout << stdout if stdout
-        @captured_stderr << stderr if stderr
-      end
-    end
-
-    it 'should have stdout' do
-      expect(output.stdout).to eq("Hello\n")
-      expect(output.stdout).to eq(@captured_stdout)
-    end
-
-    it 'should have stderr' do
-      # TODO: Option to parse CLIXML
-      # expect(output.output).to eq("Hello\n, world!")
-      # expect(output.stderr).to eq(", world!")
-      expect(output.stderr).to eq(
-        "#< CLIXML\r\n<Objs Version=\"1.1.0.1\" " \
-        "xmlns=\"http://schemas.microsoft.com/powershell/2004/04\">" \
-        "<S S=\"Error\">, world!_x000D__x000A_</S></Objs>")
-      expect(output.stderr).to eq(@captured_stderr)
-    end
-
-    it 'should have output' do
-      # TODO: Option to parse CLIXML
-      # expect(output.output).to eq("Hello\n, world!")
-      expect(output.output).to eq("Hello\n#< CLIXML\r\n<Objs Version=\"1.1.0.1\" " \
-        "xmlns=\"http://schemas.microsoft.com/powershell/2004/04\">" \
-        "<S S=\"Error\">, world!_x000D__x000A_</S></Objs>")
-    end
-  end
-
-  describe 'it should handle utf-8 characters' do
-    subject(:output) { @winrm.powershell('echo "✓1234-äöü"') }
-    it { should have_exit_code 0 }
-    it { should have_stdout_match(/✓1234-äöü/) }
-  end
-end
+# encoding: UTF-8
+describe 'winrm client powershell', integration: true do
+  before(:all) do
+    @winrm = winrm_connection
+  end
+
+  describe 'ipconfig' do
+    subject(:output) { @winrm.powershell('ipconfig') }
+    it { should have_exit_code 0 }
+    it { should have_stdout_match(/Windows IP Configuration/) }
+    it { should have_no_stderr }
+  end
+
+  describe 'echo \'hello world\' using apostrophes' do
+    subject(:output) { @winrm.powershell("echo 'hello world'") }
+    it { should have_exit_code 0 }
+    it { should have_stdout_match(/hello world/) }
+    it { should have_no_stderr }
+  end
+
+  describe 'dir with incorrect argument /z' do
+    subject(:output) { @winrm.powershell('dir /z') }
+    it { should have_exit_code 1 }
+    it { should have_no_stdout }
+  end
+
+  describe 'Math area calculation' do
+    subject(:output) do
+      @winrm.powershell(<<-EOH
+        $diameter = 4.5
+        $area = [Math]::pow([Math]::PI * ($diameter/2), 2)
+        Write-Host $area
+      EOH
+      )
+    end
+    it { should have_exit_code 0 }
+    it { should have_stdout_match(/49.9648722805149/) }
+    it { should have_no_stderr }
+  end
+
+  describe 'ipconfig with a block' do
+    subject(:stdout) do
+      outvar = ''
+      @winrm.powershell('ipconfig') do |stdout, _stderr|
+        outvar << stdout
+      end
+      outvar
+    end
+    it { should match(/Windows IP Configuration/) }
+  end
+
+  describe 'capturing output from Write-Host and Write-Error' do
+    subject(:output) do
+      script = <<-eos
+      Write-Host 'Hello'
+      $host.ui.WriteErrorLine(', world!')
+      eos
+
+      @captured_stdout = ''
+      @captured_stderr = ''
+      @winrm.powershell(script) do |stdout, stderr|
+        @captured_stdout << stdout if stdout
+        @captured_stderr << stderr if stderr
+      end
+    end
+
+    it 'should have stdout' do
+      expect(output.stdout).to eq("Hello\n")
+      expect(output.stdout).to eq(@captured_stdout)
+    end
+
+    it 'should have stderr' do
+      # TODO: Option to parse CLIXML
+      # expect(output.output).to eq("Hello\n, world!")
+      # expect(output.stderr).to eq(", world!")
+      expect(output.stderr).to eq(
+        "#< CLIXML\r\n<Objs Version=\"1.1.0.1\" " \
+        "xmlns=\"http://schemas.microsoft.com/powershell/2004/04\">" \
+        "<S S=\"Error\">, world!_x000D__x000A_</S></Objs>")
+      expect(output.stderr).to eq(@captured_stderr)
+    end
+
+    it 'should have output' do
+      # TODO: Option to parse CLIXML
+      # expect(output.output).to eq("Hello\n, world!")
+      expect(output.output).to eq("Hello\n#< CLIXML\r\n<Objs Version=\"1.1.0.1\" " \
+        "xmlns=\"http://schemas.microsoft.com/powershell/2004/04\">" \
+        "<S S=\"Error\">, world!_x000D__x000A_</S></Objs>")
+    end
+  end
+
+  describe 'it should handle utf-8 characters' do
+    subject(:output) { @winrm.powershell('echo "✓1234-äöü"') }
+    it { should have_exit_code 0 }
+    it { should have_stdout_match(/✓1234-äöü/) }
+  end
+end

data/spec/response_handler_spec.rb

@@ -1,59 +1,59 @@
-# encoding: UTF-8
-require 'winrm/http/response_handler'
-
-describe 'response handler', unit: true do
-  %w(v1, v2).each do |winrm_version|
-    let(:soap_fault) { File.read("spec/stubs/responses/soap_fault_#{winrm_version}.xml") }
-    let(:open_shell) { File.read("spec/stubs/responses/open_shell_#{winrm_version}.xml") }
-
-    describe "successful 200 #{winrm_version} response" do
-      it 'returns an xml doc' do
-        handler = WinRM::ResponseHandler.new(open_shell, 200)
-        xml_doc = handler.parse_to_xml
-        expect(xml_doc).to be_instance_of(REXML::Document)
-        expect(xml_doc.to_s).to eq(REXML::Document.new(open_shell).to_s)
-      end
-    end
-
-    describe "failed 500 #{winrm_version} response" do
-      it 'raises a WinRMHTTPTransportError' do
-        handler = WinRM::ResponseHandler.new('', 500)
-        expect { handler.parse_to_xml }.to raise_error(WinRM::WinRMHTTPTransportError)
-      end
-    end
-
-    describe "failed 401 #{winrm_version} response" do
-      it 'raises a WinRMAuthorizationError' do
-        handler = WinRM::ResponseHandler.new('', 401)
-        expect { handler.parse_to_xml }.to raise_error(WinRM::WinRMAuthorizationError)
-      end
-    end
-
-    describe "failed 400 #{winrm_version} response" do
-      it 'raises a WinRMWSManFault' do
-        handler = WinRM::ResponseHandler.new(soap_fault, 400)
-        begin
-          handler.parse_to_xml
-        rescue WinRM::WinRMWSManFault => e
-          expect(e.fault_code).to eq('2150858778')
-          expect(e.fault_description).to include(
-            'The specified class does not exist in the given namespace')
-        end
-      end
-    end
-  end
-
-  describe 'failed 500 WMI error response' do
-    let(:wmi_error) { File.read('spec/stubs/responses/wmi_error_v2.xml') }
-
-    it 'raises a WinRMWMIError' do
-      handler = WinRM::ResponseHandler.new(wmi_error, 500)
-      begin
-        handler.parse_to_xml
-      rescue WinRM::WinRMWMIError => e
-        expect(e.error_code).to eq('2150859173')
-        expect(e.error).to include('The WS-Management service cannot process the request.')
-      end
-    end
-  end
-end
+# encoding: UTF-8
+require 'winrm/http/response_handler'
+
+describe 'response handler', unit: true do
+  %w(v1, v2).each do |winrm_version|
+    let(:soap_fault) { File.read("spec/stubs/responses/soap_fault_#{winrm_version}.xml") }
+    let(:open_shell) { File.read("spec/stubs/responses/open_shell_#{winrm_version}.xml") }
+
+    describe "successful 200 #{winrm_version} response" do
+      it 'returns an xml doc' do
+        handler = WinRM::ResponseHandler.new(open_shell, 200)
+        xml_doc = handler.parse_to_xml
+        expect(xml_doc).to be_instance_of(REXML::Document)
+        expect(xml_doc.to_s).to eq(REXML::Document.new(open_shell).to_s)
+      end
+    end
+
+    describe "failed 500 #{winrm_version} response" do
+      it 'raises a WinRMHTTPTransportError' do
+        handler = WinRM::ResponseHandler.new('', 500)
+        expect { handler.parse_to_xml }.to raise_error(WinRM::WinRMHTTPTransportError)
+      end
+    end
+
+    describe "failed 401 #{winrm_version} response" do
+      it 'raises a WinRMAuthorizationError' do
+        handler = WinRM::ResponseHandler.new('', 401)
+        expect { handler.parse_to_xml }.to raise_error(WinRM::WinRMAuthorizationError)
+      end
+    end
+
+    describe "failed 400 #{winrm_version} response" do
+      it 'raises a WinRMWSManFault' do
+        handler = WinRM::ResponseHandler.new(soap_fault, 400)
+        begin
+          handler.parse_to_xml
+        rescue WinRM::WinRMWSManFault => e
+          expect(e.fault_code).to eq('2150858778')
+          expect(e.fault_description).to include(
+            'The specified class does not exist in the given namespace')
+        end
+      end
+    end
+  end
+
+  describe 'failed 500 WMI error response' do
+    let(:wmi_error) { File.read('spec/stubs/responses/wmi_error_v2.xml') }
+
+    it 'raises a WinRMWMIError' do
+      handler = WinRM::ResponseHandler.new(wmi_error, 500)
+      begin
+        handler.parse_to_xml
+      rescue WinRM::WinRMWMIError => e
+        expect(e.error_code).to eq('2150859173')
+        expect(e.error).to include('The WS-Management service cannot process the request.')
+      end
+    end
+  end
+end

data/spec/spec_helper.rb

@@ -1,73 +1,73 @@
-# encoding: UTF-8
-require 'rubygems'
-require 'bundler/setup'
-require 'winrm'
-require 'json'
-require_relative 'matchers'
-
-# Creates a WinRM connection for integration tests
-module ConnectionHelper
-  # rubocop:disable AbcSize
-  def winrm_connection
-    winrm = WinRM::WinRMWebService.new(
-      config[:endpoint], config[:auth_type].to_sym, config[:options])
-    winrm.logger.level = :error
-    winrm
-  end
-  # rubocop:enable AbcSize
-
-  def config
-    @config ||= begin
-      cfg = symbolize_keys(YAML.load(File.read(winrm_config_path)))
-      cfg[:options].merge!(basic_auth_only: true) unless cfg[:auth_type].eql? :kerberos
-      merge_environment!(cfg)
-      cfg
-    end
-  end
-
-  def merge_environment!(config)
-    merge_config_option_from_environment(config, 'user')
-    merge_config_option_from_environment(config, 'pass')
-    merge_config_option_from_environment(config, 'no_ssl_peer_verification')
-    if ENV['use_ssl_peer_fingerprint']
-      config[:options][:ssl_peer_fingerprint] = ENV['winrm_cert']
-    end
-    config[:endpoint] = ENV['winrm_endpoint'] if ENV['winrm_endpoint']
-  end
-
-  def merge_config_option_from_environment(config, key)
-    env_key = 'winrm_' + key
-    config[:options][key.to_sym] = ENV[env_key] if ENV[env_key]
-  end
-
-  def winrm_config_path
-    # Copy config-example.yml to config.yml and edit for your local configuration
-    path = File.expand_path("#{File.dirname(__FILE__)}/config.yml")
-    unless File.exist?(path)
-      # user hasn't done this, so use sane defaults for unit tests
-      path = File.expand_path("#{File.dirname(__FILE__)}/config-example.yml")
-    end
-    path
-  end
-
-  # rubocop:disable Metrics/MethodLength
-  def symbolize_keys(hash)
-    hash.each_with_object({}) do |(key, value), result|
-      new_key = case key
-                when String then key.to_sym
-                else key
-                end
-      new_value = case value
-                  when Hash then symbolize_keys(value)
-                  else value
-                  end
-      result[new_key] = new_value
-      result
-    end
-  end
-  # rubocop:enable Metrics/MethodLength
-end
-
-RSpec.configure do |config|
-  config.include(ConnectionHelper)
-end
+# encoding: UTF-8
+require 'rubygems'
+require 'bundler/setup'
+require 'winrm'
+require 'json'
+require_relative 'matchers'
+
+# Creates a WinRM connection for integration tests
+module ConnectionHelper
+  # rubocop:disable AbcSize
+  def winrm_connection
+    winrm = WinRM::WinRMWebService.new(
+      config[:endpoint], config[:auth_type].to_sym, config[:options])
+    winrm.logger.level = :error
+    winrm
+  end
+  # rubocop:enable AbcSize
+
+  def config
+    @config ||= begin
+      cfg = symbolize_keys(YAML.load(File.read(winrm_config_path)))
+      cfg[:options].merge!(basic_auth_only: true) unless cfg[:auth_type].eql? :kerberos
+      merge_environment!(cfg)
+      cfg
+    end
+  end
+
+  def merge_environment!(config)
+    merge_config_option_from_environment(config, 'user')
+    merge_config_option_from_environment(config, 'pass')
+    merge_config_option_from_environment(config, 'no_ssl_peer_verification')
+    if ENV['use_ssl_peer_fingerprint']
+      config[:options][:ssl_peer_fingerprint] = ENV['winrm_cert']
+    end
+    config[:endpoint] = ENV['winrm_endpoint'] if ENV['winrm_endpoint']
+  end
+
+  def merge_config_option_from_environment(config, key)
+    env_key = 'winrm_' + key
+    config[:options][key.to_sym] = ENV[env_key] if ENV[env_key]
+  end
+
+  def winrm_config_path
+    # Copy config-example.yml to config.yml and edit for your local configuration
+    path = File.expand_path("#{File.dirname(__FILE__)}/config.yml")
+    unless File.exist?(path)
+      # user hasn't done this, so use sane defaults for unit tests
+      path = File.expand_path("#{File.dirname(__FILE__)}/config-example.yml")
+    end
+    path
+  end
+
+  # rubocop:disable Metrics/MethodLength
+  def symbolize_keys(hash)
+    hash.each_with_object({}) do |(key, value), result|
+      new_key = case key
+                when String then key.to_sym
+                else key
+                end
+      new_value = case value
+                  when Hash then symbolize_keys(value)
+                  else value
+                  end
+      result[new_key] = new_value
+      result
+    end
+  end
+  # rubocop:enable Metrics/MethodLength
+end
+
+RSpec.configure do |config|
+  config.include(ConnectionHelper)
+end