diff options
author | Masashi Hirano <cherrydog07@gmail.com> | 2019-04-21 17:04:04 +0900 |
---|---|---|
committer | Daniel Bevenius <daniel.bevenius@gmail.com> | 2019-04-24 10:07:09 +0200 |
commit | fb3f6005c037f9af8f435edd0d1f407266af2916 (patch) | |
tree | e78d5c169e9b3a99e42bad259aaaa0d90a1d0cf5 /test/es-module | |
parent | 9b982feedf76ac51fe4ed1aaf22026cbb0df1a5e (diff) | |
download | android-node-v8-fb3f6005c037f9af8f435edd0d1f407266af2916.tar.gz android-node-v8-fb3f6005c037f9af8f435edd0d1f407266af2916.tar.bz2 android-node-v8-fb3f6005c037f9af8f435edd0d1f407266af2916.zip |
test: fix ineffective error tests
Fix tests whether errors are thrown correctly
because they are successful when error doesn't get thrown.
PR-URL: https://github.com/nodejs/node/pull/27333
Fixes: https://github.com/nodejs/node/issues/26385
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Diffstat (limited to 'test/es-module')
-rw-r--r-- | test/es-module/test-esm-error-cache.js | 12 |
1 files changed, 7 insertions, 5 deletions
diff --git a/test/es-module/test-esm-error-cache.js b/test/es-module/test-esm-error-cache.js index 79f76357ec..26e0d170ac 100644 --- a/test/es-module/test-esm-error-cache.js +++ b/test/es-module/test-esm-error-cache.js @@ -18,9 +18,11 @@ let error; assert(error); - try { - await import(file); - } catch (e) { - assert.strictEqual(error, e); - } + await assert.rejects( + () => import(file), + (e) => { + assert.strictEqual(error, e); + return true; + } + ); })(); |