Handling invoke responses with empty body

[Rodrigo Brandão (rodrigobr-msft)]

This pull request improves the handling of empty response bodies for invoke activities in the Exchange.from_request method and adds tests to ensure correct behavior. The main changes are:

Bug Fixes:

• Updated Exchange.from_request in exchange.py to return None for the body field in InvokeResponse when the response body is empty or contains only whitespace, preventing JSON parsing errors.

Testing Improvements:

• Added a parameterized test test_from_request_with_empty_invoke_response_body in test_exchange.py to verify that empty or whitespace-only response bodies are correctly handled and result in None in invoke_response.body.

[Copilot]

Pull request overview

This pull request updates the transcript transport layer’s Exchange.from_request parsing for invoke activities to safely handle responses that have an empty/whitespace-only body, and adds a regression test to ensure this behavior remains correct.

Changes:

• Adjusted Exchange.from_request to treat empty/whitespace-only invoke response bodies as {} (instead of attempting json.loads and failing).
• Added a parameterized test to cover both empty-string and whitespace-only invoke response bodies.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File	Description
dev/testing/microsoft-agents-testing/microsoft_agents/testing/core/transport/transcript/exchange.py	Avoids JSON parsing on empty/whitespace invoke response bodies by defaulting InvokeResponse.body to {}.
dev/testing/microsoft-agents-testing/tests/core/transport/transcript/test_exchange.py	Adds parameterized coverage ensuring empty/whitespace invoke bodies yield invoke_response.body == {}.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[Copilot]

Pull request overview

Copilot reviewed 2 out of 2 changed files in this pull request and generated no new comments.

[Copilot]

Pull request overview

Copilot reviewed 2 out of 2 changed files in this pull request and generated no new comments.