validator: remove an unnecessary map_err which actually deleted err information

Ivo Georgiev · Ivo Georgiev · commit c3a39d8afaf7 · 2019-11-04T21:52:07.000+02:00
diff --git a/primitives/src/validator.rs b/primitives/src/validator.rs
@@ -65,9 +65,7 @@ impl TryFrom<&str> for ValidatorId {
 impl TryFrom<&String> for ValidatorId {
     type Error = DomainError;
     fn try_from(value: &String) -> Result<Self, Self::Error> {
-        ValidatorId::try_from(value.as_str()).map_err(|_| {
-            DomainError::InvalidArgument("Failed to deserialize validator id".to_string())
-        })
+        ValidatorId::try_from(value.as_str())
     }
 }
 

Original file line number	Diff line number	Diff line change
`@@ -65,9 +65,7 @@ impl TryFrom<&str> for ValidatorId {`
`65`	`65`	`impl TryFrom<&String> for ValidatorId {`
`66`	`66`	`type Error = DomainError;`
`67`	`67`	`fn try_from(value: &String) -> Result<Self, Self::Error> {`
`68`		`- ValidatorId::try_from(value.as_str()).map_err(\|_\| {`
`69`		`- DomainError::InvalidArgument("Failed to deserialize validator id".to_string())`
`70`		`- })`
	`68`	`+ ValidatorId::try_from(value.as_str())`
`71`	`69`	`}`
`72`	`70`	`}`
`73`	`71`