Fokko commented on code in PR #2404:
URL: https://github.com/apache/iceberg-python/pull/2404#discussion_r2575577647
##########
tests/catalog/test_rest.py:
##########
@@ -955,6 +955,25 @@ def test_load_table_404(rest_mock: Mocker) -> None:
assert "Table does not exist" in str(e.value)
+def test_load_table_404_non_existent_namespace(rest_mock: Mocker) -> None:
Review Comment:
The mocked tests predate the time when we didn't have the
IcebergTestFixtures Docker image. I think it would be best to test against the
Docker image instead in the integration tests.
--
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]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
