test_runner: files without any tests matching `--test-name-pattern` shouldn't be counted or reported
Personne n'a encore pris cette issue.
- Langage dominant
- JavaScript
- Étoiles
- 122k
- Forks
- 37.3k
- Merge moyen
- 4 j 2 h
- PR mergées (30 j)
- 283
Description
Version
v26.3.0
Subsystem
test_runner
What steps will reproduce the bug?
echo "const test = require('node:test'); test('good test', (t) => { t.assert.equal(1, 1) })" > one.test.js
echo "const test = require('node:test'); test('bad test', (t) => { t.assert.equal(1, 2) })" > two.test.js
node --test --test-name-pattern=good *.test.js
How often does it reproduce? Is there a required condition?
always
What is the expected behavior? Why is that the expected behavior?
✔ good test (0.963084ms)
ℹ tests 1
ℹ suites 0
ℹ pass 1
ℹ fail 0
ℹ cancelled 0
ℹ skipped 0
ℹ todo 0
ℹ duration_ms 0.963084ms
The file two.test.js does not contain a single test that matches the --test-name-pattern, so should be skipped entirely, i.e. treated as if it doesn't exist.
What do you see instead?
✔ good test (0.963084ms)
✔ two.test.js (61.52275ms) <-- file with no matching tests reported (by file name) as passing
ℹ tests 2 <-- should be 1 test
ℹ suites 0
ℹ pass 2 <-- should be 1 pass
ℹ fail 0
ℹ cancelled 0
ℹ skipped 0
ℹ todo 0
ℹ duration_ms 68.503541 <-- should be 0.963084ms
Guide de contribution
Ouvrir le guide de contribution
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Piste de recherche
Commencez par exécuter la commande signalée avec one.test.js et two.test.js, puis examinez l’implémentation de test_runner et ses tests existants pour la gestion de --test-name-pattern. C’est terminé lorsqu’un fichier sans test correspondant est omis et que le total, le nombre de tests réussis et la durée incluent uniquement le test correspondant qui réussit.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Évaluation
- Stack technique
- javascript, node.js
- Domaine
- cli, testing-qa
- Type d'issue
- Bug
- Difficulté
- 3/5
- Temps estimé
- 1-2 jours
- Activité
- Calme
- Clarté
- Plutôt claire
- Accessibilité débutants
- 67/100