Skip to content

transport: add test case for zero second timeout #8452

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

dfawley
Copy link
Member

@dfawley dfawley commented Jul 15, 2025

(This includes the fix in #8451 in order to make the test not hang.)

RELEASE NOTES: none

@dfawley dfawley added this to the 1.75 Release milestone Jul 15, 2025
@dfawley dfawley requested a review from arjan-bal July 15, 2025 22:32
Copy link

codecov bot commented Jul 15, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 82.35%. Comparing base (bed551a) to head (fbee081).

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #8452      +/-   ##
==========================================
- Coverage   82.40%   82.35%   -0.06%     
==========================================
  Files         414      414              
  Lines       40436    40437       +1     
==========================================
- Hits        33322    33302      -20     
- Misses       5757     5775      +18     
- Partials     1357     1360       +3     
Files with missing lines Coverage Δ
internal/transport/http2_server.go 91.14% <100.00%> (+1.54%) ⬆️

... and 23 files with indirect coverage changes

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

@arjan-bal arjan-bal assigned dfawley and unassigned arjan-bal Jul 16, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants