client tests now have a proper structure

client/src/services/grpc_client.rs

@@ -14,7 +14,10 @@ use common::proto::multieko2::table_script::{
 use common::proto::multieko2::tables_data::{
     tables_data_client::TablesDataClient,
     GetTableDataByPositionRequest,
+    GetTableDataRequest,          // ADD THIS
     GetTableDataResponse,
+    DeleteTableDataRequest,       // ADD THIS
+    DeleteTableDataResponse,      // ADD THIS
     GetTableDataCountRequest,
     PostTableDataRequest, PostTableDataResponse, PutTableDataRequest,
     PutTableDataResponse,
@@ -116,7 +119,7 @@ impl GrpcClient {
         Ok(response.into_inner())
     }
 
-    // NEW Methods for TablesData service
+    // Existing TablesData methods
     pub async fn get_table_data_count(
         &mut self,
         profile_name: String,
@@ -135,7 +138,7 @@ impl GrpcClient {
         Ok(response.into_inner().count as u64)
     }
 
-pub async fn get_table_data_by_position(
+    pub async fn get_table_data_by_position(
         &mut self,
         profile_name: String,
         table_name: String,
@@ -155,18 +158,58 @@ pub async fn get_table_data_by_position(
         Ok(response.into_inner())
     }
 
+    // ADD THIS: Missing get_table_data method
+    pub async fn get_table_data(
+        &mut self,
+        profile_name: String,
+        table_name: String,
+        id: i64,
+    ) -> Result<GetTableDataResponse> {
+        let grpc_request = GetTableDataRequest {
+            profile_name,
+            table_name,
+            id,
+        };
+        let request = tonic::Request::new(grpc_request);
+        let response = self
+            .tables_data_client
+            .get_table_data(request)
+            .await
+            .context("gRPC GetTableData call failed")?;
+        Ok(response.into_inner())
+    }
+
+    // ADD THIS: Missing delete_table_data method
+    pub async fn delete_table_data(
+        &mut self,
+        profile_name: String,
+        table_name: String,
+        record_id: i64,
+    ) -> Result<DeleteTableDataResponse> {
+        let grpc_request = DeleteTableDataRequest {
+            profile_name,
+            table_name,
+            record_id,
+        };
+        let request = tonic::Request::new(grpc_request);
+        let response = self
+            .tables_data_client
+            .delete_table_data(request)
+            .await
+            .context("gRPC DeleteTableData call failed")?;
+        Ok(response.into_inner())
+    }
+
     pub async fn post_table_data(
         &mut self,
         profile_name: String,
         table_name: String,
-        // CHANGE THIS: Accept the pre-converted data
         data: HashMap<String, Value>,
     ) -> Result<PostTableDataResponse> {
-        // The conversion logic is now gone from here.
         let grpc_request = PostTableDataRequest {
             profile_name,
             table_name,
-            data, // This is now the correct type
+            data,
         };
         let request = tonic::Request::new(grpc_request);
         let response = self
@@ -182,15 +225,13 @@ pub async fn get_table_data_by_position(
         profile_name: String,
         table_name: String,
         id: i64,
-        // CHANGE THIS: Accept the pre-converted data
         data: HashMap<String, Value>,
     ) -> Result<PutTableDataResponse> {
-        // The conversion logic is now gone from here.
         let grpc_request = PutTableDataRequest {
             profile_name,
             table_name,
             id,
-            data, // This is now the correct type
+            data,
         };
         let request = tonic::Request::new(grpc_request);
         let response = self