---
type: email
from: Kazuhiro Sera <notifications@github.com>
subject: Re: [openai/openai-agents-python] test(mcp): add runner-level

 coverage for upstream tool errors (PR #2554)
received: 2026-03-06 02:23:14
priority: high
status: awaiting_approval
---

## Email Content
seratch left a comment (openai/openai-agents-python#2554)



Thanks for adding coverage here. I don't think we need this PR as-is.



The MCP failure-handling behavior is already covered in `test_mcp_util`, and this runner-level test is fairly weak because it only proves the run continues to the next turn, not that the error is actually surfaced as tool output.



I'm going to close this for now. If we decide to add runner-level coverage later, I'd prefer a test that explicitly asserts the generated `tool_call_output_item`.



-- 

Reply to this email directly or view it on GitHub:

https://github.com/openai/openai-agents-python/pull/2554#issuecomment-3969736102

You are receiving this because you are subscribed to this thread.



Message ID: <openai/openai-agents-python/pull/2554/c3969736...

## Suggested Actions
- [ ] Reply
- [ ] Archive
- [ ] Flag for review
