Refactor file io tests to support different file io storage
- Dominant language
- Rust
- Stars
- 1.4k
- Forks
- 567
- Avg merge
- 2d 2h
- Merged PRs (30d)
- 93
Description
> * Could we add some tests to verify it? I think it should be easy since we have it as default feature.
Current test is not extsiable for different storage service, please allow me to complement them in a new PR. For example, I can refactor `test_local_input_file`, `test_delete_local_file`, ... to make them work on different file io instead.
> Could we have a simple doc to explain how to use it? At least for me, it's not straight forward without refering to opendal's doc.
Sure, I will add this.
_Originally posted by @Xuanwo in https://github.com/apache/iceberg-rust/pull/481#discussion_r1693014389_
Contributor guide
Research direction
Start by locating the existing test_local_input_file and test_delete_local_file tests and review how the current default file I/O storage is configured. Refactor the relevant tests so they work with different storage services, then add a simple usage document explaining the configuration without requiring readers to consult OpenDAL's documentation.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- rust
- Domain
- backend, testing
- Issue type
- Refactor
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Quiet
- Clarity
- Mostly clear
- Newbie friendliness
- 55/100