Skip to content

Commit 24bc910

Browse files
jasnelllouwers
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: nodejs#54840 Refs: nodejs#52630 Reviewed-By: Yagiz Nizipli <[email protected]> Reviewed-By: Moshe Atlow <[email protected]> Reviewed-By: LiviaMedeiros <[email protected]> Reviewed-By: Antoine du Hamel <[email protected]> Reviewed-By: Trivikram Kamat <[email protected]>
1 parent 64483e5 commit 24bc910

File tree

2 files changed

+1
-4
lines changed

2 files changed

+1
-4
lines changed

β€Žtest/parallel/parallel.statusβ€Ž

Lines changed: 0 additions & 3 deletions
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β€Ž renamed to β€Žtest/sequential/test-error-serdes.jsβ€Ž

Lines changed: 1 addition & 1 deletion
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)