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 afd6e46

Browse filesBrowse files
puzpuzpuzaddaleax
authored andcommitted
test: fix test-heapdump-zlib
PR-URL: #34499 Refs: #34048 Reviewed-By: Richard Lau <riclau@uk.ibm.com> Reviewed-By: Anna Henningsen <anna@addaleax.net> Reviewed-By: Rich Trott <rtrott@gmail.com>
1 parent 8c0a781 commit afd6e46
Copy full SHA for afd6e46

File tree

Expand file treeCollapse file tree

1 file changed

+16
-5
lines changed
Open diff view settings
Filter options
Expand file treeCollapse file tree

1 file changed

+16
-5
lines changed
Open diff view settings
Collapse file

‎test/pummel/test-heapdump-zlib.js‎

Copy file name to clipboard
+16-5Lines changed: 16 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1,17 +1,28 @@
11
// Flags: --expose-internals
22
'use strict';
3-
require('../common');
3+
const common = require('../common');
44
const { validateSnapshotNodes } = require('../common/heap');
55
const zlib = require('zlib');
66

77
validateSnapshotNodes('Node / ZlibStream', []);
8-
// eslint-disable-next-line no-unused-vars
9-
const gunzip = zlib.createGunzip();
8+
9+
const gzip = zlib.createGzip();
1010
validateSnapshotNodes('Node / ZlibStream', [
1111
{
1212
children: [
13-
{ node_name: 'Zlib', edge_name: 'wrapped' },
14-
{ node_name: 'Node / zlib_memory', edge_name: 'zlib_memory' }
13+
{ node_name: 'Zlib', edge_name: 'wrapped' }
14+
// No entry for memory because zlib memory is initialized lazily.
1515
]
1616
}
1717
]);
18+
19+
gzip.write('hello world', common.mustCall(() => {
20+
validateSnapshotNodes('Node / ZlibStream', [
21+
{
22+
children: [
23+
{ node_name: 'Zlib', edge_name: 'wrapped' },
24+
{ node_name: 'Node / zlib_memory', edge_name: 'zlib_memory' }
25+
]
26+
}
27+
]);
28+
}));

0 commit comments

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