sjyangkevin commented on code in PR #51059:
URL: https://github.com/apache/airflow/pull/51059#discussion_r2157619129
##########
airflow-core/tests/unit/serialization/serializers/test_serializers.py:
##########
@@ -361,6 +381,32 @@ def test_kubernetes_serializer(self, monkeypatch):
assert serialize(pod) == ("", "", 0, False)
assert serialize(123) == ("", "", 0, False)
+ def test_pydantic(self):
Review Comment:
I found a set of test cases for allow list in `test_serde.py`, I've created
a test case using an existing Pydantic model `U`. This Pydantic model is not in
the allow list, and it raises `ImportError`, which is expected.
However, if we directly invoke
`airflow.serialization.serializers.pydantic.deserialize`, arbitrary Pydantic
models can be deserialized.
Will push the code soon.
--
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]
