mirror of
https://github.com/nodejs/node.git
synced 2025-12-28 07:50:41 +00:00
e50cbc1abd
Tests should be explicit regarding whether a promise is expected to settle, and the test should fail when the behavior does not meet expectations. PR-URL: https://github.com/nodejs/node/pull/60976 Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: Erick Wendel <erick.workspace@gmail.com>
47 lines
1.1 KiB
JavaScript
47 lines
1.1 KiB
JavaScript
// Flags: --expose-gc
|
||
'use strict';
|
||
const common = require('../common');
|
||
const assert = require('assert');
|
||
const async_hooks = require('async_hooks');
|
||
const v8 = require('v8');
|
||
|
||
// Regression test for https://github.com/nodejs/node/issues/28786
|
||
// Make sure that creating a heap snapshot inside an async_hooks hook
|
||
// works for Promises.
|
||
|
||
const createSnapshot = common.mustCall(() => {
|
||
v8.getHeapSnapshot().resume();
|
||
}, 8); // 2 × init + 2 × resolve + 1 × (after + before) + 2 × destroy = 8 calls
|
||
|
||
const promiseIds = [];
|
||
|
||
async_hooks.createHook({
|
||
init(id, type) {
|
||
if (type === 'PROMISE') {
|
||
createSnapshot();
|
||
promiseIds.push(id);
|
||
}
|
||
},
|
||
|
||
before(id) {
|
||
if (promiseIds.includes(id)) createSnapshot();
|
||
},
|
||
|
||
after(id) {
|
||
if (promiseIds.includes(id)) createSnapshot();
|
||
},
|
||
|
||
promiseResolve: common.mustCallAtLeast((id) => {
|
||
assert(promiseIds.includes(id));
|
||
createSnapshot();
|
||
}),
|
||
|
||
destroy(id) {
|
||
if (promiseIds.includes(id)) createSnapshot();
|
||
}
|
||
}).enable();
|
||
|
||
|
||
Promise.resolve().then(common.mustCall());
|
||
setImmediate(globalThis.gc);
|