You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@camel.apache.org by "jamesnetherton (via GitHub)" <gi...@apache.org> on 2026/07/10 07:52:10 UTC

[I] Add test coverage for recent openai component enhancements [camel-quarkus]

jamesnetherton opened a new issue, #8858:
URL: https://github.com/apache/camel-quarkus/issues/8858

   Some features were added to the openai component recently. We should add coverage here, primarily to check that they work in native mode.
   
   
   **1. Test MCP tool integration / agentic loop for camel-openai**
   
   The `camel-openai` component supports an agentic loop via `processNonStreamingAgentic()` where the producer detects tool calls in chat completion responses, executes them via MCP servers, and loops back to the model. Related configuration options include `mcpServer.*`, `autoToolExecution`, `maxToolIterations`, `mcpTimeout`, `mcpReconnect`, and `mcpProtocolVersions`.
   
   **2. Audio transcription**
   
   See details here: https://camel.apache.org/components/next/others/openai-operations.html.
   
   I don't think we need to cover all of the options. We just a test that exercises the core feature(s).


-- 
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: commits-unsubscribe@camel.apache.org.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [I] Add test coverage for recent openai component enhancements [camel-quarkus]

Posted by "jomin7 (via GitHub)" <gi...@apache.org>.
jomin7 commented on issue #8858:
URL: https://github.com/apache/camel-quarkus/issues/8858#issuecomment-5255638170

   Hi, I’d be happy to look into this issue and see how I can help.
   


-- 
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: commits-unsubscribe@camel.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [I] Add test coverage for recent openai component enhancements [camel-quarkus]

Posted by "jomin7 (via GitHub)" <gi...@apache.org>.
jomin7 commented on issue #8858:
URL: https://github.com/apache/camel-quarkus/issues/8858#issuecomment-5291955933

   Hi @jamesnetherton, I'm planning to add tests for two areas: `tool call detection` (verifying the component correctly
     surfaces tool call objects when finish_reason is "tool_calls") and `audio transcription` (sending an audio file and
     verifying the transcribed text is returned). Both will include native mode IT classes and use WireMock stubs.
   
     One thing to note — the full agentic loop with MCP server integration won't be covered, as it would require a running
     MCP server which is difficult to set up in a mocked test environment. Would this level of coverage be sufficient, or
     would you like me to explore other approaches before raising a PR?


-- 
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: commits-unsubscribe@camel.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [I] Add test coverage for recent openai component enhancements [camel-quarkus]

Posted by "jamesnetherton (via GitHub)" <gi...@apache.org>.
jamesnetherton commented on issue #8858:
URL: https://github.com/apache/camel-quarkus/issues/8858#issuecomment-5292205614

   > MCP server which is difficult to set up in a mocked test environment
   
   `quarkus-mcp-server` is available in the project if that helps.


-- 
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: commits-unsubscribe@camel.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [I] Add test coverage for recent openai component enhancements [camel-quarkus]

Posted by "jamesnetherton (via GitHub)" <gi...@apache.org>.
jamesnetherton closed issue #8858: Add test coverage for recent openai component enhancements
URL: https://github.com/apache/camel-quarkus/issues/8858


-- 
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: commits-unsubscribe@camel.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org