Skip to content

Conversation

alexander-alderman-webb
Copy link
Contributor

@alexander-alderman-webb alexander-alderman-webb commented Sep 2, 2025

To test you can revert changes in #4736, or just flip the manual_span_cleanup argument. The OpenAI tests should then fail.

Closes #4723.
Closes PY-1816.

@alexander-alderman-webb alexander-alderman-webb requested a review from a team as a code owner September 2, 2025 12:22
@alexander-alderman-webb alexander-alderman-webb changed the title test: Trigger Pytest failure when an unraisable exception occurs tests: Trigger Pytest failure when an unraisable exception occurs Sep 2, 2025
@alexander-alderman-webb alexander-alderman-webb marked this pull request as draft September 2, 2025 12:28
Copy link

codecov bot commented Sep 2, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 84.92%. Comparing base (b1a8b63) to head (9c3f3bb).
⚠️ Report is 1 commits behind head on master.
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #4738   +/-   ##
=======================================
  Coverage   84.92%   84.92%           
=======================================
  Files         156      156           
  Lines       16115    16115           
  Branches     2741     2741           
=======================================
+ Hits        13685    13686    +1     
  Misses       1643     1643           
+ Partials      787      786    -1     

see 1 file with indirect coverage changes

@alexander-alderman-webb alexander-alderman-webb marked this pull request as ready for review September 2, 2025 12:47
@alexander-alderman-webb alexander-alderman-webb merged commit 9c4eb5e into master Sep 2, 2025
135 checks passed
@alexander-alderman-webb alexander-alderman-webb deleted the webb/error-on-unraisable-exception branch September 2, 2025 13:27
Copy link

linear bot commented Sep 2, 2025

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.

Hidden AttributeError in openai tests
2 participants