provisioning: Ensure constraints are valid

Signed-off-by: Ikey Doherty <[email protected]>

Author: ikeycode

crates/provisioning/src/commands/create_partition.rs

@@ -58,6 +58,14 @@ pub(crate) fn parse(context: Context<'_>) -> Result<super::Command, crate::Error
         }
     }
 
+    if matches!(constraints, Constraints::Invalid) {
+        return Err(crate::InvalidArguments {
+            at: context.node.span(),
+            advice: Some("create-partition [disk=<disk>] [role=<role>] [constraints=<constraints>] [type=(GUID)] - you must provide constraints".into()),
+        }
+        .into());
+    }
+
     Ok(super::Command::CreatePartition(Box::new(Command {
         disk,
         id,

crates/provisioning/src/types/constraints.rs

@@ -6,7 +6,7 @@ use crate::{get_kdl_entry, kdl_value_to_storage_size};
 
 /// Constraints for partition size, 1:1 mapping to SizeRequirements in
 /// partitioning strategy internals.
-#[derive(Debug, Default, Clone, Copy)]
+#[derive(Debug, Default, Clone, Copy, PartialEq, Eq)]
 pub enum Constraints {
     /// Exact size in bytes
     Exact(u64),