Repository navigation
Add a test to JSONRPCHandlerTest for sending a streaming message where multiple events are received from the server #52
Description
The existing JSONRPCHandlerTest only tests the case where one event is received when sending a streaming message (e.g., see testOnMessageStreamNewMessageSuccess and notice that only one event is enqueued by the server and received by the client).
We should also test the case where multiple events are received when sending a streaming message.
Activity
[-]Add a test for A2AClient#sendStreamingMessage where multiple events are received from the server[/-][+]Add a test to JSONRPCHandlerTest for sending a streaming message where multiple events are received from the server[/+]misysin commentedon Oct 2, 2025
Hi @fjuma , I’d like to take this one up. Can you assign this to me? Thanks :)
kabir commentedon Oct 2, 2025
@amithsourya done - thanks :-)
Add test for streaming with multiple events Fixes a2aproject#52
misysin commentedon Oct 4, 2025
Hey @kabir @fjuma 👋
PR #326 is up and ready for review!
Had a blast working on this and learned tons about the A2A protocol
Would love to keep contributing if there are other issues that could use some help
kabir commentedon Oct 9, 2025
@amithsourya That would be great :-) I've been focussed on a tricky problem the past few days, but will open some issues once I start looking at other stuff again. Or just comment on any open issue you feel like doing?
Add test for streaming with multiple events Fixes a2aproject#52
misysin commentedon Oct 9, 2025
@kabir Great! I'll check out the open issues.
Out of curiosity, what's the tricky problem you're working on? I love solving tough puzzles, so if there's any way I could contribute ideas or just follow along to learn, I'd be interested!
Also, is there a Slack/Discord/chat for the project? Would be great to be part of any ongoing discussions!
test: add test for streaming with multiple events (#326)
kabir commentedon Oct 10, 2025
@amithsourya the first part was this #331. Although in its final form it doesn't look like much, the journey to get here was quite eventful with each fix breaking something else :-D
I am working on another PR which tightens this up further to try to resolve the occasionally failing Kafka test
kabir commentedon Oct 10, 2025
Hi, @amithsourya the whole A2A project uses Discord for discussions. https://discord.gg/aGCW27RK We are in #sdk-java.
misysin commentedon Oct 10, 2025
Thanks a lot @kabir Just joined the Discord.
Appreciate you sharing the PR. It sounds like quite a ride 😄
I'll go through it and learn from how you tackled it
test: add test for streaming with multiple events (a2aproject#326)