Merge branch 'fix/ParseError' into release/0.0.1

Author: stewones

src/ParseError.js

@@ -20,9 +20,25 @@ class ParseError extends Error {
   constructor(code, message) {
     super(message);
     this.code = code;
+
+    const stringify = obj => {
+      let log = '';
+      for (const k in obj) {
+        log += `${obj[k]} `;
+      }
+      return log.trim();
+    };
+
     Object.defineProperty(this, 'message', {
       enumerable: true,
-      value: typeof message === 'string' ? message : JSON.stringify(message),
+      value:
+        typeof message === 'string'
+          ? message
+          : typeof message === 'object' &&
+            typeof message.toString === 'function' &&
+            !message.toString().includes('[object Object]')
+            ? message.toString()
+            : stringify(message),
     });
   }
 

src/__tests__/ParseError-test.js

@@ -28,11 +28,49 @@ describe('ParseError', () => {
     });
   });
 
-  it('message must be a string', () => {
-    const someRandomError = { code: 420, message: 'time to chill' };
+  it('message can be a string', () => {
+    const someRandomError = 'oh no';
+
+    const error = new ParseError(1337, someRandomError);
+
+    expect(JSON.parse(JSON.stringify(error))).toEqual({
+      message: someRandomError,
+      code: 1337,
+    });
+  });
+
+  it('message can be an object passed trough some external dependency', () => {
+    const someRandomError = { code: '420', message: 'time to chill', status: '🎮' };
+
+    const error = new ParseError(1337, someRandomError);
+
+    expect(JSON.parse(JSON.stringify(error))).toEqual({
+      message: '420 time to chill 🎮',
+      code: 1337,
+    });
+  });
+
+  it('message can be an Error instance *receiving a string* passed trough some external dependency', () => {
+    const someRandomError = new Error('good point');
+
     const error = new ParseError(1337, someRandomError);
+
+    expect(JSON.parse(JSON.stringify(error))).toEqual({
+      message: 'Error: good point',
+      code: 1337,
+    });
+  });
+
+  it('message can be an Error instance *receiving an object* passed trough some external dependency', () => {
+    const someRandomErrorWrong = new Error({
+      code: 'WRONG',
+      message: 'this is not how errors should be handled',
+    });
+
+    const error = new ParseError(1337, someRandomErrorWrong);
+
     expect(JSON.parse(JSON.stringify(error))).toEqual({
-      message: JSON.stringify(someRandomError),
+      message: '', // <-- Yeah because we can't parse errors used like that
       code: 1337,
     });
   });