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: #48750
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Debadree Chatterjee <debadree333@gmail.com>
Reviewed-By: Yagiz Nizipli <yagiz@nizipli.com>
  • Loading branch information
StefanStojanovic authored and RafaelGSS committed Aug 15, 2023
1 parent 5aef593 commit d5e93b1
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 d5e93b1

Please sign in to comment.