github/kata-containers
1
0
Fork 2
mirror of https://github.com/kata-containers/kata-containers.git synced 2026-07-01 14:38:33 +00:00
Code Issues Projects Releases Wiki Activity

libs: Fix shim-interface tests after removing create_dir_all

Browse Source 
Two tests relied on the side-effect of create_dir_all (removed in
the previous commit) to pass:

(1) test_get_uds_with_sid_ok: use a directory name that actually
starts with the search prefix so prefix matching works without
creating dirs.
(2) test_get_uds_with_sid_with_zero: assert Err on zero matches
instead of Ok, matching the corrected lookup behavior.

Signed-off-by: Alex Lyn <alex.lyn@antgroup.com>
This commit is contained in:
Alex Lyn
2026-05-14 22:54:11 +08:00
parent 4764e31d00
commit aef3ab8f32
1 changed files with 5 additions and 14 deletions
Download Patch File Download Diff File Expand all files Collapse all files

19
src/libs/shim-interface/src/lib.rs
View File

@@ -130,7 +130,7 @@ mod tests {
#[test]
fn test_get_uds_with_sid_ok() {
let run_path = tempdir().unwrap();
let dir = run_path.path().join("aata98654dangboxpath1");
let dir = run_path.path().join("kata98654sandboxpath1");
fs::create_dir_all(dir.as_path()).unwrap();
let result = get_uds_with_sid("kata", &run_path.path().display().to_string());
@@ -141,7 +141,7 @@ mod tests {
"unix://{}",
run_path
.path()
.join("kata")
.join("kata98654sandboxpath1")
.join(SHIM_MGMT_SOCK_NAME)
.display()
)
@@ -152,18 +152,9 @@ mod tests {
fn test_get_uds_with_sid_with_zero() {
let run_path = tempdir().unwrap();
let result = get_uds_with_sid("acdsdfe", &run_path.path().display().to_string());
assert!(result.is_ok());
assert_eq!(
result.unwrap(),
format!(
"unix://{}",
run_path
.path()
.join("acdsdfe")
.join(SHIM_MGMT_SOCK_NAME)
.display()
)
)
assert!(result.is_err());
let err_msg = result.unwrap_err().to_string();
assert!(err_msg.contains("is not found"));
}
#[test]