HonahX commented on code in PR #1429: URL: https://github.com/apache/iceberg-python/pull/1429#discussion_r1906450780
########## tests/catalog/test_s3tables.py: ########## @@ -0,0 +1,227 @@ +import pytest Review Comment: Since this the file's name starts with "integration_*" instead of "test_*", it won't be collected by `make test`, so I think we're good for now. I am in favor of having both unit tests and integration tests for service integration like s3tables and glue. Although the integration tests for AWS services will probably never run with our auto CI, they are still very helpful in case when bugs are not caught by mocked tests and when verifying the release. -- 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: issues-unsubscr...@iceberg.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: issues-unsubscr...@iceberg.apache.org For additional commands, e-mail: issues-h...@iceberg.apache.org
