Skip to content

Commit a4dc342

Browse files
isaacsisaacs
committed
tests: parallel tests
Several changes to the test suite to support running tests in parallel, bringing the overall test time down considerably. - Replace all literal 1337 and 1234 ports with a custom per-process port assignment based on the TAP_CHILD_ID environment variable. - Add common.pkg as a per-test working directory instead of polluting __dirname or accidentally reusing the same working directory for multiple tests. - Rework test config handling so that tests don't rely on config setup being run in a particular order. - Remove the npm-registry-couchapp tests, since it (a) relies on CouchDB, (b) is no longer a reliable indicator of registry compatibility, and (c) is already superceded in most cases by tests that use npm-registry-mock. (A test suite that runs against a reference implementation is a thing that should exist, but not here.) - Remove the fake-registry logging when TAP_CHILD_ID is set, since this is extremely hard to make sense of when running multiple tests in parallel. When Node v6 compatibility is dropped in npm v7, we can upgrade to the latest version of tap for a bit more speed, dropping Domains (and the associated deprecation warnings), and a fancier test reporter.
1 parent 740fd95 commit a4dc342

File tree

269 files changed

+551
-1204
lines changed

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

269 files changed

+551
-1204
lines changed

‎.gitignore

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -21,3 +21,5 @@ npm-debug.log
2121
.jshintrc
2222
.eslintrc
2323
.nyc_output
24+
/test/npm_cache*
25+
/node_modules/.cache

‎node_modules/.cache/esm/.data.blob

-5.65 MB
Binary file not shown.

‎node_modules/.cache/esm/.data.json

Lines changed: 0 additions & 1 deletion
This file was deleted.

‎node_modules/.cache/esm/01261cd23ab0e665.js

Lines changed: 0 additions & 384 deletions
This file was deleted.

0 commit comments

Comments
 (0)