Brijesh-Thakkar opened a new pull request, #26704:
URL: https://github.com/apache/camel/pull/26704

   ## Description
   
   Fixes `CAMEL-24868` by correcting the Backlog Debugger removal test.
   
   The `BacklogDebuggerTest#testBacklogDebuggerRemoveBodyAndHeader` test was 
failing while verifying that message data was removed at the `foo` breakpoint.
   
   The test contained assertions checking that the message body, message 
header, and exchange property had been removed, but the corresponding Backlog 
Debugger removal operations were not being invoked before the verification. 
This made the test inconsistent with the behavior it was intended to validate.
   
   This change restores the Backlog Debugger operations that remove the message 
body, message header, and exchange property before dumping and verifying the 
traced message at the `foo` breakpoint.
   
   ### Changes
   
   - Restored `removeMessageBodyOnBreakpoint` for the `foo` breakpoint.
   - Restored `removeMessageHeaderOnBreakpoint` for the `foo` breakpoint and 
`beer` header.
   - Restored `removeExchangePropertyOnBreakpoint` for the `foo` breakpoint and 
`food` property.
   - Added verification of the traced XML at the `foo` breakpoint.
   - Verified that:
     - the `foo` node is present;
     - message headers are removed;
     - the `food` exchange property is removed;
     - the message body is empty.
   
   The change is limited to the Backlog Debugger test and does not modify the 
production Backlog Debugger implementation.
   
   ### Testing
   
   The affected Backlog Debugger test was executed to reproduce and verify the 
failure/fix.
   
   The full Maven build was also attempted from the repository root using:
   
       mvn clean install -DskipTests
   
   Maven itself was initially unavailable, so Maven 3.9.12 was installed 
locally before rerunning the build.
   
   The root build progressed through the initial modules but stopped at 
`camel-tooling-util` because the active Java runtime did not support the Java 
17 release target:
   
       error: release version 17 not supported
   
   Therefore, the full root build could not be completed successfully in the 
current environment. This is an environment/JDK configuration issue rather than 
a test failure.
   
   ### AI Assistance
   
   AI assistance was used during development through OpenCode with the Nemotron 
3.5 model.
   
   ### Checklist
   
   - [ ] I have run `mvn clean install -DskipTests` locally from root folder 
and I have committed all auto-generated changes.


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