Skip to content

Commit b2aa972

Browse files
authored
skip degenerated line in CentroidCalculatorTests (#66321)
1 parent d0c7c65 commit b2aa972

File tree

1 file changed

+10
-6
lines changed

1 file changed

+10
-6
lines changed

x-pack/plugin/spatial/src/test/java/org/elasticsearch/xpack/spatial/index/fielddata/CentroidCalculatorTests.java

+10-6
Original file line numberDiff line numberDiff line change
@@ -356,9 +356,11 @@ public void testAddDifferentDimensionalType() {
356356
CentroidCalculator calculator = new CentroidCalculator();
357357
calculator.add(point);
358358
calculator.add(line);
359-
assertThat(calculator.getX(), equalTo(lineCalculator.getX()));
360-
assertThat(calculator.getY(), equalTo(lineCalculator.getY()));
361-
assertThat(calculator.sumWeight(), equalTo(lineCalculator.sumWeight()));
359+
if (lineCalculator.getDimensionalShapeType() == LINE) { // skip degenerated line
360+
assertThat(calculator.getX(), equalTo(lineCalculator.getX()));
361+
assertThat(calculator.getY(), equalTo(lineCalculator.getY()));
362+
assertThat(calculator.sumWeight(), equalTo(lineCalculator.sumWeight()));
363+
}
362364
}
363365

364366
// line add point
@@ -368,9 +370,11 @@ public void testAddDifferentDimensionalType() {
368370
CentroidCalculator calculator = new CentroidCalculator();
369371
calculator.add(line);
370372
calculator.add(point);
371-
assertThat(calculator.getX(), equalTo(lineCalculator.getX()));
372-
assertThat(calculator.getY(), equalTo(lineCalculator.getY()));
373-
assertThat(calculator.sumWeight(), equalTo(lineCalculator.sumWeight()));
373+
if (lineCalculator.getDimensionalShapeType() == LINE) { // skip degenerated line
374+
assertThat(calculator.getX(), equalTo(lineCalculator.getX()));
375+
assertThat(calculator.getY(), equalTo(lineCalculator.getY()));
376+
assertThat(calculator.sumWeight(), equalTo(lineCalculator.sumWeight()));
377+
}
374378
}
375379

376380
// line add line

0 commit comments

Comments
 (0)