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

   ## Summary
   
   _Claude Code on behalf of Andrea Cosentino_
   
   - Fix bug in `InlineQueryResultCachedVoice.Builder.voiceFileId()` where 
`this.id` was assigned instead of `this.voiceFileId`, causing voiceFileId to 
always be null when using the builder
   - Add unit tests verifying builder assignment, id independence, JSON 
serialization, and deserialization
   
   ## Root Cause
   
   In `InlineQueryResultCachedVoice.java` line 72, the builder method was:
   ```java
   public Builder voiceFileId(String voiceFileId) {
       this.id = voiceFileId;  // BUG: overwrites id, leaves voiceFileId null
       return this;
   }
   ```
   
   All other `InlineQueryResultCached*` classes correctly assign their file ID 
fields (e.g., `this.gifFileId = gifFileId`). This was a copy-paste error from 
the original CAMEL-13224 commit.
   
   ## Test plan
   
   - [x] New `InlineQueryResultCachedVoiceTest` with 4 tests covering builder, 
id independence, JSON serialization, and deserialization
   - [x] All 93 unit tests pass, 0 failures
   - [x] Integration tests compile and are skipped (require Telegram API token)
   
   🤖 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]

Reply via email to