From 678e1be5cf9c94607677c91b7e4b889a1f6362a5 Mon Sep 17 00:00:00 2001
From: Colin Goodheart-Smithe <colings86@users.noreply.github.com>
Date: Wed, 9 Sep 2015 14:24:53 +0100
Subject: [PATCH] [TEST] fix geo query failures when no types are defined

---
 .../index/query/GeoDistanceRangeQueryTests.java     | 13 ++++++++++++-
 .../index/query/GeohashCellQueryBuilderTests.java   | 11 +++++++++++
 2 files changed, 23 insertions(+), 1 deletion(-)

diff --git a/core/src/test/java/org/elasticsearch/index/query/GeoDistanceRangeQueryTests.java b/core/src/test/java/org/elasticsearch/index/query/GeoDistanceRangeQueryTests.java
index 4f415c64209..cba0ab7b215 100644
--- a/core/src/test/java/org/elasticsearch/index/query/GeoDistanceRangeQueryTests.java
+++ b/core/src/test/java/org/elasticsearch/index/query/GeoDistanceRangeQueryTests.java
@@ -114,7 +114,7 @@ public class GeoDistanceRangeQueryTests extends BaseQueryTestCase<GeoDistanceRan
         assertThat(geoQuery.fieldName(), equalTo(queryBuilder.fieldName()));
         if (queryBuilder.point() != null) {
             assertThat(geoQuery.lat(), equalTo(queryBuilder.point().lat()));
-            assertThat(geoQuery.lat(), equalTo(queryBuilder.point().lat()));
+            assertThat(geoQuery.lon(), equalTo(queryBuilder.point().lon()));
         }
         assertThat(geoQuery.geoDistance(), equalTo(queryBuilder.geoDistance()));
         if (queryBuilder.from() != null && queryBuilder.from() instanceof Number) {
@@ -139,6 +139,17 @@ public class GeoDistanceRangeQueryTests extends BaseQueryTestCase<GeoDistanceRan
         }
     }
 
+    /**
+     * Overridden here to ensure the test is only run if at least one type is
+     * present in the mappings. Geo queries do not execute if the field is not
+     * explicitly mapped
+     */
+    @Override
+    public void testToQuery() throws IOException {
+        assumeTrue("test runs only when at least a type is registered", getCurrentTypes().length > 0);
+        super.testToQuery();
+    }
+
     @Test
     public void testNullFieldName() {
         GeoDistanceRangeQueryBuilder builder = new GeoDistanceRangeQueryBuilder(null);
diff --git a/core/src/test/java/org/elasticsearch/index/query/GeohashCellQueryBuilderTests.java b/core/src/test/java/org/elasticsearch/index/query/GeohashCellQueryBuilderTests.java
index f0bc73f1b7c..5a408631cba 100644
--- a/core/src/test/java/org/elasticsearch/index/query/GeohashCellQueryBuilderTests.java
+++ b/core/src/test/java/org/elasticsearch/index/query/GeohashCellQueryBuilderTests.java
@@ -71,6 +71,17 @@ public class GeohashCellQueryBuilderTests extends BaseQueryTestCase<GeohashCellQ
         }
     }
 
+    /**
+     * Overridden here to ensure the test is only run if at least one type is
+     * present in the mappings. Geo queries do not execute if the field is not
+     * explicitly mapped
+     */
+    @Override
+    public void testToQuery() throws IOException {
+        assumeTrue("test runs only when at least a type is registered", getCurrentTypes().length > 0);
+        super.testToQuery();
+    }
+
     @Test
     public void testNullField() {
         GeohashCellQuery.Builder builder = new Builder(null);