mobuchowski commented on code in PR #39513:
URL: https://github.com/apache/airflow/pull/39513#discussion_r1598122196
##########
tests/providers/openlineage/plugins/test_openlineage.py:
##########
@@ -131,7 +131,7 @@ def teardown_method(self):
),
],
)
- def test_plugin_disablements(self, mocks, expected):
+ def test_plugin_disablements(self, mocks, expected,
require_running_tests_from_sources):
Review Comment:
> But by moving airflow imports to inside setup/teardown methods, we delay
plugin initializiation to the setup method which is is run after patching of
the os environment is done for the test. That seems to fix the problem.
Hmm... not sure how local imports and clearing `lru_cache` interacts... but
I think tests shouldn't really rely on Airflow initialization. Not sure how to
do it yet.
--
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
