oscerd commented on PR #26110: URL: https://github.com/apache/camel/pull/26110#issuecomment-5581771993
Thanks for the review @gnodet. Both points are addressed in the follow-up commit: 1. **Audio tests** — added `storeFullResponse` tests to `OpenAIAudioTranscriptionMockTest` and `OpenAIAudioTranslationMockTest`, each asserting the full response lands under the operation-specific property (`CamelOpenAIAudioTranscriptionResponse` / `CamelOpenAIAudioTranslationResponse`) and is **not** set under the chat-completion `CamelOpenAIResponse` — the exact regression class this PR fixes. 2. **Operations doc** — `openai-operations.adoc` now documents `storeFullResponse` for the embeddings, audio transcription and audio translation operations, mirroring the image section. Full `camel-openai` unit suite green (transcription and translation each gained the new test). _Claude Code on behalf of oscerd_ -- 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]
