-
-
Notifications
You must be signed in to change notification settings - Fork 348
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
chore: Convert some tap tests to run parallel and use snapshots. #1075
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@bcoe just one question for you! otherwise I think this is amazing [ignoring my pedantry].
Create `npm run lint`, recreate `npm run report`.
@JaKXz @bcoe I've made a few tweaks, mainly moving some code from the test file to helpers. Partly this is due to the test @isaacs posted in #1077 made me realize that we might want to share more code. One internal behavior change instead of adding stdio chunks directly to a string I'm now using an array of buffers and Buffer.concat at the end. In theory this shouldn't have an effect but I just like the code better (saw it in #1077). |
Prompted by discussion in istanbuljs/nyc#1075 (comment)
Prompted by discussion in istanbuljs/nyc#1075 (comment)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm really excited about this, just a few nits.
* Split helpers into individual files, create index.js to export everything useful to tests. * Rename test files - nyc-integration-old.js for tests that have not been migrated, nyc-integration.js for tests which have been migrated.
No description provided.