azer / prova

Test runner based on Tape and Browserify
Other
333 stars 30 forks source link

Tap report for browser #45

Closed aghassemi closed 6 years ago

aghassemi commented 9 years ago

This is the same PR as https://github.com/azer/prova/pull/42 just switching from master to a branch specific for this so I can work on other things in master without affecting this PR.

aghassemi commented 9 years ago

Calling tape.createStream() twice has a side-effect where for the very first test in the suite, the state of whether it has ended or not is ignored making t.plan(x) not work properly in the first test. substack/tape#105

This PR includes work-around for this issue, it is not ideal as it accesses private tape variables however.

calvinmetcalf commented 9 years ago

any update on this?