Use non-zero exit codes when something goes wrong

Author: cfernandes

README.md

@@ -179,6 +179,9 @@ encoded.
 synchronization of these certificates will be skipped and a warning will
 be printed.
 
+### Release process
+The release process uses the `release-me` NPM module
+
 ### Notes
 To my understanding, the dependency graph can be visualized like this:
 

bin/operations/backup.js

@@ -47,13 +47,13 @@ module.exports = async function backup(filename, url, username, password) {
       snis
     };
 
-    // TODO: implement the backup process for SNIs
-
     fs.writeFileSync(adjustedFileName, JSON.stringify(results, null, 4));
-    console.log(`Backup data has been written to ${adjustedFileName}`.green)
+    console.log(`Backup data has been written to ${adjustedFileName}`.green);
+    console.log(' '.reset);
   } catch (e) {
-    console.log(e.message.red)
-  } finally {
+    console.log(e.message.red);
     console.log(' '.reset);
+    // non-zero exit code for Unix
+    process.exit(1);
   }
 };

bin/operations/synchronization.js

@@ -209,10 +209,12 @@ module.exports = async function synchronization(filename, url, username, passwor
 
     console.log('Consumer and Credentials updates complete'.green);
     console.log('Synchronization process complete'.green.bold);
+    console.log(' '.reset);
   } catch (e) {
     console.log(e);
     console.log(e.message.red);
-  } finally {
     console.log(' '.reset);
+    // non-zero exit code for Unix
+    process.exit(1);
   }
 };

bin/operations/teardown.js

@@ -33,9 +33,11 @@ module.exports = async function teardown(url, username, password) {
     const certificateIds = certificates.map(certificate => certificate.id);
     certificateIds.map(async id => await kong.certificates.removeCertificate(id).catch(err => console.log(err.message.grey)));
     console.log('Certificate deletion complete'.red.reset);
+    console.log('Teardown process complete'.reset);
   } catch (e) {
-    console.log(e.message.red)
-  } finally {
+    console.log(e.message.red);
     console.log('Teardown process complete'.reset);
+    // non-zero exit code for Unix
+    process.exit(1);
   }
 };