From c22fae088c88704634c446ec5a1eeea62096db62 Mon Sep 17 00:00:00 2001 From: James Truher Date: Tue, 20 Jun 2023 12:59:08 -0700 Subject: [PATCH 1/2] Add even more debugging to try to determine why these test fail in release build. --- .../PackageManagement.Tests.ps1 | 22 +++++++++++++++++-- 1 file changed, 20 insertions(+), 2 deletions(-) diff --git a/test/powershell/Modules/PackageManagement/PackageManagement.Tests.ps1 b/test/powershell/Modules/PackageManagement/PackageManagement.Tests.ps1 index 01b4d4738a6..112b933e645 100644 --- a/test/powershell/Modules/PackageManagement/PackageManagement.Tests.ps1 +++ b/test/powershell/Modules/PackageManagement/PackageManagement.Tests.ps1 @@ -22,12 +22,30 @@ Describe "PackageManagement Acceptance Test" -Tags "Feature" { # register the asset directory $localSourceName = [Guid]::NewGuid().ToString("n") $localSourceLocation = Join-Path $PSScriptRoot assets - $newSourceResult = Register-PackageSource -Name $localSourceName -provider NuGet -Location $localSourceLocation -Force -Trusted + $newSourceResult = Register-PackageSource -Name $localSourceName -provider NuGet -Location $localSourceLocation -Force -Trusted -Verbose Write-Verbose -Verbose -Message "Register-PackageSource -Name $localSourceName -provider NuGet -Location $localSourceLocation -Force -Trusted" $newSourceResult | Out-String -Stream | Write-Verbose -Verbose + $localSourceCheck = Get-PackageSource -Name $localSourceName -ErrorAction Ignore $skipPackageTests = $false - if ($newSourceResult.Location -ne $localSourceLocation) { + # It is possible that Register-PackageSource appears to succeed but the source is not actually registered + # collect as much information as possible to help diagnose the problem + if (($newSourceResult.Location -ne $localSourceLocation) -or ($localSourceCheck.Location -ne $localSourceLocation)) { + Write-Verbose -Verbose "Skipping tests because local source could not be correctly created" + if ( $newSourceResult ) { + $newSourceResult | out-string -str | Write-Verbose -Verbose + } + else { + Write-Verbose "newSourceResult is null" + } + + if ( $localSourceCheck ) { + $localSourceCheck | out-string -str | Write-Verbose -Verbose + } + else { + Write-Verbose "LocalSourceCheck is null" + } + Get-ChildItem "$HOME/.config" -Recurse -File | out-string -str | Write-Verbose -Verbose $skipPackageTests = $true } From e26cd375c8ff2bf931dc69723fd672017ecc8889 Mon Sep 17 00:00:00 2001 From: James Truher Date: Wed, 21 Jun 2023 11:59:51 -0700 Subject: [PATCH 2/2] Only run get-childitem HOME/.config on non-Windows --- .../Modules/PackageManagement/PackageManagement.Tests.ps1 | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/test/powershell/Modules/PackageManagement/PackageManagement.Tests.ps1 b/test/powershell/Modules/PackageManagement/PackageManagement.Tests.ps1 index 112b933e645..e6fdb981f15 100644 --- a/test/powershell/Modules/PackageManagement/PackageManagement.Tests.ps1 +++ b/test/powershell/Modules/PackageManagement/PackageManagement.Tests.ps1 @@ -45,7 +45,11 @@ Describe "PackageManagement Acceptance Test" -Tags "Feature" { else { Write-Verbose "LocalSourceCheck is null" } - Get-ChildItem "$HOME/.config" -Recurse -File | out-string -str | Write-Verbose -Verbose + + if (-not $IsWindows) { + Get-ChildItem "$HOME/.config" -Recurse -File | out-string -str | Write-Verbose -Verbose + } + $skipPackageTests = $true }