diff --git a/bin/commands/runs.js b/bin/commands/runs.js index 8bf716f9..a4e21b5a 100644 --- a/bin/commands/runs.js +++ b/bin/commands/runs.js @@ -471,8 +471,12 @@ module.exports = function run(args, rawArgs) { utils.sendUsageReport(bsConfig, args, `${err}\n${Constants.userMessages.LOCAL_START_FAILED}`, Constants.messageTypes.ERROR, 'local_start_failed', buildReportData, rawArgs); } else { logger.error(Constants.userMessages.ZIP_UPLOAD_FAILED); - fileHelpers.deleteZip(); utils.sendUsageReport(bsConfig, args, `${err}\n${Constants.userMessages.ZIP_UPLOAD_FAILED}`, Constants.messageTypes.ERROR, 'zip_upload_failed', buildReportData, rawArgs); + try { + fileHelpers.deleteZip(); + } catch (err) { + utils.sendUsageReport(bsConfig, args, Constants.userMessages.ZIP_DELETE_FAILED, Constants.messageTypes.ERROR, 'zip_deletion_failed', buildReportData, rawArgs); + } try { fileHelpers.deletePackageArchieve(); } catch (err) { diff --git a/bin/helpers/fileHelpers.js b/bin/helpers/fileHelpers.js index cdd73932..e993c667 100644 --- a/bin/helpers/fileHelpers.js +++ b/bin/helpers/fileHelpers.js @@ -27,12 +27,15 @@ exports.fileExists = function (filePath, cb) { exports.deleteZip = () => { try { + if (!fs.existsSync(config.fileName)) { + return 0; + } fs.unlinkSync(config.fileName); logger.info(Constants.userMessages.ZIP_DELETED); return 0; } catch (err) { logger.info(Constants.userMessages.ZIP_DELETE_FAILED); - return 1; + throw err; } };