move all tests to test/tap, and then make that all that runs on `npm test` #3893

Closed
isaacs opened this Issue Sep 13, 2013 · 3 comments

Comments

Projects
None yet
4 participants
Owner

isaacs commented Sep 13, 2013

We can still keep the existing tests around as a sort of "extra" test, but before we remove them, we should add tap tests to cover the additional cases.

Contributor

thomblake commented Jul 24, 2014

It looks like npm test already only runs tests in test/tap.

@othiym23 othiym23 removed the easy label Jul 24, 2014

Contributor

othiym23 commented Jul 24, 2014

npm run test-all runs a different set of tests that aren't encompassed by npm test. It's probably going to take a while to get the coverage in npm (and the other modules it uses) increased to the point where the TAP tests alone are authoritative.

@othiym23 othiym23 removed the nice to have label Oct 7, 2014

Collaborator

npm-robot commented Jun 19, 2017

We're closing this issue as it has gone thirty days without activity. In our experience if an issue has gone thirty days without any activity then it's unlikely to be addressed. In the case of bug reports, often the underlying issue will be addressed but finding related issues is quite difficult and often incomplete.

If this was a bug report and it is still relevant then we encourage you to open it again as a new issue. If this was a feature request then you should feel free to open it again, or even better open a PR.

For more information about our new issue aging policies and why we've instituted them please see our blog post.

@npm-robot npm-robot closed this Jun 19, 2017

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