Fix duration conversion error

Fixes #478

Author: dmlloyd

process/src/main/java/io/smallrye/common/process/PipelineRunner.java

@@ -26,7 +26,6 @@
 import java.lang.invoke.ConstantBootstraps;
 import java.lang.invoke.MethodHandles;
 import java.lang.invoke.VarHandle;
-import java.time.temporal.ChronoUnit;
 import java.util.ArrayList;
 import java.util.List;
 import java.util.concurrent.CopyOnWriteArrayList;
@@ -422,7 +421,7 @@ int createWaitForThread(ThreadFactory tf, ProcessRunner<?> runner) throws IOExce
                     awaitIoDone();
                     if (process.isAlive() && processBuilder.softExitTimeout != null) {
                         // start the countdown!
-                        if (ProcessUtil.stillRunningAfter(process, processBuilder.softExitTimeout.get(ChronoUnit.NANOS))) {
+                        if (ProcessUtil.stillRunningAfter(process, processBuilder.softExitTimeout)) {
                             ste = true;
                             if (process.supportsNormalTermination()) {
                                 // start knocking on the door
@@ -431,7 +430,7 @@ int createWaitForThread(ThreadFactory tf, ProcessRunner<?> runner) throws IOExce
                         }
                     }
                     if (process.isAlive() && processBuilder.hardExitTimeout != null) {
-                        if (ProcessUtil.stillRunningAfter(process, processBuilder.hardExitTimeout.get(ChronoUnit.NANOS))) {
+                        if (ProcessUtil.stillRunningAfter(process, processBuilder.hardExitTimeout)) {
                             hte = true;
                             // obliterate
                             ProcessUtil.destroyAllForcibly(process.toHandle());

process/src/main/java/io/smallrye/common/process/ProcessUtil.java

@@ -5,6 +5,7 @@
 import java.nio.charset.UnsupportedCharsetException;
 import java.nio.file.Files;
 import java.nio.file.Path;
+import java.time.Duration;
 import java.util.List;
 import java.util.Optional;
 import java.util.concurrent.ExecutionException;
@@ -138,6 +139,8 @@ public static String nameOfJava() {
         return JavaPath.javaName;
     }
 
+    private static final Duration LONGEST_NS = Duration.ofNanos(Long.MAX_VALUE);
+
     /**
      * Wait (uninterruptibly) for some amount of time for the given process to finish.
      *
@@ -146,6 +149,18 @@ public static String nameOfJava() {
      * @return {@code true} if the process is still running after the elapsed time, or {@code false} if it has exited
      */
     public static boolean stillRunningAfter(Process proc, long nanos) {
+        return stillRunningAfter(proc, Duration.ofNanos(nanos));
+    }
+
+    /**
+     * Wait (uninterruptibly) for some amount of time for the given process to finish.
+     *
+     * @param proc the process (must not be {@code null})
+     * @param time the amount of time to wait (must not be {@code null})
+     * @return {@code true} if the process is still running after the elapsed time, or {@code false} if it has exited
+     */
+    public static boolean stillRunningAfter(Process proc, Duration time) {
+        long nanos = time.compareTo(LONGEST_NS) > 0 ? Long.MAX_VALUE : time.toNanos();
         boolean intr = false;
         try {
             long start = System.nanoTime();