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

add tests for ./bin/nyc.js #128

Closed
bcoe opened this issue Jan 3, 2016 · 2 comments
Closed

add tests for ./bin/nyc.js #128

bcoe opened this issue Jan 3, 2016 · 2 comments

Comments

@bcoe
Copy link
Member

bcoe commented Jan 3, 2016

It would be nice to have some unit tests for the nyc bin, I'm fine with these simply using spawn and asserting against stdout/stderr.

@jamestalmage
Copy link
Member

I have found testing to stdout/stderr to be really problematic on Windows. Especially on Node 0.10 (though not exclusively). It's certainly worthwhile to include a handful of integration tests just to make sure all the glue works. Perhaps a better idea would be to run against a fixture with known coverage, and inspect the reports.

@bcoe
Copy link
Member Author

bcoe commented Apr 11, 2016

@addaleax thanks \o/ see:

#224

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

No branches or pull requests

2 participants