tektoncd/pipeline

refactor fatal and error in unit tests

Aberta

#6.641 aberto em 9 de mai. de 2023

Ver no GitHubGuia do colaborador
 (10 comentários) (0 reação) (1 responsável)Go (1.943 forks)auto 404
good first issuehelp wanted

Métricas do repositório

Stars
 (9.013 estrelas)
Métricas de merge de PR
 (Métricas PR pendentes)

Description

We have a mix of t.Fatalf and t.Errorf in our existing unit tests. It's most of the times user's choice to decide which one to implement. There are no specific guidelines as part a tekton pipeline development process.

As part of this issue:

  • Task 1: Define a specific guideline around t.Fatalf and t.Errorf and document them as part of our unit test best practices.

  • Task 2: Revisit our code base to make sure we are aligned and following with the guidelines documented in Task 1.

See relevant comment - https://github.com/tektoncd/pipeline/pull/6596#discussion_r1188598706

Guia do colaborador

Direção de pesquisa
Revise os testes unitários existentes para entender o uso atual de t.Fatalf e t.Errorf, depois proponha uma diretriz clara sobre quando usá los e atualize os testes de acordo.
Pilha de tecnologia
go
Domain
testing
Tipo Issue
Refatoração
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
2
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-3 horas
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
Gounit testing
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
75