Fixing the logic of capturing the error

Author: suryart

bin/helpers/reporterHTML.js

@@ -112,18 +112,6 @@ let reportGenerator = (bsConfig, buildId, args, cb) => {
       errorCode = 'api_failed_build_report';
 
       logger.error('Generating the build report failed.');
-
-      if (resp.statusCode == 422) {
-        try {
-          response = JSON.parse(body);
-          message = response.message;
-        } catch (error) {
-          logger.error(`Error generating the report: ${error}`);
-          response = {message: message};
-        }
-        logger.error(response.message);
-      }
-
       logger.error(message);
 
       utils.sendUsageReport(bsConfig, args, message, messageType, errorCode);
@@ -147,6 +135,15 @@ let reportGenerator = (bsConfig, buildId, args, cb) => {
         message = Constants.userMessages.API_DEPRECATED;
         logger.info(message);
       }
+    } else if (resp.statusCode === 422) {
+      try {
+        response = JSON.parse(body);
+        message = response.message;
+      } catch (error) {
+        logger.error(`Error generating the report: ${error}`);
+        response = {message: message};
+      }
+      logger.error(response.message);
     } else if (resp.statusCode != 200) {
       messageType = Constants.messageTypes.ERROR;
       errorCode = 'api_failed_build_generate_report';