feat: Improve error from try_deserialize

Author: barskern

src/error.rs

@@ -289,6 +289,10 @@ impl de::Error for ConfigError {
     fn custom<T: fmt::Display>(msg: T) -> Self {
         Self::Message(msg.to_string())
     }
+
+    fn missing_field(field: &'static str) -> Self {
+        Self::NotFound(field.into())
+    }
 }
 
 impl ser::Error for ConfigError {

tests/testsuite/errors.rs

@@ -158,7 +158,7 @@ fn test_get_missing_field() {
     let res = c.get::<InnerSettings>("inner");
     assert_data_eq!(
         res.unwrap_err().to_string(),
-        str!["missing field `value2` for key `inner`"]
+        str![[r#"missing configuration field "value2" for key `inner`"#]]
     );
 }
 
@@ -184,7 +184,7 @@ fn test_get_missing_field_file() {
     let res = c.get::<InnerSettings>("inner");
     assert_data_eq!(
         res.unwrap_err().to_string(),
-        str!["missing field `value2` for key `inner`"]
+        str![[r#"missing configuration field "value2" for key `inner`"#]]
     );
 }
 
@@ -436,7 +436,7 @@ fn test_deserialize_missing_field() {
     let res = c.try_deserialize::<Settings>();
     assert_data_eq!(
         res.unwrap_err().to_string(),
-        str!["missing field `value2` for key `inner`"]
+        str![[r#"missing configuration field "inner.value2""#]]
     );
 }
 
@@ -468,6 +468,6 @@ fn test_deserialize_missing_field_file() {
     let res = c.try_deserialize::<Settings>();
     assert_data_eq!(
         res.unwrap_err().to_string(),
-        str!["missing field `value2` for key `inner`"]
+        str![[r#"missing configuration field "inner.value2""#]]
     );
 }

tests/testsuite/file_corn.rs

@@ -185,7 +185,7 @@ fn test_override_uppercase_value_for_struct() {
             );
         }
         Err(e) => {
-            if e.to_string().contains("missing field `FOO`") {
+            if matches!(e, config::ConfigError::NotFound(_)) {
                 assert_eq!(
                     lower_settings.foo,
                     "I HAVE BEEN OVERRIDDEN_WITH_UPPER_CASE".to_owned()

tests/testsuite/file_ini.rs

@@ -128,7 +128,7 @@ rating = 4.5
             );
         }
         Err(e) => {
-            if e.to_string().contains("missing field `FOO`") {
+            if matches!(e, config::ConfigError::NotFound(_)) {
                 assert_eq!(
                     lower_settings.foo,
                     "I HAVE BEEN OVERRIDDEN_WITH_UPPER_CASE".to_owned()

tests/testsuite/file_json.rs

@@ -178,7 +178,7 @@ fn test_override_uppercase_value_for_struct() {
             );
         }
         Err(e) => {
-            if e.to_string().contains("missing field `FOO`") {
+            if matches!(e, config::ConfigError::NotFound(_)) {
                 println!("triggered error {e:?}");
                 assert_eq!(
                     lower_settings.foo,

tests/testsuite/file_json5.rs

@@ -187,7 +187,7 @@ fn test_override_uppercase_value_for_struct() {
             );
         }
         Err(e) => {
-            if e.to_string().contains("missing field `FOO`") {
+            if matches!(e, config::ConfigError::NotFound(_)) {
                 assert_eq!(
                     lower_settings.foo,
                     "I HAVE BEEN OVERRIDDEN_WITH_UPPER_CASE".to_owned()

tests/testsuite/file_ron.rs

@@ -179,7 +179,7 @@ fn test_override_uppercase_value_for_struct() {
             );
         }
         Err(e) => {
-            if e.to_string().contains("missing field `FOO`") {
+            if matches!(e, config::ConfigError::NotFound(_)) {
                 assert_eq!(
                     lower_settings.foo,
                     "I HAVE BEEN OVERRIDDEN_WITH_UPPER_CASE".to_owned()

tests/testsuite/file_toml.rs

@@ -266,7 +266,7 @@ down = 1
             );
         }
         Err(e) => {
-            if e.to_string().contains("missing field `FOO`") {
+            if matches!(e, config::ConfigError::NotFound(_)) {
                 assert_eq!(
                     lower_settings.foo,
                     "I HAVE BEEN OVERRIDDEN_WITH_UPPER_CASE".to_owned()

tests/testsuite/file_yaml.rs

@@ -210,7 +210,7 @@ bar: I am bar
             );
         }
         Err(e) => {
-            if e.to_string().contains("missing field `FOO`") {
+            if matches!(e, config::ConfigError::NotFound(_)) {
                 println!("triggered error {e:?}");
                 assert_eq!(
                     lower_settings.foo,