YuangGao commented on code in PR #7636: URL: https://github.com/apache/opendal/pull/7636#discussion_r3486759192
########## bindings/python/tests/test_read.py: ########## @@ -258,7 +258,11 @@ async def test_async_read_not_exists(service_name, operator, async_operator): @pytest.mark.need_capability( - "read", "read_with_if_modified_since", "read_with_if_unmodified_since" Review Comment: without this change, enabling `read_with_if_modified_since` / `read_with_if_unmodified_since` on the HTTP service makes `test_sync_conditional_reads` newly match the HTTP backend via need_capability, then crash on `operator.write()` since HTTP is read-only. Adding write / delete to the gate aligns it with what the test body actually does and keeps the test skipped on read-only backends -- 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]
