tektoncd/pipeline

All exported functions must have unit test

Offen

#6.364 geöffnet am 16.03.2023

Auf GitHub ansehenContributor Guide
 (9 Kommentare) (0 Reaktionen) (1 zugewiesene Person)Go (1.943 Forks)auto 404
area/testinggood first issuehelp wantedkind/cleanup

Repository-Metriken

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

Beschreibung

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

Contributor Guide

Research-Richtung
Überprüfe jede exportierte Funktion (großgeschrieben) im Go Code mit 'grep' oder 'go doc' und prüfe, ob eine entsprechende Unit Test Datei mit einer Testfunktion (z.B. TestFunctionName) existiert. Falls nicht, füge einen Test hinzu, der dem Muster der vorhandenen Tests im selben Paket folgt.
Tech Stack
go
Domain
devopstesting
Issue Type
Test
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-2 Tage
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
GoGit
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
70