tektoncd/pipeline

refactor fatal and error in unit tests

Offen

#6.641 geöffnet am 09.05.2023

Auf GitHub ansehenContributor Guide
 (10 Kommentare) (0 Reaktionen) (1 zugewiesene Person)Go (1.943 Forks)auto 404
good first issuehelp wanted

Repository-Metriken

Stars
 (9.013 Sterne)
PR-Merge-Metriken
 (Durchschn. Merge 9T 3h) (137 gemergte PRs in 30 T)

Beschreibung

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

Contributor Guide

Research-Richtung
Überprüfen Sie vorhandene Unit Tests, um die aktuelle Verwendung von t.Fatalf und t.Errorf zu verstehen, schlagen Sie dann eine klare Richtlinie vor, wann diese verwendet werden sollen, und aktualisieren Sie die Tests entsprechend.
Tech Stack
go
Domain
testing
Issue Type
Refactoring
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-3 Stunden
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
Gounit testing
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
75