Merge pull request #436 from a-kenji/fix-default-config-test

Enables the Default Config Test
This commit is contained in:
a-kenji 2021-05-02 19:26:17 +02:00 committed by GitHub
commit bf6386f495
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
2 changed files with 12 additions and 8 deletions

View File

@ -199,12 +199,10 @@ mod config_test {
assert_eq!(result.unwrap(), Config::default());
}
// This test needs a split somewhere between test and normal runs,
// since otherwise it would look in a local configuration and fail.
//#[test]
//fn try_from_cli_args_default() {
//let opts = CliArgs::default();
//let result = Config::try_from(&opts);
//assert_eq!(result.unwrap(), Config::default());
//}
#[test]
fn try_from_cli_args_default() {
let opts = CliArgs::default();
let result = Config::try_from(&opts);
assert_eq!(result.unwrap(), Config::default());
}
}

View File

@ -48,6 +48,7 @@ pub mod install {
}
}
#[cfg(not(test))]
pub fn find_default_config_dir() -> Option<PathBuf> {
vec![
Some(xdg_config_dir()),
@ -60,6 +61,11 @@ pub fn find_default_config_dir() -> Option<PathBuf> {
.flatten()
}
#[cfg(test)]
pub fn find_default_config_dir() -> Option<PathBuf> {
None
}
pub fn xdg_config_dir() -> PathBuf {
let project_dirs = ProjectDirs::from("org", "Zellij Contributors", "Zellij").unwrap();
project_dirs.config_dir().to_owned()