Review fixes for internal-application-logger:javaagent (#17370)

otelbot[bot] · trask · web-flow · commit 94b3d542af1d · 2026-04-16T13:04:01.000+03:00
Co-authored-by: otelbot &lt;197425009+otelbot@users.noreply.github.com&gt;
Co-authored-by: Trask Stalnaker &lt;trask.stalnaker@gmail.com&gt;
diff --git a/instrumentation/internal/internal-application-logger/javaagent/src/test/java/io/opentelemetry/javaagent/instrumentation/internal/logging/ApplicationLoggerInstrumentationTest.java b/instrumentation/internal/internal-application-logger/javaagent/src/test/java/io/opentelemetry/javaagent/instrumentation/internal/logging/ApplicationLoggerInstrumentationTest.java
@@ -72,7 +72,12 @@ private static List<String> forkAndRun(String mainClassName) throws Exception {
               }
             });
 
-    process.waitFor(10, SECONDS);
+    boolean exited = process.waitFor(10, SECONDS);
+    if (!exited) {
+      process.destroyForcibly();
+    }
+    assertThat(exited).isTrue();
+    assertThat(process.exitValue()).isZero();
     return output.join();
   }
 }

Original file line number	Diff line number	Diff line change
`@@ -72,7 +72,12 @@ private static List<String> forkAndRun(String mainClassName) throws Exception {`
`72`	`72`	`}`
`73`	`73`	`});`
`74`	`74`
`75`		`- process.waitFor(10, SECONDS);`
	`75`	`+ boolean exited = process.waitFor(10, SECONDS);`
	`76`	`+ if (!exited) {`
	`77`	`+ process.destroyForcibly();`
	`78`	`+ }`
	`79`	`+ assertThat(exited).isTrue();`
	`80`	`+ assertThat(process.exitValue()).isZero();`
`76`	`81`	`return output.join();`
`77`	`82`	`}`
`78`	`83`	`}`