From 2cade8946a399d3461ab6f786ae9cb4efa3f876f Mon Sep 17 00:00:00 2001
From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com>
Date: Mon, 2 Mar 2026 22:31:52 +0000
Subject: [PATCH 1/2] Initial plan


From 274a7eea6110ee2159d64a99e0c9420b167af415 Mon Sep 17 00:00:00 2001
From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com>
Date: Mon, 2 Mar 2026 22:34:30 +0000
Subject: [PATCH 2/2] Address review comments: fix exit code checks, remove
 dead code, replace Invoke-Expression

Co-authored-by: christianhelle <710400+christianhelle@users.noreply.github.com>
---
 test/smoke-tests.bat |  7 ++++---
 test/smoke-tests.ps1 | 25 ++++++++++++++++---------
 2 files changed, 20 insertions(+), 12 deletions(-)

diff --git a/test/smoke-tests.bat b/test/smoke-tests.bat
index 6b8e2637..71a464fc 100644
--- a/test/smoke-tests.bat
+++ b/test/smoke-tests.bat
@@ -2,14 +2,13 @@
 setlocal enabledelayedexpansion
 
 :: Parse command line arguments
-set "THROTTLE_LIMIT=4"
 set "USE_PRODUCTION=false"
 set "USE_DOCKER=false"
 
 :parse_args
 if "%~1"=="" goto :main
 if /i "%~1"=="-throttlelimit" (
-    set "THROTTLE_LIMIT=%~2"
+    :: Accepted for compatibility but not used (script runs sequentially)
     shift
     shift
     goto :parse_args
@@ -34,7 +33,6 @@ shift
 goto :parse_args
 
 :main
-echo ThrottleLimit: %THROTTLE_LIMIT%
 echo UseProduction: %USE_PRODUCTION%
 echo UseDocker: %USE_DOCKER%
 
@@ -164,8 +162,11 @@ echo.
 echo === Pre-restoring packages ===
 echo.
 dotnet restore .\ConsoleApp\ConsoleApp.slnx --nologo -v q
+call :throw_on_native_failure
 dotnet restore .\ConsoleApp\ConsoleApp.Core.slnx --nologo -v q
+call :throw_on_native_failure
 dotnet restore .\Apizr\Sample.csproj --nologo -v q
+call :throw_on_native_failure
 
 :: ==========================================
 :: Phase 2: Settings-file tests
diff --git a/test/smoke-tests.ps1 b/test/smoke-tests.ps1
index e4d48c27..3ec267c4 100644
--- a/test/smoke-tests.ps1
+++ b/test/smoke-tests.ps1
@@ -204,7 +204,9 @@ function RunTests
     if ($BuildFromSource -and -not $UseDocker)
     {
         Write-Host "dotnet publish ../src/Refitter/Refitter.csproj -c Release -o bin -f net9.0"
-        Start-Process "dotnet" -Args "publish ../src/Refitter/Refitter.csproj -c Release -o bin -f net9.0" -NoNewWindow -PassThru | Wait-Process
+        $publishProcess = Start-Process "dotnet" -Args "publish ../src/Refitter/Refitter.csproj -c Release -o bin -f net9.0" -NoNewWindow -PassThru
+        $publishProcess | Wait-Process
+        if ($publishProcess.ExitCode -ne 0) { throw "dotnet publish failed!" }
 
         Write-Host "refitter --version"
         $p = Start-Process "./bin/refitter" -Args "--version" -NoNewWindow -PassThru
@@ -216,9 +218,15 @@ function RunTests
     # Phase 1: Pre-restore packages
     # ==========================================
     Write-Host "`r`n=== Pre-restoring packages ===`r`n"
-    Start-Process "dotnet" -Args "restore ./ConsoleApp/ConsoleApp.slnx --nologo -v q" -NoNewWindow -PassThru | Wait-Process
-    Start-Process "dotnet" -Args "restore ./ConsoleApp/ConsoleApp.Core.slnx --nologo -v q" -NoNewWindow -PassThru | Wait-Process
-    Start-Process "dotnet" -Args "restore ./Apizr/Sample.csproj --nologo -v q" -NoNewWindow -PassThru | Wait-Process
+    $p = Start-Process "dotnet" -Args "restore ./ConsoleApp/ConsoleApp.slnx --nologo -v q" -NoNewWindow -PassThru
+    $p | Wait-Process
+    if ($p.ExitCode -ne 0) { throw "dotnet restore ./ConsoleApp/ConsoleApp.slnx failed!" }
+    $p = Start-Process "dotnet" -Args "restore ./ConsoleApp/ConsoleApp.Core.slnx --nologo -v q" -NoNewWindow -PassThru
+    $p | Wait-Process
+    if ($p.ExitCode -ne 0) { throw "dotnet restore ./ConsoleApp/ConsoleApp.Core.slnx failed!" }
+    $p = Start-Process "dotnet" -Args "restore ./Apizr/Sample.csproj --nologo -v q" -NoNewWindow -PassThru
+    $p | Wait-Process
+    if ($p.ExitCode -ne 0) { throw "dotnet restore ./Apizr/Sample.csproj failed!" }
 
     # ==========================================
     # Phase 2: Settings-file tests (individual generate + build)
@@ -379,7 +387,7 @@ function RunTests
     Write-Host "Standard generation tasks: $($standardTasks.Count)"
     Write-Host "NetCore generation tasks: $($netCoreTasks.Count)"
 
-    # Execute standard generation in parallel batches
+    # Execute standard generation tasks sequentially
     RunGenerationTasks -tasks $standardTasks -processPath $processPath -useDocker $UseDocker
 
     # ==========================================
@@ -397,9 +405,9 @@ function RunTests
     $customNameSpec = "./OpenAPI/v3.0/petstore.json"
     $customNameArgs = "--multiple-interfaces ByEndpoint --operation-name-template ExecuteAsync"
     $customNameOutput = "./GeneratedCode/MultipleInterfacesWithCustomName_generateonly.cs"
-    $customNameCmd = "$processPath $customNameSpec --namespace GenerateOnly.MultipleInterfacesWithCustomName --output $customNameOutput --no-logging $customNameArgs"
-    Write-Host $customNameCmd
-    Invoke-Expression $customNameCmd
+    $customNameCmdArgs = "$customNameSpec --namespace GenerateOnly.MultipleInterfacesWithCustomName --output $customNameOutput --no-logging $customNameArgs"
+    Write-Host "$processPath $customNameCmdArgs"
+    StartRefitter -processPath $processPath -arguments $customNameCmdArgs -useDocker:$UseDocker
     if (-not (Test-Path $customNameOutput)) { throw "Generate-only test failed: MultipleInterfacesWithCustomName" }
     Remove-Item $customNameOutput -Force
     Write-Host "Generate-only test passed: MultipleInterfacesWithCustomName"
@@ -425,7 +433,6 @@ function RunTests
 
     @("https://petstore3.swagger.io/api/v3/openapi.json", "https://petstore3.swagger.io/api/v3/openapi.yaml") | ForEach-Object {
         $url = $_
-        $urlFormat = if ($url.EndsWith(".json")) { "json" } else { "yaml" }
         $namespace = "PetstoreFromUri"
         $outputPath = "PetstoreFromUri.generated.cs"