Skip to content

Commit 46c8dc9

Browse files
committed
Fix deprecation about registerFeature for without java plugin applied
1 parent 7740e3c commit 46c8dc9

File tree

10 files changed

+11
-9
lines changed

10 files changed

+11
-9
lines changed

build-tools-internal/src/main/java/org/elasticsearch/gradle/internal/InternalTestArtifactBasePlugin.java

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -11,6 +11,7 @@
1111

1212
import org.gradle.api.Plugin;
1313
import org.gradle.api.Project;
14+
import org.gradle.api.plugins.JavaPlugin;
1415
import org.gradle.api.provider.ProviderFactory;
1516

1617
import javax.inject.Inject;
@@ -25,6 +26,7 @@ public InternalTestArtifactBasePlugin(ProviderFactory providerFactory) {
2526

2627
@Override
2728
public void apply(Project project) {
29+
project.getPlugins().apply(JavaPlugin.class);
2830
project.getPlugins().apply(ElasticsearchJavaBasePlugin.class);
2931
project.getExtensions().create("testArtifacts", InternalTestArtifactExtension.class, project, providerFactory);
3032
}

build-tools-internal/src/main/java/org/elasticsearch/gradle/internal/test/StandaloneRestTestPlugin.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -53,15 +53,15 @@ public void apply(final Project project) {
5353

5454
// only setup tests to build
5555
SourceSetContainer sourceSets = project.getExtensions().getByType(SourceSetContainer.class);
56-
final SourceSet testSourceSet = sourceSets.create("test");
56+
final SourceSet testSourceSet = sourceSets.maybeCreate("test");
5757

5858
project.getTasks().withType(Test.class).configureEach(test -> {
5959
test.setTestClassesDirs(testSourceSet.getOutput().getClassesDirs());
6060
test.setClasspath(testSourceSet.getRuntimeClasspath());
6161
});
6262

6363
// create a compileOnly configuration as others might expect it
64-
project.getConfigurations().create("compileOnly");
64+
project.getConfigurations().maybeCreate("compileOnly");
6565
RestTestUtil.setupJavaRestTestDependenciesDefaults(project, testSourceSet);
6666

6767
EclipseModel eclipse = project.getExtensions().getByType(EclipseModel.class);

qa/repository-multi-version/build.gradle

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -10,8 +10,8 @@
1010
import org.elasticsearch.gradle.testclusters.StandaloneRestIntegTestTask
1111

1212
apply plugin: 'elasticsearch.internal-testclusters'
13-
apply plugin: 'elasticsearch.standalone-rest-test'
1413
apply plugin: 'elasticsearch.internal-test-artifact'
14+
apply plugin: 'elasticsearch.standalone-rest-test'
1515

1616
apply plugin: 'elasticsearch.bwc-test'
1717

@@ -75,3 +75,7 @@ buildParams.bwcVersions.withIndexCompatible { bwcVersion, baseName ->
7575
dependsOn tasks.named("${baseName}#Step4NewClusterTest")
7676
}
7777
}
78+
79+
tasks.named('testTestingConventions').configure {
80+
suffixes = ["IT"]
81+
}

x-pack/plugin/build.gradle

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -8,10 +8,10 @@
88
import org.elasticsearch.gradle.testclusters.StandaloneRestIntegTestTask
99
import org.elasticsearch.gradle.util.GradleUtils
1010

11+
apply plugin: 'elasticsearch.internal-test-artifact'
1112
apply plugin: 'elasticsearch.internal-yaml-rest-test'
1213
apply plugin: 'elasticsearch.yaml-rest-compat-test'
1314
apply plugin: 'elasticsearch.validate-rest-spec'
14-
apply plugin: 'elasticsearch.internal-test-artifact'
1515

1616
base {
1717
archivesName = 'x-pack'

x-pack/plugin/downsample/qa/mixed-cluster/build.gradle

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,6 @@ import org.elasticsearch.gradle.VersionProperties
99
import org.elasticsearch.gradle.testclusters.StandaloneRestIntegTestTask
1010

1111
apply plugin: 'elasticsearch.internal-yaml-rest-test'
12-
apply plugin: 'elasticsearch.internal-test-artifact'
1312
apply plugin: 'elasticsearch.bwc-test'
1413

1514

x-pack/plugin/downsample/qa/rest/build.gradle

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,6 @@
77

88
apply plugin: 'elasticsearch.internal-yaml-rest-test'
99
apply plugin: 'elasticsearch.yaml-rest-compat-test'
10-
apply plugin: 'elasticsearch.internal-test-artifact'
1110

1211
dependencies {
1312
testImplementation project(':test:test-clusters')

x-pack/plugin/eql/qa/rest/build.gradle

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,6 @@
88
apply plugin: 'elasticsearch.internal-java-rest-test'
99
apply plugin: 'elasticsearch.internal-yaml-rest-test'
1010
apply plugin: 'elasticsearch.yaml-rest-compat-test'
11-
apply plugin: 'elasticsearch.internal-test-artifact'
1211

1312
restResources {
1413
restApi {

x-pack/plugin/esql/qa/server/mixed-cluster/build.gradle

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,6 @@ import org.elasticsearch.gradle.testclusters.StandaloneRestIntegTestTask
1212

1313
apply plugin: 'elasticsearch.internal-java-rest-test'
1414
apply plugin: 'elasticsearch.internal-yaml-rest-test'
15-
apply plugin: 'elasticsearch.internal-test-artifact'
1615
apply plugin: 'elasticsearch.bwc-test'
1716
apply plugin: 'elasticsearch.bc-upgrade-test'
1817
apply plugin: org.elasticsearch.gradle.internal.precommit.CheckstylePrecommitPlugin

x-pack/plugin/esql/qa/server/multi-node/build.gradle

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,6 @@ import org.elasticsearch.gradle.util.GradleUtils
22

33
apply plugin: 'elasticsearch.internal-java-rest-test'
44
apply plugin: 'elasticsearch.internal-yaml-rest-test'
5-
apply plugin: 'elasticsearch.internal-test-artifact'
65
apply plugin: org.elasticsearch.gradle.internal.precommit.CheckstylePrecommitPlugin
76
apply plugin: org.elasticsearch.gradle.internal.precommit.ForbiddenApisPrecommitPlugin
87
apply plugin: org.elasticsearch.gradle.internal.precommit.ForbiddenPatternsPrecommitPlugin

x-pack/plugin/esql/qa/server/single-node/build.gradle

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,4 @@
1+
apply plugin: 'java'
12
apply plugin: 'elasticsearch.internal-java-rest-test'
23
apply plugin: 'elasticsearch.internal-yaml-rest-test'
34
// Necessary to use tests in Serverless

0 commit comments

Comments
 (0)