Skip to content

gzip: add tests for memoryview with itemsize > 1 byte#141208

Open
dstndstn wants to merge 5 commits intopython:mainfrom
dstndstn:add-gzip-test-memoryview
Open

gzip: add tests for memoryview with itemsize > 1 byte#141208
dstndstn wants to merge 5 commits intopython:mainfrom
dstndstn:add-gzip-test-memoryview

Conversation

@dstndstn
Copy link
Copy Markdown

@dstndstn dstndstn commented Nov 7, 2025

This PR adds tests to test_gzip.py that an array with items larger than one byte correctly round-trips through GzipFile and gzip.compress -> gzip.decompress.

Python 3.12 broke this; these tests are to try to make sure that doesn't happen again.

…ly round-trips through GzipFile and gzip.compress -> gzip.decompress
@python-cla-bot
Copy link
Copy Markdown

python-cla-bot Bot commented Nov 7, 2025

All commit authors signed the Contributor License Agreement.

CLA signed

@bedevere-app bedevere-app Bot added awaiting review tests Tests in the Lib/test dir labels Nov 7, 2025
@bedevere-app
Copy link
Copy Markdown

bedevere-app Bot commented Nov 7, 2025

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.

@bedevere-app
Copy link
Copy Markdown

bedevere-app Bot commented Nov 7, 2025

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.

@dstndstn
Copy link
Copy Markdown
Author

dstndstn commented Nov 7, 2025

My attempt at filing a bug against Python3.12 was closed with wontfix: #141201

@bedevere-app
Copy link
Copy Markdown

bedevere-app Bot commented Nov 7, 2025

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.

@dstndstn
Copy link
Copy Markdown
Author

dstndstn commented Nov 7, 2025

the macOS CI failure looks unrelated (ssl).

@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 1, 2026

This PR is stale because it has been open for 30 days with no activity.

@github-actions github-actions Bot added the stale Stale PR or inactive for long period of time. label May 1, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

awaiting review skip issue skip news stale Stale PR or inactive for long period of time. tests Tests in the Lib/test dir

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants