diff options
| author | Rich Trott <rtrott@gmail.com> | 2018-08-10 13:00:10 -0700 |
|---|---|---|
| committer | Rich Trott <rtrott@gmail.com> | 2018-08-13 22:17:58 -0700 |
| commit | c75c917823ac49abbe99d847d20b1fe93e9872a7 (patch) | |
| tree | 065c2ac3da4dbbaebea92bc8ce09c54f73550f3a /test/parallel/test-trace-events-api.js | |
| parent | c7944a7a7bd92a446ea81d774ccad31e0e7c8a3f (diff) | |
| download | android-node-v8-c75c917823ac49abbe99d847d20b1fe93e9872a7.tar.gz android-node-v8-c75c917823ac49abbe99d847d20b1fe93e9872a7.tar.bz2 android-node-v8-c75c917823ac49abbe99d847d20b1fe93e9872a7.zip | |
test: remove common.hasTracing
`common.hasTracing` is only used in one place. `common` is bloated so
let's move that to the one test that uses it.
`hasTracing` is undocumented so there's no need to remove it from the
README file as it's not there in the first place.
Similarly, it's not included in the .mjs version of the `common` file.
PR-URL: https://github.com/nodejs/node/pull/22250
Reviewed-By: Bryan English <bryan@bryanenglish.com>
Reviewed-By: Richard Lau <riclau@uk.ibm.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com>
Reviewed-By: George Adams <george.adams@uk.ibm.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Diffstat (limited to 'test/parallel/test-trace-events-api.js')
| -rw-r--r-- | test/parallel/test-trace-events-api.js | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/test/parallel/test-trace-events-api.js b/test/parallel/test-trace-events-api.js index 44193cec0e..f7d7ec35b0 100644 --- a/test/parallel/test-trace-events-api.js +++ b/test/parallel/test-trace-events-api.js @@ -3,7 +3,7 @@ const common = require('../common'); -if (!common.hasTracing) +if (!process.binding('config').hasTracing) common.skip('missing trace events'); if (!common.isMainThread) common.skip('process.chdir is not available in Workers'); |