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

execute-Unit-Tests-parallel-with-Maven #45

Conversation

TheRealHaui
Copy link
Contributor

Maven surefire plugin executes Unit Tests in parallel now.
Further information if desired: https://dzone.com/articles/running-junit-tests-parallel

@coveralls
Copy link

coveralls commented Jun 25, 2017

Coverage Status

Coverage decreased (-0.02%) to 84.75% when pulling a30b659 on TheRealHaui:execute-Unit-Tests-parallel-with-Maven into 19e1b02 on apache:master.

@TheRealHaui
Copy link
Contributor Author

Obviously, coveralls makes any kind of mistake measuring the coverage due to the fact that I've not edited any code.

@bodewig
Copy link
Member

bodewig commented Jun 26, 2017

Coveralls seems to be sensitive on which of the several JDKs we build with on Travis finishes first. Don't worry, this seems to be normal.

@bodewig
Copy link
Member

bodewig commented Jun 26, 2017

Your change is for the IT profile tests which aren't run on a regular basis. Each of these surefire instances will run the tests of a single class only. There won't be any difference between running them parallel "per class" or not in parallel at all.

As these tests are resource hogs - file system and memory in particular - they wouldn't be good candidates for running in parallel anyway.

For "normal" tests we inherit the settings from Commons Parent,

TBH I don't know whether jacoco would be compatible with a parallel execution of tests.

@TheRealHaui
Copy link
Contributor Author

You're right the change is for the *IT tests.
Was sure I made the change for the normal Unit Tests ...

However, I've tested the change on my local machine excessively and didn't encounter any problems.

Thought this might be a good idea to incorporate.
However ... / Anyway it wouldn't have been in any way a great improvement.

Thank you for your reply.
I close the pull request.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants