tektoncd/pipeline

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

Offen

#8.921 geöffnet am 25.07.2025

Auf GitHub ansehenContributor Guide
 (6 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

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

Contributor Guide

Research-Richtung
Lesen Sie die vorhandenen Test Helfer in der Testdatei und verschieben Sie sie in die entsprechenden Dateien in pkg/reconciler/testing/. Stellen Sie sicher, dass Importe und Referenzen aktualisiert werden. Überprüfen Sie das zugehörige Issue #8906 für den Kontext.
Tech Stack
go
Domain
backendtesting
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
GitGo
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
85