[PATCH test-tools 3/4] instance: source certificate and task types directly

["Michael Köppl" <m.koeppl@proxmox.com>]

Both types pdm-api-types provided here were a detour. CertificateInfo is
its re-export of proxmox_acme_api::CertificateInfo, so take it from
there. TaskStateType is simple enough to warrant a direct declaration
here.

This leaves proxmox-test-instance free of pdm crates.

Signed-off-by: Michael Köppl <m.koeppl@proxmox.com>
---
 proxmox-test-instance/Cargo.toml            |  2 +-
 proxmox-test-instance/src/api/pve_client.rs | 26 +++++++++++++++++++--
 2 files changed, 25 insertions(+), 3 deletions(-)

diff --git a/proxmox-test-instance/Cargo.toml b/proxmox-test-instance/Cargo.toml
index 5a7c333..9cbfc9b 100644
--- a/proxmox-test-instance/Cargo.toml
+++ b/proxmox-test-instance/Cargo.toml
@@ -9,7 +9,6 @@ repository.workspace = true
 
 [dependencies]
 proxmox-test-common = { workspace = true }
-pdm-api-types = { workspace = true }
 
 anyhow.workspace = true
 dirs-next.workspace = true
@@ -28,6 +27,7 @@ termcolor.workspace = true
 tokio = { workspace = true, features = [ "rt" ]}
 toml.workspace = true
 
+proxmox-acme-api.workspace = true
 proxmox-async.workspace = true
 proxmox-auth-api = { workspace = true, features = [ "api-types" ] }
 proxmox-client = { workspace = true, features = ["hyper-client"] }
diff --git a/proxmox-test-instance/src/api/pve_client.rs b/proxmox-test-instance/src/api/pve_client.rs
index f719895..4e4c1f4 100644
--- a/proxmox-test-instance/src/api/pve_client.rs
+++ b/proxmox-test-instance/src/api/pve_client.rs
@@ -2,7 +2,7 @@ use std::net::Ipv4Addr;
 use std::{collections::HashSet, sync::Arc, time::Duration};
 
 use anyhow::{Context, Error, bail};
-use pdm_api_types::{CertificateInfo, TaskStateType};
+use proxmox_acme_api::CertificateInfo;
 use proxmox_client::{AuthenticationKind, Client, HttpApiClient, HttpApiResponse, TlsOptions};
 use proxmox_http::HttpOptions;
 use proxmox_login::Login;
@@ -33,6 +33,28 @@ const SHUTDOWN_TIMEOUT_SECS: u64 = 300;
 /// This does not cancel the server-side task; large or slow operations can outlive this deadline.
 const TASK_TIMEOUT: Duration = Duration::from_secs(60 * 60);
 
+/// The state a finished PVE task ended up in, as derived from its exit status.
+enum TaskStateType {
+    Ok,
+    Warning,
+    Error,
+    Unknown,
+}
+
+impl TaskStateType {
+    fn new_from_str(status: &str) -> Self {
+        if status == "unknown" || status.is_empty() {
+            TaskStateType::Unknown
+        } else if status == "OK" {
+            TaskStateType::Ok
+        } else if status.starts_with("WARNINGS: ") {
+            TaskStateType::Warning
+        } else {
+            TaskStateType::Error
+        }
+    }
+}
+
 #[derive(Debug)]
 struct TaskFailure {
     upid: String,
@@ -1049,7 +1071,7 @@ where
 
             let exitstatus = value["exitstatus"].as_str().unwrap_or("unknown");
             return match TaskStateType::new_from_str(exitstatus) {
-                TaskStateType::OK | TaskStateType::Warning => Ok(()),
+                TaskStateType::Ok | TaskStateType::Warning => Ok(()),
                 TaskStateType::Error | TaskStateType::Unknown => {
                     bail!(TaskFailure {
                         upid: upid.to_string(),
-- 
2.47.3