tektoncd/pipeline

Add integration tests for /tekton/debug/scripts

Offen

#4.426 geöffnet am 14.12.2021

Auf GitHub ansehenContributor Guide
 (5 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)Go (1.943 Forks)auto 404
area/testinghelp wanted

Repository-Metriken

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

Beschreibung

/area testing

AFAIK the integration tests work by checking if TaskRuns/PipelineRuns have reached a desired state, based on which the tests are marked as success or failure.

To test scripts in /tekton/debug/scripts dir in the TaskRun Pod, first we would need the TaskRun Pod running with the breakpoint(s) in place. Then, we need to run exec commands allowing us to run scripts in the container(s) with breakpoint. Thus helping us assess the state/logs of the TaskRun after, based on the commands we have run.

How can this be achieved ? What are the possible problems we could face while writing these tests ?

Contributor Guide

Research-Richtung
Verstehe zuerst das vorhandene Integrationstest Framework im Tekton Pipelines Repository. Identifiziere, wie die Zustände von TaskRun/PipelineRun überprüft werden. Entwerfe dann einen Test, der einen TaskRun mit Breakpoints erstellt, `kubectl exec` verwendet, um Skripte aus `/tekton/debug/scripts` im Container auszuführen, und erwartete Zustände oder Logs bestätigt. Berücksichtige Probleme wie Container Bereitschaft, Skriptausführungsberechtigungen und Testisolierung.
Tech Stack
go
Domain
backend
Issue Type
Test
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
3
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.
Meist klar
Voraussetzungen
GoKubernetesTekton
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
30