Thrown error is silenced when rejection is not caught

[slavafomin]

Hello!

Thank you for this great module!

However, I have some unexpected behavior with thrown errors: when I throw an error from inside of action function, it's silenced somehow when the rejection is not caught.

Here's the simplified example:

const Promise = require('bluebird');
const promiseDoWhilst = require('promise-do-whilst');


foo().then(result => console.log('Result', result));
// Error is displayed in the console.

bar().then(result => console.log('Result', result));
// Error is ignored, no output.


function foo () {
  return Promise.resolve().then(() => {
    throw new Error('Some error');
  });
}

function bar () {
  return promiseDoWhilst(() => {
    throw new Error('Some error');
  }, () => true);
}

I do prefer to see all thrown errors in the output of the Node.js application.

Thanks!

[slavafomin]

The workaround would be right now to call promiseDoWhilst() this way:

const Promise = require('bluebird');

return promiseDoWhilst(action, condition)
  .catch(rejection => Promise.reject(rejection))
;

[busterc]

I believe node v6+ processes unhandled promise exceptions in the manner you were expecting. Otherwise, as you noted, indeed the expectation is to provide a catch handler if you wish.

See also: nodejs/node#830