Skip to content

Commit 6869a5f

Browse files
author
Shoshana Berleant
committed
use mkdtemp for test_tsnr
1 parent 5f39bfc commit 6869a5f

File tree

2 files changed

+9
-11
lines changed

2 files changed

+9
-11
lines changed

nipype/algorithms/tests/test_tsnr.py

Lines changed: 9 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -10,6 +10,8 @@
1010
import nibabel as nb
1111
import numpy as np
1212
import os
13+
import tempfile
14+
import shutil
1315

1416
class TestTSNR(unittest.TestCase):
1517
''' Note: Tests currently do a poor job of testing functionality '''
@@ -26,7 +28,11 @@ class TestTSNR(unittest.TestCase):
2628
}
2729

2830
def setUp(self):
29-
# setup
31+
# setup temp folder
32+
self.orig_dir = os.getcwd()
33+
self.temp_dir = tempfile.mkdtemp()
34+
os.chdir(self.temp_dir)
35+
3036
utils.save_toy_nii(self.fake_data, self.in_filenames['in_file'])
3137

3238
def test_tsnr(self):
@@ -103,8 +109,8 @@ def assert_unchanged(self, expected_ranges):
103109
assert_almost_equal(np.amax(data), max_, decimal=1)
104110

105111
def tearDown(self):
106-
utils.remove_nii(self.in_filenames.values())
107-
utils.remove_nii(self.out_filenames.values())
112+
os.chdir(self.orig_dir)
113+
shutil.rmtree(self.temp_dir)
108114

109115
fake_data = np.array([[[[2, 4, 3, 9, 1],
110116
[3, 6, 4, 7, 4]],

nipype/testing/utils.py

Lines changed: 0 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -110,11 +110,3 @@ def save_toy_nii(ndarray, filename):
110110
toy = nb.Nifti1Image(ndarray, np.eye(4))
111111
nb.nifti1.save(toy, filename)
112112
return filename
113-
114-
def remove_nii(filenames):
115-
''' remove temporary nifti files'''
116-
for filename in filenames:
117-
try:
118-
os.remove(filename)
119-
except (OSError, TypeError) as e:
120-
pass

0 commit comments

Comments
 (0)