Peman this isn't Behrouz's change. I must have added this as a placeholder. I have now removed it on my development branch, so it can be replaced in the future with the coverage test.
I think SHA will (maybe already does) call out certain data items as critical and control measure is to have some kind of means to verify integrity. I think all of the treatment parameters that I'm adding in this story will qualify as critical so I added these critical data structures and functions.
I understand and agree, that's extra work. But that wasn't even part of the Alarm design. This is the settings feature which has not been started yet. That's exactly why we have this interview to let you (or any developer) know the changes before the Unit test/Integration Test/Code Coverage is done and also to know that against which code those test/coverage needs to be done.
You are running release
CR4.8.14
FE4.8.14
(20240111091859 2024-01-11 09:20),
please report your release number when reporting bugs.