Skip to content

Commit f236efc

Browse files
authored
Convert remaining plugin projects to new test clusters framework (elastic#125626) (elastic#125726)
(cherry picked from commit 930b4ab) # Conflicts: # plugins/discovery-azure-classic/build.gradle # plugins/discovery-gce/qa/gce/build.gradle
1 parent 5c1e17c commit f236efc

File tree

30 files changed

+215
-202
lines changed

30 files changed

+215
-202
lines changed

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

Lines changed: 0 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -30,18 +30,6 @@ private static ListMultimap<Class<?>, String> createLegacyRestTestBasePluginUsag
3030
map.put(LegacyRestTestBasePlugin.class, ":docs");
3131
map.put(LegacyRestTestBasePlugin.class, ":distribution:docker");
3232
map.put(LegacyRestTestBasePlugin.class, ":modules:reindex");
33-
map.put(LegacyRestTestBasePlugin.class, ":plugins:analysis-icu");
34-
map.put(LegacyRestTestBasePlugin.class, ":plugins:analysis-kuromoji");
35-
map.put(LegacyRestTestBasePlugin.class, ":plugins:analysis-nori");
36-
map.put(LegacyRestTestBasePlugin.class, ":plugins:analysis-phonetic");
37-
map.put(LegacyRestTestBasePlugin.class, ":plugins:analysis-smartcn");
38-
map.put(LegacyRestTestBasePlugin.class, ":plugins:analysis-stempel");
39-
map.put(LegacyRestTestBasePlugin.class, ":plugins:analysis-ukrainian");
40-
map.put(LegacyRestTestBasePlugin.class, ":plugins:discovery-azure-classic");
41-
map.put(LegacyRestTestBasePlugin.class, ":plugins:discovery-ec2");
42-
map.put(LegacyRestTestBasePlugin.class, ":plugins:discovery-gce");
43-
map.put(LegacyRestTestBasePlugin.class, ":plugins:mapper-annotated-text");
44-
map.put(LegacyRestTestBasePlugin.class, ":plugins:store-smb");
4533
map.put(LegacyRestTestBasePlugin.class, ":qa:ccs-rolling-upgrade-remote-cluster");
4634
map.put(LegacyRestTestBasePlugin.class, ":qa:mixed-cluster");
4735
map.put(LegacyRestTestBasePlugin.class, ":qa:multi-cluster-search");

build-tools/src/main/java/org/elasticsearch/gradle/plugin/BasePluginBuildPlugin.java

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -56,6 +56,7 @@ public class BasePluginBuildPlugin implements Plugin<Project> {
5656
public static final String BUNDLE_PLUGIN_TASK_NAME = "bundlePlugin";
5757
public static final String EXPLODED_BUNDLE_PLUGIN_TASK_NAME = "explodedBundlePlugin";
5858
public static final String EXPLODED_BUNDLE_CONFIG = "explodedBundleZip";
59+
public static final Attribute<Boolean> EXPLODED_PLUGIN_BUNDLE_ATTRIBUTE = Attribute.of("exploded-plugin-bundle", Boolean.class);
5960

6061
protected final ProviderFactory providerFactory;
6162

@@ -170,6 +171,7 @@ private TaskProvider<Zip> createBundleTasks(final Project project, PluginPropert
170171
explodedBundleZip.setCanBeResolved(false);
171172
explodedBundleZip.setCanBeConsumed(true);
172173
explodedBundleZip.getAttributes().attribute(ArtifactTypeDefinition.ARTIFACT_TYPE_ATTRIBUTE, ArtifactTypeDefinition.DIRECTORY_TYPE);
174+
explodedBundleZip.getAttributes().attribute(EXPLODED_PLUGIN_BUNDLE_ATTRIBUTE, true);
173175
project.getArtifacts().add(EXPLODED_BUNDLE_CONFIG, explodedBundle);
174176
return bundle;
175177
}

plugins/analysis-icu/build.gradle

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -8,8 +8,8 @@ import org.elasticsearch.gradle.internal.info.BuildParams
88
* your election, the "Elastic License 2.0", the "GNU Affero General Public
99
* License v3.0 only", or the "Server Side Public License, v 1".
1010
*/
11-
apply plugin: 'elasticsearch.legacy-yaml-rest-test'
12-
apply plugin: 'elasticsearch.legacy-yaml-rest-compat-test'
11+
apply plugin: 'elasticsearch.internal-yaml-rest-test'
12+
apply plugin: 'elasticsearch.yaml-rest-compat-test'
1313
apply plugin: 'elasticsearch.internal-cluster-test'
1414

1515
esplugin {

plugins/analysis-icu/src/yamlRestTest/java/org/elasticsearch/index/analysis/IcuClientYamlTestSuiteIT.java

Lines changed: 10 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -12,11 +12,16 @@
1212
import com.carrotsearch.randomizedtesting.annotations.Name;
1313
import com.carrotsearch.randomizedtesting.annotations.ParametersFactory;
1414

15+
import org.elasticsearch.test.cluster.ElasticsearchCluster;
1516
import org.elasticsearch.test.rest.yaml.ClientYamlTestCandidate;
1617
import org.elasticsearch.test.rest.yaml.ESClientYamlSuiteTestCase;
18+
import org.junit.ClassRule;
1719

1820
public class IcuClientYamlTestSuiteIT extends ESClientYamlSuiteTestCase {
1921

22+
@ClassRule
23+
public static ElasticsearchCluster cluster = ElasticsearchCluster.local().plugin("analysis-icu").build();
24+
2025
public IcuClientYamlTestSuiteIT(@Name("yaml") ClientYamlTestCandidate testCandidate) {
2126
super(testCandidate);
2227
}
@@ -25,4 +30,9 @@ public IcuClientYamlTestSuiteIT(@Name("yaml") ClientYamlTestCandidate testCandid
2530
public static Iterable<Object[]> parameters() throws Exception {
2631
return ESClientYamlSuiteTestCase.createParameters();
2732
}
33+
34+
@Override
35+
protected String getTestRestCluster() {
36+
return cluster.getHttpAddresses();
37+
}
2838
}

plugins/analysis-kuromoji/build.gradle

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -6,8 +6,8 @@
66
* your election, the "Elastic License 2.0", the "GNU Affero General Public
77
* License v3.0 only", or the "Server Side Public License, v 1".
88
*/
9-
apply plugin: 'elasticsearch.legacy-yaml-rest-test'
10-
apply plugin: 'elasticsearch.legacy-yaml-rest-compat-test'
9+
apply plugin: 'elasticsearch.internal-yaml-rest-test'
10+
apply plugin: 'elasticsearch.yaml-rest-compat-test'
1111

1212
esplugin {
1313
description = 'The Japanese (kuromoji) Analysis plugin integrates Lucene kuromoji analysis module into elasticsearch.'

plugins/analysis-kuromoji/src/yamlRestTest/java/org/elasticsearch/index/analysis/KuromojiClientYamlTestSuiteIT.java

Lines changed: 10 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -12,11 +12,16 @@
1212
import com.carrotsearch.randomizedtesting.annotations.Name;
1313
import com.carrotsearch.randomizedtesting.annotations.ParametersFactory;
1414

15+
import org.elasticsearch.test.cluster.ElasticsearchCluster;
1516
import org.elasticsearch.test.rest.yaml.ClientYamlTestCandidate;
1617
import org.elasticsearch.test.rest.yaml.ESClientYamlSuiteTestCase;
18+
import org.junit.ClassRule;
1719

1820
public class KuromojiClientYamlTestSuiteIT extends ESClientYamlSuiteTestCase {
1921

22+
@ClassRule
23+
public static ElasticsearchCluster cluster = ElasticsearchCluster.local().plugin("analysis-kuromoji").build();
24+
2025
public KuromojiClientYamlTestSuiteIT(@Name("yaml") ClientYamlTestCandidate testCandidate) {
2126
super(testCandidate);
2227
}
@@ -25,4 +30,9 @@ public KuromojiClientYamlTestSuiteIT(@Name("yaml") ClientYamlTestCandidate testC
2530
public static Iterable<Object[]> parameters() throws Exception {
2631
return ESClientYamlSuiteTestCase.createParameters();
2732
}
33+
34+
@Override
35+
protected String getTestRestCluster() {
36+
return cluster.getHttpAddresses();
37+
}
2838
}

plugins/analysis-nori/build.gradle

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -6,8 +6,8 @@
66
* your election, the "Elastic License 2.0", the "GNU Affero General Public
77
* License v3.0 only", or the "Server Side Public License, v 1".
88
*/
9-
apply plugin: 'elasticsearch.legacy-yaml-rest-test'
10-
apply plugin: 'elasticsearch.legacy-yaml-rest-compat-test'
9+
apply plugin: 'elasticsearch.internal-yaml-rest-test'
10+
apply plugin: 'elasticsearch.yaml-rest-compat-test'
1111

1212
esplugin {
1313
description = 'The Korean (nori) Analysis plugin integrates Lucene nori analysis module into elasticsearch.'

plugins/analysis-nori/src/yamlRestTest/java/org/elasticsearch/index/analysis/NoriClientYamlTestSuiteIT.java

Lines changed: 10 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -12,11 +12,16 @@
1212
import com.carrotsearch.randomizedtesting.annotations.Name;
1313
import com.carrotsearch.randomizedtesting.annotations.ParametersFactory;
1414

15+
import org.elasticsearch.test.cluster.ElasticsearchCluster;
1516
import org.elasticsearch.test.rest.yaml.ClientYamlTestCandidate;
1617
import org.elasticsearch.test.rest.yaml.ESClientYamlSuiteTestCase;
18+
import org.junit.ClassRule;
1719

1820
public class NoriClientYamlTestSuiteIT extends ESClientYamlSuiteTestCase {
1921

22+
@ClassRule
23+
public static ElasticsearchCluster cluster = ElasticsearchCluster.local().plugin("analysis-nori").build();
24+
2025
public NoriClientYamlTestSuiteIT(@Name("yaml") ClientYamlTestCandidate testCandidate) {
2126
super(testCandidate);
2227
}
@@ -25,4 +30,9 @@ public NoriClientYamlTestSuiteIT(@Name("yaml") ClientYamlTestCandidate testCandi
2530
public static Iterable<Object[]> parameters() throws Exception {
2631
return ESClientYamlSuiteTestCase.createParameters();
2732
}
33+
34+
@Override
35+
protected String getTestRestCluster() {
36+
return cluster.getHttpAddresses();
37+
}
2838
}

plugins/analysis-phonetic/build.gradle

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -6,8 +6,8 @@
66
* your election, the "Elastic License 2.0", the "GNU Affero General Public
77
* License v3.0 only", or the "Server Side Public License, v 1".
88
*/
9-
apply plugin: 'elasticsearch.legacy-yaml-rest-test'
10-
apply plugin: 'elasticsearch.legacy-yaml-rest-compat-test'
9+
apply plugin: 'elasticsearch.internal-yaml-rest-test'
10+
apply plugin: 'elasticsearch.yaml-rest-compat-test'
1111

1212
esplugin {
1313
description = 'The Phonetic Analysis plugin integrates phonetic token filter analysis with elasticsearch.'

plugins/analysis-phonetic/src/yamlRestTest/java/org/elasticsearch/index/analysis/PhoneticClientYamlTestSuiteIT.java

Lines changed: 10 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -12,11 +12,16 @@
1212
import com.carrotsearch.randomizedtesting.annotations.Name;
1313
import com.carrotsearch.randomizedtesting.annotations.ParametersFactory;
1414

15+
import org.elasticsearch.test.cluster.ElasticsearchCluster;
1516
import org.elasticsearch.test.rest.yaml.ClientYamlTestCandidate;
1617
import org.elasticsearch.test.rest.yaml.ESClientYamlSuiteTestCase;
18+
import org.junit.ClassRule;
1719

1820
public class PhoneticClientYamlTestSuiteIT extends ESClientYamlSuiteTestCase {
1921

22+
@ClassRule
23+
public static ElasticsearchCluster cluster = ElasticsearchCluster.local().plugin("analysis-phonetic").build();
24+
2025
public PhoneticClientYamlTestSuiteIT(@Name("yaml") ClientYamlTestCandidate testCandidate) {
2126
super(testCandidate);
2227
}
@@ -25,4 +30,9 @@ public PhoneticClientYamlTestSuiteIT(@Name("yaml") ClientYamlTestCandidate testC
2530
public static Iterable<Object[]> parameters() throws Exception {
2631
return ESClientYamlSuiteTestCase.createParameters();
2732
}
33+
34+
@Override
35+
protected String getTestRestCluster() {
36+
return cluster.getHttpAddresses();
37+
}
2838
}

0 commit comments

Comments
 (0)