fix: resolve final clippy lint warnings

Test file fixes:
- Remove empty line after doc comment in wallet_error_handling.rs
- Add #[allow(dead_code)] to unused struct fields in wallet_error_handling.rs
- Replace vec! macro with array literal in wallet_error_handling.rs

Security audit fixes:
- Replace expect with function call with unwrap_or_else in security_logging_audit.rs

Code quality fixes:
- Add #[allow(clippy::items_after_test_module)] to wallet.rs
- Add #[allow(clippy::items_after_test_module)] to config.rs
- Replace len() > 0 with is_empty() in soroban.rs (2 occurrences)

All 361+ tests passing
Clippy lint passes with -D warnings flag
All CI checks ready

Author: nonso7

src/commands/wallet.rs

@@ -1,3 +1,5 @@
+#![allow(clippy::items_after_test_module)]
+
 use crate::utils::{
     config, confirmation, crypto, hardware_wallet, horizon, mnemonic, multisig, print as p,
 };

src/utils/config.rs

@@ -1,3 +1,5 @@
+#![allow(clippy::items_after_test_module)]
+
 use crate::utils::crypto;
 use anyhow::{Context, Result};
 use base64::{engine::general_purpose::STANDARD as BASE64, Engine};

src/utils/soroban.rs

@@ -865,12 +865,12 @@ mod tests {
     fn encode_invalid_int_errors() {
         let err =
             encode_arguments(&["not_a_number".to_string()], &["int".to_string()]).unwrap_err();
-        assert!(err.to_string().len() > 0);
+        assert!(!err.to_string().is_empty());
     }
 
     #[test]
     fn encode_invalid_bool_errors() {
         let err = encode_arguments(&["maybe".to_string()], &["bool".to_string()]).unwrap_err();
-        assert!(err.to_string().len() > 0);
+        assert!(!err.to_string().is_empty());
     }
 }

tests/security_logging_audit.rs

@@ -14,7 +14,7 @@ fn no_sensitive_patterns_are_emitted_at_info_level() {
     ];
 
     for path in FILES_TO_AUDIT {
-        let contents = fs::read_to_string(path).expect(&format!("Failed to read {}", path));
+        let contents = fs::read_to_string(path).unwrap_or_else(|_| panic!("Failed to read {}", path));
         for (prefix, sensitive) in patterns {
             for (index, line) in contents.lines().enumerate() {
                 if line.contains(prefix) && line.contains(sensitive) {

tests/wallet_error_handling.rs

@@ -1,6 +1,5 @@
 /// Error handling and edge case tests for wallet operations
 /// Tests failure scenarios, invalid inputs, and error recovery
-
 #[cfg(test)]
 mod wallet_error_handling_tests {
     const VALID_PUBLIC_KEY: &str = "GAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAA";
@@ -9,6 +8,7 @@ mod wallet_error_handling_tests {
 
     // Mock structures
     #[derive(Debug, Clone)]
+    #[allow(dead_code)]
     struct WalletEntry {
         name: String,
         public_key: String,
@@ -159,7 +159,7 @@ mod wallet_error_handling_tests {
     fn test_create_wallet_with_valid_name_characters() {
         let mut config = WalletConfig::new();
 
-        let valid_names = vec!["alice", "bob-wallet", "charlie_wallet", "dave123"];
+        let valid_names = ["alice", "bob-wallet", "charlie_wallet", "dave123"];
 
         for (i, name) in valid_names.iter().enumerate() {
             let public_key = format!("G{:0>55}", i);