Copilot commented on code in PR #6365:
URL: https://github.com/apache/opendal/pull/6365#discussion_r2187268671
##########
core/tests/behavior/async_list.rs:
##########
@@ -113,6 +113,9 @@ pub async fn test_list_prefix(op: Operator) -> Result<()> {
/// listing a directory, which contains more objects than a single page can
take.
pub async fn test_list_rich_dir(op: Operator) -> Result<()> {
+ if !op.info().full_capability().create_dir {
+ return Ok(());
+ }
Review Comment:
The repeated guard clause for `create_dir` capability appears in many tests.
Consider extracting this into a shared helper function or using a custom test
attribute/macro to skip tests when the capability is unsupported, reducing
duplication and improving readability.
```suggestion
skip_if_create_dir_unsupported(&op)?;
```
--
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]
