Skip to content

bpo-41620: TestCase.run() now always return a TestResult instance#28030

Merged
ambv merged 1 commit intopython:mainfrom
serhiy-storchaka:unittest-skip-return-result
Aug 30, 2021
Merged

bpo-41620: TestCase.run() now always return a TestResult instance#28030
ambv merged 1 commit intopython:mainfrom
serhiy-storchaka:unittest-skip-return-result

Conversation

@serhiy-storchaka
Copy link
Copy Markdown
Member

@serhiy-storchaka serhiy-storchaka commented Aug 29, 2021

Previously it returned None if the test class or method was
decorated with a skipping decorator.

https://bugs.python.org/issue41620

Previously it returned None if the test class or method was
decorated with a skipping decorator.
Copy link
Copy Markdown
Member

@iritkatriel iritkatriel left a comment

Choose a reason for hiding this comment

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

LGTM. Would be nice to include Iman as co-author.

@ambv ambv merged commit 7e246a3 into python:main Aug 30, 2021
@miss-islington
Copy link
Copy Markdown
Contributor

Thanks @serhiy-storchaka for the PR, and @ambv for merging it 🌮🎉.. I'm working now to backport this PR to: 3.9, 3.10.
🐍🍒⛏🤖

miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Aug 30, 2021
…thonGH-28030)

Previously it returned None if the test class or method was
decorated with a skipping decorator.

Co-authored-by: Iman Tabrizian <iman.tabrizian@gmail.com>
(cherry picked from commit 7e246a3)

Co-authored-by: Serhiy Storchaka <storchaka@gmail.com>
@bedevere-bot bedevere-bot removed the needs backport to 3.10 only security fixes label Aug 30, 2021
@bedevere-bot
Copy link
Copy Markdown

GH-28064 is a backport of this pull request to the 3.10 branch.

@bedevere-bot
Copy link
Copy Markdown

GH-28065 is a backport of this pull request to the 3.9 branch.

@ambv
Copy link
Copy Markdown
Contributor

ambv commented Aug 30, 2021

I credited Iman in the commit message.

miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Aug 30, 2021
…thonGH-28030)

Previously it returned None if the test class or method was
decorated with a skipping decorator.

Co-authored-by: Iman Tabrizian <iman.tabrizian@gmail.com>
(cherry picked from commit 7e246a3)

Co-authored-by: Serhiy Storchaka <storchaka@gmail.com>
ambv pushed a commit that referenced this pull request Aug 30, 2021
…-28030) (GH-28065)

Previously it returned None if the test class or method was
decorated with a skipping decorator.

Co-authored-by: Iman Tabrizian <iman.tabrizian@gmail.com>
(cherry picked from commit 7e246a3)

Co-authored-by: Serhiy Storchaka <storchaka@gmail.com>
miss-islington added a commit that referenced this pull request Aug 30, 2021
…-28030)

Previously it returned None if the test class or method was
decorated with a skipping decorator.

Co-authored-by: Iman Tabrizian <iman.tabrizian@gmail.com>
(cherry picked from commit 7e246a3)

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

Labels

type-bug An unexpected behavior, bug, or error

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants