Not all reporters support recursion.
Ninguém assumiu esta issue ainda.
Avaliação
- Dificuldade
- 3/5
- Tempo estimado
- 1-2 dias
- Facilidade para iniciantes
- 35/100
- Tipo de issue
- Bug
- Clareza
- Razoavelmente clara
- Status de atividade
- Estagnada
- Stack de tecnologia
- javascript, node.js
- Domínio
- testing
Direção de pesquisa
Inspecione as chamadas a nodeunit.runFiles() nos reporters eclipse, html, junit, minimal, nested, skip-passed e verbose, comparando-as com lcov e machineout. Primeiro determine como os testes existentes dos reporters exercitam recursive runs e, em seguida, adicione cobertura mostrando que a recursive option é respeitada nos reporters afetados.
Escrita pelo modelo de indexação a partir do texto da issue.
Descrição
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.
- Linguagem predominante
- JavaScript
- Estrelas
- 1.9k
- Forks
- 359
- Métricas de merge de PRs
- Nenhum PR com merge em 30d
Preparar o ambiente
Este projeto não oferece contêiner de desenvolvimento, Dockerfile nem guia de contribuição, então a configuração fica por sua conta: comece pelo README e veja nosso guia da primeira contribuição para os passos gerais.
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 caolan/nodeunit
-
Dificuldade 4/5 3-5 dias Facilidade para iniciantes 25/100
-
Dificuldade 3/5 1-2 dias Facilidade para iniciantes 35/100
-
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 42/100
-
Dificuldade 5/5 Mais de uma semana Facilidade para iniciantes 20/100
-
ES6 Module exportAberta
Dificuldade 5/5 Mais de uma semana Facilidade para iniciantes 25/100
Todas as issues de caolan/nodeunit
Issues semelhantes
-
needs-website-changes
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 68/100
Mantenedores costumam responder em até 1 dia
-
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 75/100
Mantenedores costumam responder em até 1 dia
-
[Feature] 已有证书托管支持拖拽上传Aberta
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 63/100
-
[Bug]: [A11y] Missing accessible name on icon-only standalone links (WCAG 2.1 SC 2.4.4, 4.1.2)Abertabug
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 72/100
jaegertracing/jaeger-ui#4596 ·
Mantenedores costumam responder em até 1 dia
-
Add: CNC Region Calima [CO]Abertachannels:add check:passed
Dificuldade 1/5 Menos de uma hora Facilidade para iniciantes 72/100
Mantenedores costumam responder em até 9 dias