Add currency error handler for the API

gabrielgodoy · Jan 4, 2017 · d152bcc · d152bcc
1 parent 5022612
commit d152bcc
Showing 1 changed file with 10 additions and 3 deletions.
diff --git a/options/optCurrency.js b/options/optCurrency.js
@@ -98,12 +98,19 @@ function optCurrency(command) {
   }
 
   request(apiURL, (error, response, body) => {
-    const apiResponse = JSON.parse(body);
+    let apiResponse;
+    try {
+      apiResponse = JSON.parse(body);
+    } catch (parseError) {
+      console.log(chalk.red('Something went wrong in the API. Try in a few minutes'));
+      return parseError;
+    }
+
     if ('error' in apiResponse) {
       console.log(chalk.red('It was not possible to retrieve what you want'));
-      return;
+      return false;
     }
-    console.log(`
+    return console.log(`
 ${chalk.yellow('Base currency')} ${getCountryIcon(apiResponse.base)}  ${chalk.cyan(apiResponse.base)}
 -------------\nCurrency Rates\n
 ${formatRates(apiResponse.rates)}`);