ROKUPX-2770 - fix packager error

ViacomInc · Jul 30, 2019 · b8cd02a · b8cd02a
1 parent 0a16bd7
commit b8cd02a
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 5 deletions.
diff --git a/lib/roku_builder/plugins/packager.rb b/lib/roku_builder/plugins/packager.rb
@@ -161,9 +161,13 @@ def sign_package(app_name_version:, password:, stage: nil)
       out_file = File.join(@config.out[:folder], @config.out[:file])
       out_file = out_file+".pkg" unless out_file.end_with?(".pkg")
       File.open(out_file, 'w+b') {|fp| fp.write(response.body)}
-      pkg_size = File.size(out_file).to_f / 2**20
-      raise ExecutionError, "PKG file size is too large (#{pkg_size.round(2)} MB): #{out_file}" if pkg_size > 4.0
-      @logger.info("Outfile: #{out_file}")
+      if File.exist?(out_file)
+        pkg_size = File.size(out_file).to_f / 2**20
+        raise ExecutionError, "PKG file size is too large (#{pkg_size.round(2)} MB): #{out_file}" if pkg_size > 4.0
+        @logger.info("Outfile: #{out_file}")
+      else
+        @logger.warn("Outfile Missing: #{out_file}")
+      end
     end
 
     # Uses the device to generate a new signing key

diff --git a/test/roku_builder/plugins/test_packager.rb b/test/roku_builder/plugins/test_packager.rb
@@ -94,7 +94,7 @@ def test_packager_package
 
       logger.expect(:debug, nil, [String])
       io.expect(:each_line, nil)
-      logger.expect(:info, nil) do |message|
+      logger.expect(:warn, nil) do |message|
         assert_match(/#{tmp_folder}/, message)
       end
 
@@ -142,7 +142,7 @@ def test_packager_package_squash
 
       logger.expect(:debug, nil, [String])
       io.expect(:each_line, nil)
-      logger.expect(:info, nil) do |message|
+      logger.expect(:warn, nil) do |message|
         assert_match(/#{tmp_folder}/, message)
       end