fix: macOS CI test failures for config tests

Multiple fixes:
1. Added parent directory creation in execute_set
2. Added WAL checkpoint after metadata operations
3. Made config tests serial to avoid env var conflicts
4. Added serial_test dev-dependency
5. Increased WAL checkpoint wait times for macOS CI
6. Added UUID-based unique test directories

Co-Authored-By: Claude (glm-4.7) <noreply@anthropic.com>

Author: alphaqiu

Cargo.lock

@@ -179,6 +179,7 @@ harness = false
 [dev-dependencies]
 tempfile = "3.10"
 criterion = "0.5"
+serial_test = "3.1"
 
 [profile.release]
 opt-level = 3

Cargo.toml

@@ -73,9 +73,17 @@ async fn execute_set(key: String, value: String) -> Result<()> {
     let config = ConfigManager::new()?;
     let db_config = config.get_database_config()?;
     let db_path = PathBuf::from(db_config.path);
+
+    // Ensure parent directory exists
+    if let Some(parent) = db_path.parent() {
+        std::fs::create_dir_all(parent)?;
+    }
+
     let mut vault = Vault::open(&db_path, "")?;
 
     vault.set_metadata(&key, &value)?;
+    // Force WAL checkpoint to ensure data is persisted for subsequent reads
+    let _ = vault.conn.pragma_update(None, "wal_checkpoint", "TRUNCATE");
     println!("✓ Configuration saved successfully");
 
     Ok(())
@@ -225,6 +233,9 @@ async fn execute_reset(force: bool) -> Result<()> {
         vault.delete_metadata(key)?;
     }
 
+    // Force WAL checkpoint to ensure deletes are persisted
+    let _ = vault.conn.pragma_update(None, "wal_checkpoint", "TRUNCATE");
+
     if !custom_keys.is_empty() {
         println!(
             "   ✓ Cleared {} custom configuration value(s)",

src/cli/commands/config.rs

@@ -8,6 +8,9 @@ use keyring_cli::cli::commands::config::{execute, ConfigCommands};
 use keyring_cli::db::Vault;
 use tempfile::TempDir;
 
+// Tests must run serially because they use global environment variables
+use serial_test::serial;
+
 /// Helper to set up test environment and clean up afterwards
 struct TestEnv {
     _temp_dir: TempDir,
@@ -16,14 +19,19 @@ struct TestEnv {
 
 impl TestEnv {
     fn setup(test_name: &str) -> Self {
+        // Brief delay to ensure previous test cleanup completes
+        std::thread::sleep(std::time::Duration::from_millis(50));
+
         // Clean up any existing environment variables first
         std::env::remove_var("OK_CONFIG_DIR");
         std::env::remove_var("OK_DATA_DIR");
         std::env::remove_var("OK_MASTER_PASSWORD");
 
         let temp_dir = TempDir::new().unwrap();
-        let config_dir = temp_dir.path().join(format!("config_{}", test_name));
-        let data_dir = temp_dir.path().join(format!("data_{}", test_name));
+        // Use UUID for unique database file name to avoid conflicts between tests
+        let unique_id = uuid::Uuid::new_v4().to_string()[..8].to_string();
+        let config_dir = temp_dir.path().join(format!("config_{}_{}", test_name, unique_id));
+        let data_dir = temp_dir.path().join(format!("data_{}_{}", test_name, unique_id));
         std::env::set_var("OK_CONFIG_DIR", config_dir.to_str().unwrap());
         std::env::set_var("OK_DATA_DIR", data_dir.to_str().unwrap());
         std::env::set_var("OK_MASTER_PASSWORD", "test-password");
@@ -49,9 +57,11 @@ impl Drop for TestEnv {
 }
 
 #[test]
+#[serial]
 fn test_config_set_persists_to_metadata() {
     let _env = TestEnv::setup("set_persists");
 
+
     // Set a config value
     let set_command = ConfigCommands::Set {
         key: "clipboard.timeout".to_string(),
@@ -64,7 +74,8 @@ fn test_config_set_persists_to_metadata() {
         .unwrap();
 
     // Give time for WAL to checkpoint and for all connections to close
-    std::thread::sleep(std::time::Duration::from_millis(500));
+    // macOS CI needs more time for WAL checkpoint
+    std::thread::sleep(std::time::Duration::from_secs(1));
 
     // Drop the vault from execute() before opening a new one
     // Verify it was saved to metadata
@@ -79,9 +90,11 @@ fn test_config_set_persists_to_metadata() {
 }
 
 #[test]
+#[serial]
 fn test_config_get_reads_from_metadata() {
     let _env = TestEnv::setup("get_reads");
 
+
     // Set a value in metadata
     {
         let mut vault = Vault::open(&_env.db_path, "").unwrap();
@@ -103,18 +116,22 @@ fn test_config_get_reads_from_metadata() {
 }
 
 #[test]
+#[serial]
 fn test_config_reset_clears_custom_metadata() {
     let _env = TestEnv::setup("reset_clears");
 
+
     // Set custom values directly in metadata
     {
         let mut vault = Vault::open(&_env.db_path, "").unwrap();
         vault.set_metadata("custom.key1", "value1").unwrap();
         vault.set_metadata("custom.key2", "value2").unwrap();
+        // Force WAL checkpoint to ensure data is persisted
+        let _ = vault.conn.pragma_update(None, "wal_checkpoint", "TRUNCATE");
     }
 
-    // Give time for WAL to checkpoint
-    std::thread::sleep(std::time::Duration::from_millis(200));
+    // Give time for WAL to checkpoint - macOS CI needs more time
+    std::thread::sleep(std::time::Duration::from_millis(500));
 
     // Verify they were set
     let vault = Vault::open(&_env.db_path, "").unwrap();
@@ -129,7 +146,8 @@ fn test_config_reset_clears_custom_metadata() {
 
     // Close vault to release lock
     drop(vault);
-    std::thread::sleep(std::time::Duration::from_millis(100));
+    // Give more time for WAL checkpoint and lock release on macOS CI
+    std::thread::sleep(std::time::Duration::from_millis(300));
 
     // Reset config
     let reset_command = ConfigCommands::Reset { force: true };
@@ -140,7 +158,8 @@ fn test_config_reset_clears_custom_metadata() {
         .unwrap();
 
     // Give time for WAL to checkpoint and for all connections to close
-    std::thread::sleep(std::time::Duration::from_millis(500));
+    // macOS CI needs more time for WAL checkpoint
+    std::thread::sleep(std::time::Duration::from_secs(1));
 
     // Verify custom metadata was cleared
     let vault = Vault::open(&_env.db_path, "").unwrap();
@@ -160,9 +179,11 @@ fn test_config_reset_clears_custom_metadata() {
 }
 
 #[test]
+#[serial]
 fn test_config_set_validates_key() {
     let _env = TestEnv::setup("validates_key");
 
+
     // Try to set an invalid key (should be rejected)
     let set_command = ConfigCommands::Set {
         key: "invalid.unauthorized.key".to_string(),