From ad129057b56bb3ba3e387dec5e2791ba5c545b06 Mon Sep 17 00:00:00 2001
From: Marc Hartmayer <marc@linux.ibm.com>
Date: Fri, 24 Jul 2026 11:45:52 +0200
Subject: [PATCH] pvattest: tests: Implement TODO

Instead of blindly casting the values to u32 use try_into().unwrap().

Signed-off-by: Marc Hartmayer <marc@linux.ibm.com>
Reviewed-by: Steffen Eiden <seiden@linux.ibm.com>
Signed-off-by: Steffen Eiden <seiden@linux.ibm.com>
---
 rust/pvattest/src/exchange.rs | 9 ++++++---
 1 file changed, 6 insertions(+), 3 deletions(-)

diff --git a/rust/pvattest/src/exchange.rs b/rust/pvattest/src/exchange.rs
index 294cc449..5ee2eb93 100644
--- a/rust/pvattest/src/exchange.rs
+++ b/rust/pvattest/src/exchange.rs
@@ -623,9 +623,12 @@ mod test {
         measurement: usize,
         additional: usize,
     ) {
-        // TODO as 32 checks
-        let ctx_write = ExchangeFormatRequest::new(arcb, measurement as u32, additional as u32)
-            .expect("exchange fmt creation");
+        let ctx_write = ExchangeFormatRequest::new(
+            arcb,
+            measurement.try_into().unwrap(),
+            additional.try_into().unwrap(),
+        )
+        .expect("exchange fmt creation");
 
         // let mut out = create_file(path).unwrap();
         let mut out = vec![];