`replServer.setupHistory()` silently discards callback errors since v24.2.0
Nadie ha tomado este issue todavía.
- Lenguaje dominante
- JavaScript
- Estrellas
- 122k
- Forks
- 37.3k
- Merge medio
- 4 d 2 h
- PR fusionados (30 d)
- 283
Descripción
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?
- Add an intentional bad assert in the file of test case: test\parallel\test-repl-history-perm.js
- Run python tools/test.py --mode=release parallel
- It reported "All tests passed."
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
Guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Línea de trabajo
Comienza con test/parallel/test-repl-history-perm.js y reproduce el problema usando python tools/test.py --mode=release parallel. Compara el comportamiento entre v24.1.0 y v24.2.0, centrándote en por qué la Assertion intencionada no hace que falle la ejecución de pruebas. Se considera terminado cuando los errores de callback de replServer.setupHistory() hacen que el comando de pruebas informe de un fallo.
Escrito por el modelo de indexación a partir del texto del issue.
Evaluación
- Stack tecnológico
- javascript
- Área
- testing-qa
- Tipo de issue
- Error
- Dificultad
- 3/5
- Tiempo estimado
- 1-2 días
- Estado de actividad
- Activo
- Claridad
- Bastante claro
- Aptitud para principiantes
- 68/100