Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

tests: Add timeout to pytest #2499

Merged
merged 1 commit into from
Jul 26, 2022
Merged

Conversation

wenzeslaus
Copy link
Member

Now the individual pytest tests are terminated after 300 seconds just like the gunittest ones. New dependency pytest-timeout is needed for testing to use the timeout (or avoid the warning about unused configuration). In the future, individual tests can change the time if needed using the pytest.mark.timeout decorator.

Now the individual pytest tests are terminated after 300 seconds just like the gunittest ones. New dependency pytest-timeout is needed for testing to use the timeout (or avoid the warning about unused configuration). In the future, individual tests can change the time if needed using the pytest.mark.timeout decorator.
@wenzeslaus wenzeslaus added this to the 8.4.0 milestone Jul 26, 2022
@wenzeslaus wenzeslaus added the CI Continuous integration label Jul 26, 2022
@wenzeslaus
Copy link
Member Author

pytest checks pass (OSGeo4W is broken). Merging.

Let's see if some individual tests timeout. This should prevent CI from waiting 6 hours for stuck tests.

@wenzeslaus wenzeslaus merged commit 37d45fb into OSGeo:main Jul 26, 2022
@wenzeslaus wenzeslaus deleted the add-pytest-timeout branch July 26, 2022 16:34
ninsbl pushed a commit to ninsbl/grass that referenced this pull request Oct 26, 2022
Now the individual pytest tests are terminated after 300 seconds just like the gunittest ones. New dependency pytest-timeout is needed for testing to use the timeout (or avoid the warning about unused configuration). In the future, individual tests can change the time if needed using the pytest.mark.timeout decorator.
ninsbl pushed a commit to ninsbl/grass that referenced this pull request Feb 17, 2023
Now the individual pytest tests are terminated after 300 seconds just like the gunittest ones. New dependency pytest-timeout is needed for testing to use the timeout (or avoid the warning about unused configuration). In the future, individual tests can change the time if needed using the pytest.mark.timeout decorator.
neteler pushed a commit to nilason/grass that referenced this pull request Nov 7, 2023
Now the individual pytest tests are terminated after 300 seconds just like the gunittest ones. New dependency pytest-timeout is needed for testing to use the timeout (or avoid the warning about unused configuration). In the future, individual tests can change the time if needed using the pytest.mark.timeout decorator.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CI Continuous integration
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant