tektoncd/pipeline

Cleanup 🧹 `pipelinerunresolution_test.go` 🧹 and move mock data, status mutations and selectors to testing framework.

Ouverte

#8 921 ouverte le 25 juil. 2025

Voir sur GitHubGuide contributeur
 (6 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
 (Merge moyen 9j 3h) (137 PRs mergées en 30 j)

Description

Summary

Refactor pipelinerunresolution_test.go a bit and move testing helpers to the new testing framework location:

  • Move all mock data creation to pkg/reconciler/testing/factory.go, i.e. var p = ..., var pts = ..., newTaskRun(), ...
    • rename the single letter variables to proper names e.g. p => pipeline, pts = pipelineTasks, ...
  • Move all status transformers to pkg/reconciler/testing/status.go, i.e makeSucceed(), withCanceled()...
  • Move all selectors to pkg/reconciler/testing/selectors.go, i.e. nopGetCustomRun, ...

This is a good first issue.

Related issue #8906.

/help /good-first-issue /kind cleanup

Guide contributeur

Direction de recherche
Lisez les helpers de test existants dans le fichier de test et déplacez les vers les fichiers appropriés dans pkg/reconciler/testing/, en vous assurant que les importations et les références sont mises à jour. Consultez l'issue connexe #8906 pour le contexte.
Stack technique
go
Domaine
backendtesting
Type d'issue
Refactorisation
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-3 heures
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
GitGo
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
85