Requiring the presence of the geoip fixture to run GeoIpDownloaderSta…

…ts.testStats() (#91662) (#93459) (cherry picked from commit ab17a13) authored-by: Keith Massey <[email protected]>
elastic · Feb 2, 2023 · 21fe882 · 21fe882
1 parent 7d7a0a9
commit 21fe882
Showing 1 changed file with 5 additions and 0 deletions.
diff --git a/...p/src/internalClusterTest/java/org/elasticsearch/ingest/geoip/GeoIpDownloaderStatsIT.java b/...p/src/internalClusterTest/java/org/elasticsearch/ingest/geoip/GeoIpDownloaderStatsIT.java
@@ -66,6 +66,11 @@ public void disableDownloader() {
     }
 
     public void testStats() throws Exception {
+        /*
+         * Testing without the geoip endpoint fixture falls back to https://storage.googleapis.com/, which can cause this test to run too
+         * slowly to pass.
+         */
+        assumeTrue("only test with fixture to have stable results", ENDPOINT != null);
         GeoIpDownloaderStatsAction.Request req = new GeoIpDownloaderStatsAction.Request();
         GeoIpDownloaderStatsAction.Response response = client().execute(GeoIpDownloaderStatsAction.INSTANCE, req).actionGet();
         XContentTestUtils.JsonMapView jsonMapView = new XContentTestUtils.JsonMapView(convertToMap(response));