test_runner: files without any tests matching `--test-name-pattern` shouldn't be counted or reported
Mantenedores costumam responder em até 1 dia
Ninguém assumiu esta issue ainda.
Avaliação
- Dificuldade
- 3/5
- Tempo estimado
- 1-2 dias
- Facilidade para iniciantes
- 67/100
- Tipo de issue
- Bug
- Clareza
- Razoavelmente clara
- Status de atividade
- Pouca atividade
- Stack de tecnologia
- javascript, node.js
- Domínio
- cli, testing-qa
Direção de pesquisa
Comece executando o comando reportado com one.test.js e two.test.js e, em seguida, inspecione a implementação de test_runner e os testes existentes para o tratamento de --test-name-pattern. Está concluído quando um arquivo sem testes correspondentes é omitido e o total, a contagem de testes aprovados e a duração incluem apenas o teste correspondente bem-sucedido.
Escrita pelo modelo de indexação a partir do texto da issue.
Descrição
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
- Linguagem predominante
- JavaScript
- Estrelas
- 122k
- Forks
- 37.4k
- Merge médio
- 4d 8h
- PRs com merge (30d)
- 293
Preparar o ambiente
Primeiros passos
- Leia a issue inteira e depois o guia de contribuição do projeto.
- Comente na issue dizendo que vai assumir — evita que duas pessoas façam o mesmo trabalho.
- Faça um fork do repositório e trabalhe em uma branch.
- Abra um pull request que referencie o número da issue.
Mais de nodejs/node
-
doc
Dificuldade 1/5 Menos de uma hora Facilidade para iniciantes 90/100
Mantenedores costumam responder em até 1 dia
-
doc
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 65/100
Mantenedores costumam responder em até 1 dia
-
build
Dificuldade 1/5 Menos de uma hora Facilidade para iniciantes 88/100
nodejs/node#66076 · 2 comentários ·
Mantenedores costumam responder em até 1 dia
-
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 84/100
nodejs/node#65994 · 2 comentários · 2 reações ·
Mantenedores costumam responder em até 1 dia
-
feature request
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 68/100
nodejs/node#63841 · 1 comentário ·
Mantenedores costumam responder em até 1 dia
Todas as issues de nodejs/node
Issues semelhantes
-
新讀者走讀回報，照著一篇文章實際操作Abertadocumentation good first issue help wanted
Dificuldade 1/5 1-3 horas Facilidade para iniciantes 92/100
Mantenedores costumam responder em até 1 dia
-
automated issue report
Dificuldade 1/5 Menos de uma hora Facilidade para iniciantes 85/100
lirantal/discoprint#29 ·
Mantenedores costumam responder em até 1 dia
-
RSS feed autodiscoveryAbertafeature-request
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 68/100
louislam/uptime-kuma#7914 · 1 comentário ·
Mantenedores costumam responder em até 1 dia
-
Code Cleanup dependencies
Dificuldade 1/5 Menos de uma hora Facilidade para iniciantes 68/100
ProjectSidewalk/SidewalkWebpage#5609 ·
Mantenedores costumam responder em até 1 dia
-
help wanted javascript
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 68/100
OWASP/cornucopia#3538 ·
Mantenedores costumam responder em até 1 dia