Skip to content

Conversation

@savannahostrowski
Copy link
Member

@savannahostrowskisavannahostrowski commented Oct 12, 2025

The TestOptionalsHelpVersionActions test class is missing the @force_not_colorized_test_class decorator, causing test failures when running in terminals with TERM set and stdout as a TTY, as the tests expect plain text output.

I think this was missed in #136809

@bedevere-appbedevere-appbot added tests Tests in the Lib/test dir awaiting core review labels Oct 12, 2025
@savannahostrowskisavannahostrowski changed the title GH-139979: Add decorator to prevent colorGH-139979: Add @force_not_colorized_test_class to TestOptionalHelpVersionActionsOct 12, 2025
Copy link
Member

@serhiy-storchakaserhiy-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. 👍

@savannahostrowskisavannahostrowski merged commit 0fc5c56 into python:mainOct 12, 2025
59 checks passed
@miss-islington-app
Copy link

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

miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Oct 12, 2025
…elpVersionActions (pythonGH-139980) (cherry picked from commit 0fc5c56) Co-authored-by: Savannah Ostrowski <[email protected]>
@bedevere-app
Copy link

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

@bedevere-appbedevere-appbot removed the needs backport to 3.14 bugs and security fixes label Oct 12, 2025
savannahostrowski added a commit that referenced this pull request Oct 12, 2025
…HelpVersionActions (GH-139980) (#140005) GH-139979: Add @force_not_colorized_test_class to TestOptionalHelpVersionActions (GH-139980) (cherry picked from commit 0fc5c56) Co-authored-by: Savannah Ostrowski <[email protected]>
@savannahostrowskisavannahostrowski deleted the add-color-decorator branch December 4, 2025 22:58
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

skip newstestsTests in the Lib/test dir

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@savannahostrowski@hugovk@serhiy-storchaka