Xuanwo commented on code in PR #2950:
URL:
https://github.com/apache/incubator-opendal/pull/2950#discussion_r1306936511
##########
bindings/tests/features/binding.feature:
##########
@@ -32,3 +32,7 @@ Feature: OpenDAL Binding
Then The async file "test" entry mode must be file
Then The async file "test" content length must be 13
Then The async file "test" must have content "Hello, World!"
+
+ Scenario: OpenDAL Operations with Presign
+ Given A new OpenDAL Async Operator (s3 presign only)
Review Comment:
> So we can only write tests for `presign` after exposing `op.info()` and
`Capability`. Is it right?
No, we don't need to. We can simply call `presign` and check if its result
is successful or returns an error with the error kind `Unsupported`.
--
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]
