Big refactor to final design as discussed in VC in June

Author: BoySanic

common/src/errors/validate_submit_err.rs

@@ -7,10 +7,12 @@ pub enum ValidateSubmitError {
     InvalidAssignment,
     #[error("task already validated and this result is not valid")]
     InconsistentValidationState,
-    #[error("multi-task validation in a single request is currently not implemented")]
-    MultipleTasksDisallowed,
-    #[error("too many groups meeting quorum were provided")]
-    ValidityAmbiguous,
+    #[error("all results are inconclusive, and no new assignment has finished to solve it")]
+    ValidationImpossibleError,
+    #[error("validation group contained assignments belonging to multiple tasks")]
+    ValidationGroupTaskInconsistency,
+    #[error("assignments referred to by group id cannot refer to an assignment other than itself")]
+    ValidationGroupAssociationInconsistency,
     #[error("state transition forbidden")]
     StateTransitionForbidden,
 }

common/src/records/result.rs

@@ -13,6 +13,7 @@ pub struct Result {
     pub stdout: String,
     pub stderr: String,
     pub exit_code: Option<i32>,
+    pub group_assignment_id: Option<Id<Assignment>>
 }
 
 #[non_exhaustive]

common/src/requests/validate_submit_request.rs

@@ -6,5 +6,6 @@ use crate::{records::Assignment, types::Id};
 
 #[derive(Clone, Debug, Serialize, Deserialize)]
 pub struct ValidateSubmitRequest {
-    pub assignments: HashMap<Id<Assignment>, Option<i32>>,
+    //First id is the assignment id that will change state, second is the "group id" it belongs with
+    pub assignments: HashMap<Id<Assignment>, Id<Assignment>>,
 }