PE-38219 - Installer - Windows as Jump host support - testing

Author: cathal41

plans/util/retrieve_and_upload.pp

@@ -40,6 +40,11 @@
       path   => $local_path,
     )
   }
+
+  $result_size = run_task('peadm::filesize', 'local://localhost',
+    path => $local_path,
+  )
+  $local_size = $result_size['size']
 } else {
   $exists = without_default_logging() || {
     run_command("test -e '${local_path}'", 'local://localhost',
@@ -52,14 +57,7 @@
       path   => $local_path,
     )
   }
-}
 
-if os_string  == 'windows' {
-  $result_size = run_task('peadm::filesize', 'local://localhost',
-    path => $local_path,
-  )
-  $local_size = $result_size['size']
-} else {
   $local_size = run_task('peadm::filesize', 'local://localhost',
     path => $local_path,
   ).first['size']

spec/plans/util/retrieve_and_upload_spec.rb

@@ -5,6 +5,7 @@
   include BoltSpec::Plans
 
   it 'file needs downloaded and needs uploaded' do
+    expect_task('peadm::os_identification')
     expect_command("test -e '/tmp/download'").error_with('kind' => 'nope', 'msg' => 'The command failed with exit code 1')
     expect_task('peadm::download')
     expect_task('peadm::filesize').be_called_times(2).return_for_targets(