test: check the existence of target_path_parent before detelting (#33812)

This commit is contained in:
Yihau Chen 2023-10-23 22:14:34 +08:00 committed by GitHub
parent cb736123b2
commit 7d56c3484c
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 10 additions and 8 deletions

View File

@ -626,14 +626,16 @@ fn build_solana_package(
// The package version directory doesn't contain a valid
// installation, and it should be removed.
let target_path_parent = target_path.parent().expect("Invalid package path");
fs::remove_dir_all(target_path_parent).unwrap_or_else(|err| {
error!(
"Failed to remove {} while recovering from installation failure: {}",
target_path_parent.to_string_lossy(),
err,
);
exit(1);
});
if target_path_parent.exists() {
fs::remove_dir_all(target_path_parent).unwrap_or_else(|err| {
error!(
"Failed to remove {} while recovering from installation failure: {}",
target_path_parent.to_string_lossy(),
err,
);
exit(1);
});
}
error!("Failed to install platform-tools: {}", err);
exit(1);
});