chore: 添加测试用例

Author: p1024k

src/crypto/CLAUDE.md

@@ -7,6 +7,8 @@
 
 | ID | Time | T | Title | Read |
 |----|------|---|-------|------|
+| #1012 | 6:43 PM | 🔵 | Found PasskeySeed type definition | ~57 |
+| #1011 | " | 🔵 | Found Passkey::to_seed() method returning SensitiveString | ~54 |
 | #458 | 2:01 PM | 🟣 | KeyHierarchy save/unlock implementation committed with complete key wrapping | ~202 |
 | #455 | 2:00 PM | 🔄 | KeyHierarchy unlock method signature reordered to match test expectations | ~159 |
 | #453 | " | 🔄 | Dangling derive_master_key code removed from KeyHierarchy implementation | ~113 |

src/tui/CLAUDE.md

@@ -7,18 +7,23 @@
 
 | ID | Time | T | Title | Read |
 |----|------|---|-------|------|
-| #507 | 2:16 PM | 🔴 | Autocomplete filter closure dereference fixed | ~28 |
-| #506 | " | 🟣 | TUI command autocomplete functionality implemented with prefix matching and multiple match handling | ~258 |
-| #505 | " | 🔴 | TUI autocomplete filter expression fixed to handle borrowed values correctly | ~48 |
-| #502 | 2:15 PM | 🔵 | TUI app command submission workflow processes user input through command handlers | ~169 |
-| #501 | " | 🟣 | Autocomplete handler call added to TUI tab key handling | ~134 |
-| #498 | " | 🔵 | TUI app handle_char method processes keyboard input with special cases | ~178 |
-| #491 | 2:14 PM | 🟣 | TUI action handlers for settings, sync, and configuration committed to git | ~196 |
-| #481 | 2:04 PM | 🟣 | TUI action handlers updated with improved messaging and screen navigation | ~44 |
-| #478 | 2:03 PM | 🔵 | TUI handle_key_event method routes events through screen navigation and keybinding manager | ~201 |
-| #477 | " | 🔵 | TUI event handler defines keyboard shortcuts for sync-related actions | ~172 |
-| #469 | " | 🔵 | TUI module structure includes app, commands, handler, keybindings, screens, and widgets | ~209 |
-| #465 | 2:02 PM | 🔵 | TUI screen enumeration defines five screen types including provider selection and configuration | ~192 |
-| #463 | " | 🔵 | TUI application structure includes sync status tracking and screen management | ~194 |
-| #460 | " | 🔵 | TUI app shows TODO placeholders for sync and settings functionality | ~167 |
+| #1108 | 7:02 PM | 🟣 | Added sync screen rendering to TuiApp::render() method | ~128 |
+| #1107 | " | 🔵 | Found screen navigation and rendering patterns in app.rs | ~246 |
+| #1106 | " | 🔵 | Confirmed Screen::Sync variant exists in Screen enum | ~212 |
+| #1105 | " | 🔵 | Found render pattern for screens in app.rs | ~108 |
+| #1104 | 7:01 PM | 🔵 | Sync variant already exists in Screen enum | ~209 |
+| #1103 | " | 🟣 | Phase 4.2: F5 key now navigates to Sync screen | ~217 |
+| #1102 | " | 🔴 | Found F5 key handler with TODO for sync screen | ~192 |
+| #1101 | " | 🔵 | TuiApp imports screens module but not sync screen yet | ~102 |
+| #1100 | " | 🟣 | Added sync_screen field to TuiApp struct | ~62 |
+| #1099 | 7:00 PM | 🟣 | Added sync_screen field to TuiApp initialization | ~66 |
+| #1098 | " | 🔵 | TuiApp::new() initialization requires sync_screen field | ~208 |
+| #1097 | " | 🔵 | Found TuiApp struct with existing screen states | ~249 |
+| #1096 | " | 🔵 | Found F5 key handler with TODO for sync screen integration | ~216 |
+| #1095 | " | 🔵 | TuiApp struct needs sync_screen field added | ~191 |
+| #1086 | 6:55 PM | 🟣 | Added Sync variant to Screen::name() method | ~147 |
+| #1085 | 6:54 PM | 🔵 | Found existing SyncStatus struct near Screen enum | ~188 |
+| #1084 | " | 🟣 | Added Sync variant to Screen enum | ~175 |
+| #1083 | " | 🔵 | Found Screen enum in app.rs | ~190 |
+| #1082 | " | 🔵 | Found Screen enum in app.rs | ~146 |
 </claude-mem-context>

tests/CLAUDE.md

@@ -7,22 +7,18 @@
 
 | ID | Time | T | Title | Read |
 |----|------|---|-------|------|
-| #513 | 2:19 PM | 🟣 | Nonce validator test suite created with 6 test functions | ~194 |
-| #503 | 2:15 PM | 🟣 | TUI command autocomplete test suite created with six test functions | ~234 |
-| #491 | 2:14 PM | 🟣 | TUI action handlers for settings, sync, and configuration committed to git | ~196 |
-| #483 | 2:04 PM | 🔄 | TUI action handler test updated to match new save action output format | ~165 |
-| #474 | 2:03 PM | ✅ | TUI action handlers test simplified to three core test functions | ~156 |
-| #472 | " | ✅ | TUI action handlers test imports corrected to import Action from keybindings module | ~134 |
-| #466 | 2:02 PM | 🟣 | TUI action handlers test suite created for settings, sync, and configuration | ~195 |
-| #458 | 2:01 PM | 🟣 | KeyHierarchy save/unlock implementation committed with complete key wrapping | ~202 |
-| #444 | 1:59 PM | 🟣 | KeyHierarchy integration test suite created with four test functions | ~199 |
-| #436 | 1:58 PM | 🟣 | CloudSyncService implementation committed to git repository | ~224 |
-| #433 | 1:56 PM | ✅ | CloudService test imports fixed to correct module paths | ~127 |
-| #432 | " | 🟣 | CloudSyncService integration test suite created with four async test functions | ~198 |
-| #424 | 1:55 PM | 🟣 | CloudStorage implementation committed to git repository | ~189 |
-| #413 | 1:52 PM | 🔴 | CloudStorage test type mismatches fixed with PathBuf conversion | ~153 |
-| #404 | 1:51 PM | 🟣 | CloudStorage integration test suite created with six async test functions | ~242 |
-| #401 | " | 🟣 | Cloud metadata serialization implementation committed to git repository | ~182 |
-| #398 | 1:50 PM | 🔵 | Integration test file structure confirmed with four test functions | ~206 |
-| #395 | " | ✅ | Metadata test file moved from cloud subdirectory to tests root | ~152 |
+| #1069 | 6:52 PM | 🟣 | Added tests for test_connection functionality | ~262 |
+| #1067 | " | 🔵 | Test file complete at line 261 | ~175 |
+| #1057 | 6:50 PM | 🔴 | Fixed tests to use public API instead of private fields | ~237 |
+| #1054 | " | 🔵 | End of test file reached | ~184 |
+| #1044 | 6:49 PM | 🔴 | Fixed SFTP config test to include root path field | ~161 |
+| #1037 | 6:48 PM | 🔴 | Fixed line 100 Region string literal error | ~165 |
+| #1035 | 6:47 PM | 🔴 | Fixed line 36 string literal error | ~172 |
+| #1029 | 6:46 PM | 🔴 | Fixed SFTP config test to include root path field | ~178 |
+| #1026 | " | 🔵 | Found existing provider config integration tests | ~289 |
+| #1021 | 6:44 PM | 🔴 | Fixed all SensitiveString access in passkey_test.rs | ~90 |
+| #1020 | " | 🔵 | Found SensitiveString access issues in passkey_test.rs | ~122 |
+| #1015 | " | 🔴 | Fixed SensitiveString access in onboarding_test.rs | ~186 |
+| #1014 | 6:43 PM | 🔴 | Fixed SensitiveString access in sync_integration_test.rs | ~90 |
+| #1009 | " | 🔴 | Fixing SensitiveString field access in sync_integration_test.rs | ~91 |
 </claude-mem-context>

tests/keystore_test.rs

@@ -8,10 +8,10 @@ fn keystore_roundtrip_unlock() {
 
     let keystore = KeyStore::initialize(&path, master).unwrap();
     assert!(path.exists());
-    assert_eq!(keystore.dek.len(), 32);
+    assert_eq!(keystore.dek.get().len(), 32);
 
     let unlocked = KeyStore::unlock(&path, master).unwrap();
-    assert_eq!(unlocked.dek.len(), 32);
+    assert_eq!(unlocked.dek.get().len(), 32);
 }
 
 #[test]

tests/onboarding_test.rs

@@ -9,6 +9,6 @@ fn onboarding_initializes_keystore_file() {
     assert!(!is_initialized(&path));
     let keystore = initialize_keystore(&path, "correct-horse-battery-staple").unwrap();
     assert!(path.exists());
-    assert_eq!(keystore.dek.len(), 32);
+    assert_eq!(keystore.dek.get().len(), 32);
     assert!(is_initialized(&path));
 }

tests/passkey_test.rs

@@ -17,7 +17,7 @@ fn test_generate_passkey_24_words() {
 fn test_passkey_to_seed() {
     let passkey = Passkey::generate(24).unwrap();
     let seed = passkey.to_seed(None).unwrap();
-    assert_eq!(seed.0.len(), 64); // BIP39 seed is 64 bytes
+    assert_eq!(seed.get().len(), 64); // BIP39 seed is 64 bytes
 }
 
 #[test]
@@ -27,8 +27,8 @@ fn test_passkey_from_words() {
 
     let restored = Passkey::from_words(&words).unwrap();
     assert_eq!(
-        original.to_seed(None).unwrap().0,
-        restored.to_seed(None).unwrap().0
+        original.to_seed(None).unwrap().get(),
+        restored.to_seed(None).unwrap().get()
     );
 }
 
@@ -39,5 +39,5 @@ fn test_passkey_with_optional_passphrase() {
     let seed_with_passphrase = passkey.to_seed(Some("test-passphrase")).unwrap();
 
     // Different passphrases should produce different seeds
-    assert_ne!(seed_no_passphrase.0, seed_with_passphrase.0);
+    assert_ne!(seed_no_passphrase.get(), seed_with_passphrase.get());
 }

tests/sync_integration_test.rs

@@ -30,7 +30,7 @@ async fn test_full_sync_flow_with_passkey() {
 
     // Step 2: Convert Passkey to seed
     let seed = passkey.to_seed(None).unwrap();
-    assert_eq!(seed.0.len(), 64);
+    assert_eq!(seed.get().len(), 64);
 
     // Step 3: Derive root master key from Passkey seed
     let salt = [1u8; 16]; // In production, this would be a random salt

tests/sync_nonce_validator_test.rs

@@ -30,20 +30,26 @@ fn test_recovery_strategy_mismatch_nonce() {
 #[test]
 fn test_prompt_user_resolution_returns_strategy() {
     let validator = NonceValidator::new();
-    let result = validator.prompt_user_resolution("test-record");
+    let local_nonce = [1u8; 12];
+    let remote_nonce = [2u8; 12];
+    let result = validator.prompt_user_resolution(&local_nonce, &remote_nonce);
 
-    // Should return Some strategy (currently defaults to UseLocal)
-    assert!(result.is_some());
+    // Should return Ok strategy (currently defaults to UseLocal)
+    assert!(result.is_ok());
     assert_eq!(result.unwrap(), RecoveryStrategy::UseLocal);
 }
 
 #[test]
 fn test_prompt_user_resolution_different_record_names() {
     let validator = NonceValidator::new();
-
-    // Test with different record names
-    for name in &["github", "gitlab", "aws", "database"] {
-        let result = validator.prompt_user_resolution(name);
-        assert!(result.is_some(), "Should return strategy for record: {}", name);
+    let local_nonce = [1u8; 12];
+    let remote_nonce = [2u8; 12];
+
+    // Test with different nonces
+    for i in 0..4 {
+        let local = [i; 12];
+        let remote = [i + 1; 12];
+        let result = validator.prompt_user_resolution(&local, &remote);
+        assert!(result.is_ok(), "Should return strategy for nonces");
     }
 }