Skip to content

test(worker): cover LRU accounting by build mode - #331

Merged
beinan merged 1 commit into
milvus-io:mainfrom
beinan:fix-worker-lru-accounting
Jul 27, 2026
Merged

test(worker): cover LRU accounting by build mode#331
beinan merged 1 commit into
milvus-io:mainfrom
beinan:fix-worker-lru-accounting

Conversation

@beinan

@beinan beinan commented Jul 27, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • gate the debug_assert! panic test with cfg(debug_assertions)
  • add release-mode coverage for saturating underflow behavior

Without the build-mode gate, cargo test --release fails because debug_assert! is compiled out while #[should_panic] remains active.

Follow-up to #308.

Verification

  • cargo fmt --all --check
  • cargo clippy -p talon-worker --all-targets --all-features --locked -- -D warnings
  • cargo test -p talon-worker --all-features --locked
  • cargo test --release -p talon-worker eviction --locked

@beinan
beinan merged commit 121085d into milvus-io:main Jul 27, 2026
17 checks passed
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