Skip to content

Commit 098140e

Browse files
authored
Adjust testHKmeans assertion after #132706 (#132786)
1 parent 6cd330c commit 098140e

File tree

2 files changed

+3
-5
lines changed

2 files changed

+3
-5
lines changed

muted-tests.yml

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -501,9 +501,6 @@ tests:
501501
- class: org.elasticsearch.repositories.SnapshotMetricsIT
502502
method: testSnapshotAPMMetrics
503503
issue: https://github.com/elastic/elasticsearch/issues/132731
504-
- class: org.elasticsearch.index.codec.vectors.cluster.HierarchicalKMeansTests
505-
method: testHKmeans
506-
issue: https://github.com/elastic/elasticsearch/issues/132771
507504
- class: org.elasticsearch.xpack.esql.qa.single_node.GenerativeForkIT
508505
method: test {csv-spec:lookup-join.MvJoinKeyFromRowExpanded}
509506
issue: https://github.com/elastic/elasticsearch/issues/132778

server/src/test/java/org/elasticsearch/index/codec/vectors/cluster/HierarchicalKMeansTests.java

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -46,8 +46,9 @@ public void testHKmeans() throws IOException {
4646
assertEquals(nVectors, soarAssignments.length);
4747
// verify no duplicates exist
4848
for (int i = 0; i < assignments.length; i++) {
49-
assertTrue(soarAssignments[i] >= 0 && soarAssignments[i] < centroids.length);
50-
assertNotEquals(assignments[i], soarAssignments[i]);
49+
int soarAssignment = soarAssignments[i];
50+
assertTrue(soarAssignment == -1 || (soarAssignment >= 0 && soarAssignment < centroids.length));
51+
assertNotEquals(assignments[i], soarAssignment);
5152
}
5253
} else {
5354
assertEquals(0, soarAssignments.length);

0 commit comments

Comments
 (0)