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

Add support for parallel test execution #8

Closed
radarsh opened this issue Oct 1, 2017 · 6 comments
Closed

Add support for parallel test execution #8

radarsh opened this issue Oct 1, 2017 · 6 comments
Assignees
Labels
Milestone

Comments

@radarsh
Copy link
Owner

radarsh commented Oct 1, 2017

Currently if maxParallelForks is set to anything other than 1, the plugin throws an exception. This can be quite limiting.

@radarsh radarsh added the feature label Oct 1, 2017
@eriwen
Copy link

eriwen commented Nov 20, 2017

Thanks for providing a great plugin! I'd like to advocate for this feature, since that makes this plugin useful for a large number of projects.

If you let us (Gradle team) know what you're struggling with (maybe on the forums), we may have a good answer for you. Thanks again.

@radarsh
Copy link
Owner Author

radarsh commented Nov 24, 2017

Thanks for the feedback. The main challenge I’m facing is being able to group tests by suites when they are executed in parallel. So by ensuring that there is only one parallel fork (Gradle default) I can achieve the grouping.

I will soon post on the forum too with more details.

@radarsh
Copy link
Owner Author

radarsh commented Dec 1, 2017

@eriwen I've explained in detail on the forums.

@jake-at-work
Copy link

@radarsh I'd really like to use this plugin to get some better logging of progress in our builds when console is in plain mode (our CI chokes on Gradle's rich output). The problem I have is I need this maxParallelForks issue to work. Looking at your discussion on the forum linked above either solution would be sufficient for me to take this into production. If I was to simply fork you plugin and remove that assertion would I end up with the odd groupings at least? If so would you consider this as a sufficient step forward to just include it in your releases? I feel like something is better than nothing at this point.

@radarsh
Copy link
Owner Author

radarsh commented Jul 23, 2018

@pivotal-jbarrett let me see what I can do. This issue has been open for far too long.

@radarsh
Copy link
Owner Author

radarsh commented Jul 25, 2018

I have a working prototype using the second approach I have described on the forum before. Screenshots:

Plain (parallel) theme:

image

Standard (parallel) theme:

image

Mocha (parallel) theme:

image

Will work on getting this ready for the next release of the plugin.

@radarsh radarsh added this to the 1.4.0 milestone Jul 28, 2018
@radarsh radarsh self-assigned this Jul 28, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants