CH38497 improve error handling (#30)

* added error handling for missing req files

* updated error handling to print then throw

* updated error message, moved try/catch to app.ts

* updated changelog and version #

Author: AdamTranquilla

CHANGELOG.md

@@ -1,5 +1,9 @@
 # Changelog
 
+## 1.1.2 (May 19, 2021)
+
+Added error handling for missing configuration files
+
 ## 1.1.1 (April 14, 2020)
 
 Move `update-notifier` to dependencies and exclude from ncc build

package.json

@@ -1,7 +1,7 @@
 {
   "name": "awsx",
   "description": "AWS CLI profile switcher with MFA support",
-  "version": "1.1.1",
+  "version": "1.1.2",
   "author": "Neo Financial Engineering <engineering@neofinancial.com>",
   "license": "MIT",
   "repository": {

src/app.ts

@@ -6,6 +6,7 @@ import { STS } from 'aws-sdk';
 import { promisify } from 'util';
 
 import {
+  configFileCheck,
   backupConfig,
   initConfig,
   getProfileNames,
@@ -620,6 +621,12 @@ const status = async (): Promise<void> => {
 };
 
 const awsx = (): void => {
+  try {
+    configFileCheck();
+  } catch (error) {
+    process.exit(1);
+  }
+
   updateNotifier({ pkg }).notify();
 
   yargs

src/config.ts

@@ -12,6 +12,23 @@ const AWSX_PROFILE_PATH = path.join(AWSX_HOME, 'profiles');
 const AWS_CREDENTIALS_PATH = path.join(AWS_HOME, 'credentials');
 const AWS_CONFIG_PATH = path.join(AWS_HOME, 'config');
 
+const configFileCheck = (): void => {
+  const hasCredentials = fs.existsSync(AWS_CREDENTIALS_PATH);
+  const hasConfig = fs.existsSync(AWS_CONFIG_PATH);
+
+  !hasCredentials && console.error(`You are missing a required file at: ${AWS_CREDENTIALS_PATH}`);
+
+  !hasConfig && console.error(`You are missing a required file at: ${AWS_CONFIG_PATH}`);
+
+  if (!hasCredentials || !hasConfig) {
+    console.error(
+      `Run aws configure to fix this error: https://docs.aws.amazon.com/cli/latest/userguide/cli-configure-quickstart.html`
+    );
+
+    throw new Error('Missing a required configuration file');
+  }
+};
+
 const copyFileIfExists = (sourcePath: string, destPath: string): void => {
   if (fs.existsSync(sourcePath)) {
     fs.copyFileSync(sourcePath, destPath);
@@ -244,6 +261,7 @@ const deleteAssumeRoleProfile = (profileName: string): void => {
 };
 
 export {
+  configFileCheck,
   backupConfig,
   initConfig,
   isMfaSessionStillValid,