Bisk1 commented on code in PR #34037: URL: https://github.com/apache/airflow/pull/34037#discussion_r1313879460
########## airflow/config_templates/unit_tests.cfg: ########## @@ -59,6 +59,8 @@ unit_test_mode = True killed_task_cleanup_time = 5 # We only allow our own classes to be deserialized in tests allowed_deserialization_classes = airflow\..* tests\..* +# mocks from unittest.mock work correctly in subprocesses only if they are created by "fork" method +mp_start_method = fork Review Comment: I understand. I knew of the problem with fork in macOS but I thought it's just unstable but functionally correct for Airflow purposes, so I figured it's still useful to switch it for testing. I will review other tests to see if there is any regression on macOS after this change. I personally use breeze but it's a little cumbersome to use when debugging code with IDE. -- 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: commits-unsubscr...@airflow.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
