Use `then` instead of `done` for message promise.

jeffmo / node-worker-pool

MIT License

23 stars 10 forks source link

Use `then` instead of `done` for message promise. #4

Closed cpojer closed 9 years ago

cpojer commented 9 years ago

I'm switching to native promises for jest. Promise doesn't have done + I think this is a bad place to call done because there are already error handlers in place that bubble up.

cpojer commented 9 years ago

I also have a half-working version of node-worker-pool where Q is removed but it makes all the tests fail and I'm unsure how to fix them all because they won't depend on the timers any more. One step at a time, I guess.