From 7c17ee2f8ca283f6e0c6993d34b5dd9f5c1aeb96 Mon Sep 17 00:00:00 2001 From: Ed Savage Date: Thu, 14 Sep 2023 13:12:46 +0100 Subject: [PATCH] [ML] Fix failing JobUpdateTests.testMergeWithJob test (#99571) Ensure that a random compatible _MlConfigVersion_ is used to set the job version. --- .../xpack/core/ml/job/config/JobUpdateTests.java | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/x-pack/plugin/core/src/test/java/org/elasticsearch/xpack/core/ml/job/config/JobUpdateTests.java b/x-pack/plugin/core/src/test/java/org/elasticsearch/xpack/core/ml/job/config/JobUpdateTests.java index 64c216aff28e8..09ff29f768dce 100644 --- a/x-pack/plugin/core/src/test/java/org/elasticsearch/xpack/core/ml/job/config/JobUpdateTests.java +++ b/x-pack/plugin/core/src/test/java/org/elasticsearch/xpack/core/ml/job/config/JobUpdateTests.java @@ -7,14 +7,12 @@ package org.elasticsearch.xpack.core.ml.job.config; import org.elasticsearch.ElasticsearchStatusException; -import org.elasticsearch.Version; import org.elasticsearch.common.io.stream.Writeable; import org.elasticsearch.common.unit.ByteSizeUnit; import org.elasticsearch.common.unit.ByteSizeValue; import org.elasticsearch.core.Nullable; import org.elasticsearch.core.TimeValue; import org.elasticsearch.test.AbstractXContentSerializingTestCase; -import org.elasticsearch.test.VersionUtils; import org.elasticsearch.xcontent.XContentParser; import org.elasticsearch.xpack.core.ml.MlConfigVersion; import org.elasticsearch.xpack.core.ml.job.process.autodetect.state.ModelSnapshot; @@ -293,7 +291,7 @@ public void testMergeWithJob() { updateBuilder.setPerPartitionCategorizationConfig(new PerPartitionCategorizationConfig(true, randomBoolean())); updateBuilder.setCustomSettings(customSettings); updateBuilder.setModelSnapshotId(randomAlphaOfLength(10)); - updateBuilder.setJobVersion(MlConfigVersion.fromVersion(VersionUtils.randomCompatibleVersion(random(), Version.CURRENT))); + updateBuilder.setJobVersion(MlConfigVersionUtils.randomCompatibleVersion(random())); updateBuilder.setModelPruneWindow(TimeValue.timeValueDays(randomIntBetween(1, 100))); JobUpdate update = updateBuilder.build();