-
Notifications
You must be signed in to change notification settings - Fork 24.7k
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
Merge test runner task into RestIntegTest #60261
Merge test runner task into RestIntegTest #60261
Conversation
8ef064b
to
f41c2a5
Compare
f41c2a5
to
db0eb83
Compare
Pinging @elastic/es-core-infra (:Core/Infra/Build) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks great. Very glad to have all that junk removed from the task constructor and also no longer need the additional complexity of the "runner".
@@ -67,6 +67,8 @@ class StandaloneRestTestPlugin implements Plugin<Project> { | |||
ElasticsearchJavaPlugin.configureInputNormalization(project) | |||
ElasticsearchJavaPlugin.configureCompile(project) | |||
|
|||
project.pluginManager.apply(RestTestBasePlugin) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nit: can we group this up above with the other plugin applications?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
the original reason for this was that the plugin relies on nonInputSystemProperties.
I've redid the general augmentation of Test tasks now and moved that into its own plugin. Another benefit of this is we removed one reused static method from ElasticsearchJavaPlugin.
* Merge test runner task into RestIntegTest * Reorganizing Standalone runner and RestIntegTest task * Rework general test task configuration and extension
* Merge test runner task into RestIntegTest (#60261) * Merge test runner task into RestIntegTest * Reorganizing Standalone runner and RestIntegTest task * Rework general test task configuration and extension * Fix merge issues * use former 7.x common test configuration
This commit changes the yamlRestTest and javaRestTest tasks to be lazily created. This change requires pro-actively creating the testClusters container so that the configuration can be applied without any changes to the build.gradle files. related: elastic#60261 related: elastic#47804
RestIntegTest
RestTestRunnerTask
and renamed toStandaloneRestIntegTest
for standalone usages of the runner task without configuration conventions applied.RestIntegTest