feat(storage): Add more test coverage for initial read ranges in AsyncClient::Open() - #16357
Conversation
There was a problem hiding this comment.
Code Review
This pull request introduces comprehensive unit and integration tests for multi-range pre-warming, caching, and teardown safety in ObjectDescriptorImpl and AsyncClient. The reviewer feedback focuses on aligning the new test code with the repository style guide, specifically by replacing Abseil types (absl::optional, absl::string_view) with their standard library equivalents (std::optional, std::string_view), avoiding the use of auto when it obscures StatusOr return types, and ensuring auto is properly qualified or replaced with explicit types in range-based loops.
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #16357 +/- ##
==========================================
- Coverage 92.26% 92.26% -0.01%
==========================================
Files 2237 2237
Lines 210434 210630 +196
==========================================
+ Hits 194166 194340 +174
- Misses 16268 16290 +22 ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
f57b76c to
d89474b
Compare
bc2114b to
1a92ba4
Compare
No description provided.