fix(Error handling): Escalate unhandledpromiserejectionwarning in non…

… promise methods (#3)

hooks.js

@@ -208,7 +208,12 @@ class HooksPromise {
                 /**
                  * Get the target method response
                  */
-                let responsePromised = originalMethod.apply(self, passedArgs);
+                let responsePromised;
+                try {
+                    responsePromised = originalMethod.apply(self, passedArgs);
+                } catch (e) {
+                    responsePromised = Promise.reject(e);
+                }
 
                 /**
                  * Convert it to a Promise if it is not to be able to chain it

mocks/model.mock.js

@@ -9,6 +9,9 @@ module.exports = {
                 resolve('1234');
             }),
             saveReturnObject: () => Promise.resolve({ a: 123 }),
+            saveThrowError: () => {
+                throw new Error('Thrown Error');
+            },
         };
     },
 };

test/hooks.js

@@ -176,6 +176,14 @@ describe('hooks-promise', () => {
                     expect(check2).equal('myPreHookMethod2');
                 });
         });
+
+        it('should catch and reject thrown errors when the original method is not a promise', () => {
+            model.pre('saveThrowError', spies.preHook2);
+            return model.saveThrowError()
+                .catch((err) => {
+                    expect(err.message).equal('Thrown Error');
+                });
+        });
     });
 
     describe('post()', () => {