RanaPriyansh opened a new pull request, #859: URL: https://github.com/apache/arrow-rs-object-store/pull/859
# Which issue does this PR close? Part of #753. This does not complete all filesystem error coverage. # Rationale for this change Local filesystem error paths need deterministic tests that also work when permission checks cannot force failures. # What changes are included in this PR? Add two tests using nonempty directories as object targets. The put test checks the rename error, preserved contents, and immediate removal of the staged file. The delete test checks the error path and preserved contents. Both tests then remove the obstacle and verify a successful public operation at the same location. The tests use real filesystem operations and typed error assertions. They do not change production code or introduce failure injection. Local verification on macOS with Rust 1.97.0: both exact tests, the local filesystem suite, and default tests pass. Formatting, strict all-feature/all-target Clippy, and strict no-default fs Clippy pass. Linux, Windows, and MSRV execution have not been performed locally. OpenAI Codex assisted with the tests and this PR description. # Are there any user-facing changes? No. Production behavior is unchanged. -- 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]
