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

[DE-786] Run tests in parallel #169

Merged
merged 4 commits into from
Nov 25, 2024
Merged

[DE-786] Run tests in parallel #169

merged 4 commits into from
Nov 25, 2024

Conversation

maciej-nedza
Copy link
Contributor

@maciej-nedza maciej-nedza commented Nov 22, 2024

Make tests run in parallel:

  • runs on n threads where n is number of configured subdomains
  • each thread gets its own subdomain
  • test classes are run in parallel, but test methods are run sequentially
  • there's one additional subdomain which can be preconfigured and not cleared, using it we can test stuff we couldn't test otherwise
  • changed default test lifecycle to per class, to enable non-static BeforeAll. Without it, static parameters in base classes clashed in parallel test runs.
  • there's @ResourceLock annotation which in theory could be used to fix the problem with shared resources, but when I've tested it, it was crashing due to thread pool limits. seems like it's not working with fixed=max thread pool size. Then we should just avoid static stuff.

Copy link

swarmia bot commented Nov 22, 2024

@maciej-nedza maciej-nedza force-pushed the improve-tests-execution branch from c918e57 to 1acd3e6 Compare November 25, 2024 10:25
@maciej-nedza maciej-nedza force-pushed the improve-tests-execution branch from 1acd3e6 to d18b829 Compare November 25, 2024 11:42
@maciej-nedza maciej-nedza marked this pull request as ready for review November 25, 2024 13:10
@maciej-nedza maciej-nedza merged commit a9733ba into main Nov 25, 2024
2 checks passed
@maciej-nedza maciej-nedza deleted the improve-tests-execution branch November 25, 2024 14:40
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.

2 participants