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/common/index.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/common/index.js')
-rw-r--r-- | test/common/index.js | 7 |
1 files changed, 0 insertions, 7 deletions
diff --git a/test/common/index.js b/test/common/index.js index 66326070a0..742a78e367 100644 --- a/test/common/index.js +++ b/test/common/index.js @@ -29,7 +29,6 @@ const os = require('os'); const { exec, execSync, spawn, spawnSync } = require('child_process'); const stream = require('stream'); const util = require('util'); -const { hasTracing } = process.binding('config'); const { fixturesDir } = require('./fixtures'); const tmpdir = require('./tmpdir'); @@ -226,12 +225,6 @@ Object.defineProperty(exports, 'hasCrypto', { } }); -Object.defineProperty(exports, 'hasTracing', { - get: function() { - return Boolean(hasTracing); - } -}); - Object.defineProperty(exports, 'hasFipsCrypto', { get: function() { return exports.hasCrypto && require('crypto').fips; |