strided_scalar to return read-only array

Update tests to verify read-only, instead of checking values

Author: effigies

nibabel/fileslice.py

@@ -767,9 +767,12 @@ def strided_scalar(shape, scalar=0.):
     strided_arr : array
         Array of shape `shape` for which all values == `scalar`, built by
         setting all strides of `strided_arr` to 0, so the scalar is broadcast
-        out to the full array `shape`.
+        out to the full array `shape`. `strided_arr` is flagged as not
+        `writeable`.
     """
     shape = tuple(shape)
     scalar = np.array(scalar)
     strides = [0] * len(shape)
-    return np.lib.stride_tricks.as_strided(scalar, shape, strides)
+    strided_scalar = np.lib.stride_tricks.as_strided(scalar, shape, strides)
+    strided_scalar.flags.writeable = False
+    return strided_scalar

nibabel/tests/test_fileslice.py

@@ -639,8 +639,9 @@ def test_strided_scalar():
         assert_equal(observed.shape, shape)
         assert_equal(observed.dtype, expected.dtype)
         assert_array_equal(observed.strides, 0)
-        observed[..., 0] = 99
-        assert_array_equal(observed, expected * 0 + 99)
+        def setval(x):
+            x[..., 0] = 99
+        assert_raises(ValueError, setval, observed)
     # Default scalar value is 0
     assert_array_equal(strided_scalar((2, 3, 4)), np.zeros((2, 3, 4)))