kaxil commented on code in PR #56602:
URL: https://github.com/apache/airflow/pull/56602#discussion_r2430153515
##########
airflow-core/tests/unit/core/test_configuration.py:
##########
@@ -923,8 +923,10 @@ def test_initialize_secrets_backends_on_workers(self):
backends =
initialize_secrets_backends(DEFAULT_SECRETS_SEARCH_PATH_WORKERS)
backend_classes = [backend.__class__.__name__ for backend in backends]
- assert len(backends) == 2
+ assert len(backends) == 3
Review Comment:
No, the change in backends is for workers which is taken care of by the
right side condition.
The left is tbh a redundant check and in my bucket list to change the entire
code some point
--
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]
