[fix] Handle vows for events that execute multiple times #193

Closed
wants to merge 2 commits into
from

Conversation

Projects
None yet
2 participants
Contributor

bacchusrx commented Feb 16, 2012

Fix for issue described in #191.

Add failing test for events that fire multiple times.

In lib/vows/suite.js add a property count to each vow which is incremented every time the vow is executed.

In runTest (in addVow in lib/vows.js), on each execution increment batch.total if a vow has been executed more than once.

@bacchusrx bacchusrx [fix] Handle vows for events that execute multiple times (fixes #191)
Add failing test for events that fire multiple times.

In `lib/vows/suite.js` add a property `count` to each vow which is incremented
every time the vow is executed.

In `runTest` (in `addVow` in `lib/vows.js`), on each execution increment
`batch.total` if a vow has been executed more than once.
5da488e
Contributor

bacchusrx commented Apr 4, 2012

Yea/nay?

Owner

indexzero commented Nov 4, 2014

Looks like a solid feature addition for 0.9.0

indexzero added the v0.9.0 label Nov 4, 2014

Owner

indexzero commented Nov 22, 2014

Thanks. Merged into the v0.9.x branch.

indexzero closed this Nov 22, 2014

indexzero referenced this pull request Nov 22, 2014

Merged

v0.9.0-rc1 #328

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment