This is an automated email from the ASF dual-hosted git repository.
oscerd 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 6597f282b4b2 ci: fix CamelJfrDevConsoleTest for the registered ->
runtimeEvents status rename (#25327)
6597f282b4b2 is described below
commit 6597f282b4b26bf6cfe155aaabf0f4dd42d533a3
Author: Andrea Cosentino <[email protected]>
AuthorDate: Tue Aug 4 20:04:14 2026 +0200
ci: fix CamelJfrDevConsoleTest for the registered -> runtimeEvents status
rename (#25327)
CamelJfrDevConsole renamed its status field from "registered" to
"runtimeEvents"
(both the TEXT line and the JSON key, same isInstrumentationRegistered()
value),
but CamelJfrDevConsoleTest still asserted the old name, so its three status
tests
failed deterministically on main. Because camel-jfr is pulled into the
scalpel
build of any core-touching PR, this was failing unrelated PRs' CI. Update
the
assertions to the new field name. 17 tests pass.
Signed-off-by: Andrea Cosentino <[email protected]>
Co-authored-by: Claude Opus 4.8 (1M context) <[email protected]>
---
.../java/org/apache/camel/runtime/jfr/CamelJfrDevConsoleTest.java | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git
a/components/camel-jfr/src/test/java/org/apache/camel/runtime/jfr/CamelJfrDevConsoleTest.java
b/components/camel-jfr/src/test/java/org/apache/camel/runtime/jfr/CamelJfrDevConsoleTest.java
index eb9e105904aa..cabb6dbe087e 100644
---
a/components/camel-jfr/src/test/java/org/apache/camel/runtime/jfr/CamelJfrDevConsoleTest.java
+++
b/components/camel-jfr/src/test/java/org/apache/camel/runtime/jfr/CamelJfrDevConsoleTest.java
@@ -71,7 +71,7 @@ class CamelJfrDevConsoleTest extends CamelTestSupport {
String text = (String) console.call(DevConsole.MediaType.TEXT,
Map.of("command", "status"));
assertThat(text)
- .contains("registered: false")
+ .contains("runtimeEvents: false")
.contains("event route")
.contains("event processor")
.contains("event exchange")
@@ -92,7 +92,7 @@ class CamelJfrDevConsoleTest extends CamelTestSupport {
String text = (String)
resolveConsole(ctx).call(DevConsole.MediaType.TEXT, Map.of("command",
"status"));
- assertThat(text).contains("registered: true");
+ assertThat(text).contains("runtimeEvents: true");
}
}
@@ -180,7 +180,7 @@ class CamelJfrDevConsoleTest extends CamelTestSupport {
CamelJfrDevConsole console = resolveConsole(context);
try (Recording recording = startRecordingWithAllEvents()) {
JsonObject status = (JsonObject)
console.call(DevConsole.MediaType.JSON, Map.of("command", "status"));
- assertThat(status.getBoolean("registered")).isFalse();
+ assertThat(status.getBoolean("runtimeEvents")).isFalse();
assertThat(status.getJsonObject("events")).containsEntry("route",
true);
JsonObject toggle = (JsonObject)
console.call(DevConsole.MediaType.JSON,