Skip to content

Commit

Permalink
test: fix flaky test-string-decode.js on x86
Browse files Browse the repository at this point in the history
Removes flakiness from the mentioned test due to the x86 memory limit

PR-URL: nodejs#48750
Reviewed-By: Luigi Pinca <[email protected]>
Reviewed-By: Debadree Chatterjee <[email protected]>
Reviewed-By: Yagiz Nizipli <[email protected]>
  • Loading branch information
StefanStojanovic authored and Ceres6 committed Aug 14, 2023
1 parent 05d58d2 commit 97a6bb7
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion test/parallel/test-string-decoder.js
Original file line number Diff line number Diff line change
Expand Up @@ -203,7 +203,7 @@ assert.throws(

if (common.enoughTestMem) {
assert.throws(
() => new StringDecoder().write(Buffer.alloc(0x1fffffe8 + 1).fill('a')),
() => new StringDecoder().write(Buffer.alloc((process.arch === 'ia32' ? 0x18ffffe8 : 0x1fffffe8) + 1).fill('a')),
{
code: 'ERR_STRING_TOO_LONG',
}
Expand Down

0 comments on commit 97a6bb7

Please sign in to comment.