Skip to content

test(native): add coverage for max cache items/size/age#1892

Merged
jpnurmi merged 5 commits into
masterfrom
jpnurmi/test/native-cache
Jul 22, 2026
Merged

test(native): add coverage for max cache items/size/age#1892
jpnurmi merged 5 commits into
masterfrom
jpnurmi/test/native-cache

Conversation

@jpnurmi

@jpnurmi jpnurmi commented Jul 22, 2026

Copy link
Copy Markdown
Collaborator
  • Parameterizes test_integration_cache.py for the native backend to add missing native coverage for cache max size, max age, max items, and retry item limits.
  • Moves the remaining cache tests from test_integration_native.py to test_integration_cache.py.
  • Promotes the run_crash helper for reuse outside test_integration_native.py.

Move native cache_keep and cache_consent coverage into the cache
integration test suite, and add native backend coverage for max items
and age limit pruning tests.
Comment thread tests/__init__.py Outdated

@cursor cursor Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit 42dc78f. Configure here.

Comment thread tests/test_integration_cache.py
@codecov

codecov Bot commented Jul 22, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 75.79%. Comparing base (17b3f62) to head (4ca4bae).
⚠️ Report is 2 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #1892      +/-   ##
==========================================
+ Coverage   75.77%   75.79%   +0.01%     
==========================================
  Files          92       92              
  Lines       21985    21985              
  Branches     3917     3917              
==========================================
+ Hits        16659    16663       +4     
+ Misses       4444     4441       -3     
+ Partials      882      881       -1     
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@jpnurmi
jpnurmi merged commit 58f39be into master Jul 22, 2026
67 checks passed
@jpnurmi
jpnurmi deleted the jpnurmi/test/native-cache branch July 22, 2026 15:19
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