Merge pull request #294 from sot/close-before-unlink

in tests_acq/guide: close temp file before unlinking it
sot · Feb 27, 2024 · 52d27f9 · 52d27f9
2 parents 359a9d7 + 23a2de0
commit 52d27f9
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 0 deletions.
diff --git a/mica/stats/tests/test_acq_stats.py b/mica/stats/tests/test_acq_stats.py
@@ -42,6 +42,7 @@ def test_make_acq_stats():
     # Get a temporary file, but then delete it, because _save_acq_stats will only
     # make a new table if the supplied file doesn't exist
     fh, fn = tempfile.mkstemp(suffix='.h5')
+    os.close(fh)
     os.unlink(fn)
     acq_stats.table_file = fn
     obsid = 20001

diff --git a/mica/stats/tests/test_guide_stats.py b/mica/stats/tests/test_guide_stats.py
@@ -48,6 +48,7 @@ def test_make_gui_stats():
     # Get a temporary file, but then delete it, because _save_acq_stats will only
     # make a new table if the supplied file doesn't exist
     fh, fn = tempfile.mkstemp(suffix='.h5')
+    os.close(fh)
     os.unlink(fn)
     obsid = 20001
     obsid_info, gui, star_info, catalog, temp = update_guide_stats.calc_stats(obsid)