Xuanwo commented on code in PR #2950:
URL: 
https://github.com/apache/incubator-opendal/pull/2950#discussion_r1306912071


##########
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:
   This test violates our VISION. OpenDAL should not write code directly 
against the service. The API call should either return a presigned request or 
`Unsupported` error.



-- 
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]

Reply via email to