Skip to content

Commit

Permalink
Merge pull request #7798 from radarhere/ico
Browse files Browse the repository at this point in the history
  • Loading branch information
hugovk committed Mar 11, 2024
2 parents e093663 + 21e5d5d commit f543b03
Show file tree
Hide file tree
Showing 2 changed files with 13 additions and 0 deletions.
11 changes: 11 additions & 0 deletions Tests/test_file_ico.py
Expand Up @@ -38,6 +38,17 @@ def test_black_and_white() -> None:
assert im.size == (16, 16)


def test_palette(tmp_path: Path) -> None:
temp_file = str(tmp_path / "temp.ico")

im = Image.new("P", (16, 16))
im.save(temp_file)

with Image.open(temp_file) as reloaded:
assert reloaded.mode == "P"
assert reloaded.palette is not None


def test_invalid_file() -> None:
with open("Tests/images/flower.jpg", "rb") as fp:
with pytest.raises(SyntaxError):
Expand Down
2 changes: 2 additions & 0 deletions src/PIL/IcoImagePlugin.py
Expand Up @@ -329,6 +329,8 @@ def load(self):
self.im = im.im
self.pyaccess = None
self._mode = im.mode
if im.palette:
self.palette = im.palette
if im.size != self.size:
warnings.warn("Image was not the expected size")

Expand Down

0 comments on commit f543b03

Please sign in to comment.