winrm 1.3.0.dev.2 → 1.3.0.dev.3

data/spec/response_handler_spec.rb

@@ -41,4 +41,18 @@ describe 'response handler', :unit => true do
       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/stubs/responses/wmi_error_v2.xml

@@ -0,0 +1,41 @@
+<s:Envelope xmlns:a='http://schemas.xmlsoap.org/ws/2004/08/addressing' xmlns:e='http://schemas.xmlsoap.org/ws/2004/08/eventing' xml:lang='en-US' xmlns:n='http://schemas.xmlsoap.org/ws/2004/09/enumeration' xmlns:p='http://schemas.microsoft.com/wbem/wsman/1/wsman.xsd' xmlns:s='http://www.w3.org/2003/05/soap-envelope' xmlns:w='http://schemas.dmtf.org/wbem/wsman/1/wsman.xsd' xmlns:x='http://schemas.xmlsoap.org/ws/2004/09/transfer'>
+  <s:Header>
+    <a:Action>http://schemas.dmtf.org/wbem/wsman/1/wsman/fault</a:Action>
+    <a:MessageID>uuid:B8829021-48C3-4F27-B94B-491DAC4F29B1</a:MessageID>
+    <a:To>http://schemas.xmlsoap.org/ws/2004/08/addressing/role/anonymous</a:To>
+  </s:Header>
+  <s:Body>
+    <s:Fault>
+      <s:Code>
+        <s:Value>s:Sender</s:Value>
+        <s:Subcode>
+          <s:Value>w:QuotaLimit</s:Value>
+        </s:Subcode>
+      </s:Code>
+      <s:Reason>
+        <s:Text xml:lang='en-US'>The WS-Management service cannot process the request. The maximum number of concurrent shells for this user has been exceeded. Close existing shells or raise the quota for this user. </s:Text>
+      </s:Reason>
+      <s:Detail>
+        <p:MSFT_WmiError b:IsCIM_Error='true' xmlns:b='http://schemas.dmtf.org/wbem/wsman/1/cimbinding.xsd' xmlns:cim='http://schemas.dmtf.org/wbem/wscim/1/common' xmlns:p='http://schemas.dmtf.org/wbem/wscim/1/cim-schema/2/MSFT_WmiError' xsi:type='p:MSFT_WmiError_Type' xmlns:xsi='http://www.w3.org/2001/XMLSchema-instance'>
+          <p:CIMStatusCode xsi:type='cim:cimUnsignedInt'>27</p:CIMStatusCode>
+          <p:CIMStatusCodeDescription xsi:nil='true' xsi:type='cim:cimString'/>
+          <p:ErrorSource xsi:nil='true' xsi:type='cim:cimString'/>
+          <p:ErrorSourceFormat xsi:type='cim:cimUnsignedShort'>0</p:ErrorSourceFormat>
+          <p:ErrorType xsi:type='cim:cimUnsignedShort'>0</p:ErrorType>
+          <p:Message xsi:type='cim:cimString'>The WS-Management service cannot process the request. This user is allowed a maximum number of 30 concurrent shells, which has been exceeded. Close existing shells or raise the quota for this user. </p:Message>
+          <p:MessageID xsi:type='cim:cimString'>HRESULT 0x803381a5</p:MessageID>
+          <p:OtherErrorSourceFormat xsi:nil='true' xsi:type='cim:cimString'/>
+          <p:OtherErrorType xsi:nil='true' xsi:type='cim:cimString'/>
+          <p:OwningEntity xsi:nil='true' xsi:type='cim:cimString'/>
+          <p:PerceivedSeverity xsi:type='cim:cimUnsignedShort'>0</p:PerceivedSeverity>
+          <p:ProbableCause xsi:type='cim:cimUnsignedShort'>0</p:ProbableCause>
+          <p:ProbableCauseDescription xsi:nil='true' xsi:type='cim:cimString'/>
+          <p:error_Category xsi:type='cim:cimUnsignedInt'>30</p:error_Category>
+          <p:error_Code xsi:type='cim:cimUnsignedInt'>2150859173</p:error_Code>
+          <p:error_Type xsi:type='cim:cimString'>HRESULT</p:error_Type>
+          <p:error_WindowsErrorMessage xsi:type='cim:cimString'>The WS-Management service cannot process the request. This user is allowed a maximum number of 30 concurrent shells, which has been exceeded. Close existing shells or raise the quota for this user. </p:error_WindowsErrorMessage>
+        </p:MSFT_WmiError>
+      </s:Detail>
+    </s:Fault>
+  </s:Body>
+</s:Envelope>

data/spec/winrm_options_spec.rb

@@ -0,0 +1,75 @@
+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(614400) }
+    describe '@max_env_sz' do
+      it 'is set to 614400' do
+        expect(subject.instance_variable_get('@max_env_sz')).to eq(614400)
+      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(153600)
+      end
+    end
+  end  
+end

data/spec/winrm_primitives_spec.rb

@@ -49,22 +49,4 @@ describe "winrm client primitives" do
     end
   end
 
-  describe "simple values", :unit => true do
-    it 'should set #op_timeout' do
-      expect(@winrm.op_timeout(120)).to eq('PT2M0S')
-      expect(@winrm.op_timeout(1202)).to eq('PT20M2S')
-      expect(@winrm.op_timeout(86400)).to eq('PT24H0S')
-    end
-
-    it 'should set #max_env_size' do
-      @winrm.max_env_size(153600 * 4)
-      expect(@winrm.instance_variable_get('@max_env_sz')).to eq(614400)
-    end
-
-    it 'should set #locale' do
-      @winrm.locale('en-ca')
-      expect(@winrm.instance_variable_get('@locale')).to eq('en-ca')
-    end
-  end
-
 end

data/winrm.gemspec

@@ -26,7 +26,7 @@ Gem::Specification.new do |s|
   s.bindir = "bin"
   s.executables   = ['rwinrm']
   s.required_ruby_version	= '>= 1.9.0'
-  s.add_runtime_dependency  'gssapi', '~> 1.0'
+  s.add_runtime_dependency  'gssapi', '~> 1.2'
   s.add_runtime_dependency  'httpclient', '~> 2.2', '>= 2.2.0.2'
   s.add_runtime_dependency  'rubyntlm', '~> 0.4.0'
   s.add_runtime_dependency  'uuidtools', '~> 2.1.2'
@@ -34,8 +34,6 @@ Gem::Specification.new do |s|
   s.add_runtime_dependency  'nori', '~> 2.0'
   s.add_runtime_dependency  'gyoku', '~> 1.0'
   s.add_runtime_dependency  'builder', '>= 2.1.2'
-  s.add_runtime_dependency  'rubyzip', '~> 1.1'
-  s.add_runtime_dependency  'ruby-progressbar', '~> 1.6'
   s.add_development_dependency 'rspec', '~> 3.0.0'
   s.add_development_dependency 'rake', '~> 10.3.2'
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: winrm
 version: !ruby/object:Gem::Version
-  version: 1.3.0.dev.2
+  version: 1.3.0.dev.3
 platform: ruby
 authors:
 - Dan Wanek
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2014-11-04 00:00:00.000000000 Z
+date: 2015-01-02 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: gssapi
@@ -17,14 +17,14 @@ dependencies:
     requirements:
     - - ~>
       - !ruby/object:Gem::Version
-        version: '1.0'
+        version: '1.2'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ~>
       - !ruby/object:Gem::Version
-        version: '1.0'
+        version: '1.2'
 - !ruby/object:Gem::Dependency
   name: httpclient
   requirement: !ruby/object:Gem::Requirement
@@ -135,34 +135,6 @@ dependencies:
     - - '>='
       - !ruby/object:Gem::Version
         version: 2.1.2
-- !ruby/object:Gem::Dependency
-  name: rubyzip
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - ~>
-      - !ruby/object:Gem::Version
-        version: '1.1'
-  type: :runtime
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - ~>
-      - !ruby/object:Gem::Version
-        version: '1.1'
-- !ruby/object:Gem::Dependency
-  name: ruby-progressbar
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - ~>
-      - !ruby/object:Gem::Version
-        version: '1.6'
-  type: :runtime
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - ~>
-      - !ruby/object:Gem::Version
-        version: '1.6'
 - !ruby/object:Gem::Dependency
   name: rspec
   requirement: !ruby/object:Gem::Requirement
@@ -211,16 +183,16 @@ files:
 - README.md
 - Rakefile
 - VERSION
+- Vagrantfile
 - bin/rwinrm
 - changelog.md
 - lib/winrm.rb
 - lib/winrm/exceptions/exceptions.rb
-- lib/winrm/file_transfer.rb
-- lib/winrm/file_transfer/remote_file.rb
-- lib/winrm/file_transfer/remote_zip_file.rb
 - lib/winrm/helpers/iso8601_duration.rb
+- lib/winrm/helpers/powershell_script.rb
 - lib/winrm/http/response_handler.rb
 - lib/winrm/http/transport.rb
+- lib/winrm/output.rb
 - lib/winrm/soap_provider.rb
 - lib/winrm/winrm_service.rb
 - preamble
@@ -228,10 +200,8 @@ files:
 - spec/cmd_spec.rb
 - spec/config-example.yml
 - spec/exception_spec.rb
-- spec/file_transfer/remote_file_spec.rb
-- spec/file_transfer/remote_zip_file_spec.rb
-- spec/file_transfer_spec.rb
 - spec/matchers.rb
+- spec/output_spec.rb
 - spec/powershell_spec.rb
 - spec/response_handler_spec.rb
 - spec/spec_helper.rb
@@ -239,6 +209,8 @@ files:
 - spec/stubs/responses/open_shell_v2.xml
 - spec/stubs/responses/soap_fault_v1.xml
 - spec/stubs/responses/soap_fault_v2.xml
+- spec/stubs/responses/wmi_error_v2.xml
+- spec/winrm_options_spec.rb
 - spec/winrm_primitives_spec.rb
 - spec/wql_spec.rb
 - winrm.gemspec

data/lib/winrm/file_transfer/remote_file.rb

@@ -1,184 +0,0 @@
-require 'io/console'
-require 'json'
-require 'ruby-progressbar'
-
-module WinRM
-  class RemoteFile
-
-    attr_reader :local_path
-    attr_reader :remote_path
-    attr_reader :closed
-    attr_reader :options
-
-    def initialize(service, local_path, remote_path, opts = {})
-      @logger = Logging.logger[self]
-      @options = opts
-      @closed = false
-      @service = service
-      @shell = service.open_shell
-      @local_path = local_path
-      @remote_path = full_remote_path(local_path, remote_path)
-      @logger.debug("Creating RemoteFile of local '#{local_path}' at '#{@remote_path}'")
-    ensure
-      if !shell.nil?
-        ObjectSpace.define_finalizer( self, self.class.close(shell, service) )
-      end
-    end
-
-    def upload
-      raise WinRMUploadFailed.new("This RemoteFile is closed.") if closed
-      raise WinRMUploadFailed.new("Cannot find path: '#{local_path}'") unless File.exist?(local_path)
-
-      @remote_path, should_upload = powershell_batch do | builder |
-        builder << resolve_remote_command
-        builder << is_dirty_command
-      end
-
-      if should_upload
-        size = upload_to_remote
-        powershell_batch {|builder| builder << create_post_upload_command}
-      else
-        size = 0
-        logger.debug("Files are equal. Not copying #{local_path} to #{remote_path}")
-      end
-      size
-    end
-    
-    def close
-      service.close_shell(shell) unless shell.nil? or closed
-      @closed = true
-    end
-
-    protected
-
-    attr_reader :logger
-    attr_reader :service
-    attr_reader :shell
-
-    def self.close(shell_id, service)
-      proc { service.close_shell(shell_id) }
-    end
-
-    def full_remote_path(local_path, remote_path)
-      base_file_name = File.basename(local_path)
-      if File.basename(remote_path) != base_file_name
-        remote_path = File.join(remote_path, base_file_name)
-      end
-      remote_path
-    end
-
-    def resolve_remote_command
-      <<-EOH
-        $dest_file_path = $ExecutionContext.SessionState.Path.GetUnresolvedProviderPathFromPSPath("#{remote_path}")
-
-        if (!(Test-Path $dest_file_path)) {
-          $dest_dir = ([System.IO.Path]::GetDirectoryName($dest_file_path))
-          New-Item -ItemType directory -Force -Path $dest_dir | Out-Null
-        }
-
-        $dest_file_path
-      EOH
-    end
-
-    def is_dirty_command
-      local_md5 = Digest::MD5.file(local_path).hexdigest
-      <<-EOH
-        $dest_file_path = $ExecutionContext.SessionState.Path.GetUnresolvedProviderPathFromPSPath("#{remote_path}")
-
-        if (Test-Path $dest_file_path) {
-          $crypto_prov = new-object -TypeName System.Security.Cryptography.MD5CryptoServiceProvider
-          try {
-            $file = [System.IO.File]::Open($dest_file_path,
-              [System.IO.Filemode]::Open, [System.IO.FileAccess]::Read)
-            $guest_md5 = ([System.BitConverter]::ToString($crypto_prov.ComputeHash($file)))
-            $guest_md5 = $guest_md5.Replace("-","").ToLower()
-          }
-          finally {
-            $file.Dispose()
-          }
-          if ($guest_md5 -eq '#{local_md5}') {
-            return $false
-          }
-        }
-        if(Test-Path $dest_file_path){remove-item $dest_file_path -Force}
-        return $true
-      EOH
-    end
-
-    def upload_to_remote
-      logger.debug("Uploading '#{local_path}' to temp file '#{remote_path}'")
-      base64_host_file = Base64.encode64(IO.binread(local_path)).gsub("\n", "")
-      base64_array = base64_host_file.chars.to_a
-      unless options[:quiet]
-        console_width = IO.console.winsize[1]
-        bar = ProgressBar.create(:title => "Copying #{File.basename(local_path)}...", :total => base64_array.count, :length => console_width-1)
-      end      
-      base64_array.each_slice(8000 - remote_path.size) do |chunk|
-        cmd("echo #{chunk.join} >> \"#{remote_path}\"")
-        bar.progress += chunk.count unless options[:quiet]
-      end
-      base64_array.length
-    end
-
-    def decode_command
-      <<-EOH
-        $base64_string = Get-Content '#{remote_path}'
-        $bytes = [System.Convert]::FromBase64String($base64_string)
-        [System.IO.File]::WriteAllBytes('#{remote_path}', $bytes) | Out-Null
-      EOH
-    end
-
-    def create_post_upload_command
-      [decode_command]
-    end
-
-    def powershell_batch(&block)
-      ps_builder = []
-      yield ps_builder
-
-      commands = [ "$result = @{}" ]
-      idx = 0
-      ps_builder.flatten.each do |cmd_item|
-        commands << <<-EOH
-          $result.ret#{idx} = Invoke-Command { #{cmd_item} }
-        EOH
-        idx += 1
-      end
-      commands << "$(ConvertTo-Json $result)"
-
-      result = []
-      JSON.parse(powershell(commands.join("\n"))).each do |k,v|
-        result << v unless v.nil?
-      end
-      result unless result.empty?
-    end
-
-    def powershell(script)
-      script = "$ProgressPreference='SilentlyContinue';" + script
-      logger.debug("executing powershell script: \n#{script}")
-      script = script.encode('UTF-16LE', 'UTF-8')
-      script = Base64.strict_encode64(script)
-      cmd("powershell", ['-encodedCommand', script])
-    end
-
-    def cmd(command, arguments = [])
-      command_output = nil
-      out_stream = []
-      err_stream = []
-      service.run_command(shell, command, arguments) do |command_id|
-        command_output = service.get_command_output(shell, command_id) do |stdout, stderr|
-          out_stream << stdout if stdout
-          err_stream << stderr if stderr
-        end
-      end
-
-      if !command_output[:exitcode].zero? or !err_stream.empty?
-        raise WinRMUploadFailed,
-          :from => local_path,
-          :to => remote_path,
-          :message => command_output.inspect
-      end
-      out_stream.join.chomp
-    end
-  end
-end

data/lib/winrm/file_transfer/remote_zip_file.rb

@@ -1,60 +0,0 @@
-require 'zip'
-
-module WinRM
-  class RemoteZipFile < RemoteFile
-
-    attr_reader :archive
-
-    def initialize(service, remote_path, opts = {})
-      @logger = Logging.logger[self]
-      @archive = create_archive(remote_path)
-      @unzip_remote_path = remote_path
-      remote_path = "$env:temp/WinRM_file_transfer"
-      super(service, @archive, remote_path, opts)
-    end
-
-    def add_file(path)
-      path = path.gsub("\\","/")
-      logger.debug("adding '#{path}' to zip file")
-      raise WinRMUploadFailed.new("Cannot find path: '#{path}'") unless File.exist?(path)
-      File.directory?(path) ? glob = File.join(path, "**/*") : glob = path
-      logger.debug("iterating files in '#{glob}'")
-      Zip::File.open(archive, 'w') do |zipfile|
-        Dir.glob(glob).each do |file|
-          logger.debug("adding zip entry for '#{file}'")
-          entry = Zip::Entry.new(archive, file.sub(File.dirname(path)+'/',''), nil, nil, nil, nil, nil, nil, ::Zip::DOSTime.new(2000))
-          zipfile.add(entry,file)
-        end
-      end
-    end
-
-    protected
-
-    def create_post_upload_command
-      super << extract_zip_command
-    end
-
-    private
-
-    def create_archive(remote_path)
-      archive_folder = File.join(ENV['TMP'] || ENV['TMPDIR'] || '/tmp', 'WinRM_file_transfer_local')
-      Dir.mkdir(archive_folder) unless File.exist?(archive_folder)
-      archive = File.join(archive_folder,File.basename(remote_path))+'.zip'
-      FileUtils.rm archive, :force=>true
-
-      archive
-    end
-
-    def extract_zip_command
-      <<-EOH
-        $destination = $ExecutionContext.SessionState.Path.GetUnresolvedProviderPathFromPSPath("#{@unzip_remote_path}")
-        $shellApplication = new-object -com shell.application 
-
-        $zipPackage = $shellApplication.NameSpace('#{remote_path}') 
-        mkdir $destination -ErrorAction SilentlyContinue | Out-Null
-        $destinationFolder = $shellApplication.NameSpace($destination) 
-        $destinationFolder.CopyHere($zipPackage.Items(),0x10) | Out-Null
-      EOH
-    end
-  end
-end

data/lib/winrm/file_transfer.rb

@@ -1,39 +0,0 @@
-require 'winrm/file_transfer/remote_file'
-require 'winrm/file_transfer/remote_zip_file'
-
-module WinRM
-  # Perform file transfer operations between a local machine and winrm endpoint
-  class FileTransfer
-    # Upload one or more local files and directories to a remote directory
-    # @example copy a single directory to a winrm endpoint
-    #
-    #   WinRM::FileTransfer.upload(client, 'c:/dev/my_dir', '$env:AppData')
-    #
-    # @example copy several paths to the winrm endpoint
-    #
-    #   WinRM::FileTransfer.upload(client, ['c:/dev/file1.txt','c:/dev/dir1'], '$env:AppData')
-    #
-    # @param [WinRM::WinRMService] a winrm service client connected to the endpoint where the remote path resides
-    # @param [Array<String>] One or more paths that will be copied to the remote path. These can be files or directories to be deeply copied
-    # @param [String] The directory on the remote endpoint to copy the local items to. This path may contain powershell style environment variables
-    # @option opts [String] options to be used for the copy. Currently only :quiet is supported to suppress the progress bar
-    # @return [Fixnum] The total number of bytes copied
-    def self.upload(service, local_path, remote_path, opts = {})
-      file = nil
-      local_path = [local_path] if local_path.is_a? String
-
-      if local_path.count == 1 && !File.directory?(local_path[0])
-        file = RemoteFile.new(service, local_path[0], remote_path, opts)
-      else
-        file = RemoteZipFile.new(service, remote_path, opts)
-        local_path.each do |path|
-          file.add_file(path)
-        end
-      end
-
-      file.upload
-    ensure
-      file.close unless file.nil?
-    end
-  end
-end

data/spec/file_transfer/remote_file_spec.rb

@@ -1,71 +0,0 @@
-describe WinRM::RemoteFile, :integration => true do
-  
-  let(:this_file) { __FILE__ }
-  let(:service) { winrm_connection }
-  let(:destination) {"#{ENV['temp']}/WinRM_tests"}
-  after {
-    subject.close
-    FileUtils.rm_rf(destination)
-  }
-
-  context 'Upload a new file to directory path' do
-    subject {WinRM::RemoteFile.new(service, this_file, destination, :quiet => true)}
-
-    it 'copies the file inside the directory' do
-      expect(subject.upload).to be > 0
-      expect(File.exist?(File.join(destination, File.basename(this_file)))).to be_truthy
-    end
-    it 'copies the exact file content' do
-      expect(subject.upload).to be > 0
-      expect(File.read(File.join(destination, File.basename(this_file)))).to eq(File.read(this_file))
-    end
-
-  end
-
-  context 'Upload an identical file to directory path' do
-    subject {WinRM::RemoteFile.new(service, this_file, destination, :quiet => true)}
-    let (:next_transfer) {WinRM::RemoteFile.new(service, this_file, destination, :quiet => true)}
-
-    it 'does not copy the file' do
-      expect(subject.upload).to be > 0
-      expect(next_transfer.upload).to be == 0
-    end
-  end
-
-  context 'Upload a file to file path' do
-    subject {WinRM::RemoteFile.new(service, this_file, File.join(destination, File.basename(this_file)), :quiet => true)}
-
-    it 'copies the file to the exact path' do
-      expect(subject.upload).to be > 0
-      expect(File.exist?(File.join(destination, File.basename(this_file)))).to be_truthy
-    end
-  end
-
-  context 'Upload a new file to nested directory' do
-    let (:nested) {File.join(destination, 'nested')}
-    subject {WinRM::RemoteFile.new(service, this_file, nested, :quiet => true)}
-
-    it 'copies the file to the nested path' do
-      expect(subject.upload).to be > 0
-      expect(File.exist?(File.join(nested, File.basename(this_file)))).to be_truthy
-    end
-  end
-
-  context 'Upload a file after RemoteFile is closed' do
-    subject {WinRM::RemoteFile.new(service, this_file, destination, :quiet => true)}
-
-    it 'raises WinRMUploadFailed' do
-      expect(subject.upload).to be > 0
-      subject.close
-      expect{subject.upload}.to raise_error(WinRM::WinRMUploadFailed)
-    end
-  end
-
-  context 'Upload a bad path' do
-    subject {WinRM::RemoteFile.new(service, 'c:/some/bad/path', destination, :quiet => true)}
-
-    it 'raises WinRMUploadFailed' do
-      expect { subject.upload }.to raise_error(WinRM::WinRMUploadFailed)
-    end
-  end
-end