Fix ThreadPoolTests#testDetailedUtilizationMetric (#138393) (#138395)

(cherry picked from commit 3749b6e)

# Conflicts:
#	server/src/main/java/org/elasticsearch/common/util/concurrent/TaskExecutionTimeTrackingEsThreadPoolExecutor.java

Author: nicktindall

server/src/test/java/org/elasticsearch/threadpool/ThreadPoolTests.java

@@ -528,10 +528,12 @@ public void testDetailedUtilizationMetric() throws Exception {
             });
             safeAwait(barrier);
             safeGet(future);
-            final long maxDurationNanos = System.nanoTime() - beforeStartNanos;
-
             // Wait for TaskExecutionTimeTrackingEsThreadPoolExecutor#afterExecute to run
             assertBusy(() -> assertThat(executor.getTotalTaskExecutionTime(), greaterThan(0L)));
+            // When you call submit, the TimedRunnable wraps the FutureTask, so safeGet can return before the duration of
+            // the task is calculated. Waiting for totalTaskExecutionTime to be updated ensures maxDurationNanos is greater
+            // than the actual duration.
+            final long maxDurationNanos = System.nanoTime() - beforeStartNanos;
 
             final long beforeMetricsCollectedNanos = System.nanoTime();
             meterRegistry.getRecorder().collect();