tektoncd/pipeline

Improve unit test coverage of caching implementation.

Ouverte

#9 274 ouverte le 31 déc. 2025

Voir sur GitHubGuide contributeur
 (8 commentaires) (0 réaction) (1 personne assignée)Go (1 943 forks)auto 404
area/testinghelp wantedkind/cleanupkind/tep

Métriques du dépôt

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

Description

Unit Test Coverage

Ensure comprehensive unit test coverage for all resolver cache logic. Achieve near 100% test coverage on new cache-related code paths. Many tests are already implemented, so this one is more of a sanity check.

[Update 05.12]: Check if there are duplicated tests in any of the existing cache tests and remove those. And check if any of the existing tests are not testing actual behavior. Those can also be removed.

Areas to verify:

  • All cache operation paths in pkg/remoteresolution/resolver/framework/cache/
  • Cache mode resolution logic (parameter → ConfigMap → default hierarchy)
  • Cache key generation for different parameter types
  • Edge cases (empty cache, cache misses, eviction scenarios)
  • Concurrent access patterns

How to Contribute

Read the parent issue https://github.com/tektoncd/pipeline/issues/9228#issue-3725752696.

Guide contributeur

Direction de recherche
Inspectez les tests de cache existants dans pkg/remoteresolution/resolver/framework/cache/ pour les doublons ou les tests qui ne testent pas le comportement réel. Supprimez les et assurez une couverture complète des opérations de cache, de la résolution de mode, de la génération de clés, des cas limites et de l'accès concurrent.
Stack technique
go
Domaine
backend
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-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.
Plutôt claire
Prérequis
GitGo testing
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
75