Merge pull request #119 from lohanidamodar/feat-node-sdk-custom-exception

Feat NodeJS Sdk custom exception

Author: eldadfux

src/SDK/Language/Node.php

@@ -66,6 +66,12 @@ public function getFiles()
                 'template'      => '/node/package.json.twig',
                 'minify'        => false,
             ],
+            [
+                'scope'         => 'default',
+                'destination'   => 'lib/exception.js',
+                'template'      => '/node/lib/exception.js.twig',
+                'minify'        => false,
+            ],
             [
                 'scope'         => 'method',
                 'destination'   => 'docs/examples/{{service.name | caseLower}}/{{method.name | caseDash}}.md',

templates/node/index.js.twig

@@ -1,10 +1,12 @@
 const Client = require('./lib/client.js');
+const {{spec.title | caseUcfirst}}Exception = require('./lib/exception.js');
 {% for service in spec.services %}
 const {{service.name | caseUcfirst}} = require('./lib/services/{{service.name | caseDash}}.js');
 {% endfor %}
 
 module.exports = {
     Client,
+    {{spec.title | caseUcfirst}}Exception,
 {% for service in spec.services %}
     {{service.name | caseUcfirst}},
 {% endfor %}

templates/node/lib/client.js.twig

@@ -1,6 +1,7 @@
 const URL = require('url').URL;
 const axios = require('axios');
 const FormData = require('form-data');
+const {{spec.title | caseUcfirst}}Exception = require('./exception.js');
 
 class Client {
 
@@ -103,10 +104,20 @@ class Client {
             data: (method.toUpperCase() === 'GET' || contentType.startsWith('multipart/form-data')) ? formData : params,
             json: (contentType.startsWith('application/json'))
         };
-
-        let response = await axios(options);
-
-        return response.data;
+        try {
+            let response = await axios(options);
+            return response.data;
+        } catch(error) {
+            if('response' in error) {
+                if('data' in error.response) {
+                    throw new {{spec.title | caseUcfirst}}Exception(error.response.data.message, error.response.status, error.response.data);
+                }else{
+                    throw new {{spec.title | caseUcfirst}}Exception(error.response.statusText, error.response.status, error.response.data);
+                }
+            }else{
+                throw new {{spec.title | caseUcfirst}}Exception(error.message);
+            }
+        }
     }
 
     flatten(data, prefix = '') {

templates/node/lib/exception.js.twig

@@ -0,0 +1,9 @@
+class {{spec.title | caseUcfirst}}Exception extends Error {
+  constructor(message, code, response) {
+    super(message);
+    this.code = code;
+    this.response = response;
+  }
+}
+
+module.exports = {{spec.title | caseUcfirst}}Exception;

tests/SDKTest.php

@@ -124,7 +124,7 @@ class SDKTest extends TestCase
                 'nodejs-12' => 'docker run --rm -v $(pwd):/app -w /app node:12.12-alpine node tests/languages/node/test.js',
                 'nodejs-14' => 'docker run --rm -v $(pwd):/app -w /app node:14.5-alpine node tests/languages/node/test.js',
             ],
-            'supportException' => false,
+            'supportException' => true,
         ],
 
         'ruby' => [

tests/languages/node/test.js

@@ -54,6 +54,17 @@ async function start() {
     response = await general.upload('string', 123, ['string in array'], fs.createReadStream(__dirname + '/../../resources/file.png'));
     console.log(response.result);
 
+    try {
+        response = await general.error400();
+    } catch(error) {
+        console.log(error.message);
+    }
+    try {
+        response = await general.error500();
+    } catch(error) {
+        console.log(error.message);
+    }
+
 }
 
 start();