Skip to content

Commit c922e0b

Browse files
committed
Test: Remove broken yml test feature (#31255)
The `requires_replica` yaml test feature hasn't worked for years. This is what happens if you try to use it: ``` > Throwable #1: java.lang.NullPointerException > at __randomizedtesting.SeedInfo.seed([E6602FB306244B12:6E341069A8D826EA]:0) > at org.elasticsearch.test.rest.yaml.Features.areAllSupported(Features.java:58) > at org.elasticsearch.test.rest.yaml.section.SkipSection.skip(SkipSection.java:144) > at org.elasticsearch.test.rest.yaml.ESClientYamlSuiteTestCase.test(ESClientYamlSuiteTestCase.java:321) ``` None of our tests use it.
1 parent 1bdcadd commit c922e0b

File tree

1 file changed

+0
-5
lines changed
  • test/framework/src/main/java/org/elasticsearch/test/rest/yaml

1 file changed

+0
-5
lines changed

test/framework/src/main/java/org/elasticsearch/test/rest/yaml/Features.java

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -19,8 +19,6 @@
1919

2020
package org.elasticsearch.test.rest.yaml;
2121

22-
import org.elasticsearch.test.ESIntegTestCase;
23-
2422
import java.util.Arrays;
2523
import java.util.List;
2624

@@ -54,9 +52,6 @@ private Features() {
5452
*/
5553
public static boolean areAllSupported(List<String> features) {
5654
for (String feature : features) {
57-
if ("requires_replica".equals(feature) && ESIntegTestCase.cluster().numDataNodes() >= 2) {
58-
continue;
59-
}
6055
if (!SUPPORTED.contains(feature)) {
6156
return false;
6257
}

0 commit comments

Comments
 (0)