Conversation
|
Have a look at https://docs.junit.org/6.0.3/writing-tests/parallel-execution.html as well. Also, consider renaming the integration tests to *IT and let failsafe (it's actually meant to be used for integration tests instead of surefire for proper external resources management and to really test against the built artifact) run them, in parallel with the unit tests from surefire, instead. |
|
@SteKoe if you don't mind, I added some suggestions ;) |
…inclusions and exclusions
…:evaluate for accuracy
|
Hi @SteKoe. I don't know if you realized it, but there is
in the logs. |
|
Hey @cdprete, thanks for your comments, findings and your involvement in this open source project. It is very much appreciated! I haven't looked in the logs of the new build in depth, yet, and I will have to discuss the approach with my colleagues as well.
Parallel execution of unit tests seems a little bit tricky in our project due to a lot of common Abstract*Test classes which led to some issues in the past, already. So, thanks again, we are on it :) Slowly, but gradually. |
They should be "ignored" - only the children concrete classes should run - if those classes are indeed abstract.
I find the project valuable and I appreciate a lot that you guys are doing this for free, so I'm more than happy to contribute when I can. |
This pull request introduces several improvements to the build, test, and release workflows, with a focus on speeding up CI feedback, improving parallelization, and clarifying documentation. The main changes include enabling parallel Maven builds, refining test and integration test execution, updating workflow concurrency controls, and optimizing Node setup and caching. Additionally, documentation is updated to reflect these changes and clarify build instructions.
CI/CD Workflow Improvements:
-T 1C) in all GitHub Actions workflows for faster builds and deployments, and clarified when coverage is collected. [1] [2] [3] [4] [5]concurrencygroups to all workflows to prevent duplicate runs and reduce wasted CI resources; documentation and release deploys are not cancelled mid-run. [1] [2] [3] [4]Testing and Build Configuration:
pom.xmlto:happy-dom(faster thanjsdom) and set a worker limit for parallel test execution.@vitest-environment jsdomannotation to individual test files where needed.Documentation Updates:
.github/copilot-instructions.mdto use-Dmaven.test.skip=trueconsistently (instead of-DskipTests) for all build commands, improving clarity and consistency. [1] [2]Node.js Setup Improvements:
actions/setup-nodewith caching, replacing manual cache steps for reliability and simplicity.These changes collectively improve CI speed, reliability, and maintainability, and make the build/test process more transparent for contributors.