potiuk commented on a change in pull request #7439: [AIRFLOW-6204][depends on
6763] Create GoogleSystemTest class
URL: https://github.com/apache/airflow/pull/7439#discussion_r380183421
##########
File path: tests/conftest.py
##########
@@ -159,6 +174,15 @@ def pytest_configure(config):
config.addinivalue_line(
"markers", "runtime(name): mark test to run with named runtime"
)
+ config.addinivalue_line(
+ "markers", "system(name): mark test to run with named system"
+ )
+ config.addinivalue_line(
+ "markers", "long_lasting(name): mark test that run for a long time
(many minutes)"
+ )
+ config.addinivalue_line(
+ "markers", "credential_file(name): mark tests that require credential
file in CREDENTIALS_DIR"
Review comment:
It makes sense because they also determine whether the test is skipped or
not - if the file with credential is missing, then the test is automatically
skipped (and reason for skipping explains what credential is missing). I think
it's generic enough to be used in a number of cases (and we can include other
markers for that - credential_variable for example for env variables missing).
The point here is to skip the tests when the credentials are missing.
----------------------------------------------------------------
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]
With regards,
Apache Git Services
