From 9bad8e2c73b7a75947e231b0f9c85fc31687ccbc Mon Sep 17 00:00:00 2001
From: Joscha Schmiedt <joscha@schmiedt.dev>
Date: Mon, 2 Dec 2024 21:41:04 +0100
Subject: [PATCH] TST: Make NamedTemporaryFiles Windows-compatible

Don't open files twice (add delete=False, which causes file to not be opened)
---
 unittests/test_cfood_metadata.py | 4 ++--
 unittests/test_macros.py         | 4 ++--
 2 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/unittests/test_cfood_metadata.py b/unittests/test_cfood_metadata.py
index c606a0a1..b123f985 100644
--- a/unittests/test_cfood_metadata.py
+++ b/unittests/test_cfood_metadata.py
@@ -18,7 +18,7 @@
 # along with this program. If not, see <https://www.gnu.org/licenses/>.
 #
 from tempfile import NamedTemporaryFile
-from unittest.mock import MagicMock, Mock, patch
+from unittest.mock import patch
 
 import pytest
 import yaml
@@ -33,7 +33,7 @@ def _temp_file_load(txt: str):
     definition using load_definition from Crawler.
     """
     definition = None
-    with NamedTemporaryFile() as f:
+    with NamedTemporaryFile(delete=False) as f:
         f.write(txt.encode())
         f.flush()
         definition = load_definition(f.name)
diff --git a/unittests/test_macros.py b/unittests/test_macros.py
index a87b633e..03fe0e66 100644
--- a/unittests/test_macros.py
+++ b/unittests/test_macros.py
@@ -50,10 +50,10 @@ def _temp_file_load(txt: str):
     definition using load_definition from Crawler.
     """
     definition = None
-    with NamedTemporaryFile() as f:
+    with NamedTemporaryFile(delete=False) as f:
         f.write(txt.encode())
         f.flush()
-        definition = load_definition(f.name)
+    definition = load_definition(f.name)
     return definition
 
 
-- 
GitLab