Skip to content

Commit a1f16b8

Browse files
authored
Fix double build of test packages for Helix (#42083)
2 parents cd6536f + fa20a7c commit a1f16b8

File tree

3 files changed

+4
-4
lines changed

3 files changed

+4
-4
lines changed

test/Microsoft.NET.TestFramework/Microsoft.NET.TestFramework.csproj

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -65,8 +65,8 @@
6565
<None Include="BuildTestPackages.targets" />
6666
</ItemGroup>
6767

68-
<Import Project="SetupTestRoot.targets" />
69-
<Import Project="BuildTestPackages.targets" />
68+
<Import Project="SetupTestRoot.targets" Condition="'$(BuildTestPackages)' != 'false'" />
69+
<Import Project="BuildTestPackages.targets" Condition="'$(BuildTestPackages)' != 'false'" />
7070

7171
<Import Project="Sdk.targets" Sdk="Microsoft.NET.Sdk" />
7272

test/UnitTests.proj

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -116,7 +116,6 @@
116116
<Copy SourceFiles="@(AssetFiles)" DestinationFiles="@(AssetFiles->'$(HelixPayloadOnHost)\TestExecutionDirectoryFiles\Assets\%(RecursiveDir)%(Filename)%(Extension)')" />
117117
</Target>
118118

119-
120119
<Target Name="AppendHelixPreCommand" BeforeTargets="CoreTest" DependsOnTargets="CopyHelixFiles">
121120
<PropertyGroup>
122121
<HelixPreCommands Condition="!$(IsPosixShell)">call %HELIX_CORRELATION_PAYLOAD%\t\RunTestsOnHelix.cmd $(TestFullMSBuild);$(HelixPreCommands)</HelixPreCommands>
@@ -179,4 +178,5 @@
179178
HelixCorrelationPayload="@(HelixCorrelationPayload)"
180179
TestOutputDirectory="$(RepoRoot)artifacts\bin\localHelixTestLayout" />
181180
</Target>
181+
182182
</Project>

test/xunit-runner/XUnitRunner.targets

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -54,7 +54,7 @@
5454
<_CurrentRuntimeTargetFramework Condition="'$(_CurrentRuntimeTargetFramework)' == ''">$(SDKCustomXUnitRuntimeTargetFramework)</_CurrentRuntimeTargetFramework>
5555
<_CurrentAdditionalProperties>%(SDKCustomXUnitProject.AdditionalProperties)</_CurrentAdditionalProperties>
5656
</PropertyGroup>
57-
<MSBuild Projects="$(_CurrentSDKCustomXUnitProject)" Targets="PublishWithOutput" Properties="CustomAfterMicrosoftCommonTargets=$(_SDKCustomXUnitPublishTargetsPath);TargetFramework=$(_CurrentPublishTargetFramework);$(_CurrentAdditionalProperties)">
57+
<MSBuild Projects="$(_CurrentSDKCustomXUnitProject)" Targets="PublishWithOutput" Properties="CustomAfterMicrosoftCommonTargets=$(_SDKCustomXUnitPublishTargetsPath);TargetFramework=$(_CurrentPublishTargetFramework);BuildTestPackages=false;$(_CurrentAdditionalProperties)">
5858
<Output TaskParameter="TargetOutputs" PropertyName="_PublishOutputDir" />
5959
</MSBuild>
6060
<MSBuild Projects="$(_CurrentSDKCustomXUnitProject)" Targets="GetTargetPath" Properties="CustomAfterMicrosoftCommonTargets=$(_SDKCustomXUnitPublishTargetsPath);TargetFramework=$(_CurrentPublishTargetFramework);$(_CurrentAdditionalProperties)">

0 commit comments

Comments
 (0)