Skip to content

Commit 03b6697

Browse files
committed
Fix ClusterRequestTests
1 parent cc6e84e commit 03b6697

File tree

2 files changed

+6
-5
lines changed

2 files changed

+6
-5
lines changed

muted-tests.yml

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -122,9 +122,6 @@ tests:
122122
- class: org.elasticsearch.test.rest.yaml.CcsCommonYamlTestSuiteIT
123123
method: test {p0=search.highlight/50_synthetic_source/text multi unified from vectors}
124124
issue: https://github.com/elastic/elasticsearch/issues/117815
125-
- class: org.elasticsearch.xpack.esql.plugin.ClusterRequestTests
126-
method: testFallbackIndicesOptions
127-
issue: https://github.com/elastic/elasticsearch/issues/117937
128125
- class: org.elasticsearch.xpack.ml.integration.RegressionIT
129126
method: testTwoJobsWithSameRandomizeSeedUseSameTrainingSet
130127
issue: https://github.com/elastic/elasticsearch/issues/117805

x-pack/plugin/esql/src/test/java/org/elasticsearch/xpack/esql/plugin/ClusterRequestTests.java

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -155,8 +155,12 @@ protected ClusterComputeRequest mutateInstance(ClusterComputeRequest in) throws
155155

156156
public void testFallbackIndicesOptions() throws Exception {
157157
ClusterComputeRequest request = createTestInstance();
158-
var version = TransportVersionUtils.randomVersionBetween(random(), TransportVersions.V_8_14_0, TransportVersions.V_8_16_0);
159-
ClusterComputeRequest cloned = copyInstance(request, version);
158+
var oldVersion = TransportVersionUtils.randomVersionBetween(
159+
random(),
160+
TransportVersions.V_8_14_0,
161+
TransportVersionUtils.getPreviousVersion(TransportVersions.V_8_16_0)
162+
);
163+
ClusterComputeRequest cloned = copyInstance(request, oldVersion);
160164
assertThat(cloned.clusterAlias(), equalTo(request.clusterAlias()));
161165
assertThat(cloned.sessionId(), equalTo(request.sessionId()));
162166
assertThat(cloned.configuration(), equalTo(request.configuration()));

0 commit comments

Comments
 (0)