Add Pester test for ARMv2

Author: andyleejordan

dsc/tests/dsc_armv2.tests.ps1

@@ -0,0 +1,57 @@
+# Copyright (c) Microsoft Corporation.
+# Licensed under the MIT License.
+
+Describe 'ARM Language 2.0 tests' {
+    It 'config with ARM Language 2.0 format works' {
+<#
+This JSON config built from the following Bicep code using the
+desiredStateConfiguration and moduleExtensionConfigs experimental features:
+
+extension dsc
+targetScope = 'desiredStateConfiguration'
+
+resource echoResource 'Microsoft.DSC.Debug/Echo@1.0.0' = {
+  output: 'Hello World'
+}
+#>
+        $configJson = @'
+{
+  "$schema": "https://aka.ms/dsc/schemas/v3/bundled/config/document.json",
+  "languageVersion": "2.2-experimental",
+  "contentVersion": "1.0.0.0",
+  "metadata": {
+    "_EXPERIMENTAL_WARNING": "This template uses ARM features that are experimental. Experimental features should be enabled for testing purposes only, as there are no guarantees about the quality or stability of these features. Do not enable these settings for any production usage, or your production environment may be subject to breaking.",
+    "_EXPERIMENTAL_FEATURES_ENABLED": [
+      "Enable defining extension configs for modules"
+    ],
+    "_generator": {
+      "name": "bicep",
+      "version": "0.38.33.27573",
+      "templateHash": "5233252217641859406"
+    }
+  },
+  "extensions": {
+    "dsc": {
+      "name": "DesiredStateConfiguration",
+      "version": "0.1.0"
+    }
+  },
+  "resources": {
+    "echoResource": {
+      "extension": "dsc",
+      "type": "Microsoft.DSC.Debug/Echo",
+      "apiVersion": "1.0.0",
+      "properties": {
+        "output": "Hello World"
+      }
+    }
+  }
+}
+'@
+        $out = dsc config get -i $configJson | ConvertFrom-Json -Depth 10
+        $LASTEXITCODE | Should -Be 0
+        $out.results | Should -HaveCount 1
+        $out.results[0].type | Should -Be 'Microsoft.DSC.Debug/Echo'
+        $out.results[0].result.actualState.output | Should -Be 'Hello World'
+    }
+}

lib/dsc-lib/src/configure/config_doc.rs

@@ -204,8 +204,17 @@ where
             resources.into_iter()
                 .map(|(name, resource)| {
                     let mut resource: Resource = serde_json::from_value(resource).map_err(serde::de::Error::custom)?;
-                    // TODO: Decide what to do if resource.name is already set.
-                    resource.name = name;
+                    // Note that this is setting the symbolic name as the
+                    // resource's name property only if that isn't already set.
+                    // In the general use case from Bicep, it won't be, but
+                    // we're unsure of the implications in other use cases.
+                    //
+                    // TODO: We will need to update the 'dependsOn' logic to
+                    // accept both the symbolic name as mapped here in addition
+                    // to `resourceId()`.
+                    if resource.name.is_empty() {
+                        resource.name = name;
+                    }
                     Ok(resource)
                 })
                 .collect()