test: More integration tests of autonomous agents#1512
test: More integration tests of autonomous agents#1512pvlugter merged 1 commit intomulti-agent-devfrom
Conversation
| "send_message factory method not yet implemented in AutonomousAgentTools — see team" | ||
| + " discussion") | ||
| public void shouldSendMessageBetweenTeamMembers() { | ||
| // Placeholder: test sendMessage(String to, String message) once the factory method is added |
There was a problem hiding this comment.
How shall we support send_message in the TestModelProvider? It should take an agent instance id if I understand it correctly. How would you know what id to use from the test? That's a generated uuid.
There was a problem hiding this comment.
I think we may need to parse it from the context messages, which gives the contact list. We could create helpers to do this in some way.
* cover usages of all tool calls provided by TestModelProvider
pvlugter
force-pushed
the
wip-aa-more-tests-patriknw
branch
from
33d2928 to
df4ac4a
Compare
|
|
||
| @Test | ||
| public void shouldGetAgentState() { | ||
| testAgentModel.fixedResponse(completeTask("{\"value\":\"done\",\"score\":1}")); |
There was a problem hiding this comment.
I think I already added a variation on complete task that can take an object so that it can be serialized to JSON automatically and the integration test can feel more realistic.
| "send_message factory method not yet implemented in AutonomousAgentTools — see team" | ||
| + " discussion") | ||
| public void shouldSendMessageBetweenTeamMembers() { | ||
| // Placeholder: test sendMessage(String to, String message) once the factory method is added |
There was a problem hiding this comment.
I think we may need to parse it from the context messages, which gives the contact list. We could create helpers to do this in some way.
2 participants
On top of #1511