test(read_cache): bifurcate read_cache local modification tests for rapid writes

[vipnydav]

Description

Fixes LRU entry size calculation to prevent underflow and bifurcates read_cache local modification tests to support separate test runners for base runs and rapid-writes enabled runs.

Testing details

1. Unit tests - Done
2. Integration tests - Done

Any backward incompatible change? If so, please explain.

No

[gemini-code-assist]

Code Review

This pull request introduces several updates to the LRU cache implementation and integration tests. In lru.go, a size field is added to the entry struct to cache the entry size, preventing potential TOCTOU (Time-of-check to time-of-use) issues when entry sizes are updated dynamically. The invariant checks and cache operations are updated to use this cached size, and corresponding concurrency tests are added. Additionally, a missing file descriptor close is resolved in the integration test helpers, and integration tests are refactored to support rapid writes configurations for Pirlo buckets. As there are no review comments, I have no additional feedback to provide.

[codecov]

Codecov Report

❌ Patch coverage is 86.36364% with 3 lines in your changes missing coverage. Please review.
✅ Project coverage is 83.85%. Comparing base (93ee10b) to head (7046338).
⚠️ Report is 2 commits behind head on master.

Files with missing lines	Patch %	Lines
internal/cache/lru/lru.go	86.36%	1 Missing and 2 partials ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##           master    #4982      +/-   ##
==========================================
- Coverage   83.86%   83.85%   -0.02%     
==========================================
  Files         173      173              
  Lines       21153    21160       +7     
==========================================
+ Hits        17741    17744       +3     
- Misses       2744     2747       +3     
- Partials      668      669       +1     

Flag	Coverage Δ
unittests	83.85% <86.36%> (-0.02%)	⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[github-actions]

Hi @meet2mky, @raj-prince, your feedback is needed to move this pull request forward. This automated reminder was triggered because there has been no activity for over 24 hours. Please provide your input when you have a moment. Thank you!

[github-actions]

Hi @meet2mky, @raj-prince, your feedback is needed to move this pull request forward. This automated reminder was triggered because there has been no activity for over 24 hours. Please provide your input when you have a moment. Thank you!