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

Testing: add test method timeout + more verbose reporting #8310

Merged
merged 1 commit into from Apr 9, 2024

Conversation

snazy
Copy link
Member

@snazy snazy commented Apr 9, 2024

This change adds a default timeout of 5 minutes, which is quite long, but will catch hanging tests. Also adds the more verbose Open Test Reporting format to potentially investigate a hanging test using a better report format than the antique legacy XML format.

JUnit docs for default timeouts
JUnit docs for Open Test Reporting

This change adds a default timeout of 5 minutes, which is quite long, but will catch hanging tests.
Also adds the more verbose Open Test Reporting format to potentially investigate a hanging test using a better report format than the antique legacy XML format.

[JUnit docs for default timeouts](https://junit.org/junit5/docs/current/user-guide/#writing-tests-declarative-timeouts)
[JUnit docs for Open Test
Reporting](https://junit.org/junit5/docs/current/user-guide/#junit-platform-reporting-open-test-reporting)
@snazy snazy requested a review from dimas-b April 9, 2024 18:11
@snazy snazy merged commit 6423e4f into projectnessie:main Apr 9, 2024
15 checks passed
@snazy snazy deleted the test-timeouts-reports branch April 9, 2024 19:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants