Fix not parsing expressions for Group resources

Author: SteveL-MSFT

dsc/tests/dsc_expressions.tests.ps1

@@ -79,4 +79,32 @@ string.
 "@.Replace("`r", "")
       $out.results[1].result.actualState.output | Should -BeExactly "This is a single-quote: '"
     }
+
+    It 'Nested Group resource does not invoke expressions' {
+      $yaml = @'
+$schema: https://aka.ms/dsc/schemas/v3/bundled/config/document.json
+resources:
+- name: Nested Group
+  type: Microsoft.DSC/Group
+  properties:
+    $schema: https://aka.ms/dsc/schemas/v3/bundled/config/document.json
+    resources:
+    - name: Deeply nested OSInfo
+      type: Microsoft/OSInfo
+      properties: {}
+    - name: Deeply nested echo
+      type: Microsoft.DSC.Debug/Echo
+      properties:
+        output:  >-
+          [reference(
+            resourceId('Microsoft/OSInfo', 'Deeply nested OSInfo')
+          )]
+      dependsOn:
+        - "[resourceId('Microsoft/OSInfo', 'Deeply nested OSInfo')]"
+'@
+
+      $out = dsc config get -i $yaml | ConvertFrom-Json
+      $LASTEXITCODE | Should -Be 0
+      $out.results[0].result[1].result.actualState.output.family | Should -BeExactly $out.results[0].result[0].result.actualState.family
+    }
 }

dsc_lib/src/configure/mod.rs

@@ -248,10 +248,19 @@ impl Configurator {
         for resource in resources {
             progress.set_resource(&resource.name, &resource.resource_type);
             progress.write_activity(format!("Get '{}'", resource.name).as_str());
-            let properties = self.invoke_property_expressions(resource.properties.as_ref())?;
-            let Some(dsc_resource) = self.discovery.find_resource(&resource.resource_type) else {
+            let discovery = &self.discovery.clone();
+            let Some(dsc_resource) = discovery.find_resource(&resource.resource_type) else {
                 return Err(DscError::ResourceNotFound(resource.resource_type));
             };
+            let properties = match &dsc_resource.kind {
+                Kind::Group => {
+                    // if Group resource, we leave it to the resource to handle expressions
+                    resource.properties.clone()
+                },
+                _ => {
+                    self.invoke_property_expressions(resource.properties.as_ref())?
+                },
+            };
             debug!("resource_type {}", &resource.resource_type);
             let filter = add_metadata(&dsc_resource.kind, properties)?;
             trace!("filter: {filter}");
@@ -325,10 +334,19 @@ impl Configurator {
         for resource in resources {
             progress.set_resource(&resource.name, &resource.resource_type);
             progress.write_activity(format!("Set '{}'", resource.name).as_str());
-            let properties = self.invoke_property_expressions(resource.properties.as_ref())?;
-            let Some(dsc_resource) = self.discovery.find_resource(&resource.resource_type) else {
+            let discovery = &self.discovery.clone();
+            let Some(dsc_resource) = discovery.find_resource(&resource.resource_type) else {
                 return Err(DscError::ResourceNotFound(resource.resource_type));
             };
+            let properties = match &dsc_resource.kind {
+                Kind::Group => {
+                    // if Group resource, we leave it to the resource to handle expressions
+                    resource.properties.clone()
+                },
+                _ => {
+                    self.invoke_property_expressions(resource.properties.as_ref())?
+                },
+            };
             debug!("resource_type {}", &resource.resource_type);
 
             // see if the properties contains `_exist` and is false
@@ -469,10 +487,19 @@ impl Configurator {
         for resource in resources {
             progress.set_resource(&resource.name, &resource.resource_type);
             progress.write_activity(format!("Test '{}'", resource.name).as_str());
-            let properties = self.invoke_property_expressions(resource.properties.as_ref())?;
-            let Some(dsc_resource) = self.discovery.find_resource(&resource.resource_type) else {
+            let discovery = &self.discovery.clone();
+            let Some(dsc_resource) = discovery.find_resource(&resource.resource_type) else {
                 return Err(DscError::ResourceNotFound(resource.resource_type));
             };
+            let properties = match &dsc_resource.kind {
+                Kind::Group => {
+                    // if Group resource, we leave it to the resource to handle expressions
+                    resource.properties.clone()
+                },
+                _ => {
+                    self.invoke_property_expressions(resource.properties.as_ref())?
+                },
+            };
             debug!("resource_type {}", &resource.resource_type);
             let expected = add_metadata(&dsc_resource.kind, properties)?;
             trace!("{}", t!("configure.mod.expectedState", state = expected));
@@ -544,10 +571,19 @@ impl Configurator {
         for resource in &resources {
             progress.set_resource(&resource.name, &resource.resource_type);
             progress.write_activity(format!("Export '{}'", resource.name).as_str());
-            let properties = self.invoke_property_expressions(resource.properties.as_ref())?;
-            let Some(dsc_resource) = self.discovery.find_resource(&resource.resource_type) else {
+            let discovery = &self.discovery.clone();
+            let Some(dsc_resource) = discovery.find_resource(&resource.resource_type) else {
                 return Err(DscError::ResourceNotFound(resource.resource_type.clone()));
             };
+            let properties = match &dsc_resource.kind {
+                Kind::Group => {
+                    // if Group resource, we leave it to the resource to handle expressions
+                    resource.properties.clone()
+                },
+                _ => {
+                    self.invoke_property_expressions(resource.properties.as_ref())?
+                },
+            };
             let input = add_metadata(&dsc_resource.kind, properties)?;
             trace!("{}", t!("configure.mod.exportInput", input = input));
             let export_result = match add_resource_export_results_to_configuration(dsc_resource, Some(dsc_resource), &mut conf, input.as_str()) {

dsc_lib/src/discovery/mod.rs

@@ -9,6 +9,7 @@ use crate::{dscresources::dscresource::DscResource, dscerror::DscError, progress
 use std::collections::BTreeMap;
 use tracing::error;
 
+#[derive(Clone)]
 pub struct Discovery {
     pub resources: BTreeMap<String, DscResource>,
 }