Ensure returned arrays are not writeable

Author: AlexanderWells-diamond

CHANGELOG.rst

@@ -23,6 +23,7 @@ Removed:
 Changed:
 
 - `AsyncioDispatcher cleanup tasks atexit <../../pull/138>`_
+- `Ensure returned numpy arrays are not writeable <../../pull/164>`_
 
 Fixed:
 

softioc/device.py

@@ -473,6 +473,9 @@ def _epics_to_value(self, value):
         if self._dtype.char == 'S':
             return [_string_at(s, 40) for s in value]
         else:
+            # As we return a reference to the numpy array, ensure it cannot be
+            # modified under our noses
+            value.flags.writeable = False
             return value
 
     def _value_to_dbr(self, value):

tests/test_records.py

@@ -340,6 +340,19 @@ def test_record_wrapper_str():
     # If we never receive R it probably means an assert failed
     select_and_recv(parent_conn, "R")
 
+def test_waveform_values_not_modifiable():
+    """Test that arrays returned from waveform records are not modifiable"""
+
+    wi = builder.WaveformIn("WI", [1,2,3])
+    wo = builder.WaveformOut("WO", [1,2,3])
+
+    with pytest.raises(ValueError):
+        wi.get()[0] = 5
+
+    with pytest.raises(ValueError):
+        wo.get()[0] = 5
+
+
 def validate_fixture_names(params):
     """Provide nice names for the out_records fixture in TestValidate class"""
     return params[0].__name__