making testing better

[RoxyFarhad]

The diff looks big but it isn't:

1. the large diff comes from re-factoring the non-streaming polling to break out of the loops when the expected messages have been processed. It essentially moves the same code block into the message iterator rather than waiting for an expected time and then processing all received messages
2. added temporal tests to the testing workflow
3. fixed some small bugs with temporal testing

[socket-security]

Review the following changes in direct dependencies. Learn more about Socket for GitHub.

Diff	Package	Supply Chain Security	Vulnerability	Quality	Maintenance	License
	agentex-sdk@​0.6.7 ⏵ 0.8.0	+14

View full report

[smoreinis]

LG, I'm still not totally clear on what the actual behavior change is when it looks like we were already breaking out of the poll_messages loop when we found the expected messages but I assume I'm missing something here about why this didn't work the way I'm reading it but approving this for now and you can help me understand this when you get a chance.

I think there's also a lint error in 050_agent_chat_guardrails in case you didn't see it.

[RoxyFarhad]

LG, I'm still not totally clear on what the actual behavior change is when it looks like we were already breaking out of the poll_messages loop when we found the expected messages but I assume I'm missing something here about why this didn't work the way I'm reading it but approving this for now and you can help me understand this when you get a chance.

@smoreinis reverted a lot of these changes back because you were right -> the only thing I added in some of the tests were the early breaking if the messages were all discovered which has sped up some tests