diff options
author | isaacs <i@izs.me> | 2019-05-23 07:05:41 +0300 |
---|---|---|
committer | isaacs <i@izs.me> | 2019-06-30 06:23:54 +0300 |
commit | a4dc34243d946008815b5f6a71ac0e6a9823c14c (patch) | |
tree | 81be618101f0de5be31caa80acf295f13eb92eeb /test/tap/builtin-config.js | |
parent | 740fd954dd3456c9ffb0061d4cadeca788137907 (diff) |
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.
Diffstat (limited to 'test/tap/builtin-config.js')
-rw-r--r-- | test/tap/builtin-config.js | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/test/tap/builtin-config.js b/test/tap/builtin-config.js index b960ed038..dddd40565 100644 --- a/test/tap/builtin-config.js +++ b/test/tap/builtin-config.js @@ -10,7 +10,7 @@ var common = require('../common-tap.js') var path = require('path') var rimraf = require('rimraf') var mkdirp = require('mkdirp') -var folder = path.resolve(__dirname, 'builtin-config') +var folder = common.pkg var test = require('tap').test var npm = path.resolve(__dirname, '../..') var spawn = require('child_process').spawn |