TST: do not remove files while they are open

cgohlke · cgohlke · commit 386a088dbd7c · 2014-07-13T13:10:46.000-07:00
Fixes WindowsError: [Error 32] The process cannot access the file because it is being used by another process
diff --git a/lib/matplotlib/tests/test_backend_pdf.py b/lib/matplotlib/tests/test_backend_pdf.py
@@ -64,7 +64,7 @@ def test_multipage_keep_empty():
         with PdfPages(tmp) as pdf:
             filename = pdf._file.fh.name
         assert os.path.exists(filename)
-        os.remove(filename)
+    os.remove(filename)
     # test if an empty pdf is deleting itself afterwards with keep_empty=False
     with NamedTemporaryFile(delete=False) as tmp:
         with PdfPages(tmp, keep_empty=False) as pdf:
@@ -80,11 +80,11 @@ def test_multipage_keep_empty():
             filename = pdf._file.fh.name
             pdf.savefig()
         assert os.path.exists(filename)
-        os.remove(filename)
+    os.remove(filename)
     # test that a non-empty pdf is left behind with keep_empty=False
     with NamedTemporaryFile(delete=False) as tmp:
         with PdfPages(tmp, keep_empty=False) as pdf:
             filename = pdf._file.fh.name
             pdf.savefig()
         assert os.path.exists(filename)
-        os.remove(filename)
+    os.remove(filename)
