Skip to content

Commit b6f90c4

Browse files
authored
Merge pull request #8325 from radarhere/defusedxml
Catch defusedxml warnings
2 parents d6cfebd + fed9168 commit b6f90c4

File tree

2 files changed

+18
-2
lines changed

2 files changed

+18
-2
lines changed

Tests/test_file_webp_metadata.py

Lines changed: 9 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -116,7 +116,15 @@ def test_read_no_exif() -> None:
116116
def test_getxmp() -> None:
117117
with Image.open("Tests/images/flower.webp") as im:
118118
assert "xmp" not in im.info
119-
assert im.getxmp() == {}
119+
if ElementTree is None:
120+
with pytest.warns(
121+
UserWarning,
122+
match="XMP data cannot be read without defusedxml dependency",
123+
):
124+
xmp = im.getxmp()
125+
else:
126+
xmp = im.getxmp()
127+
assert xmp == {}
120128

121129
with Image.open("Tests/images/flower2.webp") as im:
122130
if ElementTree is None:

Tests/test_image.py

Lines changed: 9 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -938,7 +938,15 @@ def test_exif_hide_offsets(self) -> None:
938938

939939
def test_empty_xmp(self) -> None:
940940
with Image.open("Tests/images/hopper.gif") as im:
941-
assert im.getxmp() == {}
941+
if ElementTree is None:
942+
with pytest.warns(
943+
UserWarning,
944+
match="XMP data cannot be read without defusedxml dependency",
945+
):
946+
xmp = im.getxmp()
947+
else:
948+
xmp = im.getxmp()
949+
assert xmp == {}
942950

943951
def test_getxmp_padded(self) -> None:
944952
im = Image.new("RGB", (1, 1))

0 commit comments

Comments
 (0)