Skip to content

Commit 83125b5

Browse files
authored
Checks over exif instead of reloeaded_exif
test_imagefile.py checks exif instead of reloaded_exif; making the tests pass when they should not.
1 parent 5fc0e96 commit 83125b5

1 file changed

Lines changed: 5 additions & 5 deletions

File tree

Tests/test_imagefile.py

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -261,9 +261,9 @@ def test_exif_jpeg(self, tmp_path):
261261
with Image.open(out) as reloaded:
262262
reloaded_exif = reloaded.getexif()
263263
assert reloaded_exif[258] == 8
264-
assert 40960 not in exif
264+
assert 40960 not in reloaded_exif
265265
assert reloaded_exif[40963] == 455
266-
assert exif[11] == "Pillow test"
266+
assert reloaded_exif[11] == "Pillow test"
267267

268268
with Image.open("Tests/images/no-dpi-in-exif.jpg") as im: # Big endian
269269
exif = im.getexif()
@@ -281,9 +281,9 @@ def test_exif_jpeg(self, tmp_path):
281281
with Image.open(out) as reloaded:
282282
reloaded_exif = reloaded.getexif()
283283
assert reloaded_exif[258] == 8
284-
assert 40960 not in exif
284+
assert 34665 not in reloaded_exif
285285
assert reloaded_exif[40963] == 455
286-
assert exif[305] == "Pillow test"
286+
assert reloaded_exif[305] == "Pillow test"
287287

288288
@skip_unless_feature("webp")
289289
@skip_unless_feature("webp_anim")
@@ -302,7 +302,7 @@ def check_exif():
302302
reloaded_exif = reloaded.getexif()
303303
assert reloaded_exif[258] == 8
304304
assert reloaded_exif[40963] == 455
305-
assert exif[305] == "Pillow test"
305+
assert reloaded_exif[305] == "Pillow test"
306306

307307
im.save(out, exif=exif)
308308
check_exif()

0 commit comments

Comments
 (0)