[opensearch-project#7101] Fixing the GeoTileIT#testMultivaluedGeoPoin…

…tsAggregation test case. The issue was happening because we encode the GeoPoint as long and error comes in the precision due to that encoding. The error was not taken care while generating the exepected tiles count for execpected output. Signed-off-by: Navneet Verma <[email protected]>
navneet1v · Apr 14, 2023 · c041702 · c041702
1 parent 632eb44
commit c041702
Showing 1 changed file with 6 additions and 1 deletion.
diff --git a/...internalClusterTest/java/org/opensearch/geo/search/aggregations/bucket/GeoTileGridIT.java b/...internalClusterTest/java/org/opensearch/geo/search/aggregations/bucket/GeoTileGridIT.java
@@ -8,6 +8,7 @@
 
 package org.opensearch.geo.search.aggregations.bucket;
 
+import org.apache.lucene.geo.GeoEncodingUtils;
 import org.hamcrest.MatcherAssert;
 import org.opensearch.action.search.SearchResponse;
 import org.opensearch.common.geo.GeoBoundingBox;
@@ -160,7 +161,11 @@ protected Set<String> generateBucketsForGeometry(final Geometry geometry, final
     protected Set<String> generateBucketsForGeoPoint(final GeoPoint geoPoint) {
         Set<String> buckets = new HashSet<>();
         for (int precision = GEOPOINT_MAX_PRECISION; precision > 0; precision--) {
-            final String tile = GeoTileUtils.stringEncode(geoPoint.getLon(), geoPoint.getLat(), precision);
+            final String tile = GeoTileUtils.stringEncode(
+                GeoEncodingUtils.decodeLongitude(GeoEncodingUtils.encodeLongitude(geoPoint.getLon())),
+                GeoEncodingUtils.decodeLatitude(GeoEncodingUtils.encodeLatitude(geoPoint.getLat())),
+                precision
+            );
             buckets.add(tile);
         }
         return buckets;