wenjin272 commented on code in PR #447:
URL: https://github.com/apache/flink-agents/pull/447#discussion_r2781060056
##########
python/flink_agents/e2e_tests/e2e_tests_integration/e2e_tests_mcp/mcp_test.py:
##########
@@ -162,3 +226,36 @@ def test_mcp() -> None: # noqa:D103
print(f"{key}: {value}")
server_process.kill()
+
+
[email protected](
+ client is None, reason="Ollama client is not available or test model is
missing"
+)
+def test_mcp_without_prompts() -> None: # noqa:D103
Review Comment:
Could we merge `test_mcp_without_prompts` and `test_mcp`? I think the main
logic is same, the difference can be apply according to an environment
variable. You can see `test_chat_model_integration` as a reference.
--
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]
