Pass up preflight errors, fix error test

The order that the preflight errors are in is nondeterministic for dependencies of the same level in the tree

flowmachine/flowmachine/core/server/action_handlers.py

@@ -93,9 +93,7 @@ def _load_query_object(params: dict) -> "BaseExposedQuery":
             f"The original error was: '{orig_error_msg}'"
         )
         raise QueryLoadError(
-            error_msg,
-            params,
-            orig_error_msg=orig_error_msg,
+            error_msg, params, orig_error_msg=orig_error_msg, errors=exc.errors
         )
     except TypeError as exc:
         # We need to catch TypeError here, otherwise they propagate up to
@@ -110,11 +108,6 @@ def _load_query_object(params: dict) -> "BaseExposedQuery":
             params,
             orig_error_msg=orig_error_msg,
         )
-        raise QueryLoadError(
-            error_msg,
-            params,
-            orig_error_msg=orig_error_msg,
-        )
     except ValidationError as exc:
         # The dictionary of marshmallow errors can contain integers as keys,
         # which will raise an error when converting to JSON (where the keys

flowmachine/tests/server/test_action_handlers.py

@@ -187,7 +187,9 @@ async def test_run_query_error_handled(dummy_redis, server_config):
         ),
     )
     assert msg.status == ZMQReplyStatus.ERROR
-    assert msg.msg.rstrip() == "Preflight failed for 7b71413efc91213e798ca3bd53107186."
+    assert msg.msg.rstrip().startswith(
+        "Internal flowmachine server error: could not create query object using query schema. The original error was: 'Pre-flight failed for '7b71413efc91213e798ca3bd53107186'. Errors:"
+    )
     assert len(msg.payload["errors"]) == 3