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

Reply via email to