xBis7 commented on code in PR #49180:
URL: https://github.com/apache/airflow/pull/49180#discussion_r2065927196
##########
pyproject.toml:
##########
@@ -847,6 +847,9 @@ addopts = [
"--ignore-glob=**/tests/system/*",
"--ignore-glob=tests/system/*",
]
+markers = [
+ "timeout: mark tests with a timeout (provided by pytest-timeout)",
+]
Review Comment:
I see that the existing timeouts are on the test level and not on the class
level.
Instead of annotating the class, we annotate the test method like so
```
@pytest.mark.execution_timeout(5)
def test_...
```
The class annotation comes from a pytest third party plugin and that's why
we need this change. I'll clean it up and use method annotations.
--
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
