caolan / caolan/nodeunit

Not all reporters support recursion.

Ouverte
#347 7 commentaires 0 réactions 0 personnes assignées Voir sur GitHub
Langage dominant
JavaScript
Étoiles
1.9k
Forks
359
Métriques de merge des PR
Aucune PR mergée en 30 j

Description

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.

Guide de contribution

Aucun guide de contribution indexé pour ce dépôt

Piste de recherche

Examinez les appels à nodeunit.runFiles() dans les reporters eclipse, html, junit, minimal, nested, skip-passed et verbose, en les comparant à lcov et machineout. Déterminez d’abord comment les tests existants des reporters couvrent les recursive runs, puis ajoutez une couverture montrant que la recursive option est respectée dans les reporters concernés.

Rédigé par le modèle d'indexation à partir du texte de l'issue.

Évaluation

Stack technique
javascript, node.js
Domaine
testing
Type d'issue
Bug
Difficulté
3/5
Temps estimé
1-2 jours
Activité
À l'abandon
Clarté
Plutôt claire
Accessibilité débutants
35/100

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.