ferruzzi commented on code in PR #65123:
URL: https://github.com/apache/airflow/pull/65123#discussion_r3070857486
##########
airflow-core/tests/unit/executors/test_base_executor.py:
##########
@@ -411,6 +411,23 @@ def test_repr():
class TestExecutorConf:
"""Test ExecutorConf shim class that provides team-specific configuration
access."""
Review Comment:
I just ran into the bug this is fixing, thanks for fixing it so fast. I'd
suggest using `conf_vars` in the tests instead of reinventing the wheel here
though. If you use `conf_vars`, then it resets the values back after the test
finishes.
--
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]
