diff options
| author | Michaël Zasso <targos@protonmail.com> | 2018-07-17 08:23:49 +0200 |
|---|---|---|
| committer | Michaël Zasso <targos@protonmail.com> | 2018-07-19 08:47:28 +0200 |
| commit | df08779e0dfa82ceb69e129ae0548e97352aaddf (patch) | |
| tree | a589930afbeaf5481f3fe614eeee05fc04b7ec35 /test/es-module/test-esm-loader-missing-dynamic-instantiate-hook.mjs | |
| parent | 373aae1f86c1d061123e3aec28980aa2e7d1ebd4 (diff) | |
| download | android-node-v8-df08779e0dfa82ceb69e129ae0548e97352aaddf.tar.gz android-node-v8-df08779e0dfa82ceb69e129ae0548e97352aaddf.tar.bz2 android-node-v8-df08779e0dfa82ceb69e129ae0548e97352aaddf.zip | |
test: make crashOnUnhandleRejection opt-out
This commit removes `common.crashOnUnhandledRejection()` and adds
`common.disableCrashOnUnhandledRejection()`.
To reduce the risk of mistakes and make writing tests that involve
promises simpler, always install the unhandledRejection hook in tests
and provide a way to disable it for the rare cases where it's needed.
PR-URL: https://github.com/nodejs/node/pull/21849
Reviewed-By: Tobias Nießen <tniessen@tnie.de>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Gus Caplan <me@gus.host>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com>
Diffstat (limited to 'test/es-module/test-esm-loader-missing-dynamic-instantiate-hook.mjs')
| -rw-r--r-- | test/es-module/test-esm-loader-missing-dynamic-instantiate-hook.mjs | 7 |
1 files changed, 1 insertions, 6 deletions
diff --git a/test/es-module/test-esm-loader-missing-dynamic-instantiate-hook.mjs b/test/es-module/test-esm-loader-missing-dynamic-instantiate-hook.mjs index 27447b3e43..f2b37f7e8a 100644 --- a/test/es-module/test-esm-loader-missing-dynamic-instantiate-hook.mjs +++ b/test/es-module/test-esm-loader-missing-dynamic-instantiate-hook.mjs @@ -1,11 +1,6 @@ // Flags: --experimental-modules --loader ./test/fixtures/es-module-loaders/missing-dynamic-instantiate-hook.mjs -import { - crashOnUnhandledRejection, - expectsError -} from '../common'; - -crashOnUnhandledRejection(); +import { expectsError } from '../common'; import('test').catch(expectsError({ code: 'ERR_MISSING_DYNAMIC_INSTANTIATE_HOOK', |