apupier opened a new pull request, #29810:
URL: https://github.com/apache/camel/pull/29810

   `TextBodyFlowTest.testTheBodyOfMessaging` fails on `main` since #29753 
(CAMEL-26012, camel-pulsar: the producer no longer sends a body with Java 
serialization).
   
   **I do not understand the root cause.** This PR is created mainly to get 
things moving so the regression can be investigated, not because I am sure it 
is the right fix.
   
   What I observed: #29753 added `dependsOn = "allowJavaSerializedObject"` to 
the `producerIn` body type of `PulsarEndpoint`. The validator no longer reports 
`to: pulsar needs the body as text, bytes, a stream or a file` for parsed data 
sent to pulsar, so the test expecting 4 reports gets 3. My guess is that, since 
the option can also be set on the component, `BodyTypeResolver` now treats the 
body as "not known", but I have not verified whether that is the intended 
behaviour or whether the catalog entry / resolver should be changed instead.
   
   The change here only removes the pulsar expectation from the test (4 -> 3 
reports). If the real fix belongs in the component metadata or in the resolver, 
please feel free to close this PR in favour of it.
   
   _Claude Code (IBM Bob 2.2.1) on behalf of @apupier_
   


-- 
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]

Reply via email to