dragonknightk opened a new pull request, #3569: URL: https://github.com/apache/iceberg-rust/pull/3569
## Which issue does this PR close? - Closes #3385. ## What changes are included in this PR? Moves each backend's tests out of the crate-level `tests` module into the module that owns the backend, following the `hdfs_native` split in #3111: `memory.rs`, `fs.rs`, `s3.rs`, `gcs.rs`, `oss.rs` and `azdls.rs` each get their own `mod tests`. `lib.rs` keeps the cross-backend tests: client config parsing, factory construction, the storage-agnostic writer contract, and the IO timeout test, which exercises the `TimeoutLayer` that `resolve` applies to every backend. Feature gates are not repeated on the moved tests. `lib.rs` already guards each backend module behind its `opendal-*` feature in `cfg_if!`, so a test in `gcs.rs` can only compile when that module exists. No test was added, removed or renamed; the moved bodies are unchanged. ## Are these changes tested? `cargo fmt --all -- --check`, `cargo clippy -p iceberg-storage-opendal --all-targets -- -D warnings` and the same with `--all-features`, plus `cargo test -p iceberg-storage-opendal --lib` (17 tests) and with `--all-features` (74 tests). ## AI Disclosure Codex and Grok Bot, working for dragonknightk, contributed to this code. -- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
