Revert exception types to ValueError

This commit is contained in:
Piolie 2021-01-06 01:15:07 -03:00
parent 4dbe244e42
commit 5d0ad5e2e9
2 changed files with 7 additions and 7 deletions

View File

@ -64,7 +64,7 @@ def test_nondecimal_header(tmp_path):
with open(path, "wb") as f: with open(path, "wb") as f:
f.write(b"P6\n128\x00") f.write(b"P6\n128\x00")
with pytest.raises(UnidentifiedImageError): with pytest.raises(ValueError):
Image.open(path) Image.open(path)
@ -73,7 +73,7 @@ def test_truncated_file(tmp_path):
with open(path, "w") as f: with open(path, "w") as f:
f.write("P6") f.write("P6")
with pytest.raises(UnidentifiedImageError): with pytest.raises(ValueError):
Image.open(path) Image.open(path)
@ -83,7 +83,7 @@ def test_neg_ppm():
# has been removed. The default opener doesn't accept negative # has been removed. The default opener doesn't accept negative
# sizes. # sizes.
with pytest.raises(UnidentifiedImageError): with pytest.raises(OSError):
Image.open("Tests/images/negative_size.ppm") Image.open("Tests/images/negative_size.ppm")

View File

@ -73,7 +73,7 @@ class PpmImageFile(ImageFile.ImageFile):
continue continue
if c in B_WHITESPACE: # found whitespace, ignore it if c in B_WHITESPACE: # found whitespace, ignore it
if c == b"": # reached EOF if c == b"": # reached EOF
raise EOFError("Reached EOF while reading header") raise ValueError("Reached EOF while reading header")
continue continue
break break
@ -115,7 +115,7 @@ class PpmImageFile(ImageFile.ImageFile):
try: # check token sanity try: # check token sanity
token = int(token) token = int(token)
except ValueError: except ValueError:
raise SyntaxError("Non-decimal-ASCII found in header") raise ValueError(f"Non-decimal-ASCII found in header: {token}")
if ix == 0: # token is the x size if ix == 0: # token is the x size
xsize = token xsize = token
elif ix == 1: # token is the y size elif ix == 1: # token is the y size
@ -125,7 +125,7 @@ class PpmImageFile(ImageFile.ImageFile):
elif ix == 2: # token is maxval elif ix == 2: # token is maxval
if token > 255: if token > 255:
if not mode == "L": if not mode == "L":
raise SyntaxError(f"Too many colors for band: {token}") raise ValueError(f"Too many colors for band: {token}")
if token < 2 ** 16: if token < 2 ** 16:
self.mode = "I" self.mode = "I"
rawmode = "I;16B" rawmode = "I;16B"
@ -156,7 +156,7 @@ def _save(im, fp, filename):
elif im.mode == "RGBA": elif im.mode == "RGBA":
rawmode, head = "RGB", b"P6" rawmode, head = "RGB", b"P6"
else: else:
raise OSError(f"cannot write mode {im.mode} as PPM") raise OSError(f"Cannot write mode {im.mode} as PPM")
fp.write(head + ("\n%d %d\n" % im.size).encode("ascii")) fp.write(head + ("\n%d %d\n" % im.size).encode("ascii"))
if head == b"P6": if head == b"P6":
fp.write(b"255\n") fp.write(b"255\n")