Skip to content

gh-150313: Add tests for computed flow control write limits#150314

Closed
azibom wants to merge 2 commits into
python:mainfrom
azibom:asyncio-flowcontrol-tests
Closed

gh-150313: Add tests for computed flow control write limits#150314
azibom wants to merge 2 commits into
python:mainfrom
azibom:asyncio-flowcontrol-tests

Conversation

@azibom
Copy link
Copy Markdown

@azibom azibom commented May 23, 2026

Adds test coverage for computed write buffer limit behavior in
asyncio._FlowControlMixin.

The new tests verify:

  • default computed limits: (16 * 1024, 64 * 1024)
  • high computed from low
  • low computed from high

These behaviors are implemented in _set_write_buffer_limits()
but were not explicitly covered by tests.

Tests help protect against future regressions in flow control
buffer calculations.

Closes gh-150313.

@bedevere-app bedevere-app Bot added the tests Tests in the Lib/test dir label May 23, 2026
@bedevere-app
Copy link
Copy Markdown

bedevere-app Bot commented May 23, 2026

Most changes to Python require a NEWS entry. Add one using the blurb_it web app or the blurb command-line tool.

If this change has little impact on Python users, wait for a maintainer to apply the skip news label instead.

@python-cla-bot
Copy link
Copy Markdown

The following commit authors need to sign the Contributor License Agreement:

CLA not signed

@bedevere-app
Copy link
Copy Markdown

bedevere-app Bot commented May 23, 2026

Most changes to Python require a NEWS entry. Add one using the blurb_it web app or the blurb command-line tool.

If this change has little impact on Python users, wait for a maintainer to apply the skip news label instead.

@azibom azibom closed this May 23, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

awaiting review tests Tests in the Lib/test dir

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add tests for computed write buffer limits in asyncio FlowControlMixin

1 participant