Skip to content

Commit b875f2d

Browse files
jasnelladuh95
authored andcommitted
test: reduce stack size for test-error-serdes
Hopefully reduces the run time and the likelihood of the test failing with a flaky timeout error. remove test-error-serdes from flaky list move test-error-serdes to sequential PR-URL: #54840 Refs: #52630 Reviewed-By: Yagiz Nizipli <yagiz@nizipli.com> Reviewed-By: Moshe Atlow <moshe@atlow.co.il> Reviewed-By: LiviaMedeiros <livia@cirno.name> Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com> Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com>
1 parent a624002 commit b875f2d

File tree

2 files changed

+1
-4
lines changed

2 files changed

+1
-4
lines changed

test/parallel/parallel.status

-3
Original file line numberDiff line numberDiff line change
@@ -16,9 +16,6 @@ test-worker-arraybuffer-zerofill: PASS, FLAKY
1616
# https://github.com/nodejs/node/issues/51862
1717
test-fs-read-stream-concurrent-reads: PASS, FLAKY
1818

19-
# https://github.com/nodejs/node/issues/52630
20-
test-error-serdes: PASS, FLAKY
21-
2219
# Until V8 provides a better way to check for flag mismatch without
2320
# making the code cache/snapshot unreproducible, disable the test
2421
# for a preemptive check now. It should idealy fail more gracefully

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

+1-1
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
// Flags: --expose-internals
1+
// Flags: --expose-internals --stack-size=64
22
'use strict';
33
require('../common');
44
const assert = require('assert');

0 commit comments

Comments
 (0)