From 6658e3d7625cd4dc62db71ff939a26424f50b7dc Mon Sep 17 00:00:00 2001
From: David Turner <david.turner@elastic.co>
Date: Sun, 17 Dec 2017 20:43:06 +0000
Subject: [PATCH] Map Long.MIN_VALUE onto 0 rather than retrying

---
 .../main/java/org/elasticsearch/test/ESTestCase.java   | 10 +++++-----
 1 file changed, 5 insertions(+), 5 deletions(-)

diff --git a/test/framework/src/main/java/org/elasticsearch/test/ESTestCase.java b/test/framework/src/main/java/org/elasticsearch/test/ESTestCase.java
index 84e750f6e2880..d2712e268ade2 100644
--- a/test/framework/src/main/java/org/elasticsearch/test/ESTestCase.java
+++ b/test/framework/src/main/java/org/elasticsearch/test/ESTestCase.java
@@ -509,12 +509,12 @@ public static int randomInt() {
         return random().nextInt();
     }
 
+    /**
+     * @return a <code>long</code> between <code>0</code> and <code>Long.MAX_VALUE</code> (inclusive) chosen uniformly at random.
+     */
     public static long randomNonNegativeLong() {
-        long randomLong;
-        do {
-            randomLong = randomLong();
-        } while (randomLong == Long.MIN_VALUE);
-        return Math.abs(randomLong);
+        long randomLong = randomLong();
+        return randomLong == Long.MIN_VALUE ? 0 : Math.abs(randomLong);
     }
 
     public static float randomFloat() {