TST: Fix side effect #2379 (#2395)

Issue due to unclosure of temporary file

Closes #2394

tests/test_writer.py

@@ -224,7 +224,8 @@ def writer_operate(writer: PdfWriter) -> None:
 )
 def test_writer_operations_by_traditional_usage(convert, needs_cleanup):
     if callable(convert):
-        write_data_here = convert(NamedTemporaryFile(suffix=".pdf", delete=False).name)
+        with NamedTemporaryFile(suffix=".pdf", delete=False) as fo:
+            write_data_here = convert(fo.name)
     else:
         write_data_here = convert
 
@@ -254,7 +255,8 @@ def test_writer_operations_by_traditional_usage(convert, needs_cleanup):
 )
 def test_writer_operations_by_semi_traditional_usage(convert, needs_cleanup):
     if callable(convert):
-        write_data_here = convert(NamedTemporaryFile(suffix=".pdf", delete=False).name)
+        with NamedTemporaryFile(suffix=".pdf", delete=False) as fo:
+            write_data_here = convert(fo.name)
     else:
         write_data_here = convert
 
@@ -281,11 +283,10 @@ def test_writer_operations_by_semi_traditional_usage(convert, needs_cleanup):
         (BytesIO(), False),
     ],
 )
-def test_writer_operations_by_semi_new_traditional_usage(
-    convert, needs_cleanup
-):
+def test_writer_operations_by_semi_new_traditional_usage(convert, needs_cleanup):
     if callable(convert):
-        write_data_here = convert(NamedTemporaryFile(suffix=".pdf", delete=False).name)
+        with NamedTemporaryFile(suffix=".pdf", delete=False) as fo:
+            write_data_here = convert(fo.name)
     else:
         write_data_here = convert
 
@@ -309,7 +310,8 @@ def test_writer_operations_by_semi_new_traditional_usage(
 )
 def test_writer_operation_by_new_usage(convert, needs_cleanup):
     if callable(convert):
-        write_data_here = convert(NamedTemporaryFile(suffix=".pdf", delete=False).name)
+        with NamedTemporaryFile(suffix=".pdf", delete=False) as fo:
+            write_data_here = convert(fo.name)
     else:
         write_data_here = convert