Skip to content

[volumecache 11/12] Add cache path normalization tests - #381

Open
toloudis wants to merge 1 commit into
feature/volumecache-pr10-disk-cache-testsfrom
feature/volumecache-pr11-path-normalization-tests
Open

toloudis wants to merge 1 commit into
feature/volumecache-pr10-disk-cache-testsfrom
feature/volumecache-pr11-path-normalization-tests

Conversation

@toloudis

@toloudis toloudis commented Jun 7, 2026 •

Copy link
Copy Markdown
Contributor

Stacked draft PR 11 of 12 for the refreshed volume cache work.

This PR adds tests that equivalent local paths normalize to the same cache key while distinct subpaths, scenes, times, channels, regions, sequence flags, and remote URLs remain distinct.

Basically we are just making sure that if you specify the same source path in different ways, and it's cached, the cache will find it and return it.

@toloudis
toloudis force-pushed the feature/volumecache-pr11-path-normalization-tests branch from 1595a80 to eeb7294 Compare June 22, 2026 03:28
@toloudis
toloudis force-pushed the feature/volumecache-pr11-path-normalization-tests branch from eeb7294 to cc61b49 Compare August 14, 2026 21:45
@toloudis
toloudis force-pushed the feature/volumecache-pr11-path-normalization-tests branch from cc61b49 to 5aadfe4 Compare August 15, 2026 15:34
@toloudis
toloudis force-pushed the feature/volumecache-pr11-path-normalization-tests branch from 5aadfe4 to 0a1ce83 Compare August 27, 2026 18:19
@toloudis
toloudis force-pushed the feature/volumecache-pr11-path-normalization-tests branch from 0a1ce83 to e383fad Compare August 27, 2026 18:25
@toloudis
toloudis force-pushed the feature/volumecache-pr11-path-normalization-tests branch 2 times, most recently from f66f5c9 to d16ade9 Compare September 27, 2026 19:55
@toloudis
toloudis force-pushed the feature/volumecache-pr11-path-normalization-tests branch from d16ade9 to 48bb729 Compare October 2, 2026 21:11
@toloudis
toloudis marked this pull request as ready for review October 2, 2026 21:28

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant