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 f815565

Browse filesBrowse files
Lxxyxdanielleadams
authored andcommitted
test: increase coverage for internal/error_serdes.js
test serializeError with getter property Refs: https://coverage.nodejs.org/coverage-73a21e4c06d5781d/lib/internal/error_serdes.js.html#L49 PR-URL: #36628 Reviewed-By: Rich Trott <rtrott@gmail.com> Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
1 parent 181bad5 commit f815565
Copy full SHA for f815565

File tree

Expand file treeCollapse file tree

1 file changed

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

1 file changed

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

‎test/parallel/test-error-serdes.js‎

Copy file name to clipboardExpand all lines: test/parallel/test-error-serdes.js
+17Lines changed: 17 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -49,3 +49,20 @@ assert.strictEqual(cycle(Function), '[Function: Function]');
4949
assert.strictEqual(err.name, 'TypeError');
5050
assert.strictEqual(err.code, 'ERR_INVALID_ARG_TYPE');
5151
}
52+
53+
{
54+
let called = false;
55+
class DynamicError extends Error {
56+
get type() {
57+
called = true;
58+
return 'dynamic';
59+
}
60+
61+
get shouldIgnoreError() {
62+
throw new Error();
63+
}
64+
}
65+
66+
serializeError(new DynamicError());
67+
assert.deepStrictEqual(called, true);
68+
}

0 commit comments

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