Skip to content

Do not break long test names in the regrtest output - #155714

Merged
serhiy-storchaka merged 1 commit into
python:mainfrom
serhiy-storchaka:regrtest-printlist-wrap
Aug 13, 2026
Merged

Do not break long test names in the regrtest output#155714
serhiy-storchaka merged 1 commit into
python:mainfrom
serhiy-storchaka:regrtest-printlist-wrap

Conversation

@serhiy-storchaka

@serhiy-storchaka serhiy-storchaka commented Aug 13, 2026

Copy link
Copy Markdown
Member

printlist() passes the joined test names to textwrap.fill(), which breaks words longer than the line width. Test case identifiers are usually longer:

23 tests failed:
    test.test_os.test_posix.PosixTester.test_getgroups test.test_pyrep
    l.test_pyrepl.TestPyReplModuleCompleter.test_attribute_completion

A line break which replaced a space is indistinguishable from one inside a name, so the names cannot even be read back.

@serhiy-storchaka
Do not break long test names in the regrtest output
9cea274 
printlist() used textwrap.fill(), which breaks words longer than the
line width.
@serhiy-storchaka serhiy-storchaka added needs backport to 3.13 bugs and security fixes needs backport to 3.14 bugs and security fixes needs backport to 3.15 pre-release feature fixes, bugs and security fixes labels Aug 13, 2026
@serhiy-storchaka serhiy-storchaka added skip news tests Tests in the Lib/test dir skip issue labels Aug 13, 2026
@serhiy-storchaka
serhiy-storchaka merged commit ceb3518 into python:main Aug 13, 2026
71 checks passed
@miss-islington-app

miss-islington-app Bot commented Aug 13, 2026

Copy link
Copy Markdown

Thanks @serhiy-storchaka for the PR 🌮🎉.. I'm working now to backport this PR to: 3.13, 3.14, 3.15.
🐍🍒⛏🤖

@bedevere-app

bedevere-app Bot commented Aug 13, 2026

Copy link
Copy Markdown

GH-155722 is a backport of this pull request to the 3.15 branch.

@bedevere-app bedevere-app Bot removed the needs backport to 3.15 pre-release feature fixes, bugs and security fixes label Aug 13, 2026
@bedevere-app

bedevere-app Bot commented Aug 13, 2026

Copy link
Copy Markdown

GH-155723 is a backport of this pull request to the 3.14 branch.

@bedevere-app bedevere-app Bot removed the needs backport to 3.14 bugs and security fixes label Aug 13, 2026
@bedevere-app

bedevere-app Bot commented Aug 13, 2026

Copy link
Copy Markdown

GH-155724 is a backport of this pull request to the 3.13 branch.

@serhiy-storchaka
serhiy-storchaka deleted the regrtest-printlist-wrap branch August 13, 2026 15:08
@bedevere-app bedevere-app Bot removed the needs backport to 3.13 bugs and security fixes label Aug 13, 2026
serhiy-storchaka added a commit that referenced this pull request Aug 13, 2026
@miss-islington @serhiy-storchaka
[3.13] Do not break long test names in the regrtest output (GH-155714) (
412feb8 
GH-155724)

printlist() used textwrap.fill(), which breaks words longer than the
line width.
(cherry picked from commit ceb3518)

Co-authored-by: Serhiy Storchaka <storchaka@gmail.com>
serhiy-storchaka added a commit that referenced this pull request Aug 13, 2026
@miss-islington @serhiy-storchaka
[3.14] Do not break long test names in the regrtest output (GH-155714) (
65fa1a0 
GH-155723)

printlist() used textwrap.fill(), which breaks words longer than the
line width.
(cherry picked from commit ceb3518)

Co-authored-by: Serhiy Storchaka <storchaka@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

skip issue 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.

1 participant

@serhiy-storchaka