nodejs / nodejs/node

`replServer.setupHistory()` silently discards callback errors since v24.2.0

Aperta
#60,837 3 commenti 0 reazioni 0 assegnatari Vedi su GitHub

Nessuno ha ancora preso questa issue.

repl stale
Lingua principale
JavaScript
Stelle
122k
Fork
37.3k
Merge medio
4g 2h
PR unite (30g)
283

Descrizione

Version

26.0.0-pre

Platform
Linux ubuntu 6.8.0-87-generic #88-Ubuntu SMP PREEMPT_DYNAMIC Sat Oct 11 09:28:41 UTC 2025 x86_64 x86_64 x86_64 GNU/Linux

Microsoft Windows NT 10.0.26200.0 x64
Subsystem

No response

What steps will reproduce the bug?
  1. Add an intentional bad assert in the file of test case: test\parallel\test-repl-history-perm.js
Image
  1. Run python tools/test.py --mode=release parallel
  2. It reported "All tests passed."
Image
How often does it reproduce? Is there a required condition?

I can reproduce it everytime

What is the expected behavior? Why is that the expected behavior?

Test failed as 1 not equal 2 obviously.

What do you see instead?

[00:51]% 100|+ 3953|- 0]: Done

All tests passed.

Additional information

I try several versions of node. It seemed to happen from v24.2.0.
This is screenshot of v24.1.0

Image

Guida per i contributori

Apri la guida per i contributori

Come iniziare

  1. Leggi tutta la issue e poi la guida ai contributi del progetto.
  2. Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
  3. Fai un fork del repository e lavora su un branch.
  4. Apri una pull request che faccia riferimento al numero della issue.

Direzione di ricerca

Inizia da test/parallel/test-repl-history-perm.js e riproduci il problema usando python tools/test.py --mode=release parallel. Confronta il comportamento tra v24.1.0 e v24.2.0, concentrandoti sul motivo per cui l’Assertion intenzionale non fa fallire l’esecuzione dei test. Il lavoro è completato quando gli errori di callback di replServer.setupHistory() fanno sì che il comando di test segnali un fallimento.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Valutazione

Stack tecnologico
javascript
Ambito
testing-qa
Tipo di issue
Bug
Difficoltà
3/5
Tempo stimato
1-2 giorni
Stato di attività
Attiva
Chiarezza
Abbastanza chiara
Idoneità per principianti
68/100

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.