chore: test cleanup

Author: uroybd

src/config/tests.rs

@@ -1,185 +1,3 @@
-// FIXME: These tests are for obsolete set_profile_context method
-// #[cfg(test)]
-// mod profile_context_tests {
-//     use crate::config::Config;
-//     use crate::context::Context;
-//     use std::env;
-//     use toml::Table;
-
-//     #[test]
-//     fn test_set_profile_context_with_explicit_profile() {
-//         let temp_dir = env::temp_dir().join(format!("dotr_test_{}", uuid::Uuid::new_v4()));
-//         std::fs::create_dir_all(&temp_dir).unwrap();
-
-//         let mut config = Config::new();
-//         config.profiles.insert(
-//             "test-profile".to_string(),
-//             crate::profile::Profile::new("test-profile"),
-//         );
-
-//         let mut ctx = Context::new(&temp_dir, &config, &None, false).unwrap();
-//         let result = config.update_profiles(&Some("test-profile".to_string()), &mut ctx, false);
-
-//         assert!(result.is_ok());
-//         assert!(ctx.profile.is_some());
-//         assert_eq!(ctx.profile.unwrap().name, "test-profile");
-
-//         std::fs::remove_dir_all(&temp_dir).ok();
-//     }
-
-//     #[test]
-//     fn test_set_profile_context_falls_back_to_default() {
-//         let temp_dir = env::temp_dir().join(format!("dotr_test_{}", uuid::Uuid::new_v4()));
-//         std::fs::create_dir_all(&temp_dir).unwrap();
-
-//         let mut config = Config::new();
-//         let mut ctx = Context::new(&temp_dir, &config, &None, false).unwrap();
-
-//         let result = config.update_profiles(&None, &mut ctx, false);
-
-//         assert!(result.is_ok());
-//         assert!(ctx.profile.is_some());
-//         assert_eq!(ctx.profile.unwrap().name, "default");
-
-//         std::fs::remove_dir_all(&temp_dir).ok();
-//     }
-
-//     #[test]
-//     fn test_set_profile_context_uses_dotr_profile_env_var() {
-//         let temp_dir = env::temp_dir().join(format!("dotr_test_{}", uuid::Uuid::new_v4()));
-//         std::fs::create_dir_all(&temp_dir).unwrap();
-
-//         let mut config = Config::new();
-//         config.profiles.insert(
-//             "env-profile".to_string(),
-//             crate::profile::Profile::new("env-profile"),
-//         );
-
-//         let mut ctx = Context::new(&temp_dir, &config, &None, false).unwrap();
-//         let mut vars = ctx.get_context_variables();
-//         vars.insert(
-//             "DOTR_PROFILE".to_string(),
-//             toml::Value::String("env-profile".to_string()),
-//         );
-//         ctx.extend_variables(vars);
-
-//         let result = config.update_profiles(&None, &mut ctx, false);
-
-//         assert!(result.is_ok());
-//         assert!(ctx.profile.is_some());
-//         assert_eq!(ctx.profile.unwrap().name, "env-profile");
-
-//         std::fs::remove_dir_all(&temp_dir).ok();
-//     }
-
-//     #[test]
-//     fn test_set_profile_context_always_create_missing_default_profile() {
-//         let temp_dir = env::temp_dir().join(format!("dotr_test_{}", uuid::Uuid::new_v4()));
-//         std::fs::create_dir_all(&temp_dir).unwrap();
-
-//         let mut config = Config::new();
-//         let mut ctx = Context::new(&temp_dir, &config, &None, false).unwrap();
-
-//         let result = config.update_profiles(&Some("default".to_string()), &mut ctx, false);
-
-//         assert!(result.is_ok());
-//         assert!(config.profiles.contains_key("default"));
-//         assert!(ctx.profile.is_some());
-//         assert_eq!(ctx.profile.unwrap().name, "default");
-
-//         std::fs::remove_dir_all(&temp_dir).ok();
-//     }
-
-//     #[test]
-//     fn test_set_profile_context_creates_missing_profile() {
-//         let temp_dir = env::temp_dir().join(format!("dotr_test_{}", uuid::Uuid::new_v4()));
-//         std::fs::create_dir_all(&temp_dir).unwrap();
-
-//         let mut config = Config::new();
-//         let mut ctx = Context::new(&temp_dir, &config, &None, false).unwrap();
-
-//         let result = config.update_profiles(&Some("new-profile".to_string()), &mut ctx, true);
-
-//         assert!(result.is_ok());
-//         assert!(config.profiles.contains_key("new-profile"));
-//         assert!(ctx.profile.is_some());
-//         assert_eq!(ctx.profile.unwrap().name, "new-profile");
-
-//         std::fs::remove_dir_all(&temp_dir).ok();
-//     }
-
-//     #[test]
-//     fn test_set_profile_context_fails_when_profile_not_found() {
-//         let temp_dir = env::temp_dir().join(format!("dotr_test_{}", uuid::Uuid::new_v4()));
-//         std::fs::create_dir_all(&temp_dir).unwrap();
-
-//         let mut config = Config::new();
-//         let mut ctx = Context::new(&temp_dir, &config, &None, false).unwrap();
-
-//         let result = config.update_profiles(&Some("nonexistent".to_string()), &mut ctx, false);
-
-//         assert!(result.is_err());
-//         assert!(
-//             result
-//                 .unwrap_err()
-//                 .to_string()
-//                 .contains("Profile nonexistent not found")
-//         );
-
-//         std::fs::remove_dir_all(&temp_dir).ok();
-//     }
-
-//     #[test]
-//     fn test_set_profile_context_explicit_overrides_env() {
-//         let temp_dir = env::temp_dir().join(format!("dotr_test_{}", uuid::Uuid::new_v4()));
-//         std::fs::create_dir_all(&temp_dir).unwrap();
-
-//         let mut config = Config::new();
-//         config.profiles.insert(
-//             "explicit-profile".to_string(),
-//             crate::profile::Profile::new("explicit-profile"),
-//         );
-//         config.profiles.insert(
-//             "env-profile".to_string(),
-//             crate::profile::Profile::new("env-profile"),
-//         );
-
-//         let mut ctx = Context::new(&temp_dir, &config, &None, false).unwrap();
-//         let mut vars = ctx.get_context_variables();
-//         vars.insert(
-//             "DOTR_PROFILE".to_string(),
-//             toml::Value::String("env-profile".to_string()),
-//         );
-//         ctx.extend_variables(vars);
-
-//         let result = config.update_profiles(&Some("explicit-profile".to_string()), &mut ctx, false);
-
-//         assert!(result.is_ok());
-//         assert_eq!(ctx.profile.unwrap().name, "explicit-profile");
-
-//         std::fs::remove_dir_all(&temp_dir).ok();
-//     }
-
-//     #[test]
-//     fn test_config_new_creates_default_profile() {
-//         let config = Config::new();
-
-//         assert!(config.profiles.contains_key("default"));
-//         assert_eq!(config.profiles.get("default").unwrap().name, "default");
-//     }
-
-//     #[test]
-//     fn test_config_from_table_preserves_default_profile() {
-//         let mut table = Table::new();
-//         table.insert("banner".to_string(), toml::Value::Boolean(true));
-
-//         let config = Config::from_table(&table).unwrap();
-
-//         // from_table doesn't auto-create default, that's only in new()
-//         assert!(!config.profiles.contains_key("default"));
-//     }
-// }
-
 #[cfg(test)]
 mod filter_packages_tests {
     use crate::config::Config;