Make felt_from_number report errors properly

instead of returning Ok(None) in case of an error
lambdaclass · Apr 20, 2023 · ee0a5b3 · ee0a5b3
1 parent d545372
commit ee0a5b3
Showing 1 changed file with 4 additions and 1 deletion.
diff --git a/src/serde/deserialize_program.rs b/src/serde/deserialize_program.rs
@@ -162,7 +162,10 @@ where
     D: Deserializer<'de>,
 {
     let n = Number::deserialize(deserializer)?;
-    Ok(Felt252::parse_bytes(n.to_string().as_bytes(), 10))
+    match Felt252::parse_bytes(n.to_string().as_bytes(), 10) {
+        Some(x) => Ok(Some(x)),
+        None => Err(String::from("felt_from_number parse error")).map_err(de::Error::custom),
+    }
 }
 
 #[derive(Serialize, Deserialize, Debug, PartialEq, Eq, Clone)]