forked from Proxmox/proxmox
8ada17854d
Get handles for the available repositories along with their current configuration status and make it possible to add them. Signed-off-by: Fabian Ebner <f.ebner@proxmox.com>
347 lines
11 KiB
Rust
347 lines
11 KiB
Rust
use std::path::PathBuf;
|
|
|
|
use anyhow::{bail, format_err, Error};
|
|
|
|
use proxmox_apt::repositories::{
|
|
check_repositories, standard_repositories, APTRepositoryFile, APTRepositoryHandle,
|
|
APTRepositoryInfo, APTStandardRepository,
|
|
};
|
|
|
|
#[test]
|
|
fn test_parse_write() -> Result<(), Error> {
|
|
let test_dir = std::env::current_dir()?.join("tests");
|
|
let read_dir = test_dir.join("sources.list.d");
|
|
let write_dir = test_dir.join("sources.list.d.actual");
|
|
let expected_dir = test_dir.join("sources.list.d.expected");
|
|
|
|
if write_dir.is_dir() {
|
|
std::fs::remove_dir_all(&write_dir)
|
|
.map_err(|err| format_err!("unable to remove dir {:?} - {}", write_dir, err))?;
|
|
}
|
|
|
|
std::fs::create_dir_all(&write_dir)
|
|
.map_err(|err| format_err!("unable to create dir {:?} - {}", write_dir, err))?;
|
|
|
|
let mut files = vec![];
|
|
let mut errors = vec![];
|
|
|
|
for entry in std::fs::read_dir(read_dir)? {
|
|
let path = entry?.path();
|
|
|
|
match APTRepositoryFile::new(&path)? {
|
|
Some(mut file) => match file.parse() {
|
|
Ok(()) => files.push(file),
|
|
Err(err) => errors.push(err),
|
|
},
|
|
None => bail!("unexpected None for '{:?}'", path),
|
|
}
|
|
}
|
|
|
|
assert!(errors.is_empty());
|
|
|
|
for file in files.iter_mut() {
|
|
let path = PathBuf::from(&file.path);
|
|
let new_path = write_dir.join(path.file_name().unwrap());
|
|
file.path = new_path.into_os_string().into_string().unwrap();
|
|
file.digest = None;
|
|
file.write()?;
|
|
}
|
|
|
|
let mut expected_count = 0;
|
|
|
|
for entry in std::fs::read_dir(expected_dir)? {
|
|
expected_count += 1;
|
|
|
|
let expected_path = entry?.path();
|
|
let actual_path = write_dir.join(expected_path.file_name().unwrap());
|
|
|
|
let expected_contents = std::fs::read(&expected_path)
|
|
.map_err(|err| format_err!("unable to read {:?} - {}", expected_path, err))?;
|
|
|
|
let actual_contents = std::fs::read(&actual_path)
|
|
.map_err(|err| format_err!("unable to read {:?} - {}", actual_path, err))?;
|
|
|
|
assert_eq!(
|
|
expected_contents, actual_contents,
|
|
"Use\n\ndiff {:?} {:?}\n\nif you're not fluent in byte decimals",
|
|
expected_path, actual_path
|
|
);
|
|
}
|
|
|
|
let actual_count = std::fs::read_dir(write_dir)?.count();
|
|
|
|
assert_eq!(expected_count, actual_count);
|
|
|
|
Ok(())
|
|
}
|
|
|
|
#[test]
|
|
fn test_digest() -> Result<(), Error> {
|
|
let test_dir = std::env::current_dir()?.join("tests");
|
|
let read_dir = test_dir.join("sources.list.d");
|
|
let write_dir = test_dir.join("sources.list.d.digest");
|
|
|
|
if write_dir.is_dir() {
|
|
std::fs::remove_dir_all(&write_dir)
|
|
.map_err(|err| format_err!("unable to remove dir {:?} - {}", write_dir, err))?;
|
|
}
|
|
|
|
std::fs::create_dir_all(&write_dir)
|
|
.map_err(|err| format_err!("unable to create dir {:?} - {}", write_dir, err))?;
|
|
|
|
let path = read_dir.join("standard.list");
|
|
|
|
let mut file = APTRepositoryFile::new(&path)?.unwrap();
|
|
file.parse()?;
|
|
|
|
let new_path = write_dir.join(path.file_name().unwrap());
|
|
file.path = new_path.clone().into_os_string().into_string().unwrap();
|
|
|
|
let old_digest = file.digest.unwrap();
|
|
|
|
// file does not exist yet...
|
|
assert!(file.read_with_digest().is_err());
|
|
assert!(file.write().is_err());
|
|
|
|
// ...but it should work if there's no digest
|
|
file.digest = None;
|
|
file.write()?;
|
|
|
|
// overwrite with old contents...
|
|
std::fs::copy(path, new_path)?;
|
|
|
|
// modify the repo
|
|
let mut repo = file.repositories.first_mut().unwrap();
|
|
repo.enabled = !repo.enabled;
|
|
|
|
// ...then it should work
|
|
file.digest = Some(old_digest);
|
|
file.write()?;
|
|
|
|
// expect a different digest, because the repo was modified
|
|
let (_, new_digest) = file.read_with_digest()?;
|
|
assert_ne!(old_digest, new_digest);
|
|
|
|
assert!(file.write().is_err());
|
|
|
|
Ok(())
|
|
}
|
|
|
|
#[test]
|
|
fn test_empty_write() -> Result<(), Error> {
|
|
let test_dir = std::env::current_dir()?.join("tests");
|
|
let read_dir = test_dir.join("sources.list.d");
|
|
let write_dir = test_dir.join("sources.list.d.remove");
|
|
|
|
if write_dir.is_dir() {
|
|
std::fs::remove_dir_all(&write_dir)
|
|
.map_err(|err| format_err!("unable to remove dir {:?} - {}", write_dir, err))?;
|
|
}
|
|
|
|
std::fs::create_dir_all(&write_dir)
|
|
.map_err(|err| format_err!("unable to create dir {:?} - {}", write_dir, err))?;
|
|
|
|
let path = read_dir.join("standard.list");
|
|
|
|
let mut file = APTRepositoryFile::new(&path)?.unwrap();
|
|
file.parse()?;
|
|
|
|
let new_path = write_dir.join(path.file_name().unwrap());
|
|
file.path = new_path.clone().into_os_string().into_string().unwrap();
|
|
|
|
file.digest = None;
|
|
|
|
file.write()?;
|
|
|
|
assert!(file.exists());
|
|
|
|
file.repositories.clear();
|
|
|
|
file.write()?;
|
|
|
|
assert!(!file.exists());
|
|
|
|
Ok(())
|
|
}
|
|
|
|
#[test]
|
|
fn test_check_repositories() -> Result<(), Error> {
|
|
let test_dir = std::env::current_dir()?.join("tests");
|
|
let read_dir = test_dir.join("sources.list.d");
|
|
|
|
let absolute_suite_list = read_dir.join("absolute_suite.list");
|
|
let mut file = APTRepositoryFile::new(&absolute_suite_list)?.unwrap();
|
|
file.parse()?;
|
|
|
|
let infos = check_repositories(&vec![file])?;
|
|
|
|
assert_eq!(infos.is_empty(), true);
|
|
let pve_list = read_dir.join("pve.list");
|
|
let mut file = APTRepositoryFile::new(&pve_list)?.unwrap();
|
|
file.parse()?;
|
|
|
|
let path_string = pve_list.into_os_string().into_string().unwrap();
|
|
|
|
let mut expected_infos = vec![];
|
|
for n in 0..=5 {
|
|
expected_infos.push(APTRepositoryInfo {
|
|
path: path_string.clone(),
|
|
index: n,
|
|
property: Some("URIs".to_string()),
|
|
kind: "badge".to_string(),
|
|
message: "official host name".to_string(),
|
|
});
|
|
}
|
|
expected_infos.sort();
|
|
|
|
let mut infos = check_repositories(&vec![file])?;
|
|
infos.sort();
|
|
|
|
assert_eq!(infos, expected_infos);
|
|
|
|
let bad_sources = read_dir.join("bad.sources");
|
|
let mut file = APTRepositoryFile::new(&bad_sources)?.unwrap();
|
|
file.parse()?;
|
|
|
|
let path_string = bad_sources.into_os_string().into_string().unwrap();
|
|
|
|
let mut expected_infos = vec![
|
|
APTRepositoryInfo {
|
|
path: path_string.clone(),
|
|
index: 0,
|
|
property: Some("Suites".to_string()),
|
|
kind: "warning".to_string(),
|
|
message: "suite 'sid' should not be used in production!".to_string(),
|
|
},
|
|
APTRepositoryInfo {
|
|
path: path_string.clone(),
|
|
index: 1,
|
|
property: Some("Suites".to_string()),
|
|
kind: "warning".to_string(),
|
|
message: "old suite 'lenny' configured!".to_string(),
|
|
},
|
|
APTRepositoryInfo {
|
|
path: path_string.clone(),
|
|
index: 2,
|
|
property: Some("Suites".to_string()),
|
|
kind: "warning".to_string(),
|
|
message: "old suite 'stretch' configured!".to_string(),
|
|
},
|
|
APTRepositoryInfo {
|
|
path: path_string.clone(),
|
|
index: 3,
|
|
property: Some("Suites".to_string()),
|
|
kind: "warning".to_string(),
|
|
message: "use the name of the stable distribution instead of 'stable'!".to_string(),
|
|
},
|
|
APTRepositoryInfo {
|
|
path: path_string.clone(),
|
|
index: 4,
|
|
property: Some("Suites".to_string()),
|
|
kind: "ignore-pre-upgrade-warning".to_string(),
|
|
message: "suite 'bookworm' should not be used in production!".to_string(),
|
|
},
|
|
APTRepositoryInfo {
|
|
path: path_string.clone(),
|
|
index: 5,
|
|
property: Some("Suites".to_string()),
|
|
kind: "warning".to_string(),
|
|
message: "suite 'testing' should not be used in production!".to_string(),
|
|
},
|
|
];
|
|
for n in 0..=5 {
|
|
expected_infos.push(APTRepositoryInfo {
|
|
path: path_string.clone(),
|
|
index: n,
|
|
property: Some("URIs".to_string()),
|
|
kind: "badge".to_string(),
|
|
message: "official host name".to_string(),
|
|
});
|
|
}
|
|
expected_infos.sort();
|
|
|
|
let mut infos = check_repositories(&vec![file])?;
|
|
infos.sort();
|
|
|
|
assert_eq!(infos, expected_infos);
|
|
|
|
Ok(())
|
|
}
|
|
#[test]
|
|
fn test_standard_repositories() -> Result<(), Error> {
|
|
let test_dir = std::env::current_dir()?.join("tests");
|
|
let read_dir = test_dir.join("sources.list.d");
|
|
|
|
let mut expected = vec![
|
|
APTStandardRepository {
|
|
handle: APTRepositoryHandle::Enterprise,
|
|
status: None,
|
|
name: APTRepositoryHandle::Enterprise.name("pve"),
|
|
},
|
|
APTStandardRepository {
|
|
handle: APTRepositoryHandle::NoSubscription,
|
|
status: None,
|
|
name: APTRepositoryHandle::NoSubscription.name("pve"),
|
|
},
|
|
APTStandardRepository {
|
|
handle: APTRepositoryHandle::Test,
|
|
status: None,
|
|
name: APTRepositoryHandle::Test.name("pve"),
|
|
},
|
|
APTStandardRepository {
|
|
handle: APTRepositoryHandle::CephPacific,
|
|
status: None,
|
|
name: APTRepositoryHandle::CephPacific.name("pve"),
|
|
},
|
|
APTStandardRepository {
|
|
handle: APTRepositoryHandle::CephPacificTest,
|
|
status: None,
|
|
name: APTRepositoryHandle::CephPacificTest.name("pve"),
|
|
},
|
|
APTStandardRepository {
|
|
handle: APTRepositoryHandle::CephOctopus,
|
|
status: None,
|
|
name: APTRepositoryHandle::CephOctopus.name("pve"),
|
|
},
|
|
APTStandardRepository {
|
|
handle: APTRepositoryHandle::CephOctopusTest,
|
|
status: None,
|
|
name: APTRepositoryHandle::CephOctopusTest.name("pve"),
|
|
},
|
|
];
|
|
|
|
let absolute_suite_list = read_dir.join("absolute_suite.list");
|
|
let mut file = APTRepositoryFile::new(&absolute_suite_list)?.unwrap();
|
|
file.parse()?;
|
|
|
|
let std_repos = standard_repositories("pve", &vec![file]);
|
|
|
|
assert_eq!(std_repos, expected);
|
|
|
|
let pve_list = read_dir.join("pve.list");
|
|
let mut file = APTRepositoryFile::new(&pve_list)?.unwrap();
|
|
file.parse()?;
|
|
|
|
let file_vec = vec![file];
|
|
|
|
let std_repos = standard_repositories("pbs", &file_vec);
|
|
|
|
expected[0].name = APTRepositoryHandle::Enterprise.name("pbs");
|
|
expected[1].name = APTRepositoryHandle::NoSubscription.name("pbs");
|
|
expected[2].name = APTRepositoryHandle::Test.name("pbs");
|
|
|
|
assert_eq!(&std_repos, &expected[0..=2]);
|
|
|
|
expected[0].status = Some(false);
|
|
expected[1].status = Some(true);
|
|
expected[0].name = APTRepositoryHandle::Enterprise.name("pve");
|
|
expected[1].name = APTRepositoryHandle::NoSubscription.name("pve");
|
|
expected[2].name = APTRepositoryHandle::Test.name("pve");
|
|
|
|
let std_repos = standard_repositories("pve", &file_vec);
|
|
|
|
assert_eq!(std_repos, expected);
|
|
|
|
Ok(())
|
|
}
|