From ed10d9d711910008c1d9fbcd4dedd1ef55e0daf2 Mon Sep 17 00:00:00 2001 From: Gary Gregory Date: Mon, 25 May 2026 06:41:13 -0400 Subject: [PATCH] Make the StopWatch test more reliable. --- .../java/org/apache/commons/lang3/time/StopWatchTest.java | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/src/test/java/org/apache/commons/lang3/time/StopWatchTest.java b/src/test/java/org/apache/commons/lang3/time/StopWatchTest.java index 07f4ab81b4c..d42b790322b 100644 --- a/src/test/java/org/apache/commons/lang3/time/StopWatchTest.java +++ b/src/test/java/org/apache/commons/lang3/time/StopWatchTest.java @@ -69,12 +69,12 @@ private StopWatch createMockStopWatch(final long nanos) { return set(watch, nanos); } - private StopWatch set(final StopWatch watch, final long nanos) { + private StopWatch set(final StopWatch watch, final long elapsed) { try { final long currentNanos = System.nanoTime(); final List splits = new ArrayList<>(); - splits.add(new StopWatch.Split(String.valueOf(0), Duration.ofNanos(nanos))); - FieldUtils.writeField(watch, "startTimeNanos", currentNanos - nanos, true); + splits.add(new StopWatch.Split(String.valueOf(0), Duration.ofNanos(elapsed))); + FieldUtils.writeField(watch, "startTimeNanos", currentNanos - elapsed, true); FieldUtils.writeField(watch, "stopTimeNanos", currentNanos, true); FieldUtils.writeField(watch, "splits", splits, true); } catch (final IllegalAccessException e) { @@ -488,9 +488,9 @@ void testSuspend() throws InterruptedException { final Duration sleepDuration = MIN_DURATION; final long sleepMillis = sleepDuration.toMillis(); sleepPlus1(sleepDuration); - watch.suspend(); final long testSuspendMillis = System.currentTimeMillis(); final long testSuspendNanos = System.nanoTime(); + watch.suspend(); final long testSuspendTimeNanos = testSuspendNanos - testStartNanos; // See sleepPlus1 final Duration testSuspendDuration = Duration.ofNanos(testSuspendTimeNanos).plusMillis(1);