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 34ef9e4

Browse filesBrowse files
cjihrigtargos
authored andcommitted
test: detect missing postmortem metadata
This commit updates test-postmortem-metadata to provide a more useful error message in the scenario where Node is compiled without postmortem support. PR-URL: #27828 Reviewed-By: Richard Lau <riclau@uk.ibm.com> Reviewed-By: Michaël Zasso <targos@protonmail.com> Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de> Reviewed-By: Rich Trott <rtrott@gmail.com>
1 parent 1da5acb commit 34ef9e4
Copy full SHA for 34ef9e4

File tree

Expand file treeCollapse file tree

1 file changed

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

1 file changed

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

‎test/v8-updates/test-postmortem-metadata.js‎

Copy file name to clipboardExpand all lines: test/v8-updates/test-postmortem-metadata.js
+3Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -41,6 +41,9 @@ const symbols = nm.stdout.toString().split('\n').reduce((filtered, line) => {
4141

4242
return filtered;
4343
}, []);
44+
45+
assert.notStrictEqual(symbols.length, 0, 'No postmortem metadata detected');
46+
4447
const missing = getExpectedSymbols().filter((symbol) => {
4548
return !symbols.includes(symbol);
4649
});

0 commit comments

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