nailo2c opened a new issue, #51915:
URL: https://github.com/apache/airflow/issues/51915

   ### Apache Airflow version
   
   main (development)
   
   ### If "Other Airflow 2 version" selected, which one?
   
   _No response_
   
   ### What happened?
   
   I added a new unit test, 
[test_serde_serialize_recursion_limit](https://github.com/apache/airflow/pull/51419/files#diff-a74fdab60d6b8940a5673aeaa60e066348216a94cc8bc9cf0eb7d51af6ac92e5R593-R596),
 in PR #51419
   
   ```python
   def test_serde_serialize_recursion_limit(self):
       depth = sys.getrecursionlimit() - 1
       with pytest.raises(RecursionError, match="maximum recursion depth 
reached for serialization"):
           serialize(object(), depth=depth)
   ```
   
   It passed the CI checks in the PR but failed after being merged into the 
main branch, so I removed it in PR #51737
   
   Some error logs:
   https://github.com/apache/airflow/actions/runs/15655403802/job/44105937266
   https://github.com/apache/airflow/actions/runs/15653524051/job/44101637421
   
   This unit test is for the following code snippet:
   
https://github.com/apache/airflow/blob/323076a00edd641926d2bd4fa32c7fd0231be57e/airflow-core/src/airflow/serialization/serde.py#L88-L115
   
   ### What you think should happen instead?
   
   It should be works, but I'm not sure why it failed.
   
   ### How to reproduce
   
   Add test function `test_serde_serialize_recursion_limit` to file 
`airflow-core/tests/unit/serialization/serializers/test_serializers.py`.
   
   Run `breeze testing core-tests --use-xdist --skip-db-tests --no-db-cleanup 
--backend none`, then you'll see
   
   ```console
   FAILED 
airflow-core/tests/unit/serialization/serializers/test_serializers.py::TestSerializers::test_serde_serialize_recursion_limit
 - TypeError: cannot serialize object of type <class 'object'>
   FAILED 
airflow-core/tests/unit/always/test_project_structure.py::TestProjectStructure::test_providers_modules_should_have_tests
 - Failed: Timeout >60.0s
   = 2 failed, 1652 passed, 4986 skipped, 1 xfailed, 9 warnings in 463.86s 
(0:07:43) =
   ```
   
   ### Operating System
   
   macos
   
   ### Versions of Apache Airflow Providers
   
   _No response_
   
   ### Deployment
   
   Other
   
   ### Deployment details
   
   _No response_
   
   ### Anything else?
   
   _No response_
   
   ### Are you willing to submit PR?
   
   - [ ] Yes I am willing to submit a PR!
   
   ### Code of Conduct
   
   - [x] I agree to follow this project's [Code of 
Conduct](https://github.com/apache/airflow/blob/main/CODE_OF_CONDUCT.md)
   


-- 
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]

Reply via email to