tektoncd/pipeline

All exported functions must have unit test

Aperta

#6364 aperta il 16 mar 2023

Vedi su GitHubGuida contributor
 (9 commenti) (0 reazioni) (1 assegnatario)Go (1943 fork)auto 404
area/testinggood first issuehelp wantedkind/cleanup

Metriche repository

Star
 (9013 stelle)
Metriche merge PR
 (Metriche PR in attesa)

Descrizione

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

Guida contributor

Direzione di ricerca
Ispeziona ogni funzione esportata (in maiuscolo) nel codice Go usando 'grep' o 'go doc' e verifica se esiste un file di test corrispondente con una funzione di test (es. TestFunctionName). Se manca, aggiungi un test seguendo il pattern dei test esistenti nello stesso pacchetto.
Tech stack
go
Dominio
devopstesting
Tipo issue
Test
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-2 giorni
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
GoGit
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
70