MenelikV commented on code in PR #3012:
URL: https://github.com/apache/iceberg-python/pull/3012#discussion_r2786586561
##########
tests/catalog/test_rest.py:
##########
@@ -1641,6 +1641,29 @@ def
test_update_namespace_properties_invalid_namespace(rest_mock: Mocker) -> Non
assert "Empty namespace identifier" in str(e.value)
+def test_with_disabled_ssl_ca_bundle(rest_mock: Mocker) -> None:
+ from pydantic import ValidationError
+ def ssl_check_callback(req, _):
+ if req.verify:
+ raise AssertionError("SSL verification is still enabled")
+ # Given
+ rest_mock.get(
+ f"{TEST_URI}v1/config",
+ json=ssl_check_callback,
+ status_code=200,
+ )
+ # Given
+ catalog_properties = {
+ "uri": TEST_URI,
+ "token": TEST_TOKEN,
+ "ssl": {
+ "cabundle": False,
+ }
+ }
+ with pytest.raises(ValidationError) as _:
Review Comment:
Okay, works for me.
--
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]
