Fix progress display when using JSON, add None option; exhance progress schema

Author: SteveL-MSFT

dsc/Cargo.lock

@@ -4,7 +4,7 @@
 use clap::{Parser, Subcommand, ValueEnum};
 use clap_complete::Shell;
 use dsc_lib::dscresources::command_resource::TraceLevel;
-use dsc_lib::util::ProgressFormat;
+use dsc_lib::progress::ProgressFormat;
 use rust_i18n::t;
 use serde::Deserialize;
 

dsc/src/args.rs

@@ -8,7 +8,7 @@ use rust_i18n::{i18n, t};
 use std::{io, process::exit};
 use sysinfo::{Process, RefreshKind, System, get_current_pid, ProcessRefreshKind};
 use tracing::{error, info, warn, debug};
-use dsc_lib::util::ProgressFormat;
+use dsc_lib::progress::ProgressFormat;
 
 #[cfg(debug_assertions)]
 use crossterm::event;

dsc/src/main.rs

@@ -25,7 +25,7 @@ use dsc_lib::{
     },
     dscresources::dscresource::{Capability, ImplementedAs, Invoke},
     dscresources::resource_manifest::{import_manifest, ResourceManifest},
-    util::ProgressFormat,
+    progress::ProgressFormat,
 };
 use rust_i18n::t;
 use std::{
@@ -288,16 +288,14 @@ pub fn config(subcommand: &ConfigSubCommand, parameters: &Option<String>, mounte
         }
     };
 
-    let mut configurator = match Configurator::new(&json_string) {
+    let mut configurator = match Configurator::new(&json_string, progress_format) {
         Ok(configurator) => configurator,
         Err(err) => {
             error!("Error: {err}");
             exit(EXIT_DSC_ERROR);
         }
     };
 
-    configurator.set_progress_format(progress_format);
-
     if let ConfigSubCommand::Set { what_if , .. } = subcommand {
         if *what_if {
             configurator.context.execution_type = ExecutionKind::WhatIf;

dsc/src/subcommand.rs

@@ -303,4 +303,10 @@ resources:
         $LASTEXITCODE | Should -Be 1
         "$TestDrive/tracing.txt" | Should -FileContentMatchExactly "Target path does not exist: '/invalid/path'"
     }
+
+    It '--progress-format can be None' {
+        dsc -p none resource list 2> $TestDrive/tracing.txt
+        $LASTEXITCODE | Should -Be 0
+        (Get-Content $TestDrive/tracing.txt -Raw) | Should -BeNullOrEmpty
+    }
 }

dsc/tests/dsc_args.tests.ps1

@@ -68,15 +68,25 @@ Describe 'dsc config get tests' {
         $config_yaml | dsc --progress-format json config get -f - 2> $TestDrive/ErrorStream.txt
         $LASTEXITCODE | Should -Be 0
         $lines = Get-Content $TestDrive/ErrorStream.txt
-        $ProgressMessagesFound = $False
+        $ProgressMessagesFound = $false
+        $ProgressResultFound = $false
         foreach ($line in $lines) {
             $jp = $line | ConvertFrom-Json
             if ($jp.activity) { # if line is a progress message
+                $jp.id | Should -Not -BeNullOrEmpty
                 $jp.percent_complete | Should -BeIn (0..100)
-                $ProgressMessagesFound = $True
+                $ProgressMessagesFound = $true
+            }
+
+            if ($jp.percent_complete -eq 100 -and $jp.resourceType -eq 'Microsoft.DSC.Debug/Echo') {
+                $ProgressResultFound = $true
+                $jp.resourceName | Should -BeExactly 'Echo'
+                $jp.result | Should -Not -BeNullOrEmpty
+                $jp.result.output | Should -BeExactly 'hello'
             }
         }
         $ProgressMessagesFound | Should -BeTrue
+        $ProgressResultFound | Should -BeTrue
     }
 
     It 'contentVersion is ignored' {

dsc/tests/dsc_config_get.tests.ps1

@@ -66,6 +66,7 @@ Describe 'Tests for listing resources' {
         foreach ($line in $lines) {
             $jp = $line | ConvertFrom-Json
             if ($jp.activity) { # if line is a progress message
+                $jp.id | Should -Not -BeNullOrEmpty
                 $jp.percent_complete | Should -BeIn (0..100)
                 $ProgressMessagesFound = $True
             }

dsc/tests/dsc_resource_list.tests.ps1

@@ -35,6 +35,7 @@ tracing-indicatif = { version = "0.3" }
 tree-sitter = "0.25"
 tree-sitter-rust = "0.23"
 tree-sitter-dscexpression = { path = "../tree-sitter-dscexpression" }
+uuid = { version = "1.13", features = ["v4"] }
 
 [dev-dependencies]
 serde_yaml = "0.9"

dsc_lib/Cargo.lock

@@ -314,6 +314,9 @@ unknown = "Unknown"
 validation = "Validation"
 setting = "Setting"
 
+[progress]
+failedToSerialize = "Failed to serialize progress JSON: %{json}"
+
 [util]
 foundSetting = "Found setting '%{name}' in %{path}"
 notFoundSetting = "Setting '%{name}' not found in %{path}"