Avoid one more atexit in copy tests.

QuLogic · QuLogic · commit ff154badb00e · 2019-04-01T04:25:10.000-04:00
diff --git a/zarr/tests/test_convenience.py b/zarr/tests/test_convenience.py
@@ -2,7 +2,6 @@
 from __future__ import absolute_import, print_function, division
 import tempfile
 import atexit
-import os
 import unittest
 from numbers import Integral
 
@@ -601,17 +600,16 @@ def test_copy_group_dry_run(self, source, dest):
         assert 0 == n_bytes_copied
         assert_array_equal(baz, dest['foo/bar/baz'])
 
-    def test_logging(self, source, dest):
+    def test_logging(self, source, dest, tmpdir):
         # callable log
         copy(source['foo'], dest, dry_run=True, log=print)
 
         # file name
-        fn = tempfile.mktemp()
-        atexit.register(os.remove, fn)
+        fn = str(tmpdir.join('log_name'))
         copy(source['foo'], dest, dry_run=True, log=fn)
 
         # file
-        with tempfile.TemporaryFile(mode='w') as f:
+        with tmpdir.join('log_file').open(mode='w') as f:
             copy(source['foo'], dest, dry_run=True, log=f)
 
         # bad option
