Skip to content

Removed print statement from tests#6660

Merged
hugovk merged 1 commit intopython-pillow:mainfrom
radarhere:print
Oct 13, 2022
Merged

Removed print statement from tests#6660
hugovk merged 1 commit intopython-pillow:mainfrom
radarhere:print

Conversation

@radarhere
Copy link
Copy Markdown
Member

@radarhere radarhere commented Oct 13, 2022

The following test function

Pillow/Tests/test_imagemath.py

Lines 6 to 12 in f63cc58

def pixel(im):
if hasattr(im, "im"):
return f"{im.mode} {repr(im.getpixel((0, 0)))}"
else:
if isinstance(im, int):
return int(im) # hack to deal with booleans
print(im)

is used like so

Pillow/Tests/test_imagemath.py

Line 30 in f63cc58

assert pixel(ImageMath.eval("A+B", A=A, B=B)) == "I 3"

If the print statement is reached, then the assertion will fail anyway, so it is just debugging code. This PR removes it.

@radarhere radarhere changed the title Removed print statement Removed print statement from tests Oct 13, 2022
@radarhere radarhere mentioned this pull request Oct 13, 2022
Merged
@hugovk hugovk merged commit 44510d2 into python-pillow:main Oct 13, 2022
@hugovk hugovk added the Cleanup label Oct 13, 2022
@radarhere radarhere deleted the print branch October 13, 2022 09:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

Cleanup Testing

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@radarhere @hugovk