New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

use co-mocha for test #279

Closed
dead-horse opened this Issue Mar 16, 2014 · 4 comments

Comments

Projects
None yet
2 participants
@dead-horse
Member

dead-horse commented Mar 16, 2014

link: https://github.com/Qard/co-mocha

--require co-mocha
@fengmk2

This comment has been minimized.

Member

fengmk2 commented Mar 17, 2014

这样就不需要在外面包一层co了?
2014年3月17日 AM12:26于 "dead_horse" notifications@github.com写道:

link: https://github.com/Qard/co-mocha

--require co-mocha


Reply to this email directly or view it on GitHubhttps://github.com//issues/279
.

@dead-horse

This comment has been minimized.

Member

dead-horse commented Mar 17, 2014

应该是的------------------ 原始邮件 ------------------
发件人: "fengmk2"notifications@github.com
发送时间: 2014年3月17日(星期一) 上午8:24
收件人: "cnpm/cnpmjs.org"cnpmjs.org@noreply.github.com;
抄送: "dead_horse"dead_horse@qq.com;
主题: Re: [cnpmjs.org] use co-mocha for test (#279)

这样就不需要在外面包一层co了?
2014年3月17日 AM12:26于 "dead_horse" notifications@github.com写道:

link: https://github.com/Qard/co-mocha

--require co-mocha


Reply to this email directly or view it on GitHubhttps://github.com//issues/279
.


Reply to this email directly or view it on GitHub.

@fengmk2

This comment has been minimized.

Member

fengmk2 commented Mar 17, 2014

搜索了一下, 蛮多 co-mocha的, https://github.com/blakeembrey/co-mocha
你推荐的那个写得比较妥当, 只有异步的才会有timeout, 在timeout hack也比较轻.


@fengmk2 (https://twitter.com/fengmk2)
@python发烧友 (http://weibo.com/imk2/profile)
http://fengmk2.github.com (http://fengmk2.github.com/)


记得当时年纪小/
你爱谈天,我爱笑/
有一回并肩坐在桃树下/
风在林梢鸟儿在叫/
我们不知怎样睡着了/
梦里花落知多少

On Monday, March 17, 2014 at 9:21 AM, dead_horse wrote:

应该是的------------------ 原始邮件 ------------------
发件人: "fengmk2"<notifications@github.com (mailto:notifications@github.com)>
发送时间: 2014年3月17日(星期一) 上午8:24
收件人: "cnpm/cnpmjs.org"<cnpmjs.org@noreply.github.com (mailto:cnpmjs.org@noreply.github.com)>;
抄送: "dead_horse"<dead_horse@qq.com (mailto:dead_horse@qq.com)>;
主题: Re: [cnpmjs.org (http://cnpmjs.org)] use co-mocha for test (#279)

这样就不需要在外面包一层co了?
2014年3月17日 AM12:26于 "dead_horse" <notifications@github.com (mailto:notifications@github.com)>写道:

link: https://github.com/Qard/co-mocha

--require co-mocha


Reply to this email directly or view it on GitHubhttps://github.com//issues/279
.


Reply to this email directly or view it on GitHub.


Reply to this email directly or view it on GitHub (#279 (comment)).

@fengmk2

This comment has been minimized.

Member

fengmk2 commented Mar 17, 2014

但是 npm info co-mocha 是 https://github.com/blakeembrey/co-mocha 这个, 哈哈, 它直接hack run方法的, 也没什么大碍

fengmk2 added a commit that referenced this issue Mar 18, 2014

Merge pull request #281 from cnpm/issue279-co-mocha
use co-mocha for test, fixed #279
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment