tektoncd/pipeline

refactor fatal and error in unit tests

Aperta

#6641 aperta il 9 mag 2023

Vedi su GitHubGuida contributor
 (10 commenti) (0 reazioni) (1 assegnatario)Go (1943 fork)auto 404
good first issuehelp wanted

Metriche repository

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

Descrizione

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

Guida contributor

Direzione di ricerca
Rivedi i test unitari esistenti per capire l'uso corrente di t.Fatalf e t.Errorf, quindi proponi una linea guida chiara su quando usarli e aggiorna i test di conseguenza.
Tech stack
go
Dominio
testing
Tipo issue
Refactoring
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
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
Gounit testing
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
75