From c778df6ff684435df76bd81e4c972cccf8da6cba Mon Sep 17 00:00:00 2001 From: Inga Lovinde <52715130+inga-lovinde@users.noreply.github.com> Date: Thu, 10 Mar 2016 08:50:06 +0300 Subject: [PATCH] Fixed commenting problems --- BuildServer/lib/commenter.js | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/BuildServer/lib/commenter.js b/BuildServer/lib/commenter.js index 293e823..917d170 100644 --- a/BuildServer/lib/commenter.js +++ b/BuildServer/lib/commenter.js @@ -152,7 +152,7 @@ var getStatusMessageFromRelease = function (app, options, callback) { if (!dirExists) { return callback("Release directory not found. Probably repository hooks are not configured"); } - if (options.attemptsGetReport > 10) { + if (options.attemptsGetReport > 100) { return callback("Report file not found"); } @@ -177,14 +177,14 @@ var getStatusMessageFromRelease = function (app, options, callback) { if (report.result === "MBSNotFound") { return callback("mbs.json is not found"); } - if ((report.result.errors.$allMessages || []).length + (report.result.warns.$allMessages || []).length > 0) { + if (report.result && ((report.result.errors || {}).$allMessages || []).length + ((report.result.warns || {}).$allMessages || []).length > 0) { return callback(_.map( - report.result.errors.$allMessages || [], function(message) { return "ERR: " + message.message } + (report.result.errors || {}).$allMessages || [], function(message) { return "ERR: " + message.message } ).concat(_.map( - report.result.warns.$allMessages || [], function(message) { return "WARN: " + message.message } + (report.result.warns || {}).$allMessages || [], function(message) { return "WARN: " + message.message } )).join("\r\n")); } - if (report.err) { + if (!report.result || report.err) { return callback("CRITICAL ERROR: " + report.err); } if ((report.result.infos.$allMessages || []).length > 0) {