fix: Add key tracking for all error types

rust-cli · Jan 10, 2025 · fd55144 · fd55144
1 parent 5852262
commit fd55144
Show file tree

Hide file tree

Showing 3 changed files with 46 additions and 5 deletions.
diff --git a/src/error.rs b/src/error.rs
@@ -78,6 +78,16 @@ pub enum ConfigError {
         key: Option<String>,
     },
 
+    /// Custom message
+    At {
+        /// Error being extended with a path
+        error: Box<ConfigError>,
+
+        /// The key in the configuration hash of this value (if available where the
+        /// error is generated).
+        key: Option<String>,
+    },
+
     /// Custom message
     Message(String),
 
@@ -130,7 +140,15 @@ impl ConfigError {
                 key: Some(key.into()),
             },
 
-            _ => self,
+            Self::At { error, .. } => Self::At {
+                error,
+                key: Some(key.into()),
+            },
+
+            other => Self::At {
+                error: Box::new(other),
+                key: Some(key.into()),
+            },
         }
     }
 
@@ -157,8 +175,15 @@ impl ConfigError {
                 expected,
                 key: Some(concat(key)),
             },
+            Self::At { error, key } => Self::At {
+                error,
+                key: Some(concat(key)),
+            },
             Self::NotFound(key) => Self::NotFound(concat(Some(key))),
-            _ => self,
+            other => Self::At {
+                error: Box::new(other),
+                key: Some(concat(None)),
+            },
         }
     }
 
@@ -217,6 +242,16 @@ impl fmt::Display for ConfigError {
                 Ok(())
             }
 
+            ConfigError::At { ref error, ref key } => {
+                write!(f, "{error}")?;
+
+                if let Some(ref key) = *key {
+                    write!(f, " for key `{key}`")?;
+                }
+
+                Ok(())
+            }
+
             ConfigError::FileParse { ref cause, ref uri } => {
                 write!(f, "{cause}")?;
 

diff --git a/tests/testsuite/errors.rs b/tests/testsuite/errors.rs
@@ -134,7 +134,10 @@ fn test_get_missing_field() {
         .unwrap();
 
     let res = c.get::<InnerSettings>("inner");
-    assert_data_eq!(res.unwrap_err().to_string(), str!["missing field `value2`"]);
+    assert_data_eq!(
+        res.unwrap_err().to_string(),
+        str!["missing field `value2` for key `inner`"]
+    );
 }
 
 #[test]
@@ -342,5 +345,8 @@ fn test_deserialize_missing_field() {
         .unwrap();
 
     let res = c.try_deserialize::<Settings>();
-    assert_data_eq!(res.unwrap_err().to_string(), str!["missing field `value2`"]);
+    assert_data_eq!(
+        res.unwrap_err().to_string(),
+        str!["missing field `value2` for key `inner`"]
+    );
 }
diff --git a/tests/testsuite/log.rs b/tests/testsuite/log.rs
@@ -53,6 +53,6 @@ fn test_load_level_lowercase() {
     assert!(s.is_err());
     assert_data_eq!(
         s.unwrap_err().to_string(),
-        str!["enum Level does not have variant constructor error"]
+        str!["enum Level does not have variant constructor error for key `log`"]
     );
 }