Skip to content

[Backport] Replace usages RandomizedTestingTask with built-in Gradle Test (#40978) #40995

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 2 commits into from
Apr 9, 2019
Merged

[Backport] Replace usages RandomizedTestingTask with built-in Gradle Test (#40978) #40995

merged 2 commits into from
Apr 9, 2019

Conversation

@mark-vieira
Copy link
Contributor

@mark-vieira mark-vieira commented Apr 9, 2019

This is a backport of #40978

@mark-vieira
Replace usages RandomizedTestingTask with built-in Gradle Test (elast…
ab99bfa 
…ic#40978)

This commit replaces the existing RandomizedTestingTask and supporting code with Gradle's built-in JUnit support via the Test task type. Additionally, the previous workaround to disable all tasks named "test" and create new unit testing tasks named "unitTest" has been removed such that the "test" task now runs unit tests as per the normal Gradle Java plugin conventions.

(cherry picked from commit 323f312)
@mark-vieira mark-vieira self-assigned this Apr 9, 2019
@mark-vieira mark-vieira changed the title Replace usages RandomizedTestingTask with built-in Gradle Test (#40978) [Backport] Replace usages RandomizedTestingTask with built-in Gradle Test (#40978) Apr 9, 2019
@mark-vieira
Copy link
Contributor Author

mark-vieira commented Apr 9, 2019

@alpar @rjernst I've created a back port PR for this to 7.0. I wasn't sure if we want to port these changes back that far but it might be nice to have. The benefit would be slightly limited as this branch doesn't include the TestClusters work so it won't benefit from any parallel improvements.

@mark-vieira mark-vieira merged commit 5ebf9dd into elastic:7.0 Apr 9, 2019
@mark-vieira mark-vieira deleted the gradle-test-runner-7.0 branch April 17, 2019 23:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

@mark-vieira mark-vieira

Labels

backport >non-issue v7.0.1

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@mark-vieira @polyfractal