Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

Commit bda34ea

Browse filesBrowse files
apapirovskitargos
authored andcommitted
test: check gc does not resurrect the loop
PR-URL: #21093 Fixes: #18190 Refs: #18307 Reviewed-By: Jeremiah Senkpiel <fishrock123@rocketmail.com> Reviewed-By: Anna Henningsen <anna@addaleax.net> Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl> Reviewed-By: Joyee Cheung <joyeec9h3@gmail.com> Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
1 parent 4f01168 commit bda34ea
Copy full SHA for bda34ea

File tree

Expand file treeCollapse file tree

1 file changed

+6
-0
lines changed
Open diff view settings
Filter options
Expand file treeCollapse file tree

1 file changed

+6
-0
lines changed
Open diff view settings
Collapse file

‎test/sequential/test-async-wrap-getasyncid.js‎

Copy file name to clipboardExpand all lines: test/sequential/test-async-wrap-getasyncid.js
+6Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,5 @@
11
'use strict';
2+
// Flags: --expose-gc
23

34
const common = require('../common');
45
const assert = require('assert');
@@ -22,6 +23,11 @@ common.crashOnUnhandledRejection();
2223
},
2324
}).enable();
2425
process.on('beforeExit', common.mustCall(() => {
26+
// This garbage collection call verifies that the wraps being garbage
27+
// collected doesn't resurrect the process again due to weirdly timed
28+
// uv_close calls and other similar instruments in destructors.
29+
global.gc();
30+
2531
process.removeAllListeners('uncaughtException');
2632
hooks.disable();
2733
delete providers.NONE; // Should never be used.

0 commit comments

Comments
0 (0)
Morty Proxy This is a proxified and sanitized view of the page, visit original site.