o-nikolas commented on code in PR #38665:
URL: https://github.com/apache/airflow/pull/38665#discussion_r1548152476
##########
tests/system/conftest.py:
##########
@@ -49,9 +49,9 @@ def skip_if_env_var_not_set(provider_env_vars):
def pytest_collection_modifyitems(config, items):
"""Add @pytest.mark.system(provider_name) for every system test."""
- rootdir = Path(config.rootdir)
+ rootdir = Path(config.rootpath)
Review Comment:
I think Path is going to return a PosixPath object, which it was using
before. Just a matter of where the input comes from.
This this change is blocking system tests from passing so I'm going to merge
this, but if it turns out we can drop the `Path` we can circle back.
--
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]
