Skip to content

Commit 0b9e87d

Browse files
SouravSourav
Sourav
authored and
Sourav
committed
change to logger.error incase of errors
1 parent 1da9162 commit 0b9e87d

File tree

5 files changed

+16
-16
lines changed

5 files changed

+16
-16
lines changed

‎bin/commands/info.js‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -47,7 +47,7 @@ module.exports = function info(args, rawArgs) {
4747
errorCode = 'api_failed_build_info';
4848

4949
logger.info(message);
50-
logger.info(utils.formatRequest(err, resp, body));
50+
logger.error(utils.formatRequest(err, resp, body));
5151
} else {
5252
let build = null;
5353
try {
@@ -82,7 +82,7 @@ module.exports = function info(args, rawArgs) {
8282
message = Constants.userMessages.BUILD_INFO_FAILED;
8383
logger.error(message);
8484
}
85-
logger.info(utils.formatRequest(err, resp, body));
85+
logger.error(utils.formatRequest(err, resp, body));
8686
} else {
8787
messageType = Constants.messageTypes.SUCCESS;
8888
message = `Build info for build id: \n ${JSON.stringify(

‎bin/helpers/build.js‎

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -25,7 +25,7 @@ const createBuild = (bsConfig, zip) => {
2525

2626
request.post(options, function (err, resp, body) {
2727
if (err) {
28-
logger.info(utils.formatRequest(err, resp, body));
28+
logger.error(utils.formatRequest(err, resp, body));
2929
reject(err);
3030
} else {
3131
let build = null;
@@ -39,11 +39,11 @@ const createBuild = (bsConfig, zip) => {
3939
if (build) {
4040
resolve(build.message);
4141
} else {
42-
logger.info(utils.formatRequest(err, resp, body));
42+
logger.error(utils.formatRequest(err, resp, body));
4343
reject(Constants.userMessages.API_DEPRECATED);
4444
}
4545
} else if (resp.statusCode != 201) {
46-
logger.info(utils.formatRequest(err, resp, body));
46+
logger.error(utils.formatRequest(err, resp, body));
4747
if (build) {
4848
reject(`${Constants.userMessages.BUILD_FAILED} Error: ${build.message}`);
4949
} else {

‎bin/helpers/buildArtifacts.js‎

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -162,7 +162,7 @@ const sendUpdatesToBstack = async (bsConfig, buildId, args, options, rawArgs) =>
162162
request.post(options, function (err, resp, data) {
163163
if(err) {
164164
utils.sendUsageReport(bsConfig, args, err, Constants.messageTypes.ERROR, 'api_failed_build_artifacts_status_update', null, rawArgs);
165-
logger.info(utils.formatRequest(err, resp, body));
165+
logger.error(utils.formatRequest(err, resp, body));
166166
reject(err);
167167
} else {
168168
try {
@@ -204,7 +204,7 @@ exports.downloadBuildArtifacts = async (bsConfig, buildId, args, rawArgs) => {
204204
let buildDetails = null;
205205
request.get(options, async function (err, resp, body) {
206206
if(err) {
207-
logger.info(utils.formatRequest(err, resp, body));
207+
logger.error(utils.formatRequest(err, resp, body));
208208
utils.sendUsageReport(bsConfig, args, err, Constants.messageTypes.ERROR, 'api_failed_build_artifacts', null, rawArgs);
209209
process.exitCode = Constants.ERROR_EXIT_CODE;
210210
} else {
@@ -213,7 +213,7 @@ exports.downloadBuildArtifacts = async (bsConfig, buildId, args, rawArgs) => {
213213
if(resp.statusCode != 200) {
214214
logger.error('Downloading the build artifacts failed.');
215215
logger.error(`Error: Request failed with status code ${resp.statusCode}`)
216-
logger.info(utils.formatRequest(err, resp, body));
216+
logger.error(utils.formatRequest(err, resp, body));
217217
utils.sendUsageReport(bsConfig, args, buildDetails, Constants.messageTypes.ERROR, 'api_failed_build_artifacts', null, rawArgs);
218218
process.exitCode = Constants.ERROR_EXIT_CODE;
219219
} else {
@@ -223,7 +223,7 @@ exports.downloadBuildArtifacts = async (bsConfig, buildId, args, rawArgs) => {
223223
messageType = Constants.messageTypes.ERROR;
224224
message = Constants.userMessages.DOWNLOAD_BUILD_ARTIFACTS_FAILED.replace('<build-id>', buildId).replace('<machine-count>', BUILD_ARTIFACTS_FAIL_COUNT);
225225
logger.error(message);
226-
logger.info(utils.formatRequest(err, resp, body));
226+
logger.error(utils.formatRequest(err, resp, body));
227227
process.exitCode = Constants.ERROR_EXIT_CODE;
228228
} else {
229229
messageType = Constants.messageTypes.SUCCESS;
@@ -245,7 +245,7 @@ exports.downloadBuildArtifacts = async (bsConfig, buildId, args, rawArgs) => {
245245
}
246246
utils.sendUsageReport(bsConfig, args, err, messageType, errorCode, null, rawArgs);
247247
logger.error(`Error: Request failed with status code ${resp.statusCode}`)
248-
logger.info(utils.formatRequest(err, resp, body));
248+
logger.error(utils.formatRequest(err, resp, body));
249249
process.exitCode = Constants.ERROR_EXIT_CODE;
250250
}
251251
}

‎bin/helpers/reporterHTML.js‎

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -113,7 +113,7 @@ let reportGenerator = (bsConfig, buildId, args, rawArgs, cb) => {
113113

114114
logger.error('Generating the build report failed.');
115115
logger.error(message);
116-
logger.info(utils.formatRequest(err, resp, body));
116+
logger.error(utils.formatRequest(err, resp, body));
117117

118118
utils.sendUsageReport(bsConfig, args, message, messageType, errorCode, null, rawArgs);
119119
return;
@@ -136,7 +136,7 @@ let reportGenerator = (bsConfig, buildId, args, rawArgs, cb) => {
136136
message = Constants.userMessages.API_DEPRECATED;
137137
logger.info(message);
138138
}
139-
logger.info(utils.formatRequest(err, resp, body));
139+
logger.error(utils.formatRequest(err, resp, body));
140140
} else if (resp.statusCode === 422) {
141141
messageType = Constants.messageTypes.ERROR;
142142
errorCode = 'api_failed_build_generate_report';
@@ -148,7 +148,7 @@ let reportGenerator = (bsConfig, buildId, args, rawArgs, cb) => {
148148
response = {message: message};
149149
}
150150
logger.error(response.message);
151-
logger.info(utils.formatRequest(err, resp, body));
151+
logger.error(utils.formatRequest(err, resp, body));
152152
} else if (resp.statusCode != 200) {
153153
messageType = Constants.messageTypes.ERROR;
154154
errorCode = 'api_failed_build_generate_report';
@@ -163,7 +163,7 @@ let reportGenerator = (bsConfig, buildId, args, rawArgs, cb) => {
163163
message = Constants.userMessages.BUILD_GENERATE_REPORT_FAILED.replace('<build-id>', buildId);
164164
logger.error(message);
165165
}
166-
logger.info(utils.formatRequest(err, resp, body));
166+
logger.error(utils.formatRequest(err, resp, body));
167167
} else {
168168
messageType = Constants.messageTypes.SUCCESS;
169169
message = `Report for build: ${buildId} was successfully created.`;

‎bin/helpers/zipUpload.js‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -52,7 +52,7 @@ const uploadSuits = (bsConfig, filePath, opts, obj) => {
5252
var r = request.post(options, function (err, resp, body) {
5353

5454
if (err) {
55-
logger.info(utils.formatRequest(err, resp, body));
55+
logger.error(utils.formatRequest(err, resp, body));
5656
reject(err);
5757
} else {
5858
try {
@@ -61,7 +61,7 @@ const uploadSuits = (bsConfig, filePath, opts, obj) => {
6161
responseData = {};
6262
}
6363
if (resp.statusCode != 200) {
64-
logger.info(utils.formatRequest(err, resp, body));
64+
logger.error(utils.formatRequest(err, resp, body));
6565
if (resp.statusCode == 401) {
6666
if (responseData && responseData["error"]) {
6767
responseData["time"] = Date.now() - obj.startTime;

0 commit comments

Comments
 (0)