From 686cde3af3470d96f6447ed1edac229f5429ebe5 Mon Sep 17 00:00:00 2001
From: Paul McCarthy <pauldmccarthy@gmail.com>
Date: Mon, 27 Jan 2020 14:00:40 +0000
Subject: [PATCH] TEST: Image.save semantics have changed
---
tests/test_image_advanced.py | 33 ++++++++++++++-------------------
1 file changed, 14 insertions(+), 19 deletions(-)
diff --git a/tests/test_image_advanced.py b/tests/test_image_advanced.py
index 2be8678b6..fd7f64359 100644
--- a/tests/test_image_advanced.py
+++ b/tests/test_image_advanced.py
@@ -187,6 +187,15 @@ def _test_image_indexed_save(threaded):
# make sure the data range is correct
assert img.dataRange == (0, 40)
+ # Save the image to a different
+ # location (no changes to data though)
+ filename = op.join(testdir, 'image2.nii.gz')
+ img.save(filename)
+
+ # known data range
+ # should not have changed
+ assert img.dataRange == (0, 40)
+
# change some data
data = np.zeros((100, 100, 100))
data[:] = 45
@@ -195,35 +204,21 @@ def _test_image_indexed_save(threaded):
if threaded:
img.getImageWrapper().getTaskThread().waitUntilIdle()
- # save the image
+ # save the image - this will
+ # cause the image data to be
+ # loaded into memory
img.save()
- assert img.dataRange == (0, 45)
-
- # access the data - index should
- # get rebuilt to this point
- img[..., 0]
- img[..., 40]
-
if threaded:
img.getImageWrapper().getTaskThread().waitUntilIdle()
- # make sure we got the modified data
- assert img.dataRange == (0, 45)
-
- img[..., 49]
-
- if threaded:
- img.getImageWrapper().getTaskThread().waitUntilIdle()
-
- # make sure we got the modified data
assert img.dataRange == (0, 49)
-
+ assert np.all(img[..., 40] == data)
# Finally, reload, and verify the change
img = fslimage.Image(filename)
- assert np.all(img[..., 40] == 45)
+ assert np.all(img[..., 40] == data)
@pytest.mark.longtest
--
GitLab