tektoncd/pipeline

All exported functions must have unit test

Ouverte

#6 364 ouverte le 16 mars 2023

Voir sur GitHubGuide contributeur
 (9 commentaires) (0 réaction) (1 personne assignée)Go (1 943 forks)auto 404
area/testinggood first issuehelp wantedkind/cleanup

Métriques du dépôt

Stars
 (9 013 étoiles)
Métriques de merge PR
 (Métriques PR en attente)

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

Guide contributeur

Direction de recherche
Inspectez chaque fonction exportée (commençant par une majuscule) dans le code Go à l'aide de 'grep' ou 'go doc', et vérifiez s'il existe un fichier de test unitaire correspondant avec une fonction de test (par exemple TestFunctionName). Si manquant, ajoutez un test en suivant le modèle des tests existants dans le même package.
Stack technique
go
Domaine
devopstesting
Type d'issue
Test
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-2 jours
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
GoGit
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
70