Skip to content
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

GH-106160: Fix test_gzip failing under WASI, which does not have zlib. #106167

Merged
merged 2 commits into from
Jun 28, 2023

Conversation

Yhg1s
Copy link
Member

@Yhg1s Yhg1s commented Jun 28, 2023

Fix test_gzip by using test.support.import_helper.import_module to import zlib. (gzip unconditionally imports zlib, so this does not cause any new skips.)

test.support.import_helper.import_module to import zlib. (gzip
unconditionally imports zlib, so this does not cause any new skips.)
@Yhg1s Yhg1s enabled auto-merge (squash) June 28, 2023 00:32
@Yhg1s Yhg1s merged commit 161012f into python:main Jun 28, 2023
@miss-islington
Copy link
Contributor

Thanks @Yhg1s for the PR 🌮🎉.. I'm working now to backport this PR to: 3.12.
🐍🍒⛏🤖

@bedevere-bot
Copy link

GH-106170 is a backport of this pull request to the 3.12 branch.

@bedevere-bot bedevere-bot removed the needs backport to 3.12 bug and security fixes label Jun 28, 2023
miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Jun 28, 2023
…e zlib. (pythonGH-106167)

Fix test_gzip's failure under WASI, which does not have zlib, by using
test.support.import_helper.import_module to import zlib. (gzip
unconditionally imports zlib, so this does not cause any new skips.)
(cherry picked from commit 161012f)

Co-authored-by: T. Wouters <[email protected]>
kumaraditya303 pushed a commit that referenced this pull request Jun 28, 2023
…ve zlib. (GH-106167) (#106170)

GH-106160: Fix test_gzip failing under WASI, which does not have zlib. (GH-106167)

Fix test_gzip's failure under WASI, which does not have zlib, by using
test.support.import_helper.import_module to import zlib. (gzip
unconditionally imports zlib, so this does not cause any new skips.)
(cherry picked from commit 161012f)

Co-authored-by: T. Wouters <[email protected]>
@Yhg1s Yhg1s deleted the wasi_zlib branch November 22, 2023 15:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
skip news tests Tests in the Lib/test dir
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants