From 58fc168807458df7fd6991a0712836a997c734fe Mon Sep 17 00:00:00 2001 From: cjihrig Date: Wed, 5 Jun 2019 15:35:13 -0400 Subject: 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 Reviewed-By: Yongsheng Zhang Reviewed-By: Ruben Bridgewater Reviewed-By: Rich Trott Reviewed-By: Anto Aravinth Reviewed-By: James M Snell --- test/fixtures/require-resolve.js | 6 ++++++ 1 file changed, 6 insertions(+) (limited to 'test/fixtures') 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') +); -- cgit v1.2.3