X Tutup
Skip to content

test(openai-agents): Set streaming header when library uses with_streaming_response()#5583

Merged
alexander-alderman-webb merged 6 commits intomasterfrom
webb/pass-streamed-header
Mar 9, 2026
Merged

test(openai-agents): Set streaming header when library uses with_streaming_response()#5583
alexander-alderman-webb merged 6 commits intomasterfrom
webb/pass-streamed-header

Conversation

@alexander-alderman-webb
Copy link
Contributor

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

Description

Addresses test failures in #5574

Issues

Reminders

@alexander-alderman-webb alexander-alderman-webb requested a review from a team as a code owner March 3, 2026 14:20
@github-actions
Copy link
Contributor

github-actions bot commented Mar 3, 2026 •
edited
Loading

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

  • Set streaming header when library uses with_streaming_response() by alexander-alderman-webb in #5583
  • Replace mocks with httpx for streamed responses by alexander-alderman-webb in #5580
  • Replace mocks with httpx in non-MCP tool tests by alexander-alderman-webb in #5602
  • Replace mocks with httpx in MCP tool tests by alexander-alderman-webb in #5605
  • Replace mocks with httpx in handoff tests by alexander-alderman-webb in #5604
  • Replace mocks with httpx in API error test by alexander-alderman-webb in #5601
  • Replace mocks with httpx in non-error single-response tests by alexander-alderman-webb in #5600
  • Remove test for unreachable state by alexander-alderman-webb in #5584
  • Expect namespace tool field for new openai versions by alexander-alderman-webb in #5599

Other

  • Do not run actions on potel-base by sentrivana in #5614

🤖 This preview updates automatically when you update the PR.

@github-actions
Copy link
Contributor

github-actions bot commented Mar 3, 2026 •
edited
Loading

Codecov Results 📊

✅ 31 passed | Total: 31 | Pass Rate: 100% | Execution Time: 10.12s

📊 Comparison with Base Branch

Metric Change
Total Tests 📉 -33
Passed Tests 📉 -33
Failed Tests —
Skipped Tests —

All tests are passing successfully.

✅ Patch coverage is 100.00%. Project has 14491 uncovered lines.
✅ Project coverage is 27.15%. Comparing base (base) to head (head).

Coverage diff 
@@            Coverage Diff             @@
##          main       #PR       +/-##
==========================================
+ Coverage    26.46%    27.15%    +0.69%
==========================================
  Files          189       189         —
  Lines        20016     19891      -125
  Branches      6476      6468        -8
==========================================
+ Hits          5297      5400      +103
- Misses       14719     14491      -228
- Partials       457       488       +31

Generated by Codecov Action

Base automatically changed from webb/remove-streamed-mocks to master March 9, 2026 12:18
@alexander-alderman-webb alexander-alderman-webb merged commit a04ba6e into master Mar 9, 2026
160 checks passed
@alexander-alderman-webb alexander-alderman-webb deleted the webb/pass-streamed-header branch March 9, 2026 13:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@ericapisani ericapisani ericapisani approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@alexander-alderman-webb @ericapisani
X Tutup