[tests] Mark tests inconclusive on premature response termination. Fixes #25873#25875
[tests] Mark tests inconclusive on premature response termination. Fixes #25873#25875rolfbjarne wants to merge 1 commit into
Conversation
… the response
Add IgnoreInCIIfResponseEndedPrematurely to handle HttpIOException
("response ended prematurely") in IgnoreInCIIfBadNetwork. This marks
such tests as Inconclusive in CI instead of failing, since premature
response termination from external services like httpbin.org is a
network issue, not a test failure.
Fixes #25873
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
✅ [PR Build #e40e95e] Build passed (Build packages) ✅Pipeline on Agent |
This comment has been minimized.
This comment has been minimized.
✅ [PR Build #e40e95e] Build passed (Detect API changes) ✅Pipeline on Agent |
✅ [PR Build #e40e95e] Build passed (Build macOS tests) ✅Pipeline on Agent |
✅ API diff for current PR / commitNET (empty diffs)✅ API diff vs stableNET (empty diffs)ℹ️ Generator diffGenerator Diff: vsdrops (html) vsdrops (raw diff) gist (raw diff) - Please review changes) Pipeline on Agent |
🔥 [CI Build #e40e95e] Test results 🔥Test results❌ Tests failed on VSTS: test results 1 tests crashed, 0 tests failed, 194 tests passed. Failures❌ generator tests🔥 Failed catastrophically on VSTS: test results - generator (no summary found). Html Report (VSDrops) Download Successes✅ assembly-processing: All 1 tests passed. Html Report (VSDrops) Download macOS tests✅ Tests on macOS Monterey (12): All 5 tests passed. Html Report (VSDrops) Download Linux Build VerificationPipeline on Agent |
2 participants
The
MessageHandlerTest.AcceptSslCertificatesServicePointManagertest intermittently fails in CI because httpbin.org drops the TCP connection mid-response, producing anHttpIOException: The response ended prematurely. The existingIgnoreInCIIfBadNetworkhelper didn't handle this exception type, so the test was reported as a hard failure rather than being marked inconclusive.This adds an
IgnoreInCIIfResponseEndedPrematurelycheck that catchesHttpIOExceptionand marks the testInconclusivewhen running in CI, matching the pattern used for other transient network errors (DNS failures, timeouts, connection lost).Fixes #25873
🤖 Pull request created by Copilot