tektoncd/pipeline

Improve unit test coverage of caching implementation.

Open

#9,274 opened on Dec 31, 2025

View on GitHub
 (8 comments) (0 reactions) (1 assignee)Go (1,943 forks)auto 404
area/testinghelp wantedkind/cleanupkind/tep

Repository metrics

Stars
 (9,013 stars)
PR merge metrics
 (PR metrics pending)

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.

Contributor guide