Skip to content

test(data): remove cache_dir deprecation warnings in cache tests#36

Merged
lilinoct18-coder merged 1 commit into
devfrom
chore/recover-tests-data-stash
Feb 16, 2026
Merged

test(data): remove cache_dir deprecation warnings in cache tests#36
lilinoct18-coder merged 1 commit into
devfrom
chore/recover-tests-data-stash

Conversation

@novis10813

Copy link
Copy Markdown
Owner

Summary

  • Migrate tests/data/test_cache.py and tests/data/test_cache_polars.py from deprecated BarCache(cache_dir=...) usage to BarCache(storage=..., cache_prefix="").
  • Add shared fixtures using LocalStorageBackend so tests remain isolated by temporary directory while using the supported API.
  • Keep test behavior unchanged (cache key/date-range/clear semantics), while eliminating deprecated constructor usage.

Verification

  • uv run pytest tests/data/test_cache.py tests/data/test_cache_polars.py -W error::DeprecationWarning (14 passed)

Notes

  • This PR intentionally touches only tests/data/* and does not include uv.lock or other stash contents.

@lilinoct18-coder lilinoct18-coder left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

it's a descent job, pretty good

@lilinoct18-coder lilinoct18-coder merged commit 65641be into dev Feb 16, 2026
4 checks passed
@lilinoct18-coder lilinoct18-coder deleted the chore/recover-tests-data-stash branch February 16, 2026 02:38
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.

2 participants