Skip to content

Commit 80d539d

Browse files
authored
[ML] Fix failing test DetectionRulesTests.testEqualsAndHashcode (#111351)
Fixes #111308
1 parent 1a939e9 commit 80d539d

File tree

2 files changed

+2
-3
lines changed

2 files changed

+2
-3
lines changed

muted-tests.yml

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -108,9 +108,6 @@ tests:
108108
- class: org.elasticsearch.upgrades.LogsIndexModeFullClusterRestartIT
109109
method: testLogsIndexing {cluster=UPGRADED}
110110
issue: https://github.com/elastic/elasticsearch/issues/111306
111-
- class: org.elasticsearch.xpack.core.ml.job.config.DetectionRuleTests
112-
method: testEqualsAndHashcode
113-
issue: https://github.com/elastic/elasticsearch/issues/111308
114111
- class: org.elasticsearch.xpack.ml.integration.DatafeedJobsRestIT
115112
issue: https://github.com/elastic/elasticsearch/issues/111319
116113
- class: org.elasticsearch.xpack.esql.analysis.VerifierTests

x-pack/plugin/core/src/test/java/org/elasticsearch/xpack/core/ml/job/config/DetectionRuleTests.java

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -139,6 +139,8 @@ protected DetectionRule mutateInstance(DetectionRule instance) {
139139

140140
if (actions.contains(RuleAction.FORCE_TIME_SHIFT) && params.getForceTimeShift() == null) {
141141
params = new RuleParams(new RuleParamsForForceTimeShift(randomLong()));
142+
} else if (actions.contains(RuleAction.FORCE_TIME_SHIFT) == false && params.getForceTimeShift() != null) {
143+
params = new RuleParams();
142144
}
143145

144146
return new DetectionRule.Builder(conditions).setActions(actions).setScope(scope).setParams(params).build();

0 commit comments

Comments
 (0)