diff options
| author | cjihrig <cjihrig@gmail.com> | 2019-06-05 15:35:13 -0400 |
|---|---|---|
| committer | cjihrig <cjihrig@gmail.com> | 2019-06-11 10:20:31 -0700 |
| commit | 58fc168807458df7fd6991a0712836a997c734fe (patch) | |
| tree | e43537c6d02ff58b22abf341da2a44c47d491950 /test/fixtures | |
| parent | 0cd112a07ec3ee71e56df7a04353ae99ca84b5b5 (diff) | |
| download | android-node-v8-58fc168807458df7fd6991a0712836a997c734fe.tar.gz android-node-v8-58fc168807458df7fd6991a0712836a997c734fe.tar.bz2 android-node-v8-58fc168807458df7fd6991a0712836a997c734fe.zip | |
module: handle empty require.resolve() options
If require.resolve() is passed an options object, but
the paths option is not present, then use the default
require.resolve() paths.
PR-URL: https://github.com/nodejs/node/pull/28078
Fixes: https://github.com/nodejs/node/issues/28077
Reviewed-By: Richard Lau <riclau@uk.ibm.com>
Reviewed-By: Yongsheng Zhang <zyszys98@gmail.com>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Anto Aravinth <anto.aravinth.cse@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Diffstat (limited to 'test/fixtures')
| -rw-r--r-- | test/fixtures/require-resolve.js | 6 |
1 files changed, 6 insertions, 0 deletions
diff --git a/test/fixtures/require-resolve.js b/test/fixtures/require-resolve.js index 3d8500e509..66521d12e9 100644 --- a/test/fixtures/require-resolve.js +++ b/test/fixtures/require-resolve.js @@ -92,3 +92,9 @@ common.expectsError(() => { code: 'ERR_INVALID_OPT_VALUE', type: TypeError, }); + +// Verify that the default require.resolve() is used for empty options. +assert.strictEqual( + require.resolve('./printA.js', {}), + require.resolve('./printA.js') +); |