oscerd opened a new pull request, #25327:
URL: https://github.com/apache/camel/pull/25327
### Problem
`CamelJfrDevConsoleTest` fails **deterministically on `main`** — 3 of its
status tests (`statusReportsNotRegisteredByDefault`,
`statusReportsRegisteredWhenRuntimeInstrumentationEnabled`,
`jsonStatusAndToggleReflectLiveState`).
`CamelJfrDevConsole` renamed its status field from `registered` to
`runtimeEvents` — both the TEXT line (`sb.append("runtimeEvents: ")…`) and the
JSON key (`root.put("runtimeEvents", …)`), keeping the same
`isInstrumentationRegistered()` value — but the test still asserted the old
`registered` name (TEXT `contains("registered: …")` and JSON
`getBoolean("registered")`, which now returns null).
Because `camel-jfr` is transitively pulled into the scalpel build of any PR
that touches a core module, this was failing the CI of unrelated PRs.
### Fix
Update the three assertions to the new `runtimeEvents` field name.
Test-only, no production change. `CamelJfrDevConsoleTest` — **17 tests pass**.
_Claude Code on behalf of Andrea Cosentino (@oscerd)._
🤖 Generated with [Claude Code](https://claude.com/claude-code)
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
