From 0d8e40dc6ba7c5e9c88a56f591eddf7b38b2a6dc Mon Sep 17 00:00:00 2001
From: Alex Archambault <alexandre.archambault@gmail.com>
Date: Fri, 9 Jun 2023 11:20:24 +0200
Subject: [PATCH] Add logging / checks when spawning kernel in ITs

---
 .../src/main/scala/almond/integration/KernelLauncher.scala      | 1 +
 .../launcher/src/main/scala/almond/launcher/Launcher.scala      | 2 ++
 2 files changed, 3 insertions(+)

diff --git a/modules/scala/integration/src/main/scala/almond/integration/KernelLauncher.scala b/modules/scala/integration/src/main/scala/almond/integration/KernelLauncher.scala
index 0d0251a22..dd5d567ba 100644
--- a/modules/scala/integration/src/main/scala/almond/integration/KernelLauncher.scala
+++ b/modules/scala/integration/src/main/scala/almond/integration/KernelLauncher.scala
@@ -469,6 +469,7 @@ class KernelLauncher(
           else
             Map.empty[String, String]
 
+        assert(proc == null)
         proc = os.proc(command).spawn(
           cwd = dir,
           env = extraEnv ++ specExtraEnv,
diff --git a/modules/scala/launcher/src/main/scala/almond/launcher/Launcher.scala b/modules/scala/launcher/src/main/scala/almond/launcher/Launcher.scala
index e20f86c99..bf705c7c8 100644
--- a/modules/scala/launcher/src/main/scala/almond/launcher/Launcher.scala
+++ b/modules/scala/launcher/src/main/scala/almond/launcher/Launcher.scala
@@ -198,6 +198,7 @@ object Launcher extends CaseApp[LauncherOptions] {
         setDaemon(true)
         override def run(): Unit =
           if (p.isAlive()) {
+            System.err.println("Shutting down underlying kernel")
             p.close()
             val timeout = 500.millis
             if (!p.waitFor(timeout.toMillis)) {
@@ -206,6 +207,7 @@ object Launcher extends CaseApp[LauncherOptions] {
               )
               p.destroyForcibly()
             }
+            System.err.println("Shut down underlying kernel")
           }
       }
     Runtime.getRuntime.addShutdownHook(hook)