diff options
author | Guy Bedford <guybedford@gmail.com> | 2019-07-30 21:41:58 -0400 |
---|---|---|
committer | Rich Trott <rtrott@gmail.com> | 2019-08-01 20:36:27 -0700 |
commit | 452b393c1fc993c50acd447eb84eeea529b89111 (patch) | |
tree | 8be7041710e9bd07c4f8622c6f2684515145e81f /test/es-module | |
parent | ed138baff395ffc2ee8f41e99012866fdc27b4f4 (diff) | |
download | android-node-v8-452b393c1fc993c50acd447eb84eeea529b89111.tar.gz android-node-v8-452b393c1fc993c50acd447eb84eeea529b89111.tar.bz2 android-node-v8-452b393c1fc993c50acd447eb84eeea529b89111.zip |
module: exports error as MODULE_NOT_FOUND
PR-URL: https://github.com/nodejs/node/pull/28905
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Jan Krems <jan.krems@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com>
Diffstat (limited to 'test/es-module')
-rw-r--r-- | test/es-module/test-esm-exports.mjs | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/test/es-module/test-esm-exports.mjs b/test/es-module/test-esm-exports.mjs index ca10097aa9..6c42d3b64e 100644 --- a/test/es-module/test-esm-exports.mjs +++ b/test/es-module/test-esm-exports.mjs @@ -29,7 +29,7 @@ import { requireFixture, importFixture } from '../fixtures/pkgexports.mjs'; // There's no such export - so there's nothing to do. loadFixture('pkgexports/missing').catch(mustCall((err) => { - strictEqual(err.code, 'ERR_PATH_NOT_EXPORTED'); + strictEqual(err.code, (isRequire ? '' : 'ERR_') + 'MODULE_NOT_FOUND'); assertStartsWith(err.message, 'Package exports'); assertIncludes(err.message, 'do not define a \'./missing\' subpath'); })); @@ -37,7 +37,7 @@ import { requireFixture, importFixture } from '../fixtures/pkgexports.mjs'; // The file exists but isn't exported. The exports is a number which counts // as a non-null value without any properties, just like `{}`. loadFixture('pkgexports-number/hidden.js').catch(mustCall((err) => { - strictEqual(err.code, 'ERR_PATH_NOT_EXPORTED'); + strictEqual(err.code, (isRequire ? '' : 'ERR_') + 'MODULE_NOT_FOUND'); assertStartsWith(err.message, 'Package exports'); assertIncludes(err.message, 'do not define a \'./hidden.js\' subpath'); })); @@ -57,7 +57,7 @@ import { requireFixture, importFixture } from '../fixtures/pkgexports.mjs'; // Even though 'pkgexports/sub/asdf.js' works, alternate "path-like" variants // do not to prevent confusion and accidental loopholes. loadFixture('pkgexports/sub/./../asdf.js').catch(mustCall((err) => { - strictEqual(err.code, 'ERR_PATH_NOT_EXPORTED'); + strictEqual(err.code, (isRequire ? '' : 'ERR_') + 'MODULE_NOT_FOUND'); assertStartsWith(err.message, 'Package exports'); assertIncludes(err.message, 'do not define a \'./sub/./../asdf.js\' subpath'); |