◐ Shell
reader mode source ↗
Skip to content

gh-89392: Make test_decimal discoverable#106209

Merged
serhiy-storchaka merged 1 commit into
python:mainfrom
serhiy-storchaka:test_decimal
Jun 29, 2023
Merged

gh-89392: Make test_decimal discoverable#106209
serhiy-storchaka merged 1 commit into
python:mainfrom
serhiy-storchaka:test_decimal

Conversation

@serhiy-storchaka

@serhiy-storchaka serhiy-storchaka commented Jun 28, 2023 •
edited by bedevere-bot
Loading

Copy link
Copy Markdown
Member

It is very peculiar test file. It provides additional CLI interface and programming interface which accept options to control what to test what print during testing.

This PR is different from Zachary's patch for #64207. It only makes minimal changes necessary to make tests discoverable, use standard unittest methods to collect and run tests, and avoid changing the current behavior. The code of the module can be improved further, but this is out of the scope of this PR.

The two modified tests depended on the order in which the tests were ran.

@serhiy-storchaka serhiy-storchaka added Tests in the Lib/test dir skip news needs backport to 3.11 only security fixes needs backport to 3.12 only security fixes labels Jun 28, 2023
@serhiy-storchaka serhiy-storchaka requested a review from zware June 28, 2023 17:06
@bedevere-bot bedevere-bot mentioned this pull request Jun 28, 2023
Closed
@serhiy-storchaka serhiy-storchaka force-pushed the test_decimal branch 2 times, most recently from 6dbb8a5 to 9e7f09a Compare June 28, 2023 17:14
zware
zware approved these changes Jun 28, 2023
View reviewed changes

@zware zware 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.

Hide comment

LGTM. As you said, there's more cleanup to be done here, but splitting out just the discoverability fixes makes a lot of sense; I tried to do too much in my old attempt :)

@zware zware mentioned this pull request Jun 28, 2023
Closed
@erlend-aasland erlend-aasland mentioned this pull request Jun 29, 2023
Merged
erlend-aasland
erlend-aasland approved these changes Jun 29, 2023
View reviewed changes

@erlend-aasland erlend-aasland left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Hide comment

This is a very nice cleanup; thanks, Serhiy!

python -m test test_decimal -m "<regex>" -v now finally behaves as expected.

@serhiy-storchaka serhiy-storchaka merged commit 0e24499 into python:main Jun 29, 2023
@miss-islington

miss-islington commented Jun 29, 2023

Copy link
Copy Markdown
Contributor

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

@miss-islington

miss-islington commented Jun 29, 2023

Copy link
Copy Markdown
Contributor

Sorry @serhiy-storchaka, I had trouble checking out the 3.12 backport branch.
Please retry by removing and re-adding the "needs backport to 3.12" label.
Alternatively, you can backport using cherry_picker on the command line.
cherry_picker 0e24499129f3917b199a6d46fa33eeedd2c447fc 3.12

@serhiy-storchaka serhiy-storchaka deleted the test_decimal branch June 29, 2023 09:53
miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Jun 29, 2023
@serhiy-storchaka @miss-islington
pythongh-89392: Make test_decimal discoverable (pythonGH-106209) …
1f7fe73 
(cherry picked from commit 0e24499)

Co-authored-by: Serhiy Storchaka <storchaka@gmail.com>
@bedevere-bot

bedevere-bot commented Jun 29, 2023

Copy link
Copy Markdown

GH-106229 is a backport of this pull request to the 3.11 branch.

@bedevere-bot bedevere-bot removed the needs backport to 3.11 only security fixes label Jun 29, 2023
@serhiy-storchaka serhiy-storchaka removed the needs backport to 3.12 only security fixes label Jun 29, 2023
@serhiy-storchaka serhiy-storchaka added the needs backport to 3.12 only security fixes label Jun 29, 2023
@miss-islington

miss-islington commented Jun 29, 2023

Copy link
Copy Markdown
Contributor

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

miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Jun 29, 2023
@serhiy-storchaka @miss-islington
pythongh-89392: Make test_decimal discoverable (pythonGH-106209) …
132a20b 
(cherry picked from commit 0e24499)

Co-authored-by: Serhiy Storchaka <storchaka@gmail.com>
@bedevere-bot

bedevere-bot commented Jun 29, 2023

Copy link
Copy Markdown

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

@bedevere-bot bedevere-bot removed the needs backport to 3.12 only security fixes label Jun 29, 2023
@zware

zware commented Jun 29, 2023

Copy link
Copy Markdown
Member

Only took 10 years, but we got there! Thanks Serhiy :)

ambv pushed a commit that referenced this pull request Jul 5, 2023
@miss-islington @serhiy-storchaka
[3.12] gh-89392: Make test_decimal discoverable (GH-106209) (#106230) …
334b95b 
gh-89392: Make test_decimal discoverable (GH-106209)
(cherry picked from commit 0e24499)

Co-authored-by: Serhiy Storchaka <storchaka@gmail.com>
ambv pushed a commit that referenced this pull request Jul 5, 2023
@miss-islington @serhiy-storchaka
[3.11] gh-89392: Make test_decimal discoverable (GH-106209) (#106229) …
33fb895 
(cherry picked from commit 0e24499)

Co-authored-by: Serhiy Storchaka <storchaka@gmail.com>
@serhiy-storchaka serhiy-storchaka removed their assignment Jul 25, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@erlend-aasland erlend-aasland erlend-aasland approved these changes

@zware zware zware approved these changes

@rhettinger rhettinger Awaiting requested review from rhettinger

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.

5 participants

@serhiy-storchaka @miss-islington @bedevere-bot @zware @erlend-aasland