akashchamp opened a new pull request, #11063: URL: https://github.com/apache/arrow-rs/pull/11063
- Closes #9348. # Rationale for this change The async reader implementation module contains a large group of tests that exercise only public APIs. Moving those tests to the existing Parquet integration target keeps production code and tests easier to navigate while preserving the tests that intentionally exercise crate-private helpers. # What changes are included in this PR? - Move 16 public-API async reader tests into `parquet/tests/arrow_reader/async_reader.rs`. - Register the module behind the existing `async` feature. - Keep four tests in the source module because they require crate-private helpers or fields. # Are these changes tested? - Extracted integration target: 25 tests passed. - Retained source-module tests: 4 tests passed. - Full `cargo test -p parquet --features async`: 1,395 unit tests, 142 integration tests (1 ignored), and 81 doctests (7 ignored) passed. - `cargo clippy -p parquet --all-targets --features async -- -D warnings`, repository formatting, and the pinned typos check passed. - Manual validation listed the moved `async_reader` tests from the integration binary and directly ran `async_reader::test_async_reader` successfully. # Are there any user-facing changes? No. This reorganizes tests only and does not change production behavior or public APIs. # AI assistance Assisted-by: Codex AI assistance was used to prepare the initial test extraction. I reviewed every moved test, retained the crate-private coverage in the source module, and manually validated the extracted integration target. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
