tektoncd/pipeline

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

Aperta

#8921 aperta il 25 lug 2025

Vedi su GitHubGuida contributor
 (6 commenti) (0 reazioni) (1 assegnatario)Go (1943 fork)auto 404
area/testinggood first issuehelp wantedkind/cleanup

Metriche repository

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

Descrizione

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

Guida contributor

Direzione di ricerca
Leggi gli helper di test esistenti nel file di test e spostali nei file appropriati in pkg/reconciler/testing/, assicurandoti che import e riferimenti siano aggiornati. Controlla l'issue correlata #8906 per contesto.
Tech stack
go
Dominio
backendtesting
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
GitGo
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
85