From 47966cfdd2db04f47a0fea02641bd8b746ba97e6 Mon Sep 17 00:00:00 2001
From: Gal Lalouche <gal.lalouche@elastic.co>
Date: Tue, 17 Dec 2024 14:32:17 +0200
Subject: [PATCH] Fix TODO

---
 .../index/mapper/AbstractShapeGeometryFieldMapperTests.java    | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/server/src/test/java/org/elasticsearch/index/mapper/AbstractShapeGeometryFieldMapperTests.java b/server/src/test/java/org/elasticsearch/index/mapper/AbstractShapeGeometryFieldMapperTests.java
index 8ca412405a062..9a4cbcebd9d27 100644
--- a/server/src/test/java/org/elasticsearch/index/mapper/AbstractShapeGeometryFieldMapperTests.java
+++ b/server/src/test/java/org/elasticsearch/index/mapper/AbstractShapeGeometryFieldMapperTests.java
@@ -36,7 +36,6 @@
 import java.util.stream.IntStream;
 
 public class AbstractShapeGeometryFieldMapperTests extends ESTestCase {
-    // TODO handle geo as well, this is actually bugged, since extracting the result ignores minneg etc.
     public void testCartesianBoundsBlockLoader() throws IOException {
         testBoundsBlockLoaderAux(
             CoordinateEncoder.CARTESIAN,
@@ -46,7 +45,7 @@ public void testCartesianBoundsBlockLoader() throws IOException {
         );
     }
 
-    // TODO when we turn this optimization on for geo, handle this as well.
+    // TODO when we turn this optimization on for geo, this test should pass.
     public void ignoreTestGeoBoundsBlockLoader() throws IOException {
         testBoundsBlockLoaderAux(
             CoordinateEncoder.GEO,