nodejs / nodejs/node

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

Offen
#60,837 3 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Dieses Issue hat noch niemand übernommen.

repl stale
Vorherrschende Sprache
JavaScript
Sterne
122k
Forks
37.3k
Ø Merge
4 T. 2 Std.
Gemergte PRs (30 T.)
283

Beschreibung

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

Beitragsleitfaden

Beitragsleitfaden öffnen

Erste Schritte

  1. Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
  2. Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
  3. Forke das Repository und arbeite in einem Branch.
  4. Öffne einen Pull Request, der die Issue-Nummer nennt.

Rechercherichtung

Beginne mit test/parallel/test-repl-history-perm.js und reproduziere das Problem mit python tools/test.py --mode=release parallel. Vergleiche das Verhalten zwischen v24.1.0 und v24.2.0 und konzentriere dich darauf, warum die absichtliche Assertion den Testlauf nicht fehlschlagen lässt. Erledigt ist es, wenn Callback-Fehler von replServer.setupHistory() dazu führen, dass der Testbefehl einen Fehler meldet.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
javascript
Bereich
testing-qa
Issue-Typ
Bug
Schwierigkeit
3/5
Geschätzter Aufwand
1-2 Tage
Aktivitätsstatus
Aktiv
Klarheit
Größtenteils klar
Anfängerfreundlichkeit
68/100

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.