explicitly terminate jvm in mutation minion

hcoles · Oct 30, 2023 · c55ee14 · c55ee14
1 parent 495a8de
commit c55ee14
Show file tree

Hide file tree

Showing 3 changed files with 7 additions and 102 deletions.
diff --git a/pitest-entry/src/main/java/org/pitest/mutationtest/execute/Receive.java b/pitest-entry/src/main/java/org/pitest/mutationtest/execute/Receive.java
@@ -29,6 +29,8 @@ public void apply(final byte control, final SafeDataInputStream is) {
             case Id.REPORT:
                 handleReport(is);
                 break;
+            default:
+                LOG.severe("Unknown control byte " + control);
         }
     }
 

diff --git a/pitest/src/main/java/org/pitest/mutationtest/execute/MutationTestMinion.java b/pitest/src/main/java/org/pitest/mutationtest/execute/MutationTestMinion.java
@@ -99,6 +99,11 @@ public void run() {
               tests, this.reporter));
 
       this.reporter.done(ExitCode.OK);
+
+      // rudely kill the vm in case it is kept alive
+      // by threads launched by client
+      System.exit(0);
+
     } catch (final Throwable ex) {
       ex.printStackTrace(System.out);
       LOG.log(Level.WARNING, "Error during mutation test", ex);

diff --git a/pitest/src/test/java/org/pitest/mutationtest/execute/MutationTestMinionTest.java b/pitest/src/test/java/org/pitest/mutationtest/execute/MutationTestMinionTest.java