diff options
| author | Rich Trott <rtrott@gmail.com> | 2018-09-19 16:33:18 -0700 |
|---|---|---|
| committer | Rich Trott <rtrott@gmail.com> | 2018-09-21 19:20:26 -0700 |
| commit | 2b29df71ebc07cea87e14ed7f2d93e78f8b04b3c (patch) | |
| tree | 38cf5dd09556a0d00bbc4f87143fb7b476f49b98 /lib/tty.js | |
| parent | 5942a3447a8b74c0030ed43cff440dbc93c1845a (diff) | |
| download | android-node-v8-2b29df71ebc07cea87e14ed7f2d93e78f8b04b3c.tar.gz android-node-v8-2b29df71ebc07cea87e14ed7f2d93e78f8b04b3c.tar.bz2 android-node-v8-2b29df71ebc07cea87e14ed7f2d93e78f8b04b3c.zip | |
test: do not export common.leakedGlobals()
common.leakedGlobals() was exposed only to test its logic. The logic can
instead be tested by running a fixture file that leaks a global and
seeing if `common` causes an AssertionError on exit. This way, the
entire functionality of leak detection is tested rather than just the
leakedGlobals() function. It also reduces API surface area for the
common monolith by one function.
PR-URL: https://github.com/nodejs/node/pull/22965
Reviewed-By: Richard Lau <riclau@uk.ibm.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Jeremiah Senkpiel <fishrock123@rocketmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Diffstat (limited to 'lib/tty.js')
0 files changed, 0 insertions, 0 deletions