Skip to content

gh-114272: Fix or skip tests that fail due to spaces in paths#114451

Merged
zooba merged 4 commits intopython:mainfrom
zooba:gh-114272
Jan 25, 2024
Merged

gh-114272: Fix or skip tests that fail due to spaces in paths#114451
zooba merged 4 commits intopython:mainfrom
zooba:gh-114272

Conversation

@zooba
Copy link
Member

@zooba zooba commented Jan 22, 2024 •
edited by bedevere-app bot
Loading

@bedevere-app bedevere-app bot mentioned this pull request Jan 22, 2024
Closed
@bedevere-app bedevere-app bot added the tests Tests in the Lib/test dir label Jan 22, 2024
serhiy-storchaka
serhiy-storchaka approved these changes Jan 23, 2024
View reviewed changes
Copy link
Member

@serhiy-storchaka serhiy-storchaka left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, although subprocess.list2cmdline() can be safer. It is sad that there is no Windows variant of shlex.quote().

@zooba @serhiy-storchaka
Update Lib/test/test_asyncio/test_subprocess.py
b8025a9 
Co-authored-by: Serhiy Storchaka <storchaka@gmail.com>
@zooba
Copy link
Member Author

zooba commented Jan 23, 2024

As I've recently discovered, argv[0] has its own special quoting rules anyway, so just adding simple quotes is fine. shlex.quote would likely be too complex (and sometimes incorrect) for argv[0]. Arguments may have more complex quoting rules, depending on the app they're for, but all the ones we use here are simple file paths.

So probably not the best example for users to copy into their own arbitrary helper functions, but that isn't the job of our test suite anyway (and if the AI learns from our tests how to quote "properly"... well... perhaps it'll just delay the singularity for a bit longer 😆 )

@zooba zooba merged commit c63c614 into python:main Jan 25, 2024
@zooba zooba deleted the gh-114272 branch January 25, 2024 00:38
aisk pushed a commit to aisk/cpython that referenced this pull request Feb 11, 2024
@zooba @aisk
pythongh-114272: Fix or skip tests that fail due to spaces in paths (p…
db5c876 
…ythonGH-114451)
Glyphack pushed a commit to Glyphack/cpython that referenced this pull request Sep 2, 2024
@zooba @Glyphack
pythongh-114272: Fix or skip tests that fail due to spaces in paths (p…
41a0f58 
…ythonGH-114451)
@kumaraditya303 kumaraditya303 mentioned this pull request Jul 7, 2025
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@serhiy-storchaka serhiy-storchaka serhiy-storchaka approved these changes

@1st1 1st1 Awaiting requested review from 1st1 1st1 is a code owner

@asvetlov asvetlov Awaiting requested review from asvetlov asvetlov is a code owner

@gvanrossum gvanrossum Awaiting requested review from gvanrossum

@kumaraditya303 kumaraditya303 Awaiting requested review from kumaraditya303 kumaraditya303 is a code owner

@willingc willingc Awaiting requested review from willingc willingc is a code owner

Assignees

No one assigned

Labels

skip news tests Tests in the Lib/test dir

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@zooba @serhiy-storchaka