Skip to content

Commit f815565

Browse 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 <[email protected]> Reviewed-By: Antoine du Hamel <[email protected]>
1 parent 181bad5 commit f815565

File tree

1 file changed

+17
-0
lines changed

1 file changed

+17
-0
lines changed

test/parallel/test-error-serdes.js

+17
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)