Skip to content

Commit 663f500

Browse files
committed
TEST: Make sure Nifti1Pair.get_sform/get_qform methods return integer types.
1 parent f8820c7 commit 663f500

File tree

1 file changed

+20
-1
lines changed

1 file changed

+20
-1
lines changed

nibabel/tests/test_nifti1.py

Lines changed: 20 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -12,6 +12,8 @@
1212
import warnings
1313
import struct
1414

15+
import six
16+
1517
import numpy as np
1618

1719
from nibabel import nifti1 as nifti1
@@ -595,7 +597,7 @@ def test_intents(self):
595597
assert_raises(HeaderDataError, ehdr.set_intent, 999, (1,),
596598
allow_unknown=True)
597599
assert_raises(HeaderDataError, ehdr.set_intent, 999, (1,2),
598-
allow_unknown=True)
600+
allow_unknown=True)
599601

600602
def test_set_slice_times(self):
601603
hdr = self.header_class()
@@ -900,6 +902,23 @@ def test_set_sform(self):
900902
img.set_sform(new_affine, 2)
901903
assert_array_almost_equal(img.affine, new_affine)
902904

905+
def test_sqform_code_type(self):
906+
# make sure get_s/qform returns codes as integers
907+
img = self.image_class(np.zeros((2, 3, 4)), None)
908+
assert isinstance(img.get_sform(coded=True)[1], six.integer_types)
909+
assert isinstance(img.get_qform(coded=True)[1], six.integer_types)
910+
img.set_sform(None, 3)
911+
img.set_qform(None, 3)
912+
assert isinstance(img.get_sform(coded=True)[1], six.integer_types)
913+
assert isinstance(img.get_qform(coded=True)[1], six.integer_types)
914+
img.set_sform(None, 2.0)
915+
img.set_qform(None, 4.0)
916+
assert isinstance(img.get_sform(coded=True)[1], six.integer_types)
917+
assert isinstance(img.get_qform(coded=True)[1], six.integer_types)
918+
img.set_sform(None, img.get_sform(coded=True)[1])
919+
img.set_qform(None, img.get_qform(coded=True)[1])
920+
921+
903922
def test_hdr_diff(self):
904923
# Check an offset beyond data does not raise an error
905924
img = self.image_class(np.zeros((2, 3, 4)), np.eye(4))

0 commit comments

Comments
 (0)