tektoncd/pipeline

All exported functions must have unit test

Aberta

#6.364 aberto em 16 de mar. de 2023

Ver no GitHubGuia do colaborador
 (9 comentários) (0 reação) (1 responsável)Go (1.943 forks)auto 404
area/testinggood first issuehelp wantedkind/cleanup

Métricas do repositório

Stars
 (9.013 estrelas)
Métricas de merge de PR
 (Mesclagem média 9d 3h) (137 fundiu PRs em 30d)

Description

As discussed in this PR https://github.com/tektoncd/pipeline/pull/6165#discussion_r1138710049, our standards restricts PR merges if an exported function does not have a unit test.

Let’s audit pipeline repo and make sure we comply to this standard across the board. Let’s check if an exported function has a unit test associated with it, if not, please add one.

/cc @jerop

Guia do colaborador

Direção de pesquisa
Inspecione cada função exportada (começando com letra maiúscula) no código Go usando 'grep' ou 'go doc' e verifique se existe um arquivo de teste unitário correspondente com uma função de teste (exemplo: TestFunctionName). Se estiver faltando, adicione um teste seguindo o padrão dos testes existentes no mesmo pacote.
Pilha de tecnologia
go
Domain
devopstesting
Tipo Issue
Teste
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-2 dias
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
GoGit
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
70