ashb commented on a change in pull request #8909:
URL: https://github.com/apache/airflow/pull/8909#discussion_r427228723
##########
File path: tests/executors/test_celery_executor.py
##########
@@ -75,6 +75,17 @@ def _prepare_app(broker_url=None, execute=None):
patch_app = mock.patch('airflow.executors.celery_executor.app', test_app)
patch_execute =
mock.patch('airflow.executors.celery_executor.execute_command', test_execute)
+ backend = test_app.backend
Review comment:
Ah okay, I know what's gone on.
Looking closer at the logs in CI I see
> E ModuleNotFoundError: No module named '# overridden by startup scripts'
Previously we weren't ever actually creating a backend, so it didn't matter.
I think the fix here is to limit this test to mysql+postgres backends only
Sorry for the ping @auvipy for what was (clearly) going to be something
we've done.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
[email protected]
