From 8fd9b54f851b923c2ffdfce8bd7fbeee844fd9e9 Mon Sep 17 00:00:00 2001
From: kakha urigashvili <kakuri@amazon.com>
Date: Wed, 20 May 2020 14:32:11 -0700
Subject: [PATCH] fix: add handling for error http status code for version
 check

---
 lib/commands/abstract-command.js            |  5 +++--
 test/unit/commands/abstract-command-test.js | 18 ++++++++++++++++++
 2 files changed, 21 insertions(+), 2 deletions(-)

diff --git a/lib/commands/abstract-command.js b/lib/commands/abstract-command.js
index f7b76a25..80618783 100644
--- a/lib/commands/abstract-command.js
+++ b/lib/commands/abstract-command.js
@@ -189,8 +189,9 @@ class AbstractCommand {
             url: `${CONSTANTS.NPM_REGISTRY_URL_BASE}/${CONSTANTS.APPLICATION_NAME}/latest`,
             method: CONSTANTS.HTTP_REQUEST.VERB.GET
         }, 'GET_NPM_REGISTRY', doDebug, (err, response) => {
-            if (err) {
-                Messenger.getInstance().error(`Failed to get the latest version for ${CONSTANTS.APPLICATION_NAME} from NPM registry.\n${err}\n`);
+            if (err || response.statusCode > 300) {
+                const error = err || `Http Status Code: ${response.statusCode}.`;
+                Messenger.getInstance().error(`Failed to get the latest version for ${CONSTANTS.APPLICATION_NAME} from NPM registry.\n${error}\n`);
             } else {
                 const BANNER_WITH_HASH = '##########################################################################';
                 const latestVersion = JSON.parse(response.body).version;
diff --git a/test/unit/commands/abstract-command-test.js b/test/unit/commands/abstract-command-test.js
index 5069b682..b0cbaa34 100644
--- a/test/unit/commands/abstract-command-test.js
+++ b/test/unit/commands/abstract-command-test.js
@@ -342,6 +342,24 @@ describe('Command test - AbstractCommand class', () => {
             });
         });
 
+        it('| http client request error status code , should warn it out and pass the process', (done) => {
+            // setup
+            httpClient.request.yields(undefined, { statusCode: 400 });
+            // call
+            AbstractCommand.prototype._remindsIfNewVersion(TEST_DO_DEBUG_FALSE, undefined, (err) => {
+                // verify
+                expect(httpClient.request.args[0][0].url).equal(
+                    `${CONSTANTS.NPM_REGISTRY_URL_BASE}/${CONSTANTS.APPLICATION_NAME}/latest`
+                );
+                expect(httpClient.request.args[0][0].method).equal(CONSTANTS.HTTP_REQUEST.VERB.GET);
+                expect(errorStub.args[0][0]).equal(
+                    `Failed to get the latest version for ${CONSTANTS.APPLICATION_NAME} from NPM registry.\nHttp Status Code: 400.\n`
+                );
+                expect(err).equal(undefined);
+                done();
+            });
+        });
+
         it('| new major version released, should error out and pass the process', (done) => {
             // setup
             const latestVersion = `${currentMajor + 1}.0.0`;