add tests

Author: SteveL-MSFT

dsc/examples/hello_world.dsc.bicep

@@ -10,4 +10,5 @@ resource echo 'Microsoft.DSC.Debug/Echo@2025-08-27' = {
   }
 }
 
-output exampleOutput string = echo.properties.output
+// This is waiting on https://github.com/Azure/bicep/issues/17670 to be resolved
+// output exampleOutput string = echo.properties.output

dsc/tests/dsc_osinfo.tests.ps1

@@ -1,3 +1,6 @@
+# Copyright (c) Microsoft Corporation.
+# Licensed under the MIT License.
+
 Describe 'Tests for osinfo examples' {
     It 'Config with default parameters and get works' {
         $out = dsc config get -f $PSScriptRoot/../examples/osinfo_parameters.dsc.yaml | ConvertFrom-Json -Depth 10

dsc/tests/dsc_output.tests.ps1

@@ -0,0 +1,41 @@
+# Copyright (c) Microsoft Corporation.
+# Licensed under the MIT License.
+
+Describe 'output tests' {
+    It 'config with output property works' {
+        $configYaml = @'
+        $schema: https://aka.ms/dsc/schemas/v3/bundled/config/document.json
+        variables:
+          arrayVar:
+            - 1
+            - 2
+            - 3
+        resources:
+          - name: echo
+            type: Microsoft.DSC.Debug/Echo
+            properties:
+              output: This is a test
+        outputs:
+          simpleText:
+            type: string
+            value: Hello World
+          expression:
+            type: string
+            value: "[reference(resourceId('Microsoft.DSC.Debug/Echo', 'echo')).output]"
+          conditionSucceed:
+            type: int
+            condition: "[equals(1, 1)]"
+            value: "[variables('arrayVar')[1]]"
+          conditionFail:
+            type: int
+            condition: "[equals(1, 2)]"
+            value: "[variables('arrayVar')[1]]"
+'@
+        $out = dsc config get -i $configYaml | ConvertFrom-Json -Depth 10
+        $LASTEXITCODE | Should -Be 0
+        $out.outputs.simpleText | Should -Be 'Hello World'
+        $out.outputs.expression | Should -Be 'This is a test'
+        $out.outputs.conditionSucceed | Should -Be 2
+        $out.outputs.conditionFail | Should -BeNullOrEmpty
+    }
+}

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

@@ -7,7 +7,6 @@ use schemars::{JsonSchema, json_schema};
 use serde::{Deserialize, Serialize};
 use serde_json::{Map, Value};
 use std::{collections::HashMap, fmt::Display};
-use std::fmt::Display;
 
 use crate::{dscerror::DscError, schemas::DscRepoSchema};
 
@@ -131,7 +130,7 @@ pub struct UserFunctionOutput {
 }
 
 #[derive(Debug, Clone, PartialEq, Deserialize, Serialize, JsonSchema)]
-#[serde(untagged)]
+#[serde(rename_all = "camelCase")]
 pub enum ValueOrCopy {
     Value(String),
     Copy(Copy),
@@ -142,6 +141,7 @@ pub enum ValueOrCopy {
 pub struct Output {
     pub condition: Option<String>,
     pub r#type: DataType,
+    #[serde(flatten)]
     pub value_or_copy: ValueOrCopy,
 }
 
@@ -156,14 +156,14 @@ pub struct Configuration {
     #[serde(skip_serializing_if = "Option::is_none")]
     pub functions: Option<Vec<UserFunction>>,
     #[serde(skip_serializing_if = "Option::is_none")]
-    pub parameters: Option<HashMap<String, Parameter>>,
-    #[serde(skip_serializing_if = "Option::is_none")]
-    pub variables: Option<Map<String, Value>>,
-    pub resources: Vec<Resource>,
-    #[serde(skip_serializing_if = "Option::is_none")]
     pub metadata: Option<Metadata>,
     #[serde(skip_serializing_if = "Option::is_none")]
     pub outputs: Option<HashMap<String, Output>>,
+    #[serde(skip_serializing_if = "Option::is_none")]
+    pub parameters: Option<HashMap<String, Parameter>>,
+    pub resources: Vec<Resource>,
+    #[serde(skip_serializing_if = "Option::is_none")]
+    pub variables: Option<Map<String, Value>>,
 }
 
 #[derive(Debug, Clone, PartialEq, Deserialize, Serialize, JsonSchema)]

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

@@ -25,6 +25,7 @@ pub struct Context {
     pub dsc_version: Option<String>,
     pub execution_type: ExecutionKind,
     pub extensions: Vec<DscExtension>,
+    pub outputs: Map<String, Value>,
     pub parameters: HashMap<String, (Value, DataType)>,
     pub process_expressions: bool,
     pub process_mode: ProcessMode,
@@ -36,7 +37,6 @@ pub struct Context {
     pub system_root: PathBuf,
     pub user_functions: HashMap<String, UserFunctionDefinition>,
     pub variables: Map<String, Value>,
-    pub outputs: Map<String, Value>,
 }
 
 impl Context {
@@ -48,6 +48,7 @@ impl Context {
             dsc_version: None,
             execution_type: ExecutionKind::Actual,
             extensions: Vec::new(),
+            outputs: Map::new(),
             parameters: HashMap::new(),
             process_expressions: true,
             process_mode: ProcessMode::Normal,
@@ -62,8 +63,6 @@ impl Context {
             system_root: get_default_os_system_root(),
             user_functions: HashMap::new(),
             variables: Map::new(),
-            restart_required: None,
-            outputs: Map::new(),
         }
     }
 }