Skip to content

Move end-to-end test HTTP server to a thread to support 3.14+ - #65

Merged
aiudirog merged 4 commits into
mainfrom
e2e-http-server-thread
Sep 20, 2026
Merged

aiudirog merged 4 commits into
mainfrom
e2e-http-server-thread

Conversation

@aiudirog

Copy link
Copy Markdown
Member

This is an alternative to #61 & #63 to instead move the testing HTTP server to a thread in the same process.

In addition to avoiding to multiprocessing migration from fork to spawn entirely, it also provides a few other benefits such as not needing to rebind the server port and proper shutdown of the server instead of just terminating the child process.

@aiudirog aiudirog self-assigned this Sep 20, 2026
@codecov-commenter

Copy link
Copy Markdown

⚠️ Please install the 'codecov app svg image' to ensure uploads and comments are reliably processed by Codecov.

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 95.48%. Comparing base (912c9ec) to head (f560ee7).
❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files
@@           Coverage Diff           @@
##             main      #65   +/-   ##
=======================================
  Coverage   95.48%   95.48%           
=======================================
  Files           3        3           
  Lines         155      155           
=======================================
  Hits          148      148           
  Misses          7        7           

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@aiudirog
aiudirog merged commit ef9a503 into main Sep 20, 2026
20 checks passed
@aiudirog
aiudirog deleted the e2e-http-server-thread branch September 29, 2026 01:48
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.

2 participants