Refresh the execution during resume_execution to avoid using a detached SqlAlchemy instance

shared/workspaces/test/test_execution_manager.py

@@ -214,6 +214,7 @@ def test_resume_execution(
             }
         )
         fake_execution.capability = capability
+        mock_capability_info.save_execution(fake_execution)
         dummy_execution_manager.queue_execution(fake_execution)
         assert fake_execution.queue_state == QueueState.Queued.name
         dummy_execution_manager.resume_execution(fake_execution)

shared/workspaces/workspaces/capability/services/execution_manager.py

@@ -87,6 +87,11 @@ class ExecutionManager(ExecutionManagerIF):
         :param execution:  the execution whose time has come
         """
         # Mark this execution as no longer queued
+        # this is important for the tests
+        execution.queue_state = QueueState.Running.name
+
+        # load the execution afresh
+        execution = self.capability_info.lookup_execution(execution.id)
         execution.queue_state = QueueState.Running.name
         self.capability_info.save_execution(execution)
         subject = execution.__json__()