From c7b981244cfe0dbca2284690bd5091aeda410de0 Mon Sep 17 00:00:00 2001 From: Bob Arnson Date: Wed, 11 Jun 2025 22:37:39 -0400 Subject: Fix unit tests to account for Actions runners: - Handle extra-long paths being normalized to either \\?\\ and "normal." - Rewrite PathGetTempPathTest test because it's not a good test: - By overwriting `TMP` with an extra-long path, it causes `::GetTempPath[2]W` to fall back to the value of `TEMP` instead. - But that's testing `::GetTempPath[2]W`, not `PathGetTempPath`. - It also assumes `TMP` and `TEMP` are set typically. - It also assumes it's not running as LocalSystem. - Instead, just compare the values you get from `::GetTempPathW` and `PathGetTempPath`. Fixes https://github.com/wixtoolset/issues/issues/9087 --- src/libs/libs.cmd | 1 + 1 file changed, 1 insertion(+) (limited to 'src/libs/libs.cmd') diff --git a/src/libs/libs.cmd b/src/libs/libs.cmd index d8b81f54..acf5ff55 100644 --- a/src/libs/libs.cmd +++ b/src/libs/libs.cmd @@ -22,6 +22,7 @@ msbuild -Restore libs_t.proj -p:Configuration=%_C% -tl -nologo -m -warnaserror -bl:%_L%\libs_build.binlog || exit /b dotnet test ^ + --results-directory %_L%\TestResults --blame-hang-timeout 1min --blame-hang-dump-type mini -l:"console;verbosity=detailed" ^ %_B%\net6.0\WixToolsetTest.Versioning.dll ^ %_B%\x86\DUtilUnitTest.dll ^ %_B%\x64\DUtilUnitTest.dll ^ -- cgit v1.2.3-55-g6feb