Pranaykarvi commented on PR #64022: URL: https://github.com/apache/airflow/pull/64022#issuecomment-4183233384
Fixed the failing `test_execute_sets_server_process_context` test. The test was asserting `_AIRFLOW_PROCESS_CONTEXT` **after** `_execute()` returns, but the `finally` block already restored it to `None` by then. Fixed by using a `side_effect` on `mock_supervisor_start` to capture the env var **during** execution, then asserting both that it was set to `"server"` mid-run and restored to `None` afterwards. -- 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]
