fix: acceptance tests should build and run when repo path has whitesp…

…ace (#49)
spekt · Sep 7, 2023 · a53ca13 · a53ca13
1 parent 4a2cf7c
commit a53ca13
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/test/Xunit.Xml.TestLogger.AcceptanceTests/DotnetTestFixture.cs b/test/Xunit.Xml.TestLogger.AcceptanceTests/DotnetTestFixture.cs
@@ -39,7 +39,7 @@ public DotnetTestFixture()
                 p.StartInfo.UseShellExecute = false;
                 p.StartInfo.RedirectStandardOutput = true;
                 p.StartInfo.FileName = "dotnet";
-                p.StartInfo.Arguments = $"test --no-build {testLogger} {testProject}";
+                p.StartInfo.Arguments = $"test --no-build {testLogger} \"{testProject}\"";
 
                 // Required to run netcoreapp3.1 without icu support on devbox (linux)
                 p.StartInfo.EnvironmentVariables.Add("DOTNET_SYSTEM_GLOBALIZATION_INVARIANT", "1");

diff --git a/test/Xunit.Xml.TestLogger.AcceptanceTests/Xunit.Xml.TestLogger.AcceptanceTests.csproj b/test/Xunit.Xml.TestLogger.AcceptanceTests/Xunit.Xml.TestLogger.AcceptanceTests.csproj
@@ -20,9 +20,9 @@
 
   <!-- Build test assets for acceptance tests -->
   <PropertyGroup>
-    <TestRestoreConfig>$(MSBuildThisFileDirectory)../assets/NuGet.$(Configuration).config</TestRestoreConfig>
-    <TestFullProject>$(MSBuildThisFileDirectory)../assets/Xunit.Xml.TestLogger.NetFull.Tests/Xunit.Xml.TestLogger.NetFull.Tests.csproj</TestFullProject>
-    <TestCoreProject>$(MSBuildThisFileDirectory)../assets/Xunit.Xml.TestLogger.NetCore.Tests/Xunit.Xml.TestLogger.NetCore.Tests.csproj</TestCoreProject>
+    <TestRestoreConfig>"$(MSBuildThisFileDirectory)../assets/NuGet.$(Configuration).config"</TestRestoreConfig>
+    <TestFullProject>"$(MSBuildThisFileDirectory)../assets/Xunit.Xml.TestLogger.NetFull.Tests/Xunit.Xml.TestLogger.NetFull.Tests.csproj"</TestFullProject>
+    <TestCoreProject>"$(MSBuildThisFileDirectory)../assets/Xunit.Xml.TestLogger.NetCore.Tests/Xunit.Xml.TestLogger.NetCore.Tests.csproj"</TestCoreProject>
   </PropertyGroup>
   <Target Name="TestTarget" AfterTargets="Build">
     <Message Importance="High" Text="... Building test assets" />