test: skip the test if the buffer allocation fails #58738

lpinca · 2025-06-17T08:28:50Z

Use the error message as another condition to skip the test when the buffer allocation fails.

Refs: 795dd8eb7988ae38553e
Refs: e9c6004a2d580008082b

lpinca · 2025-06-17T08:32:51Z

This should fix

https://ci.nodejs.org/job/node-test-binary-windows-js-suites/34803/RUN_SUBSET=3,nodes=win11-COMPILED_BY-vs2022_clang/testReport/junit/(root)/pummel/test_buffer_large_size_buffer_write/

---
duration_ms: 128.011
exitcode: 1
severity: fail
stack: |-
  c:\workspace\node-test-binary-windows-js-suites\node\test\pummel\test-buffer-large-size-buffer-write.js:19
      throw e;
      ^

  RangeError: Array buffer allocation failed
      at new ArrayBuffer (<anonymous>)
      at new Uint8Array (<anonymous>)
      at new FastBuffer (node:internal/buffer:968:5)
      at Buffer.alloc (node:buffer:399:10)
      at Object.<anonymous> (c:\workspace\node-test-binary-windows-js-suites\node\test\pummel\test-buffer-large-size-buffer-write.js:14:22)
      at Module._compile (node:internal/modules/cjs/loader:1691:14)
      at Object..js (node:internal/modules/cjs/loader:1823:10)
      at Module.load (node:internal/modules/cjs/loader:1426:32)
      at Module._load (node:internal/modules/cjs/loader:1249:12)
      at TracingChannel.traceSync (node:diagnostics_channel:322:14)

  Node.js v25.0.0-pre
...

https://ci.nodejs.org/job/node-test-binary-windows-js-suites/34840/RUN_SUBSET=2,nodes=win11-COMPILED_BY-vs2022_clang/testReport/junit/(root)/pummel/test_buffer_large_size_buffer_alloc_unsafe_slow/

---
duration_ms: 161.012
exitcode: 1
severity: fail
stack: |-
  c:\workspace\node-test-binary-windows-js-suites\node\test\pummel\test-buffer-large-size-buffer-alloc-unsafe-slow.js:15
      throw e;
      ^

  AssertionError [ERR_ASSERTION]: Expected values to be strictly deep-equal:
  + actual - expected

  + Comparison {}
  - Comparison {
  -   code: 'ERR_STRING_TOO_LONG'
  - }

      at Object.<anonymous> (c:\workspace\node-test-binary-windows-js-suites\node\test\pummel\test-buffer-large-size-buffer-alloc-unsafe-slow.js:12:10)
      at Module._compile (node:internal/modules/cjs/loader:1691:14)
      at Object..js (node:internal/modules/cjs/loader:1823:10)
      at Module.load (node:internal/modules/cjs/loader:1426:32)
      at Module._load (node:internal/modules/cjs/loader:1249:12)
      at TracingChannel.traceSync (node:diagnostics_channel:322:14)
      at wrapModuleLoad (node:internal/modules/cjs/loader:235:24)
      at Module.executeUserEntryPoint [as runMain] (node:internal/modules/run_main:152:5)
      at node:internal/main/run_main_module:33:47 {
    generatedMessage: true,
    code: 'ERR_ASSERTION',
    actual: RangeError: Array buffer allocation failed
        at new ArrayBuffer (<anonymous>)
        at new Uint8Array (<anonymous>)
        at new FastBuffer (node:internal/buffer:968:5)
        at createUnsafeBuffer (node:internal/buffer:1114:12)
        at Buffer.allocUnsafeSlow (node:buffer:418:10)
        at assert.throws.code (c:\workspace\node-test-binary-windows-js-suites\node\test\pummel\test-buffer-large-size-buffer-alloc-unsafe-slow.js:12:30)
        at getActual (node:assert:505:5)
        at ok.throws (node:assert:651:24)
        at Object.<anonymous> (c:\workspace\node-test-binary-windows-js-suites\node\test\pummel\test-buffer-large-size-buffer-alloc-unsafe-slow.js:12:10)
        at Module._compile (node:internal/modules/cjs/loader:1691:14),
    expected: { code: 'ERR_STRING_TOO_LONG' },
    operator: 'throws'
  }

  Node.js v25.0.0-pre
...

https://ci.nodejs.org/job/node-test-binary-windows-js-suites/34840/RUN_SUBSET=2,nodes=win11-COMPILED_BY-vs2022_clang/testReport/junit/(root)/pummel/test_buffer_large_size_slowbuffer/

---
duration_ms: 168.372
exitcode: 1
severity: fail
stack: |-
  c:\workspace\node-test-binary-windows-js-suites\node\test\pummel\test-buffer-large-size-slowbuffer.js:19
      throw e;
      ^

  AssertionError [ERR_ASSERTION]: Expected values to be strictly deep-equal:
  + actual - expected

  + Comparison {}
  - Comparison {
  -   code: 'ERR_STRING_TOO_LONG'
  - }

      at Object.<anonymous> (c:\workspace\node-test-binary-windows-js-suites\node\test\pummel\test-buffer-large-size-slowbuffer.js:14:10)
      at Module._compile (node:internal/modules/cjs/loader:1691:14)
      at Object..js (node:internal/modules/cjs/loader:1823:10)
      at Module.load (node:internal/modules/cjs/loader:1426:32)
      at Module._load (node:internal/modules/cjs/loader:1249:12)
      at TracingChannel.traceSync (node:diagnostics_channel:322:14)
      at wrapModuleLoad (node:internal/modules/cjs/loader:235:24)
      at Module.executeUserEntryPoint [as runMain] (node:internal/modules/run_main:152:5)
      at node:internal/main/run_main_module:33:47 {
    generatedMessage: true,
    code: 'ERR_ASSERTION',
    actual: RangeError: Array buffer allocation failed
        at new ArrayBuffer (<anonymous>)
        at new Uint8Array (<anonymous>)
        at new FastBuffer (node:internal/buffer:968:5)
        at createUnsafeBuffer (node:internal/buffer:1114:12)
        at Buffer.allocUnsafeSlow (node:buffer:418:10)
        at assert.throws.code (c:\workspace\node-test-binary-windows-js-suites\node\test\pummel\test-buffer-large-size-slowbuffer.js:14:30)
        at getActual (node:assert:505:5)
        at ok.throws (node:assert:651:24)
        at Object.<anonymous> (c:\workspace\node-test-binary-windows-js-suites\node\test\pummel\test-buffer-large-size-slowbuffer.js:14:10)
        at Module._compile (node:internal/modules/cjs/loader:1691:14),
    expected: { code: 'ERR_STRING_TOO_LONG' },
    operator: 'throws'
  }

  Node.js v25.0.0-pre
...

https://ci.nodejs.org/job/node-test-binary-windows-js-suites/34840/RUN_SUBSET=2,nodes=win11-COMPILED_BY-vs2022_clang/testReport/junit/(root)/pummel/test_string_decoder_large_buffer/

---
duration_ms: 174.014
exitcode: 1
severity: fail
stack: |-
  c:\workspace\node-test-binary-windows-js-suites\node\test\pummel\test-string-decoder-large-buffer.js:26
      throw e;
      ^

  RangeError: Array buffer allocation failed
      at new ArrayBuffer (<anonymous>)
      at new Uint8Array (<anonymous>)
      at new FastBuffer (node:internal/buffer:968:5)
      at createUnsafeBuffer (node:internal/buffer:1114:12)
      at allocate (node:buffer:433:10)
      at Buffer.allocUnsafe (node:buffer:408:10)
      at Object.<anonymous> (c:\workspace\node-test-binary-windows-js-suites\node\test\pummel\test-string-decoder-large-buffer.js:21:22)
      at Module._compile (node:internal/modules/cjs/loader:1691:14)
      at Object..js (node:internal/modules/cjs/loader:1823:10)
      at Module.load (node:internal/modules/cjs/loader:1426:32)

  Node.js v25.0.0-pre
...

Use the error message as another condition to skip the test when the buffer allocation fails. Refs: nodejs@795dd8eb7988ae38553e Refs: nodejs@e9c6004a2d580008082b

codecov · 2025-06-17T09:25:29Z

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 90.12%. Comparing base (0722023) to head (fdb2c4b).
Report is 9 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main   #58738      +/-   ##
==========================================
- Coverage   90.16%   90.12%   -0.04%     
==========================================
  Files         637      639       +2     
  Lines      188126   188201      +75     
  Branches    36886    36909      +23     
==========================================
- Hits       169620   169617       -3     
- Misses      11272    11330      +58     
- Partials     7234     7254      +20

see 46 files with indirect coverage changes

🚀 New features to boost your workflow:

❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

test/pummel/test-string-decoder-large-buffer.js

Co-authored-by: Livia Medeiros <livia@cirno.name>

nodejs-github-bot · 2025-06-17T12:59:52Z

CI: https://ci.nodejs.org/job/node-test-pull-request/67497/

LiviaMedeiros · 2025-06-17T17:36:46Z

https://ci.nodejs.org/job/node-test-binary-windows-js-suites/34850/RUN_SUBSET=3,nodes=win11-COMPILED_BY-vs2022_clang/testReport/junit/(root)/pummel/test_string_decoder_large_buffer/

---
duration_ms: 203.164
exitcode: 134
severity: fail
stack: |2-

  <--- Last few GCs --->

  FATAL ERROR: MarkCompactCollector: young object promotion failed Allocation failed - JavaScript heap out of memory
  ----- Native stack trace -----

   1: 00007FF6857BD5B9
   2: 00007FF686409447
   3: 00007FF686208E77
   4: 00007FF6861E8E1E
   5: 00007FF6861DE4AC
   6: 00007FF6861DE15C
   7: 00007FF6861DDDA1
   8: 00007FF6861F0058
   9: 00007FF686AB6D7D
  10: 00007FF686AB7092
  11: 00007FF6861E0E14
  12: 00007FF6861C8733
  13: 00007FF6861C26D4
  14: 00007FF68620B3DA
  15: 00007FF68620A8C6
  16: 00007FF68621BE1E
  17: 00007FF68621B93B
  18: 00007FF68756CAAD
  19: 00007FF6862068E6
  20: 00007FF6862085ED
  21: 00007FF685FF37DA
  22: 00007FF685FF366B
  23: 00007FF6863DFB8A
  24: 00007FF6863DF03B
  25: 0000019737E78AFA
...

joyeecheung · 2025-06-17T21:36:50Z

Regarding the OOM issue - maybe adding something like --max-old-space-size=8000 would help, the default size is ~4GB AFAICT, which doesn't give a lot of wiggle room since the buffer alone is ~4GB.

nodejs-github-bot · 2025-06-19T18:00:43Z

CI: https://ci.nodejs.org/job/node-test-pull-request/67539/

nodejs-github-bot · 2025-06-20T06:12:33Z

Landed in 3d608bb

nodejs-github-bot added needs-ci PRs that need a full CI run. test Issues and PRs related to the tests. labels Jun 17, 2025

test: skip the test if the buffer allocation fails

b65339b

Use the error message as another condition to skip the test when the buffer allocation fails. Refs: nodejs@795dd8eb7988ae38553e Refs: nodejs@e9c6004a2d580008082b

lpinca force-pushed the fix/flaky-tests branch from e4e25da to b65339b Compare June 17, 2025 08:34

lpinca added the request-ci Add this label to start a Jenkins CI on a PR. label Jun 17, 2025

github-actions bot added request-ci-failed An error occurred while starting CI via request-ci label, and manual interventon is needed. and removed request-ci Add this label to start a Jenkins CI on a PR. labels Jun 17, 2025

This comment was marked as outdated.

Sign in to view

LiviaMedeiros approved these changes Jun 17, 2025

View reviewed changes

LiviaMedeiros added request-ci Add this label to start a Jenkins CI on a PR. and removed request-ci-failed An error occurred while starting CI via request-ci label, and manual interventon is needed. labels Jun 17, 2025

LiviaMedeiros reviewed Jun 17, 2025

View reviewed changes

test/pummel/test-string-decoder-large-buffer.js Outdated Show resolved Hide resolved

Update test/pummel/test-string-decoder-large-buffer.js

fdb2c4b

Co-authored-by: Livia Medeiros <livia@cirno.name>

lpinca added the commit-queue-squash Add this label to instruct the Commit Queue to squash all the PR commits into the first one. label Jun 17, 2025

LiviaMedeiros approved these changes Jun 17, 2025

View reviewed changes

github-actions bot removed the request-ci Add this label to start a Jenkins CI on a PR. label Jun 17, 2025

joyeecheung approved these changes Jun 17, 2025

View reviewed changes

lpinca added the commit-queue Add this label to land a pull request using GitHub Actions. label Jun 20, 2025

nodejs-github-bot removed the commit-queue Add this label to land a pull request using GitHub Actions. label Jun 20, 2025

nodejs-github-bot merged commit 3d608bb into nodejs:main Jun 20, 2025
60 checks passed

lpinca deleted the fix/flaky-tests branch June 20, 2025 06:18

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Uh oh!

test: skip the test if the buffer allocation fails #58738

test: skip the test if the buffer allocation fails #58738

lpinca commented Jun 17, 2025

Uh oh!

lpinca commented Jun 17, 2025

Uh oh!

codecov bot commented Jun 17, 2025 •

edited

Loading

Uh oh!

This comment was marked as outdated.

Uh oh!

nodejs-github-bot commented Jun 17, 2025

Uh oh!

LiviaMedeiros commented Jun 17, 2025

Uh oh!

joyeecheung commented Jun 17, 2025

Uh oh!

nodejs-github-bot commented Jun 19, 2025

Uh oh!

Uh oh!

nodejs-github-bot commented Jun 20, 2025

Uh oh!

Uh oh!

Uh oh!

test: skip the test if the buffer allocation fails #58738

test: skip the test if the buffer allocation fails #58738

Conversation

lpinca commented Jun 17, 2025

Uh oh!

lpinca commented Jun 17, 2025

Uh oh!

codecov bot commented Jun 17, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Codecov Report

Uh oh!

This comment was marked as outdated.

Uh oh!

nodejs-github-bot commented Jun 17, 2025

Uh oh!

LiviaMedeiros commented Jun 17, 2025

Uh oh!

joyeecheung commented Jun 17, 2025

Uh oh!

nodejs-github-bot commented Jun 19, 2025

Uh oh!

Uh oh!

nodejs-github-bot commented Jun 20, 2025

Uh oh!

Uh oh!

codecov bot commented Jun 17, 2025 •

edited

Loading