[Maps] Move elasticsearch_geo_utils to common (#75508)

Author: thomasneirynck

x-pack/plugins/maps/public/elasticsearch_geo_utils.d.ts renamed to x-pack/plugins/maps/common/elasticsearch_geo_utils.d.ts

@@ -4,8 +4,12 @@
  * you may not use this file except in compliance with the Elastic License.
  */
 
-import { MapExtent } from '../common/descriptor_types';
+import { MapExtent } from './descriptor_types';
 
 export function scaleBounds(bounds: MapExtent, scaleFactor: number): MapExtent;
 
 export function turfBboxToBounds(turfBbox: unknown): MapExtent;
+
+export function clampToLatBounds(lat: number): number;
+
+export function clampToLonBounds(lon: number): number;

x-pack/plugins/maps/public/elasticsearch_geo_utils.js renamed to x-pack/plugins/maps/common/elasticsearch_geo_utils.js

@@ -16,10 +16,12 @@ import {
   LON_INDEX,
   LAT_INDEX,
 } from '../common/constants';
-import { getEsSpatialRelationLabel } from '../common/i18n_getters';
-import { SPATIAL_FILTER_TYPE } from './kibana_services';
+import { getEsSpatialRelationLabel } from './i18n_getters';
+import { FILTERS } from '../../../../src/plugins/data/common';
 import turfCircle from '@turf/circle';
 
+const SPATIAL_FILTER_TYPE = FILTERS.SPATIAL_FILTER;
+
 function ensureGeoField(type) {
   const expectedTypes = [ES_GEO_FIELD_TYPE.GEO_POINT, ES_GEO_FIELD_TYPE.GEO_SHAPE];
   if (!expectedTypes.includes(type)) {

x-pack/plugins/maps/public/elasticsearch_geo_utils.test.js renamed to x-pack/plugins/maps/common/elasticsearch_geo_utils.test.js

@@ -4,14 +4,6 @@
  * you may not use this file except in compliance with the Elastic License.
  */
 
-jest.mock('ui/new_platform');
-
-jest.mock('./kibana_services', () => {
-  return {
-    SPATIAL_FILTER_TYPE: 'spatial_filter',
-  };
-});
-
 import {
   hitsToGeoJson,
   geoPointToGeometry,
@@ -22,7 +14,7 @@ import {
   makeESBbox,
   scaleBounds,
 } from './elasticsearch_geo_utils';
-import { indexPatterns } from '../../../../src/plugins/data/public';
+import _ from 'lodash';
 
 const geoFieldName = 'location';
 
@@ -173,19 +165,14 @@ describe('hitsToGeoJson', () => {
   });
 
   describe('dot in geoFieldName', () => {
-    const indexPatternMock = {
-      fields: {
-        getByName: (name) => {
-          const fields = {
-            ['my.location']: {
-              type: 'geo_point',
-            },
-          };
-          return fields[name];
-        },
-      },
+    // This essentially should test the implmentation of index-pattern.flattenHit, rather than anything in geo_utils.
+    // Leaving this here for reference.
+    const geoFieldName = 'my.location';
+    const indexPatternFlattenHit = (hit) => {
+      return {
+        [geoFieldName]: _.get(hit._source, geoFieldName),
+      };
     };
-    const indexPatternFlattenHit = indexPatterns.flattenHitWrapper(indexPatternMock);
 
     it('Should handle geoField being an object', () => {
       const hits = [

x-pack/plugins/maps/public/actions/data_request_actions.ts

@@ -41,7 +41,7 @@ import { ILayer } from '../classes/layers/layer';
 import { IVectorLayer } from '../classes/layers/vector_layer/vector_layer';
 import { DataMeta, MapExtent, MapFilters } from '../../common/descriptor_types';
 import { DataRequestAbortError } from '../classes/util/data_request';
-import { scaleBounds, turfBboxToBounds } from '../elasticsearch_geo_utils';
+import { scaleBounds, turfBboxToBounds } from '../../common/elasticsearch_geo_utils';
 
 const FIT_TO_BOUNDS_SCALE_FACTOR = 0.1;
 

x-pack/plugins/maps/public/actions/map_actions.ts

@@ -54,7 +54,7 @@ import {
   MapRefreshConfig,
 } from '../../common/descriptor_types';
 import { INITIAL_LOCATION } from '../../common/constants';
-import { scaleBounds } from '../elasticsearch_geo_utils';
+import { scaleBounds } from '../../common/elasticsearch_geo_utils';
 
 export function setMapInitError(errorMessage: string) {
   return {

x-pack/plugins/maps/public/classes/sources/es_geo_grid_source/convert_to_geojson.js

@@ -8,7 +8,7 @@ import _ from 'lodash';
 import { RENDER_AS } from '../../../../common/constants';
 import { getTileBoundingBox } from './geo_tile_utils';
 import { extractPropertiesFromBucket } from '../../util/es_agg_utils';
-import { clamp } from '../../../elasticsearch_geo_utils';
+import { clamp } from '../../../../common/elasticsearch_geo_utils';
 
 const GRID_BUCKET_KEYS_TO_IGNORE = ['key', 'gridCentroid'];
 

x-pack/plugins/maps/public/classes/sources/es_geo_grid_source/es_geo_grid_source.js

@@ -21,7 +21,7 @@ import { getDataSourceLabel } from '../../../../common/i18n_getters';
 import { AbstractESAggSource, DEFAULT_METRIC } from '../es_agg_source';
 import { DataRequestAbortError } from '../../util/data_request';
 import { registerSource } from '../source_registry';
-import { makeESBbox } from '../../../elasticsearch_geo_utils';
+import { makeESBbox } from '../../../../common/elasticsearch_geo_utils';
 
 export const MAX_GEOTILE_LEVEL = 29;
 

x-pack/plugins/maps/public/classes/sources/es_geo_grid_source/geo_tile_utils.js

@@ -6,7 +6,7 @@
 
 import _ from 'lodash';
 import { DECIMAL_DEGREES_PRECISION } from '../../../../common/constants';
-import { clampToLatBounds } from '../../../elasticsearch_geo_utils';
+import { clampToLatBounds } from '../../../../common/elasticsearch_geo_utils';
 
 const ZOOM_TILE_KEY_INDEX = 0;
 const X_TILE_KEY_INDEX = 1;

x-pack/plugins/maps/public/classes/sources/es_pew_pew_source/es_pew_pew_source.js

@@ -16,7 +16,7 @@ import { getDataSourceLabel } from '../../../../common/i18n_getters';
 import { convertToLines } from './convert_to_lines';
 import { AbstractESAggSource, DEFAULT_METRIC } from '../es_agg_source';
 import { registerSource } from '../source_registry';
-import { turfBboxToBounds } from '../../../elasticsearch_geo_utils';
+import { turfBboxToBounds } from '../../../../common/elasticsearch_geo_utils';
 import { DataRequestAbortError } from '../../util/data_request';
 
 const MAX_GEOTILE_LEVEL = 29;

x-pack/plugins/maps/public/classes/sources/es_search_source/es_search_source.js

@@ -9,7 +9,7 @@ import React from 'react';
 
 import { AbstractESSource } from '../es_source';
 import { getSearchService } from '../../../kibana_services';
-import { hitsToGeoJson } from '../../../elasticsearch_geo_utils';
+import { hitsToGeoJson } from '../../../../common/elasticsearch_geo_utils';
 import { UpdateSourceEditor } from './update_source_editor';
 import {
   SOURCE_TYPES,