jayceslesar commented on issue #813: URL: https://github.com/apache/iceberg-python/issues/813#issuecomment-2960856449
We can delete mountains of code if this is done correctly.... What about a strategy of each test_catalog providing a suite of basic fixtures that can be passed into a parameterized suite of functions? Something like... ```py @mock_aws @pytest.fixture def catalog_list_tables_fixture(...) ... ``` Like the glue test, and dynamo test (ect) could all provide a fixture like that? Unsure the best architecture but I think doing it correctly can vastly reduce the amount of code in the catalog test suite -- 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