davsclaus opened a new pull request, #29807: URL: https://github.com/apache/camel/pull/29807
# Description Fixes `TextBodyFlowTest.testTheBodyOfMessaging` on main, which fails on every PR whose build runs `camel-yaml-dsl-validator` (seen on #29594 and #29706). Since #29753 ([CAMEL-26012](https://issues.apache.org/jira/browse/CAMEL-26012)) the `producerIn` body type of pulsar has `dependsOn: [allowJavaSerializedObject]`, and that option exists on the component as well as on the endpoint. `BodyTypeResolver` treats a `dependsOn` option that the step does not set, but that can be set elsewhere, as not known, so the validator says nothing about the body a route sends to pulsar, as it already does for spring-rabbitmq. The test still expected the `to: pulsar needs the body as text, bytes, a stream or a file` finding (4 findings, CI found 3). The test now expects 3 findings and asserts that the pulsar route is not reported; smtp and google-pubsub keep covering a producer that needs text. #29753's own build and the main build passed because neither ran this module. Whether options such as `allowJavaSerializedObject` should be endpoint-only, so they cannot be switched on for a whole component (which would also bring the pulsar finding back), is checked separately. ## Testing - The 3 findings the test now expects are exactly what CI reported on #29594 and #29706 (mqtt, service, mail). - Not run locally: a local catalog build regenerated from stale component jars; this PR's CI runs the test. _Claude Code on behalf of davsclaus_ 🤖 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]
