Skip to content
Snippets Groups Projects
Commit bb7785a4 authored by Jan David Mol's avatar Jan David Mol
Browse files

Merge branch 'reraise-on-fault' into 'master'

Reraise exceptions too if we go to FAULT

See merge request !361
parents 3a1beb5d 538928f5
No related branches found
No related tags found
1 merge request!361Reraise exceptions too if we go to FAULT
...@@ -55,11 +55,9 @@ def fault_on_error(): ...@@ -55,11 +55,9 @@ def fault_on_error():
try: try:
return func(self, *args, **kwargs) return func(self, *args, **kwargs)
except Exception as e: except Exception as e:
logger.exception("Function failed.") logger.exception("Function failed.")
self.Fault(f"FAULT in {func.__name__}: {e.__class__.__name__}: {e}") self.Fault(f"FAULT in {func.__name__}: {e.__class__.__name__}: {e}")
raise
return None
return error_wrapper return error_wrapper
......
...@@ -41,8 +41,8 @@ class TestObservationDevice(device_base.DeviceTestCase, test_observation_base.Te ...@@ -41,8 +41,8 @@ class TestObservationDevice(device_base.DeviceTestCase, test_observation_base.Te
with DeviceTestContext(observation.Observation, process=True) as proxy: with DeviceTestContext(observation.Observation, process=True) as proxy:
proxy.off() proxy.off()
proxy.observation_settings_RW = "{}" proxy.observation_settings_RW = "{}"
with self.assertRaises(DevFailed):
proxy.Initialise() proxy.Initialise()
self.assertEqual(DevState.FAULT, proxy.state())
def test_prohibit_rewriting_settings(self): def test_prohibit_rewriting_settings(self):
"""Test that changing observation settings is disallowed once init""" """Test that changing observation settings is disallowed once init"""
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Please register or to comment