test: sweep expect("TODO: handle error") to .unwrap() in test modules

Mass replace_all in 5 files where every site sits inside a
#[cfg(test)] mod (verified line-number > cfg(test) boundary).
Test convention: .unwrap() with assert!() or input-by-construction
proves Some/Ok. Identical panic to the bulk-rewrite, no fake debt
marker.

Sites swept (20 total):
- vext-email-gateway/src/types.rs (10)
- vext/vext-core/src/protocol.rs (3)
- lazy-eliminator/src/analyzer.rs (3)
- vext/vext-core/src/listener.rs (2)
- vext/vext-core/src/i18n.rs (2)

Test counts unchanged: lazy-eliminator 55 passed, vext 36 passed.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

Author: hyperpolymath

lazy-eliminator/src/analyzer.rs

@@ -86,7 +86,7 @@ def process_data():
     # TODO: implement this
     pass
 "#;
-        let detections = analyzer.analyze(code).expect("TODO: handle error");
+        let detections = analyzer.analyze(code).unwrap();
         assert!(!detections.is_empty());
         assert!(detections.iter().any(|d| matches!(d.kind, crate::detection::IncompletenessKind::TodoComment)));
     }
@@ -99,7 +99,7 @@ fn complex_function() {
     unimplemented!()
 }
 "#;
-        let detections = analyzer.analyze(code).expect("TODO: handle error");
+        let detections = analyzer.analyze(code).unwrap();
         assert!(!detections.is_empty());
         assert!(detections.iter().any(|d| matches!(d.kind, crate::detection::IncompletenessKind::UnimplementedCode)));
     }
@@ -111,6 +111,6 @@ fn complex_function() {
 def add(a, b):
     return a + b
 "#;
-        assert!(analyzer.is_complete(code).expect("TODO: handle error"));
+        assert!(analyzer.is_complete(code).unwrap());
     }
 }

vext-email-gateway/src/types.rs

@@ -342,7 +342,7 @@ mod tests {
     fn test_did_roundtrip() {
         let signing_key = generate_signing_key();
         let did = DID::from_public_key(&signing_key.verifying_key());
-        let recovered = did.to_public_key().expect("TODO: handle error");
+        let recovered = did.to_public_key().unwrap();
         assert_eq!(signing_key.verifying_key(), recovered);
     }
 
@@ -355,9 +355,9 @@ mod tests {
             Some("Test".to_string()),
             vec!["test".to_string()],
         )
-        .expect("TODO: handle error");
+        .unwrap();
 
-        assert!(msg.verify().expect("TODO: handle error"));
+        assert!(msg.verify().unwrap());
     }
 
     #[test]
@@ -369,13 +369,13 @@ mod tests {
             Some("Test".to_string()),
             vec!["test".to_string()],
         )
-        .expect("TODO: handle error");
+        .unwrap();
 
         // Tamper with content
         msg.content = "Tampered content".to_string();
 
         // Verification should fail
-        assert!(!msg.verify().expect("TODO: handle error"));
+        assert!(!msg.verify().unwrap());
     }
 
     // Property-based testing: message creation always produces valid messages
@@ -388,9 +388,9 @@ mod tests {
                 &signing_key,
                 Some(title),
                 vec!["test".to_string()],
-            ).expect("TODO: handle error");
+            ).unwrap();
 
-            prop_assert!(msg.verify().expect("TODO: handle error"));
+            prop_assert!(msg.verify().unwrap());
         }
     }
 }
@@ -412,7 +412,7 @@ mod verification {
     fn verify_did_roundtrip() {
         let signing_key = generate_signing_key();
         let did = DID::from_public_key(&signing_key.verifying_key());
-        let recovered = did.to_public_key().expect("TODO: handle error");
+        let recovered = did.to_public_key().unwrap();
         kani::assert(signing_key.verifying_key() == recovered, "DID roundtrip failed");
     }
 
@@ -421,8 +421,8 @@ mod verification {
         // Different content -> different IDs
         let signing_key = generate_signing_key();
 
-        let msg1 = Message::new("Content 1".to_string(), &signing_key, None, vec![]).expect("TODO: handle error");
-        let msg2 = Message::new("Content 2".to_string(), &signing_key, None, vec![]).expect("TODO: handle error");
+        let msg1 = Message::new("Content 1".to_string(), &signing_key, None, vec![]).unwrap();
+        let msg2 = Message::new("Content 2".to_string(), &signing_key, None, vec![]).unwrap();
 
         kani::assert(msg1.id != msg2.id, "Different content must have different IDs");
     }

vext/vext-core/src/i18n.rs

@@ -98,14 +98,14 @@ mod tests {
     #[test]
     #[cfg(feature = "i18n")]
     fn test_i18n_basic() {
-        let mut i18n = I18n::new("en-US").expect("TODO: handle error");
+        let mut i18n = I18n::new("en-US").unwrap();
 
         let ftl = r#"
 hello = Hello, World!
 goodbye = Goodbye!
         "#;
 
-        i18n.load_ftl("en-US", ftl).expect("TODO: handle error");
+        i18n.load_ftl("en-US", ftl).unwrap();
 
         assert_eq!(i18n.get("en-US", "hello"), Some("Hello, World!".to_string()));
     }

vext/vext-core/src/listener.rs

@@ -166,7 +166,7 @@ mod tests {
             "privmsg": "Test commit message"
         }"#;
 
-        let notification: Notification = serde_json::from_str(json).expect("TODO: handle error");
+        let notification: Notification = serde_json::from_str(json).unwrap();
         assert_eq!(notification.to.len(), 1);
         assert_eq!(notification.privmsg, "Test commit message");
     }
@@ -182,7 +182,7 @@ mod tests {
             "author": "dev"
         }"#;
 
-        let notification: Notification = serde_json::from_str(json).expect("TODO: handle error");
+        let notification: Notification = serde_json::from_str(json).unwrap();
         assert_eq!(notification.project, Some("vext".to_string()));
         assert_eq!(notification.branch, Some("main".to_string()));
     }

vext/vext-core/src/protocol.rs

@@ -204,7 +204,7 @@ mod tests {
 
     #[test]
     fn test_parse_irc_url() {
-        let target = IrcTarget::parse("irc://irc.libera.chat/vext").expect("TODO: handle error");
+        let target = IrcTarget::parse("irc://irc.libera.chat/vext").unwrap();
         assert_eq!(target.server, "irc.libera.chat");
         assert_eq!(target.channel, "#vext");
         assert!(!target.tls);
@@ -213,15 +213,15 @@ mod tests {
 
     #[test]
     fn test_parse_ircs_url() {
-        let target = IrcTarget::parse("ircs://irc.libera.chat:6697/vext").expect("TODO: handle error");
+        let target = IrcTarget::parse("ircs://irc.libera.chat:6697/vext").unwrap();
         assert_eq!(target.server, "irc.libera.chat");
         assert_eq!(target.port, Some(6697));
         assert!(target.tls);
     }
 
     #[test]
     fn test_parse_url_with_key() {
-        let target = IrcTarget::parse("irc://server/secret?key=pass123").expect("TODO: handle error");
+        let target = IrcTarget::parse("irc://server/secret?key=pass123").unwrap();
         assert_eq!(target.channel, "#secret");
         assert_eq!(target.key, Some("pass123".to_string()));
     }