o-nikolas commented on code in PR #40285: URL: https://github.com/apache/airflow/pull/40285#discussion_r1644790025
########## tests/jobs/test_backfill_job.py: ########## @@ -79,11 +80,38 @@ def dag_bag(): return DagBag(include_examples=True) -# Patch the MockExecutor into the dict of known executors in the Loader -@patch.dict( - ExecutorLoader.executors, {MOCK_EXECUTOR: f"{MockExecutor.__module__}.{MockExecutor.__qualname__}"} -) class TestBackfillJob: + def _mock_executor(self, executor=None): Review Comment: Thanks, figuring out how to make this logic accessible as a fixture and as a generator used within tests when mocking needs to happen multiple times was a very long and frustrating couple hours of development haha 😅 -- 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