This is an automated email from the ASF dual-hosted git repository.

davsclaus pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/camel.git


The following commit(s) were added to refs/heads/main by this push:
     new b6a3128884f0 CAMEL-26377: Fix flaky MonitorStartupTest (#29173)
b6a3128884f0 is described below

commit b6a3128884f081b92870dc12260ca293ac7df0a0
Author: Guillaume Nodet <[email protected]>
AuthorDate: Wed Nov 4 16:26:22 2026 +0100

    CAMEL-26377: Fix flaky MonitorStartupTest (#29173)
---
 .../jbang/core/commands/tui/MonitorStartupTest.java    | 18 ++++++++++++++----
 1 file changed, 14 insertions(+), 4 deletions(-)

diff --git 
a/dsl/camel-jbang/camel-jbang-plugin-tui/src/test/java/org/apache/camel/dsl/jbang/core/commands/tui/MonitorStartupTest.java
 
b/dsl/camel-jbang/camel-jbang-plugin-tui/src/test/java/org/apache/camel/dsl/jbang/core/commands/tui/MonitorStartupTest.java
index 2a0dd3b3e58e..5ac7d7bd2ba4 100644
--- 
a/dsl/camel-jbang/camel-jbang-plugin-tui/src/test/java/org/apache/camel/dsl/jbang/core/commands/tui/MonitorStartupTest.java
+++ 
b/dsl/camel-jbang/camel-jbang-plugin-tui/src/test/java/org/apache/camel/dsl/jbang/core/commands/tui/MonitorStartupTest.java
@@ -30,11 +30,17 @@ import org.apache.camel.dsl.jbang.run.common.Printer;
 import org.apache.camel.test.AvailablePortFinder;
 import org.apache.camel.test.AvailablePortFinder.Port;
 import org.junit.jupiter.api.Test;
+import org.junit.jupiter.api.parallel.Isolated;
 
 import static org.assertj.core.api.Assertions.assertThat;
 import static org.assertj.core.api.Assertions.assertThatThrownBy;
 import static org.awaitility.Awaitility.await;
 
+/**
+ * Tests for {@link MonitorStartup} that bind a real network port: isolated 
from parallel test forks to avoid port races
+ * (CAMEL-26377).
+ */
+@Isolated
 class MonitorStartupTest {
 
     @Test
@@ -59,12 +65,16 @@ class MonitorStartupTest {
     }
 
     @Test
-    void browserOnlyRefusesMcp() {
+    void browserOnlyRefusesMcp() throws Exception {
         TuiWebServer server = newServer(0);
 
-        assertThatThrownBy(() -> MonitorStartup.serveBrowserOnly(server, 0, 
true, new ListPrinter()))
-                .isInstanceOf(CliException.class)
-                .hasMessageContaining("--mcp needs an interactive terminal");
+        try {
+            assertThatThrownBy(() -> MonitorStartup.serveBrowserOnly(server, 
0, true, new ListPrinter()))
+                    .isInstanceOf(CliException.class)
+                    .hasMessageContaining("--mcp needs an interactive 
terminal");
+        } finally {
+            server.stop();
+        }
     }
 
     @Test

Reply via email to