Fix clippy and split test

Author: Gijsreyn

dsc/src/mcp/invoke_dsc_resource.rs

@@ -7,7 +7,6 @@ use dsc_lib::{
     dscresources::{
         dscresource::Invoke,
         invoke_result::{
-            DeleteResult,
             ExportResult,
             GetResult,
             SetResult,
@@ -39,7 +38,7 @@ pub enum ResourceOperationResult {
     SetResult(SetResult),
     TestResult(TestResult),
     ExportResult(ExportResult),
-    DeleteResult(DeleteResult),
+    DeleteResult { success: bool },
 }
 
 #[derive(Serialize, JsonSchema)]
@@ -98,11 +97,10 @@ impl McpServer {
                     Ok(ResourceOperationResult::TestResult(result))
                 },
                 DscOperation::Delete => {
-                    let result = match resource.delete(&properties_json, &ExecutionKind::Actual) {
-                        Ok(res) => res,
+                    match resource.delete(&properties_json) {
+                        Ok(()) => Ok(ResourceOperationResult::DeleteResult { success: true }),
                         Err(e) => return Err(McpError::internal_error(e.to_string(), None)),
-                    };
-                    Ok(ResourceOperationResult::DeleteResult(result))
+                    }
                 },
                 DscOperation::Export => {
                     let result = match resource.export(&properties_json) {

dsc/tests/dsc_mcp.tests.ps1

@@ -304,10 +304,9 @@ Describe 'Tests for MCP server' {
         @{ operation = 'get'; property = 'actualState' }
         @{ operation = 'set'; property = 'beforeState' }
         @{ operation = 'test'; property = 'desiredState' }
-        @{ operation = 'delete'; property = 'beforeState' }
         @{ operation = 'export'; property = 'actualState' }
     ) {
-        param($operation)
+        param($operation, $property)
 
         $mcpRequest = @{
             jsonrpc = "2.0"
@@ -334,4 +333,30 @@ Describe 'Tests for MCP server' {
         $response.result.structuredContent.result.$property.action | Should -BeExactly $operation -Because $because
         $response.result.structuredContent.result.$property.hello | Should -BeExactly "World" -Because $because
     }
+
+    It 'Calling invoke_dsc_resource for delete operation' {
+        $mcpRequest = @{
+            jsonrpc = "2.0"
+            id = 12
+            method = "tools/call"
+            params = @{
+                name = "invoke_dsc_resource"
+                arguments = @{
+                    type = 'Test/Operation'
+                    operation = 'delete'
+                    resource_type = 'Test/Operation'
+                    properties_json = (@{
+                        hello = "World"
+                        action = 'delete'
+                    } | ConvertTo-Json -Depth 20)
+                }
+            }
+        }
+
+        $response = Send-McpRequest -request $mcpRequest
+        $response.id | Should -Be 12
+        $because = ($response | ConvertTo-Json -Depth 20 | Out-String)
+        ($response.result.structuredContent.psobject.properties | Measure-Object).Count | Should -Be 1 -Because $because
+        $response.result.structuredContent.result.success | Should -Be $true -Because $because
+    }
 }