Skip to content

gh-130655: Increase test coverage of gettext._expand_lang#130656

Merged
serhiy-storchaka merged 1 commit into
python:mainfrom
tomasr8:gettext-expand-lang
Feb 28, 2025
Merged

gh-130655: Increase test coverage of gettext._expand_lang#130656
serhiy-storchaka merged 1 commit into
python:mainfrom
tomasr8:gettext-expand-lang

Conversation

@tomasr8

@tomasr8 tomasr8 commented Feb 27, 2025 •
edited by bedevere-app Bot
Loading

Copy link
Copy Markdown
Member

This will help with properly testing gettext.find which calls _expand_lang.

With these tests, the function is fully covered:

obrazek

@tomasr8 tomasr8 added tests Tests in the Lib/test dir skip news needs backport to 3.12 only security fixes needs backport to 3.13 bugs and security fixes labels Feb 27, 2025
@bedevere-app bedevere-app Bot mentioned this pull request Feb 27, 2025
Open
serhiy-storchaka
serhiy-storchaka approved these changes Feb 28, 2025
View reviewed changes

@serhiy-storchaka serhiy-storchaka left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM. 👍

@serhiy-storchaka serhiy-storchaka merged commit 24c52cb into python:main Feb 28, 2025
@miss-islington-app

miss-islington-app Bot commented Feb 28, 2025

Copy link
Copy Markdown

Thanks @tomasr8 for the PR, and @serhiy-storchaka for merging it 🌮🎉.. I'm working now to backport this PR to: 3.12, 3.13.
🐍🍒⛏🤖 I'm not a witch! I'm not a witch!

miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Feb 28, 2025
@tomasr8 @miss-islington
pythongh-130655: Increase test coverage of gettext._expand_lang() (py…
535f12d 
…thonGH-130656)

(cherry picked from commit 24c52cb)

Co-authored-by: Tomas R. <tomas.roun8@gmail.com>
@bedevere-app

bedevere-app Bot commented Feb 28, 2025

Copy link
Copy Markdown

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

@bedevere-app bedevere-app Bot removed the needs backport to 3.13 bugs and security fixes label Feb 28, 2025
miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Feb 28, 2025
@tomasr8 @miss-islington
pythongh-130655: Increase test coverage of gettext._expand_lang() (py…
8a5be31 
…thonGH-130656)

(cherry picked from commit 24c52cb)

Co-authored-by: Tomas R. <tomas.roun8@gmail.com>
@bedevere-app

bedevere-app Bot commented Feb 28, 2025

Copy link
Copy Markdown

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

@bedevere-app bedevere-app Bot removed the needs backport to 3.12 only security fixes label Feb 28, 2025
@tomasr8 tomasr8 deleted the gettext-expand-lang branch February 28, 2025 08:44
serhiy-storchaka pushed a commit that referenced this pull request Feb 28, 2025
@miss-islington @tomasr8
[3.12] gh-130655: Increase test coverage of gettext._expand_lang() (G…
aa91a11 
…H-130656) (GH-130672)

(cherry picked from commit 24c52cb)

Co-authored-by: Tomas R <tomas.roun8@gmail.com>
serhiy-storchaka pushed a commit that referenced this pull request Feb 28, 2025
@miss-islington @tomasr8
[3.13] gh-130655: Increase test coverage of gettext._expand_lang() (G…
b223dc7 
…H-130656) (GH-130671)

(cherry picked from commit 24c52cb)

Co-authored-by: Tomas R <tomas.roun8@gmail.com>
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

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

@tomasr8 @serhiy-storchaka