angular / angular/angular-cli

Unit test: Add reporter watermark configuration to angular.json

Aberta
#31,998 3 comentários 0 reações 0 responsáveis Ver no GitHub
angular/build:unit-test area: @angular/build feature feature: insufficient votes
Linguagem predominante
TypeScript
Estrelas
27k
Forks
11.8k
Merge médio
14h 23min
PRs com merge (30d)
162

Descrição

### Command

test

### Description

After taking advantage of all the new Vitest unit test features in 21.0.0, there is only one configuration item left in the vitest-base.config.ts:

```
export default defineConfig({
test: {
slowTestThreshold: 1000
}
});
```

This is required since the Vitest's [slowTestThreshold](https://vitest.dev/guide/cli.html#slowtestthreshold) default of 300 ms is not adequate for extended test cases.

Ideally this configuration should be available through the angular.json configuration unit-test options, since it would allow the complete removal of the vitest-base.config.ts file.

Maybe follow the nomenclature that you introduced with "coverageWatermarks" and call it "reporterWatermark".

### Describe the solution you'd like

_No response_

### Describe alternatives you've considered

_No response_

Guia de contribuição

Abrir o guia de contribuição

Direção de pesquisa

Comece pelas opções de testes unitários de angular.json e vitest-base.config.ts e, em seguida, execute o comando de testes unitários do repositório para rastrear como as opções chegam ao Vitest. Confirme que a configuração desejada do limite para testes lentos é aceita por meio de angular.json e que vitest-base.config.ts pode ser removido sem perder a configuração.

Escrita pelo modelo de indexação a partir do texto da issue.

Avaliação

Stack de tecnologia
angular, typescript
Domínio
cli, testing, tooling
Tipo de issue
Funcionalidade
Dificuldade
4/5
Tempo estimado
3-5 dias
Status de atividade
Estagnada
Clareza
Razoavelmente clara
Facilidade para iniciantes
45/100

Receba novas issues na sua caixa de entrada

Um resumo curto de issues do GitHub para quem está começando.