From aa1b83e85938c8908270f721819bf36caa3433a5 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Jakub=20Jare=C5=A1?= <me@jakubjares.com>
Date: Mon, 14 Oct 2024 10:12:24 +0200
Subject: [PATCH] Fix publishing as docker image via /t:PublishContainer
 (#3929)

---
 .../Microsoft.Testing.Platform.MSBuild.targets     | 14 ++++++++++++++
 1 file changed, 14 insertions(+)

diff --git a/src/Platform/Microsoft.Testing.Platform.MSBuild/buildMultiTargeting/Microsoft.Testing.Platform.MSBuild.targets b/src/Platform/Microsoft.Testing.Platform.MSBuild/buildMultiTargeting/Microsoft.Testing.Platform.MSBuild.targets
index 37c0a13b02..a8ff84d21d 100644
--- a/src/Platform/Microsoft.Testing.Platform.MSBuild/buildMultiTargeting/Microsoft.Testing.Platform.MSBuild.targets
+++ b/src/Platform/Microsoft.Testing.Platform.MSBuild/buildMultiTargeting/Microsoft.Testing.Platform.MSBuild.targets
@@ -330,4 +330,18 @@
   </Target>
 
   <Target Name="_TestRunStart" />
+
+  <Target Name="_SetTestingPlatformDefaultsForContainerBuild" BeforeTargets="ComputeContainerConfig"
+    Condition="'$(IsPublishable)' == 'true' AND '$(EnableSdkContainerSupport)' == 'true' AND !$(RuntimeIdentifier.StartsWith('win'))">
+    <!--
+    When PublishContainer target is used to create a docker image for the test project, it will try to write /app/TestResults folder
+    which is a non-writable folder. The environment variable below (notice double underscore) is used to override option defaults, and is picked up
+    EnvironmentVariablesConfigurationProvider.
+    
+    ComputeContainerConfig is target that PublishContainer depends on to figure out the configuration for the container.
+    -->
+    <ItemGroup>
+      <ContainerEnvironmentVariable Include="platformOptions__resultDirectory" Value="/tmp/TestResults"></ContainerEnvironmentVariable>
+    </ItemGroup>
+  </Target>
 </Project>