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