Skip to content
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

Tap report for browser #45

Closed
wants to merge 3 commits into from

Conversation

aghassemi
Copy link
Contributor

This is the same PR as #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 and others added 3 commits September 25, 2014 13:07
… raw TAP output as well as structured results
…ice creating weird side-effects. For now accesssing the private stream directly. Bug to be filed with TAPE
@aghassemi
Copy link
Contributor Author

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. tape-testing/tape#105

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

@calvinmetcalf
Copy link

any update on this?

@aghassemi aghassemi closed this Jun 28, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants