turbaszek commented on a change in pull request #10304:
URL: https://github.com/apache/airflow/pull/10304#discussion_r477325874
##########
File path: tests/conftest.py
##########
@@ -251,6 +253,9 @@ def pytest_configure(config):
config.addinivalue_line(
"markers", "airflow_2: mark tests that works only on Airflow 2.0 /
master"
)
+ config.addinivalue_line(
+ "markers", "credential_env(name): mark tests that require credential
env in CREDENTIALS_ENV"
+ )
Review comment:
I think this is indeed an over-engineering. To run this test I have to
create a file that requires much more work than exporting an env variable. It
also assumes that the test is run from breeze environment or that I set up
another env variable that points to other directory...
----------------------------------------------------------------
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]
