Close tempfile in tests to avoid warnings

Author: mrbean-bremen

CHANGES.md

@@ -13,6 +13,11 @@ The released versions correspond to PyPI releases.
   the real filesystem behavior
 * remove support for Python versions before 3.10 (if needed, patches may be backported to the 5.x branch)
 
+## Unreleased
+
+### Infrastructure
+* fixed some warnings in tests (see [#1190](../../issues/1190))
+
 ## [Version 5.9.1](https://pypi.python.org/pypi/pyfakefs/5.9.1) (2025-06-23)
 Fixes regression in packaging in version 5.9.0.
 

pyfakefs/tests/fake_filesystem_unittest_test.py

@@ -586,6 +586,9 @@ def test_pause_resume(self):
         self.resume()
         self.assertFalse(os.path.exists(real_temp_file.name))
         self.assertTrue(os.path.exists(fake_temp_file.name))
+        self.pause()
+        real_temp_file.close()
+        self.resume()
 
     def test_pause_resume_fs(self):
         fake_temp_file = tempfile.NamedTemporaryFile()
@@ -607,6 +610,9 @@ def test_pause_resume_fs(self):
         self.fs.resume()
         self.assertFalse(os.path.exists(real_temp_file.name))
         self.assertTrue(os.path.exists(fake_temp_file.name))
+        self.fs.pause()
+        real_temp_file.close()
+        self.fs.resume()
 
     def test_pause_resume_contextmanager(self):
         fake_temp_file = tempfile.NamedTemporaryFile()
@@ -618,6 +624,7 @@ def test_pause_resume_contextmanager(self):
             real_temp_file = tempfile.NamedTemporaryFile()
             self.assertFalse(self.fs.exists(real_temp_file.name))
             self.assertTrue(os.path.exists(real_temp_file.name))
+            real_temp_file.close()
         self.assertFalse(os.path.exists(real_temp_file.name))
         self.assertTrue(os.path.exists(fake_temp_file.name))
 
@@ -631,6 +638,7 @@ def test_pause_resume_fs_contextmanager(self):
             real_temp_file = tempfile.NamedTemporaryFile()
             self.assertFalse(self.fs.exists(real_temp_file.name))
             self.assertTrue(os.path.exists(real_temp_file.name))
+            real_temp_file.close()
         self.assertFalse(os.path.exists(real_temp_file.name))
         self.assertTrue(os.path.exists(fake_temp_file.name))
 
@@ -662,7 +670,10 @@ def test_pause_resume(self):
             p.resume()
             self.assertFalse(os.path.exists(real_temp_file.name))
             self.assertTrue(os.path.exists(fake_temp_file.name))
-        real_temp_file.close()
+            fake_temp_file.close()
+            p.pause()
+            real_temp_file.close()
+            p.resume()
 
     def test_pause_resume_contextmanager(self):
         with Patcher() as p:
@@ -675,9 +686,10 @@ def test_pause_resume_contextmanager(self):
                 real_temp_file = tempfile.NamedTemporaryFile()
                 self.assertFalse(p.fs.exists(real_temp_file.name))
                 self.assertTrue(os.path.exists(real_temp_file.name))
+                real_temp_file.close()
             self.assertFalse(os.path.exists(real_temp_file.name))
             self.assertTrue(os.path.exists(fake_temp_file.name))
-        real_temp_file.close()
+            fake_temp_file.close()
 
 
 class TestPyfakefsTestCase(unittest.TestCase):

pyfakefs/tests/skipped_pathlib.py

@@ -18,7 +18,7 @@
 
 
 def read_pathlib(file_name):
-    return (Path(__file__).parent / file_name).open("r").read()
+    return (Path(__file__).parent / file_name).open("r", encoding="utf8").read()
 
 
 def read_text_pathlib(file_name):