WhoopInc / supertest-as-promised

Supercharge supertest with a promise interface

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

use native-or-bluebird module?

nightink opened this issue · comments

commented

Hi @benesch

I like you module, but my environment use iojs or node@~0.12.x.
I do not want to install bluebird module, so you can use native-or-bluebird module?
You agree, I can Pull request.

commented

wow!!!!

supertest@1.1.0 PR-268 - Bump superagent to 1.3.0
https://github.com/visionmedia/supertest/blob/master/History.md#110--2015-08-26

superagent@1.3.0 add basic promise support via a then function
https://github.com/visionmedia/superagent/blob/master/History.md#130-2015-08-05

Sorry I use supertest module.

Thanks you module~

Good catch! That issue is now fixed on master.

Do note that the .then() method provided by SuperTest is not a full promise implementation: ladjs/superagent#722

Also, I prefer shipping Bluebird by default so that this library "just works" out of the box. The BYOP API allows you to swap in native ES6 promises if you prefer: do you have a compelling reason not to depend on Bluebird? It's only a few kilobytes of disk space.

commented

ladjs/superagent#709

because I'm not going to be using it that way. catch is not useful in the co generator environment

+1 co generator environment
Please ignore me.