tektoncd/pipeline

Improve unit test coverage of caching implementation.

Aperta

#9274 aperta il 31 dic 2025

 (8 commenti) (0 reazioni) (1 assegnatario)Go (1943 fork)auto 404
area/testinghelp wantedkind/cleanupkind/tep

Metriche repository

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

Descrizione

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.

Guida contributor