winrm-fs 1.0.1 → 1.0.2

data/lib/winrm-fs/scripts/check_files.ps1.erb

@@ -1,49 +1,49 @@
-$hash_file = <%= hash_file %>
-
-Function Cleanup($disposable) {
-  if (($disposable -ne $null) -and ($disposable.GetType().GetMethod("Dispose") -ne $null)) {
-    $disposable.Dispose()
-  }
-}
-
-Function Check-Files($h) {
-  return $h.GetEnumerator() | ForEach-Object {
-    $dst = Unresolve-Path $_.Value.target
-    $dst_changed = $false
-    if(Test-Path $dst -PathType Container) {
-      $dst_changed = $true
-      $dst = Join-Path $dst $_.Value.src_basename
-    }
-    New-Object psobject -Property @{
-      chk_exists = ($exists = Test-Path $dst -PathType Leaf)
-      src_md5 = ($sMd5 = $_.Key)
-      dst_md5 = ($dMd5 = if ($exists) { Get-MD5Sum $dst } else { $null })
-      chk_dirty = ($dirty = if ($sMd5 -ne $dMd5) { $true } else { $false })
-      verifies = if ($dirty -eq $false) { $true } else { $false }
-      target_is_folder = $dst_changed
-    }
-  } | Select-Object -Property chk_exists,src_md5,dst_md5,chk_dirty,verifies,target_is_folder
-}
-
-Function Get-MD5Sum($src) {
-  Try {
-    $c = [System.Security.Cryptography.MD5]::Create()
-    $bytes = $c.ComputeHash(($in = (Get-Item $src).OpenRead()))
-    return ([System.BitConverter]::ToString($bytes)).Replace("-", "").ToLower()
-  } 
-  Finally {
-    Cleanup $c
-    Cleanup $in
-  }
-}
-
-Function Unresolve-Path($path) {
-  if ($path -eq $null) {
-    return $null
-  }
-  else {
-    return $ExecutionContext.SessionState.Path.GetUnresolvedProviderPathFromPSPath($path)
-  }
-}
-
-Check-Files $hash_file | ConvertTo-Csv -NoTypeInformation
+$hash_file = <%= hash_file %>
+
+Function Cleanup($disposable) {
+  if (($disposable -ne $null) -and ($disposable.GetType().GetMethod("Dispose") -ne $null)) {
+    $disposable.Dispose()
+  }
+}
+
+Function Check-Files($h) {
+  return $h.GetEnumerator() | ForEach-Object {
+    $dst = Unresolve-Path $_.Value.target
+    $dst_changed = $false
+    if(Test-Path $dst -PathType Container) {
+      $dst_changed = $true
+      $dst = Join-Path $dst $_.Value.src_basename
+    }
+    New-Object psobject -Property @{
+      chk_exists = ($exists = Test-Path $dst -PathType Leaf)
+      src_md5 = ($sMd5 = $_.Key)
+      dst_md5 = ($dMd5 = if ($exists) { Get-MD5Sum $dst } else { $null })
+      chk_dirty = ($dirty = if ($sMd5 -ne $dMd5) { $true } else { $false })
+      verifies = if ($dirty -eq $false) { $true } else { $false }
+      target_is_folder = $dst_changed
+    }
+  } | Select-Object -Property chk_exists,src_md5,dst_md5,chk_dirty,verifies,target_is_folder
+}
+
+Function Get-MD5Sum($src) {
+  Try {
+    $c = [System.Security.Cryptography.MD5]::Create()
+    $bytes = $c.ComputeHash(($in = (Get-Item $src).OpenRead()))
+    return ([System.BitConverter]::ToString($bytes)).Replace("-", "").ToLower()
+  } 
+  Finally {
+    Cleanup $c
+    Cleanup $in
+  }
+}
+
+Function Unresolve-Path($path) {
+  if ($path -eq $null) {
+    return $null
+  }
+  else {
+    return $ExecutionContext.SessionState.Path.GetUnresolvedProviderPathFromPSPath($path)
+  }
+}
+
+Check-Files $hash_file | ConvertTo-Csv -NoTypeInformation

data/lib/winrm-fs/scripts/checksum.ps1.erb

@@ -1,13 +1,13 @@
-$p = $ExecutionContext.SessionState.Path
-$path = $p.GetUnresolvedProviderPathFromPSPath("<%= path %>")
-
-if (Test-Path $path -PathType Leaf) {
-  $cryptoProv = [System.Security.Cryptography.MD5]::Create()
-  $file = [System.IO.File]::Open($path,
-      [System.IO.Filemode]::Open, [System.IO.FileAccess]::Read)
-  $md5 = ([System.BitConverter]::ToString($cryptoProv.ComputeHash($file)))
-  $md5 = $md5.Replace("-","").ToLower()
-  $file.Close()
-
-  Write-Output $md5
-}
+$p = $ExecutionContext.SessionState.Path
+$path = $p.GetUnresolvedProviderPathFromPSPath("<%= path %>")
+
+if (Test-Path $path -PathType Leaf) {
+  $cryptoProv = [System.Security.Cryptography.<%= digest %>]::Create()
+  $file = [System.IO.File]::Open($path,
+      [System.IO.Filemode]::Open, [System.IO.FileAccess]::Read)
+  $digest = ([System.BitConverter]::ToString($cryptoProv.ComputeHash($file)))
+  $digest = $digest.Replace("-","").ToLower()
+  $file.Close()
+
+  Write-Output $digest
+}

data/lib/winrm-fs/scripts/create_dir.ps1.erb

@@ -1,6 +1,6 @@
-$p = $ExecutionContext.SessionState.Path
-$path = $p.GetUnresolvedProviderPathFromPSPath("<%= path %>")
-if (!(Test-Path $path)) {
-  New-Item -ItemType Directory -Force -Path $path | Out-Null
-}
-exit 0
+$p = $ExecutionContext.SessionState.Path
+$path = $p.GetUnresolvedProviderPathFromPSPath("<%= path %>")
+if (!(Test-Path $path)) {
+  New-Item -ItemType Directory -Force -Path $path | Out-Null
+}
+exit 0

data/lib/winrm-fs/scripts/delete.ps1.erb

@@ -1,6 +1,6 @@
-$p = $ExecutionContext.SessionState.Path
-$path = $p.GetUnresolvedProviderPathFromPSPath("<%= path %>")
-if (Test-Path $path) {
-  Remove-Item $path -Force -Recurse
-}
-exit 0
+$p = $ExecutionContext.SessionState.Path
+$path = $p.GetUnresolvedProviderPathFromPSPath("<%= path %>")
+if (Test-Path $path) {
+  Remove-Item $path -Force -Recurse
+}
+exit 0

data/lib/winrm-fs/scripts/download.ps1.erb

@@ -1,8 +1,8 @@
-$p = $ExecutionContext.SessionState.Path
-$path = $p.GetUnresolvedProviderPathFromPSPath("<%= path %>")
-if (Test-Path $path -PathType Leaf) {
-  $bytes = [System.convert]::ToBase64String([System.IO.File]::ReadAllBytes($path))
-  Write-Host $bytes
-  exit 0
-}
-exit 1
+$p = $ExecutionContext.SessionState.Path
+$path = $p.GetUnresolvedProviderPathFromPSPath("<%= path %>")
+if (Test-Path $path -PathType Leaf) {
+  $bytes = [System.convert]::ToBase64String([System.IO.File]::ReadAllBytes($path))
+  Write-Host $bytes
+  exit 0
+}
+exit 1

data/lib/winrm-fs/scripts/exists.ps1.erb

@@ -1,10 +1,10 @@
-$p = $ExecutionContext.SessionState.Path
-$path = $p.GetUnresolvedProviderPathFromPSPath("<%= path %>")
-if (Test-Path $path) {
-  $true
-  exit 0
-}
-else {
-  $false
-  exit 1
-}
+$p = $ExecutionContext.SessionState.Path
+$path = $p.GetUnresolvedProviderPathFromPSPath("<%= path %>")
+if (Test-Path $path) {
+  $true
+  exit 0
+}
+else {
+  $false
+  exit 1
+}

data/lib/winrm-fs/scripts/extract_files.ps1.erb

@@ -1,52 +1,52 @@
-trap {
-  $e = $_.Exception
-  $e.InvocationInfo.ScriptName
-  do {
-    $e.Message
-    $e = $e.InnerException
-  } while ($e)
-  break
-}
-
-function folder($path){
-  $path | ? {-not (test-path $_)} | % {$null = mkdir $_}
-}
-
-Function Decode-Files($hash) {
-  foreach ($key in $hash.keys) {
-    $value = $hash[$key]
-    $tzip, $dst = $Value["tmpzip"], $Value["dst"]
-    if ($tzip) {Unzip-File $tzip $dst}
-    New-Object psobject -Property @{dst=$dst;src_md5=$key;tmpzip=$tzip}
-  }
-}
-
-Function Unzip-File($src, $dst) {
-  $unpack = $src -replace '\.zip'
-  $dst_parent = Split-Path -Path $dst -Parent
-  if(!(Test-Path $dst_parent)) { $dst = $dst_parent }
-  folder $unpack, $dst
-  try {
-    try{
-      [IO.Compression.ZipFile]::ExtractToDirectory($src, $unpack)
-    }
-    catch {
-      Add-Type -AssemblyName System.IO.Compression.FileSystem -ErrorAction Stop
-      [IO.Compression.ZipFile]::ExtractToDirectory($src, $unpack)
-    }
-  }
-  catch {
-    Try {
-      $s = New-Object -ComObject Shell.Application
-      ($s.NameSpace($unpack)).CopyHere(($s.NameSpace($src)).Items(), 0x610)
-    }
-    Finally {
-      [void][Runtime.Interopservices.Marshal]::ReleaseComObject($s)
-    }
-  }
-  dir $unpack | cp -dest "$dst/" -force -recurse
-  rm $unpack -recurse -force
-}
-
-$hash_file = <%= hash_file %>
-Decode-Files $hash_file | ConvertTo-Csv -NoTypeInformation
+trap {
+  $e = $_.Exception
+  $e.InvocationInfo.ScriptName
+  do {
+    $e.Message
+    $e = $e.InnerException
+  } while ($e)
+  break
+}
+
+function folder($path){
+  $path | ? {-not (test-path $_)} | % {$null = mkdir $_}
+}
+
+Function Decode-Files($hash) {
+  foreach ($key in $hash.keys) {
+    $value = $hash[$key]
+    $tzip, $dst = $Value["tmpzip"], $Value["dst"]
+    if ($tzip) {Unzip-File $tzip $dst}
+    New-Object psobject -Property @{dst=$dst;src_md5=$key;tmpzip=$tzip}
+  }
+}
+
+Function Unzip-File($src, $dst) {
+  $unpack = $src -replace '\.zip'
+  $dst_parent = Split-Path -Path $dst -Parent
+  if(!(Test-Path $dst_parent)) { $dst = $dst_parent }
+  folder $unpack, $dst
+  try {
+    try{
+      [IO.Compression.ZipFile]::ExtractToDirectory($src, $unpack)
+    }
+    catch {
+      Add-Type -AssemblyName System.IO.Compression.FileSystem -ErrorAction Stop
+      [IO.Compression.ZipFile]::ExtractToDirectory($src, $unpack)
+    }
+  }
+  catch {
+    Try {
+      $s = New-Object -ComObject Shell.Application
+      ($s.NameSpace($unpack)).CopyHere(($s.NameSpace($src)).Items(), 0x610)
+    }
+    Finally {
+      [void][Runtime.Interopservices.Marshal]::ReleaseComObject($s)
+    }
+  }
+  dir $unpack | cp -dest "$dst/" -force -recurse
+  rm $unpack -recurse -force
+}
+
+$hash_file = <%= hash_file %>
+Decode-Files $hash_file | ConvertTo-Csv -NoTypeInformation

data/lib/winrm-fs/scripts/scripts.rb

@@ -1,31 +1,31 @@
-# encoding: UTF-8
-#
-# Copyright 2015 Shawn Neal <sneal@sneal.net>
-#
-# 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 'erubis'
-
-module WinRM
-  module FS
-    # PS1 scripts
-    module Scripts
-      def self.render(template, context)
-        template_path = File.expand_path(
-          "#{File.dirname(__FILE__)}/#{template}.ps1.erb")
-        template = File.read(template_path)
-        Erubis::Eruby.new(template).result(context)
-      end
-    end
-  end
-end
+# encoding: UTF-8
+#
+# Copyright 2015 Shawn Neal <sneal@sneal.net>
+#
+# 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 'erubis'
+
+module WinRM
+  module FS
+    # PS1 scripts
+    module Scripts
+      def self.render(template, context)
+        template_path = File.expand_path(
+          "#{File.dirname(__FILE__)}/#{template}.ps1.erb")
+        template = File.read(template_path)
+        Erubis::Eruby.new(template).result(context)
+      end
+    end
+  end
+end

data/spec/config-example.yml

@@ -1,3 +1,3 @@
-endpoint: "http://localhost:55985/wsman"
-user: vagrant
+endpoint: "http://localhost:55985/wsman"
+user: vagrant
 password: vagrant

data/spec/integration/file_manager_spec.rb

@@ -1,224 +1,224 @@
-# encoding: UTF-8
-require 'pathname'
-
-describe WinRM::FS::FileManager do
-  let(:dest_dir) { File.join(subject.temp_dir, "winrm_#{rand(2**16)}") }
-  let(:temp_upload_dir) { '$env:TEMP/winrm-upload' }
-  let(:spec_dir) { File.expand_path(File.dirname(File.dirname(__FILE__))) }
-  let(:this_file) { Pathname.new(__FILE__) }
-  let(:service) { winrm_connection }
-
-  subject { WinRM::FS::FileManager.new(service) }
-
-  before(:each) do
-    expect(subject.delete(dest_dir)).to be true
-    expect(subject.delete(temp_upload_dir)).to be true
-  end
-
-  context 'exists?' do
-    it 'should exist' do
-      expect(subject.exists?('c:/windows')).to be true
-      expect(subject.exists?('c:/foobar')).to be false
-    end
-  end
-
-  context 'create and delete dir' do
-    it 'should create the directory recursively' do
-      subdir = File.join(dest_dir, 'subdir1', 'subdir2')
-      expect(subject.create_dir(subdir)).to be true
-      expect(subject.exists?(subdir)).to be true
-      expect(subject.create_dir(subdir)).to be true
-      expect(subject.delete(subdir)).to be true
-      expect(subject.exists?(subdir)).to be false
-    end
-  end
-
-  context 'temp_dir' do
-    it 'should return the remote users temp dir' do
-      expect(subject.temp_dir).to match(%r{C:/Users/\S+/AppData/Local/Temp})
-    end
-  end
-
-  context 'upload file' do
-    let(:dest_file) { Pathname.new(File.join(dest_dir, File.basename(this_file))) }
-
-    before(:each) do
-      expect(subject.delete(dest_dir)).to be true
-    end
-
-    it 'should upload the specified file' do
-      subject.upload(this_file, dest_file)
-      expect(subject).to have_created(dest_file).with_content(this_file)
-    end
-
-    it 'should upload to root of the c: drive' do
-      subject.upload(this_file, 'c:/winrmtest.rb')
-      expect(subject).to have_created('c:/winrmtest.rb').with_content(this_file)
-      subject.delete('c:/winrmtest.rb')
-    end
-
-    it 'should upload just filename' do
-      subject.upload(this_file, 'winrmtest.rb')
-      expect(subject).to have_created('winrmtest.rb').with_content(this_file)
-      subject.delete('winrmtest.rb')
-    end
-
-    it 'should upload using relative file path' do
-      subject.upload('./spec/integration/file_manager_spec.rb', dest_file)
-      expect(subject).to have_created(dest_file).with_content(this_file)
-    end
-
-    it 'should upload to the specified directory' do
-      subject.create_dir(dest_dir)
-      subject.upload(this_file, dest_dir)
-      expect(subject).to have_created(dest_file).with_content(this_file)
-    end
-
-    it 'should treat extensionless target as file if not an existing directory' do
-      subject.upload(this_file, dest_dir)
-      expect(subject).to have_created(dest_dir).with_content(this_file)
-    end
-
-    it 'should create extensionless source under target dir if target dir exists' do
-      subject.create_dir(dest_dir)
-      src_file = File.expand_path('../../Gemfile', File.dirname(__FILE__))
-      subject.upload(src_file, dest_dir)
-      expect(subject).to have_created(File.join(dest_dir, 'Gemfile')).with_content(src_file)
-    end
-
-    it 'should upload to the specified directory with env var' do
-      subject.upload(this_file, '$env:Temp')
-      expected_dest_file = File.join(subject.temp_dir, File.basename(this_file))
-      expect(subject).to have_created(expected_dest_file).with_content(this_file)
-    end
-
-    it 'should upload to Program Files sub dir' do
-      subject.create_dir('$env:ProgramFiles/foo')
-      subject.upload(this_file, '$env:ProgramFiles/foo')
-      expect(subject).to have_created('c:/Program Files/foo/file_manager_spec.rb') \
-        .with_content(this_file)
-    end
-
-    it 'should upload to the specified nested directory' do
-      dest_sub_dir = File.join(dest_dir, 'subdir')
-      subject.create_dir(dest_sub_dir)
-      dest_sub_dir_file = File.join(dest_sub_dir, File.basename(this_file))
-      subject.upload(this_file, dest_sub_dir)
-      expect(subject).to have_created(dest_sub_dir_file).with_content(this_file)
-    end
-
-    it 'yields progress data' do
-      block_called = false
-      total_bytes_copied = 0
-      total = subject.upload(this_file, dest_file) do \
-        |bytes_copied, total_bytes, local_path, remote_path|
-        expect(total_bytes).to be > 0
-        total_bytes_copied = bytes_copied
-        expect(local_path).to eq(this_file.to_s)
-        expect(remote_path).to eq(dest_file.to_s)
-        block_called = true
-      end
-      expect(total_bytes_copied).to eq(total)
-      expect(block_called).to be true
-      expect(total).to be > 0
-    end
-
-    it 'should not upload when content matches' do
-      subject.upload(this_file, dest_dir)
-      bytes_uploaded = subject.upload(this_file, dest_dir)
-      expect(bytes_uploaded).to eq 0
-    end
-
-    it 'should upload when content differs' do
-      matchers_file = File.join(spec_dir, 'matchers.rb')
-      subject.upload(matchers_file, dest_file)
-      bytes_uploaded = subject.upload(this_file, dest_file)
-      expect(bytes_uploaded).to be > 0
-    end
-
-    it 'raises WinRMUploadError when a bad source path is specified' do
-      expect { subject.upload('c:/some/non-existant/path/foo', dest_file) }.to raise_error
-    end
-  end
-
-  context 'upload empty file' do
-    let(:empty_src_file) { Tempfile.new('empty') }
-    let(:dest_file) { File.join(dest_dir, 'emptyfile.txt') }
-
-    it 'creates a new empty file' do
-      expect(subject.upload(empty_src_file.path, dest_file)).to be 0
-      expect(subject).to have_created(dest_file).with_content('')
-    end
-
-    it 'overwrites an existing file' do
-      expect(subject.upload(this_file, dest_file)).to be > 0
-      expect(subject.upload(empty_src_file.path, dest_file)).to be 0
-      expect(subject).to have_created(dest_file).with_content('')
-    end
-  end
-
-  context 'upload directory' do
-    let(:root_dir) { File.expand_path('../../', File.dirname(__FILE__)) }
-    let(:winrm_fs_dir) { File.join(root_dir, 'lib/winrm-fs') }
-    let(:core_dir) { File.join(root_dir, 'lib/winrm-fs/core') }
-    let(:scripts_dir) { File.join(root_dir, 'lib/winrm-fs/scripts') }
-
-    it 'copies the directory contents recursively when directory does not exist' do
-      bytes_uploaded = subject.upload(winrm_fs_dir, dest_dir)
-      expect(bytes_uploaded).to be > 0
-
-      Dir.glob(winrm_fs_dir + '/**/*.rb').each do |host_file|
-        host_file_rel = Pathname.new(host_file).relative_path_from(
-          Pathname.new(winrm_fs_dir)
-        ).to_s
-        remote_file = File.join(dest_dir, host_file_rel)
-        expect(subject).to have_created(remote_file).with_content(host_file)
-      end
-    end
-
-    it 'copies the directory recursively when directory does exist' do
-      subject.create_dir(dest_dir)
-      bytes_uploaded = subject.upload(winrm_fs_dir, dest_dir)
-      expect(bytes_uploaded).to be > 0
-
-      Dir.glob(winrm_fs_dir + '/**/*.rb').each do |host_file|
-        host_file_rel = Pathname.new(host_file).relative_path_from(
-          Pathname.new(winrm_fs_dir).dirname
-        ).to_s
-        remote_file = File.join(dest_dir, host_file_rel)
-        expect(subject).to have_created(remote_file).with_content(host_file)
-      end
-    end
-
-    it 'does not copy the directory when content is the same' do
-      subject.upload(winrm_fs_dir, dest_dir)
-      bytes_uploaded = subject.upload(winrm_fs_dir, dest_dir)
-      expect(bytes_uploaded).to eq 0
-    end
-
-    it 'unzips the directory when cached content is the same' do
-      subject.upload(winrm_fs_dir, dest_dir)
-      subject.delete(dest_dir)
-      expect(subject.exists?(dest_dir)).to be false
-      subject.upload(winrm_fs_dir, dest_dir)
-      expect(subject.exists?(dest_dir)).to be true
-    end
-
-    it 'unzips multiple directories when cached content is the same for all' do
-      subject.create_dir(dest_dir)
-      subject.upload([core_dir, scripts_dir], dest_dir)
-      subject.delete(dest_dir)
-      expect(subject.exists?(dest_dir)).to be false
-      subject.create_dir(dest_dir)
-      subject.upload([core_dir, scripts_dir], dest_dir)
-      expect(subject.exists?(File.join(dest_dir, 'core'))).to be true
-      expect(subject.exists?(File.join(dest_dir, 'scripts'))).to be true
-    end
-
-    it 'copies the directory when content differs' do
-      subject.upload(winrm_fs_dir, dest_dir)
-      bytes_uploaded = subject.upload(core_dir, dest_dir)
-      expect(bytes_uploaded).to be > 0
-    end
-  end
-end
+# encoding: UTF-8
+require 'pathname'
+
+describe WinRM::FS::FileManager do
+  let(:dest_dir) { File.join(subject.temp_dir, "winrm_#{rand(2**16)}") }
+  let(:temp_upload_dir) { '$env:TEMP/winrm-upload' }
+  let(:spec_dir) { File.expand_path(File.dirname(File.dirname(__FILE__))) }
+  let(:this_file) { Pathname.new(__FILE__) }
+  let(:service) { winrm_connection }
+
+  subject { WinRM::FS::FileManager.new(service) }
+
+  before(:each) do
+    expect(subject.delete(dest_dir)).to be true
+    expect(subject.delete(temp_upload_dir)).to be true
+  end
+
+  context 'exists?' do
+    it 'should exist' do
+      expect(subject.exists?('c:/windows')).to be true
+      expect(subject.exists?('c:/foobar')).to be false
+    end
+  end
+
+  context 'create and delete dir' do
+    it 'should create the directory recursively' do
+      subdir = File.join(dest_dir, 'subdir1', 'subdir2')
+      expect(subject.create_dir(subdir)).to be true
+      expect(subject.exists?(subdir)).to be true
+      expect(subject.create_dir(subdir)).to be true
+      expect(subject.delete(subdir)).to be true
+      expect(subject.exists?(subdir)).to be false
+    end
+  end
+
+  context 'temp_dir' do
+    it 'should return the remote users temp dir' do
+      expect(subject.temp_dir).to match(%r{C:/Users/\S+/AppData/Local/Temp})
+    end
+  end
+
+  context 'upload file' do
+    let(:dest_file) { Pathname.new(File.join(dest_dir, File.basename(this_file))) }
+
+    before(:each) do
+      expect(subject.delete(dest_dir)).to be true
+    end
+
+    it 'should upload the specified file' do
+      subject.upload(this_file, dest_file)
+      expect(subject).to have_created(dest_file).with_content(this_file)
+    end
+
+    it 'should upload to root of the c: drive' do
+      subject.upload(this_file, 'c:/winrmtest.rb')
+      expect(subject).to have_created('c:/winrmtest.rb').with_content(this_file)
+      subject.delete('c:/winrmtest.rb')
+    end
+
+    it 'should upload just filename' do
+      subject.upload(this_file, 'winrmtest.rb')
+      expect(subject).to have_created('winrmtest.rb').with_content(this_file)
+      subject.delete('winrmtest.rb')
+    end
+
+    it 'should upload using relative file path' do
+      subject.upload('./spec/integration/file_manager_spec.rb', dest_file)
+      expect(subject).to have_created(dest_file).with_content(this_file)
+    end
+
+    it 'should upload to the specified directory' do
+      subject.create_dir(dest_dir)
+      subject.upload(this_file, dest_dir)
+      expect(subject).to have_created(dest_file).with_content(this_file)
+    end
+
+    it 'should treat extensionless target as file if not an existing directory' do
+      subject.upload(this_file, dest_dir)
+      expect(subject).to have_created(dest_dir).with_content(this_file)
+    end
+
+    it 'should create extensionless source under target dir if target dir exists' do
+      subject.create_dir(dest_dir)
+      src_file = File.expand_path('../../Gemfile', File.dirname(__FILE__))
+      subject.upload(src_file, dest_dir)
+      expect(subject).to have_created(File.join(dest_dir, 'Gemfile')).with_content(src_file)
+    end
+
+    it 'should upload to the specified directory with env var' do
+      subject.upload(this_file, '$env:Temp')
+      expected_dest_file = File.join(subject.temp_dir, File.basename(this_file))
+      expect(subject).to have_created(expected_dest_file).with_content(this_file)
+    end
+
+    it 'should upload to Program Files sub dir' do
+      subject.create_dir('$env:ProgramFiles/foo')
+      subject.upload(this_file, '$env:ProgramFiles/foo')
+      expect(subject).to have_created('c:/Program Files/foo/file_manager_spec.rb') \
+        .with_content(this_file)
+    end
+
+    it 'should upload to the specified nested directory' do
+      dest_sub_dir = File.join(dest_dir, 'subdir')
+      subject.create_dir(dest_sub_dir)
+      dest_sub_dir_file = File.join(dest_sub_dir, File.basename(this_file))
+      subject.upload(this_file, dest_sub_dir)
+      expect(subject).to have_created(dest_sub_dir_file).with_content(this_file)
+    end
+
+    it 'yields progress data' do
+      block_called = false
+      total_bytes_copied = 0
+      total = subject.upload(this_file, dest_file) do \
+        |bytes_copied, total_bytes, local_path, remote_path|
+        expect(total_bytes).to be > 0
+        total_bytes_copied = bytes_copied
+        expect(local_path).to eq(this_file.to_s)
+        expect(remote_path).to eq(dest_file.to_s)
+        block_called = true
+      end
+      expect(total_bytes_copied).to eq(total)
+      expect(block_called).to be true
+      expect(total).to be > 0
+    end
+
+    it 'should not upload when content matches' do
+      subject.upload(this_file, dest_dir)
+      bytes_uploaded = subject.upload(this_file, dest_dir)
+      expect(bytes_uploaded).to eq 0
+    end
+
+    it 'should upload when content differs' do
+      matchers_file = File.join(spec_dir, 'matchers.rb')
+      subject.upload(matchers_file, dest_file)
+      bytes_uploaded = subject.upload(this_file, dest_file)
+      expect(bytes_uploaded).to be > 0
+    end
+
+    it 'raises WinRMUploadError when a bad source path is specified' do
+      expect { subject.upload('c:/some/non-existant/path/foo', dest_file) }.to raise_error
+    end
+  end
+
+  context 'upload empty file' do
+    let(:empty_src_file) { Tempfile.new('empty') }
+    let(:dest_file) { File.join(dest_dir, 'emptyfile.txt') }
+
+    it 'creates a new empty file' do
+      expect(subject.upload(empty_src_file.path, dest_file)).to be 0
+      expect(subject).to have_created(dest_file).with_content('')
+    end
+
+    it 'overwrites an existing file' do
+      expect(subject.upload(this_file, dest_file)).to be > 0
+      expect(subject.upload(empty_src_file.path, dest_file)).to be 0
+      expect(subject).to have_created(dest_file).with_content('')
+    end
+  end
+
+  context 'upload directory' do
+    let(:root_dir) { File.expand_path('../../', File.dirname(__FILE__)) }
+    let(:winrm_fs_dir) { File.join(root_dir, 'lib/winrm-fs') }
+    let(:core_dir) { File.join(root_dir, 'lib/winrm-fs/core') }
+    let(:scripts_dir) { File.join(root_dir, 'lib/winrm-fs/scripts') }
+
+    it 'copies the directory contents recursively when directory does not exist' do
+      bytes_uploaded = subject.upload(winrm_fs_dir, dest_dir)
+      expect(bytes_uploaded).to be > 0
+
+      Dir.glob(winrm_fs_dir + '/**/*.rb').each do |host_file|
+        host_file_rel = Pathname.new(host_file).relative_path_from(
+          Pathname.new(winrm_fs_dir)
+        ).to_s
+        remote_file = File.join(dest_dir, host_file_rel)
+        expect(subject).to have_created(remote_file).with_content(host_file)
+      end
+    end
+
+    it 'copies the directory recursively when directory does exist' do
+      subject.create_dir(dest_dir)
+      bytes_uploaded = subject.upload(winrm_fs_dir, dest_dir)
+      expect(bytes_uploaded).to be > 0
+
+      Dir.glob(winrm_fs_dir + '/**/*.rb').each do |host_file|
+        host_file_rel = Pathname.new(host_file).relative_path_from(
+          Pathname.new(winrm_fs_dir).dirname
+        ).to_s
+        remote_file = File.join(dest_dir, host_file_rel)
+        expect(subject).to have_created(remote_file).with_content(host_file)
+      end
+    end
+
+    it 'does not copy the directory when content is the same' do
+      subject.upload(winrm_fs_dir, dest_dir)
+      bytes_uploaded = subject.upload(winrm_fs_dir, dest_dir)
+      expect(bytes_uploaded).to eq 0
+    end
+
+    it 'unzips the directory when cached content is the same' do
+      subject.upload(winrm_fs_dir, dest_dir)
+      subject.delete(dest_dir)
+      expect(subject.exists?(dest_dir)).to be false
+      subject.upload(winrm_fs_dir, dest_dir)
+      expect(subject.exists?(dest_dir)).to be true
+    end
+
+    it 'unzips multiple directories when cached content is the same for all' do
+      subject.create_dir(dest_dir)
+      subject.upload([core_dir, scripts_dir], dest_dir)
+      subject.delete(dest_dir)
+      expect(subject.exists?(dest_dir)).to be false
+      subject.create_dir(dest_dir)
+      subject.upload([core_dir, scripts_dir], dest_dir)
+      expect(subject.exists?(File.join(dest_dir, 'core'))).to be true
+      expect(subject.exists?(File.join(dest_dir, 'scripts'))).to be true
+    end
+
+    it 'copies the directory when content differs' do
+      subject.upload(winrm_fs_dir, dest_dir)
+      bytes_uploaded = subject.upload(core_dir, dest_dir)
+      expect(bytes_uploaded).to be > 0
+    end
+  end
+end