Skip to content

bpo-38870: Use subTest in test_unparse for better error reporting#20141

Merged
pablogsal merged 4 commits into
python:masterfrom
pablogsal:unparse
May 17, 2020
Merged

bpo-38870: Use subTest in test_unparse for better error reporting#20141
pablogsal merged 4 commits into
python:masterfrom
pablogsal:unparse

Conversation

@pablogsal

@pablogsal pablogsal commented May 17, 2020

Copy link
Copy Markdown
Member

Comment thread Lib/test/test_unparse.py Outdated
Comment thread Lib/test/test_unparse.py
Comment thread Lib/test/test_unparse.py
Co-authored-by: Batuhan Taskaya <isidentical@gmail.com>

@isidentical isidentical 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 (since you marked raises= as resolved)

Comment thread Lib/test/test_unparse.py Outdated
@pablogsal

Copy link
Copy Markdown
Member Author

(since you marked raises= as resolved)

That was an error, it should have been removed :S

@isidentical

Copy link
Copy Markdown
Member

That was an error, it should have been removed :S

Ah, I see. Thanks

@pablogsal pablogsal merged commit 6341fc7 into python:master May 17, 2020
@pablogsal pablogsal deleted the unparse branch May 19, 2021 18:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

skip news tests Tests in the Lib/test dir

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants