Commit 9b7d2c2
test: guard large string decoder allocation
Use common.enoughTestMem to avoid "Array buffer allocation failed"
error on low memory devices.
Fixes: #36792
PR-URL: #36795
Reviewed-By: Gireesh Punathil <gpunathi@in.ibm.com>
Reviewed-By: Pooja D P <Pooja.D.P@ibm.com>
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>1 parent bff201a commit 9b7d2c2
1 file changed
Lines changed: 8 additions & 6 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
@@ -201,12 +201,14 @@ assert.throws( | |||
| 201 | 201 | } | |
| 202 | 202 | ); | |
| 203 | 203 | ||
| 204 | - assert.throws( | ||
| 205 | - () => new StringDecoder().write(Buffer.alloc(0x1fffffe8 + 1).fill('a')), | ||
| 206 | - { | ||
| 207 | - code: 'ERR_STRING_TOO_LONG', | ||
| 208 | - } | ||
| 209 | - ); | ||
| 204 | + if (common.enoughTestMem) { | ||
| 205 | + assert.throws( | ||
| 206 | + () => new StringDecoder().write(Buffer.alloc(0x1fffffe8 + 1).fill('a')), | ||
| 207 | + { | ||
| 208 | + code: 'ERR_STRING_TOO_LONG', | ||
| 209 | + } | ||
| 210 | + ); | ||
| 211 | + } | ||
| 210 | 212 | ||
| 211 | 213 | // Test verifies that StringDecoder will correctly decode the given input | |
| 212 | 214 | // buffer with the given encoding to the expected output. It will attempt all | |
0 commit comments