autotyping: --annotate-named-param tmp_path:pathlib.Path

python-pillow · radarhere · Jan 31, 2024 · Jan 27, 2024 · Jan 27, 2024 · Jan 27, 2024
commit e3e83b9fab5fb3a61ec1db06541feaee5246da69
diff --git a/Tests/test_file_apng.py b/Tests/test_file_apng.py
@@ -1,5 +1,7 @@
 from __future__ import annotations
 
+from pathlib import Path
+
 import pytest
 
 from PIL import Image, ImageSequence, PngImagePlugin
@@ -343,7 +345,7 @@ def test_apng_sequence_errors(test_file) -> None:
             im.load()
 
 
-def test_apng_save(tmp_path) -> None:
+def test_apng_save(tmp_path: Path) -> None:
     with Image.open("Tests/images/apng/single_frame.png") as im:
         test_file = str(tmp_path / "temp.png")
         im.save(test_file, save_all=True)
@@ -374,7 +376,7 @@ def test_apng_save(tmp_path) -> None:
         assert im.getpixel((64, 32)) == (0, 255, 0, 255)
 
 
-def test_apng_save_alpha(tmp_path) -> None:
+def test_apng_save_alpha(tmp_path: Path) -> None:
     test_file = str(tmp_path / "temp.png")
 
     im = Image.new("RGBA", (1, 1), (255, 0, 0, 255))
@@ -388,7 +390,7 @@ def test_apng_save_alpha(tmp_path) -> None:
         assert reloaded.getpixel((0, 0)) == (255, 0, 0, 127)
 
 
-def test_apng_save_split_fdat(tmp_path) -> None:
+def test_apng_save_split_fdat(tmp_path: Path) -> None:
     # test to make sure we do not generate sequence errors when writing
     # frames with image data spanning multiple fdAT chunks (in this case
     # both the default image and first animation frame will span multiple
@@ -412,7 +414,7 @@ def test_apng_save_split_fdat(tmp_path) -> None:
         assert exception is None
 
 
-def test_apng_save_duration_loop(tmp_path) -> None:
+def test_apng_save_duration_loop(tmp_path: Path) -> None:
     test_file = str(tmp_path / "temp.png")
     with Image.open("Tests/images/apng/delay.png") as im:
         frames = []
@@ -475,7 +477,7 @@ def test_apng_save_duration_loop(tmp_path) -> None:
         assert im.info["duration"] == 600
 
 
-def test_apng_save_disposal(tmp_path) -> None:
+def test_apng_save_disposal(tmp_path: Path) -> None:
     test_file = str(tmp_path / "temp.png")
     size = (128, 64)
     red = Image.new("RGBA", size, (255, 0, 0, 255))
@@ -576,7 +578,7 @@ def test_apng_save_disposal(tmp_path) -> None:
         assert im.getpixel((64, 32)) == (0, 0, 0, 0)
 
 
-def test_apng_save_disposal_previous(tmp_path) -> None:
+def test_apng_save_disposal_previous(tmp_path: Path) -> None:
     test_file = str(tmp_path / "temp.png")
     size = (128, 64)
     blue = Image.new("RGBA", size, (0, 0, 255, 255))
@@ -598,7 +600,7 @@ def test_apng_save_disposal_previous(tmp_path) -> None:
         assert im.getpixel((64, 32)) == (0, 255, 0, 255)
 
 
-def test_apng_save_blend(tmp_path) -> None:
+def test_apng_save_blend(tmp_path: Path) -> None:
     test_file = str(tmp_path / "temp.png")
     size = (128, 64)
     red = Image.new("RGBA", size, (255, 0, 0, 255))
@@ -679,7 +681,7 @@ def test_seek_after_close() -> None:
 @pytest.mark.parametrize("default_image", (True, False))
 @pytest.mark.parametrize("duplicate", (True, False))
 def test_different_modes_in_later_frames(
-    mode, default_image, duplicate, tmp_path
+    mode, default_image, duplicate, tmp_path: Path
 ) -> None:
     test_file = str(tmp_path / "temp.png")
 

diff --git a/Tests/test_file_blp.py b/Tests/test_file_blp.py
@@ -1,5 +1,7 @@
 from __future__ import annotations
 
+from pathlib import Path
+
 import pytest
 
 from PIL import Image
@@ -35,7 +37,7 @@ def test_load_blp2_dxt1a() -> None:
         assert_image_equal_tofile(im, "Tests/images/blp/blp2_dxt1a.png")
 
 
-def test_save(tmp_path) -> None:
+def test_save(tmp_path: Path) -> None:
     f = str(tmp_path / "temp.blp")
 
     for version in ("BLP1", "BLP2"):

diff --git a/Tests/test_file_bmp.py b/Tests/test_file_bmp.py
@@ -1,6 +1,7 @@
 from __future__ import annotations
 
 import io
+from pathlib import Path
 
 import pytest
 
@@ -14,7 +15,7 @@
 )
 
 
-def test_sanity(tmp_path) -> None:
+def test_sanity(tmp_path: Path) -> None:
     def roundtrip(im) -> None:
         outfile = str(tmp_path / "temp.bmp")
 
@@ -60,7 +61,7 @@ def test_save_to_bytes() -> None:
         assert reloaded.format == "BMP"
 
 
-def test_small_palette(tmp_path) -> None:
+def test_small_palette(tmp_path: Path) -> None:
     im = Image.new("P", (1, 1))
     colors = [0, 0, 0, 125, 125, 125, 255, 255, 255]
     im.putpalette(colors)
@@ -72,7 +73,7 @@ def test_small_palette(tmp_path) -> None:
         assert reloaded.getpalette() == colors
 
 
-def test_save_too_large(tmp_path) -> None:
+def test_save_too_large(tmp_path: Path) -> None:
     outfile = str(tmp_path / "temp.bmp")
     with Image.new("RGB", (1, 1)) as im:
         im._size = (37838, 37838)
@@ -92,7 +93,7 @@ def test_dpi() -> None:
         assert reloaded.info["dpi"] == (72.008961115161, 72.008961115161)
 
 
-def test_save_bmp_with_dpi(tmp_path) -> None:
+def test_save_bmp_with_dpi(tmp_path: Path) -> None:
     # Test for #1301
     # Arrange
     outfile = str(tmp_path / "temp.jpg")
@@ -110,7 +111,7 @@ def test_save_bmp_with_dpi(tmp_path) -> None:
             assert reloaded.format == "JPEG"
 
 
-def test_save_float_dpi(tmp_path) -> None:
+def test_save_float_dpi(tmp_path: Path) -> None:
     outfile = str(tmp_path / "temp.bmp")
     with Image.open("Tests/images/hopper.bmp") as im:
         im.save(outfile, dpi=(72.21216100543306, 72.21216100543306))
@@ -127,7 +128,7 @@ def test_load_dib() -> None:
         assert_image_equal_tofile(im, "Tests/images/clipboard_target.png")
 
 
-def test_save_dib(tmp_path) -> None:
+def test_save_dib(tmp_path: Path) -> None:
     outfile = str(tmp_path / "temp.dib")
 
     with Image.open("Tests/images/clipboard.dib") as im:

diff --git a/Tests/test_file_bufrstub.py b/Tests/test_file_bufrstub.py
@@ -1,5 +1,7 @@
 from __future__ import annotations
 
+from pathlib import Path
+
 import pytest
 
 from PIL import BufrStubImagePlugin, Image
@@ -37,7 +39,7 @@ def test_load() -> None:
             im.load()
 
 
-def test_save(tmp_path) -> None:
+def test_save(tmp_path: Path) -> None:
     # Arrange
     im = hopper()
     tmpfile = str(tmp_path / "temp.bufr")
@@ -47,7 +49,7 @@ def test_save(tmp_path) -> None:
         im.save(tmpfile)
 
 
-def test_handler(tmp_path) -> None:
+def test_handler(tmp_path: Path) -> None:
     class TestHandler:
         opened = False
         loaded = False

diff --git a/Tests/test_file_dds.py b/Tests/test_file_dds.py
@@ -2,6 +2,7 @@
 from __future__ import annotations
 
 from io import BytesIO
+from pathlib import Path
 
 import pytest
 
@@ -363,7 +364,7 @@ def test_not_implemented(test_file) -> None:
             pass
 
 
-def test_save_unsupported_mode(tmp_path) -> None:
+def test_save_unsupported_mode(tmp_path: Path) -> None:
     out = str(tmp_path / "temp.dds")
     im = hopper("HSV")
     with pytest.raises(OSError):
@@ -379,7 +380,7 @@ def test_save_unsupported_mode(tmp_path) -> None:
         ("RGBA", "Tests/images/pil123rgba.png"),
     ],
 )
-def test_save(mode, test_file, tmp_path) -> None:
+def test_save(mode, test_file, tmp_path: Path) -> None:
     out = str(tmp_path / "temp.dds")
     with Image.open(test_file) as im:
         assert im.mode == mode

diff --git a/Tests/test_file_eps.py b/Tests/test_file_eps.py
@@ -1,6 +1,7 @@
 from __future__ import annotations
 
 import io
+from pathlib import Path
 
 import pytest
 
@@ -225,7 +226,7 @@ def test_transparency() -> None:
 
 
 @pytest.mark.skipif(not HAS_GHOSTSCRIPT, reason="Ghostscript not available")
-def test_file_object(tmp_path) -> None:
+def test_file_object(tmp_path: Path) -> None:
     # issue 479
     with Image.open(FILE1) as image1:
         with open(str(tmp_path / "temp.eps"), "wb") as fh:
@@ -251,7 +252,7 @@ def test_1_mode() -> None:
         assert im.mode == "1"
 
 
-def test_image_mode_not_supported(tmp_path) -> None:
+def test_image_mode_not_supported(tmp_path: Path) -> None:
     im = hopper("RGBA")
     tmpfile = str(tmp_path / "temp.eps")
     with pytest.raises(ValueError):
@@ -328,7 +329,7 @@ def test_read_binary_preview() -> None:
         pass
 
 
-def test_readline_psfile(tmp_path) -> None:
+def test_readline_psfile(tmp_path: Path) -> None:
     # check all the freaking line endings possible from the spec
     # test_string = u'something\r\nelse\n\rbaz\rbif\n'
     line_endings = ["\r\n", "\n", "\n\r", "\r"]