Not all reporters support recursion.
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 3/5
- Tiempo estimado
- 1-2 días
- Aptitud para principiantes
- 35/100
- Tipo de issue
- Error
- Claridad
- Bastante claro
- Estado de actividad
- Estancado
- Stack tecnológico
- javascript, node.js
- Área
- testing
Línea de trabajo
Inspecciona las llamadas a nodeunit.runFiles() en los reporters eclipse, html, junit, minimal, nested, skip-passed y verbose, comparándolas con lcov y machineout. Primero determina cómo los tests existentes de los reporters ejercitan recursive runs y, después, añade cobertura que muestre que la recursive option se respeta en los reporters afectados.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Just started using nodeunit. It seems not all the reporters support the recursive option.
It seems recursive: options.recursive, is missing from the call to nodeunit.runFiles() in at least eclipse, html, junit, minimal, nested, skip-passed and verbose.
I can see where the missing option is missing from lcov and machineout, but in my limited testing, I couldn't see a difference ... or ANY output!
I have amended my installed nodeunit and get the recursion as expected.
I can create a patch (https://github.com/caolan/nodeunit/pull/348), but I'm not sure about proof/testing. Ideally, I'd like to know how to add tests to prove that the recursion is operational, but if the fix is correct and a super obvious oversight, then maybe it wouldn't be necessary.
- Lenguaje dominante
- JavaScript
- Estrellas
- 1.9k
- Forks
- 359
- Métricas de merge de PR
- Sin PR fusionados en 30 d
Preparar el entorno
Este proyecto no incluye contenedor de desarrollo, Dockerfile ni guía de contribución, así que la configuración corre por tu cuenta: empieza por su README y consulta nuestra guía para la primera contribución para los pasos generales.
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.
Más de caolan/nodeunit
-
Dificultad 4/5 3-5 días Aptitud para principiantes 25/100
-
Dificultad 3/5 1-2 días Aptitud para principiantes 35/100
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 42/100
-
Dificultad 5/5 Más de una semana Aptitud para principiantes 20/100
-
ES6 Module exportAbierto
Dificultad 5/5 Más de una semana Aptitud para principiantes 25/100
Todos los issues de caolan/nodeunit
Issues similares
-
area/frontend good first issue kind/cooldown
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
kestra-io/kestra#20702 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
Form-data fields are not parsed when importing a cURL requestPosiblemente ocupada @aayushbaluni la tomó hoy. Abiertobug status-new
Dificultad 2/5 1-3 horas Aptitud para principiantes 69/100
Los mantenedores suelen responder en 1 día
-
bug MVP
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
saayam-for-all/webapp#1911 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
vshulcz/deja-vu#4937 · 2 comentarios ·
Los mantenedores suelen responder en 1 día
-
[Focus] Accessibility (a11y) [Package] Grid [Type] Bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 76/100
Los mantenedores suelen responder en 1 día