Skip to content

test(openai-agents): Replace mocks with httpx in non-error single-response tests#5600

Open
alexander-alderman-webb wants to merge 8 commits intomasterfrom
webb/openai-agents/non-error-single-response
Open

test(openai-agents): Replace mocks with httpx in non-error single-response tests#5600
alexander-alderman-webb wants to merge 8 commits intomasterfrom
webb/openai-agents/non-error-single-response

Conversation

@alexander-alderman-webb
Copy link
Contributor

@alexander-alderman-webb alexander-alderman-webb commented Mar 6, 2026

Description

Replace mocks with httpx types to avoid test failures when library internals change.

Issues

Reminders

@alexander-alderman-webb alexander-alderman-webb changed the title test(openai-agents): Replace mocks with httpx in non-error single-res… test(openai-agents): Replace mocks with httpx in non-error single-response tests Mar 6, 2026
@alexander-alderman-webb alexander-alderman-webb changed the title test(openai-agents): Replace mocks with httpx in non-error single-response tests test(openai-agents): Replace mocks with httpx in non-error single-response tests Mar 6, 2026
@github-actions
Copy link
Contributor

github-actions bot commented Mar 6, 2026

Codecov Results 📊

13 passed | Total: 13 | Pass Rate: 100% | Execution Time: 6.60s

All tests are passing successfully.

✅ Patch coverage is 100.00%. Project has 13797 uncovered lines.


Generated by Codecov Action

@github-actions
Copy link
Contributor

github-actions bot commented Mar 6, 2026

Semver Impact of This PR

🟢 Patch (bug fixes)

📋 Changelog Preview

This is how your changes will appear in the changelog.
Entries from this PR are highlighted with a left border (blockquote style).


Bug Fixes 🐛

  • (celery) Propagate user-set headers by sentrivana in #5581
  • (utils) Avoid double serialization of strings in safe_serialize by ericapisani in #5587

Documentation 📚

  • (openai-agents) Remove inapplicable comment by alexander-alderman-webb in #5495
  • Add AGENTS.md by sentrivana in #5579
  • Add set_attribute example to changelog by sentrivana in #5578

Internal Changes 🔧

Openai Agents

  • Replace mocks with httpx in non-error single-response tests by alexander-alderman-webb in #5600
  • Expect namespace tool field for new openai versions by alexander-alderman-webb in #5599

🤖 This preview updates automatically when you update the PR.

@alexander-alderman-webb alexander-alderman-webb marked this pull request as ready for review March 6, 2026 11:59
@alexander-alderman-webb alexander-alderman-webb requested a review from a team as a code owner March 6, 2026 11:59
Copy link

@cursor cursor bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants