test failing: testConstantToDevice, testConstantToDeviceFanOut.

Test still does not work.
-Unable to get the exception status for the device throwing exception.
-Second device only works it has it's own trigger.

added test for exceptions on write

do not use a device register as a trigger any more

According to ChimeraTK/DeviceAccess#61 Devices will no longer close on their own (e.g. when throwing an exception). The exception handling now takes into account that the device always stays open

Martin Killenberg authored 5 years ago and Martin Christoph Hierholzer committed 5 years ago

added more exception test code which should work but is but. Commented because it completely blocks the test.

Martin Killenberg authored 5 years ago and Martin Christoph Hierholzer committed 5 years ago

Creation of fanOuts is based on externalTriggerImpl (=trigger consumer), not externalTriggerNode (=trigger feeder).
- Fixes connection network but not exception handling.
- Lots of debug output left in the code for further investigation.

Martin Killenberg authored 5 years ago and Martin Christoph Hierholzer committed 5 years ago

extended test with a second device. check that it is still working if the first one is throwing is not working with the old impl.

Martin Killenberg authored 5 years ago and Martin Christoph Hierholzer committed 5 years ago

  no change to the test sequence yet

- add test for full exception handling

  DeviceModule and ControlSystemModule now do not take the optional prefix argument in their constructors any more. Use the [] operator instead to get the submodule.

- Add ExceptionHandlingDecorator and decorate all device variables with it.

add test for findTag() (was never really tested before), including advanced features like HierarchyModifier::moveToRoot

- added more comments
- need to catch exception by reference, else the message seems to be lost
- test if the message is correct
- test if device is really closed and opened again
- changed names of status variables in control system to match the general naming conventions