Use ReferenceTrimmer

Directory.Build.props

@@ -40,12 +40,11 @@
 
   <!-- Versioning -->
   <PropertyGroup>
-    <MSTestEnginePreReleaseVersionLabel>alpha</MSTestEnginePreReleaseVersionLabel>
-
     <!--
       This is a early preview package, keep 1.0.0-alpha or similar suffix even in official builds.
     -->
     <MSTestEngineVersionPrefix>1.0.0</MSTestEngineVersionPrefix>
+    <MSTestEnginePreReleaseVersionLabel>alpha</MSTestEnginePreReleaseVersionLabel>
   </PropertyGroup>
 
   <!-- Pack config -->
@@ -70,6 +69,12 @@
     <PolyGuard>true</PolyGuard>
   </PropertyGroup>
 
+  <!-- Test config -->
+  <PropertyGroup>
+    <!-- Set the test runner here and not in test folder so that arcade doesn't force add xunit to any folder of the repo -->
+    <TestRunnerName>TestingPlatformRunner</TestRunnerName>
+  </PropertyGroup>
+
   <ItemGroup>
     <Using Include="System.Collections" />
     <Using Include="System.Collections.Concurrent" />

Directory.Packages.props

@@ -37,6 +37,7 @@
     <PackageVersion Include="Microsoft.CodeAnalysis.PublicApiAnalyzers" Version="$(MicrosoftCodeAnalysisPublicApiAnalyzersVersion)" />
     <PackageVersion Include="Microsoft.VisualStudio.Threading.Analyzers" Version="17.13.61" />
     <PackageVersion Include="StyleCop.Analyzers" Version="1.2.0-beta.556" />
+    <GlobalPackageReference Include="ReferenceTrimmer" Version="3.3.11" />
   </ItemGroup>
   <ItemGroup Label="Product dependencies">
     <PackageVersion Include="Microsoft.ApplicationInsights" Version="2.23.0" />

TestFx.sln

@@ -72,6 +72,7 @@ Project("{2150E333-8FDC-42A3-9474-1A3956D46DE8}") = "IntegrationTests", "Integra
 EndProject
 Project("{2150E333-8FDC-42A3-9474-1A3956D46DE8}") = "TestAssets", "TestAssets", "{C9F82701-0E0F-4E61-B05B-AE387E7631F6}"
 	ProjectSection(SolutionItems) = preProject
+		test\IntegrationTests\TestAssets\Directory.Build.props = test\IntegrationTests\TestAssets\Directory.Build.props
 		test\IntegrationTests\TestAssets\Directory.Build.targets = test\IntegrationTests\TestAssets\Directory.Build.targets
 	EndProjectSection
 EndProject

samples/Playground/Playground.csproj

@@ -20,7 +20,6 @@
     <ProjectReference Include="$(RepoRoot)src\Analyzers\MSTest.Analyzers.CodeFixes\MSTest.Analyzers.CodeFixes.csproj" PrivateAssets="all" ReferenceOutputAssembly="false" OutputItemType="Analyzer" />
     <ProjectReference Include="$(RepoRoot)src\Analyzers\MSTest.Analyzers\MSTest.Analyzers.csproj" PrivateAssets="all" ReferenceOutputAssembly="false" OutputItemType="Analyzer" />
     <PackageReference Include="StreamJsonRpc" />
-    <ProjectReference Include="$(RepoRoot)src\Platform\Microsoft.Testing.Extensions.Telemetry\Microsoft.Testing.Extensions.Telemetry.csproj" />
   </ItemGroup>
 
   <ItemGroup>

src/Adapter/MSTest.TestAdapter/Helpers/FixtureMethodRunner.cs

@@ -12,6 +12,24 @@ namespace Microsoft.VisualStudio.TestPlatform.MSTest.TestAdapter.Helpers;
 
 internal static class FixtureMethodRunner
 {
+    internal static void RunOnContext(ExecutionContext? executionContext, Action action)
+    {
+        if (executionContext is null)
+        {
+            action();
+        }
+        else
+        {
+            // CreateCopy doesn't do anything on .NET Core as ExecutionContexts are immutable.
+            // But it's important on .NET Framework.
+            // On .NET Framework, ExecutionContext.Run cannot be called twice with the same ExecutionContext.
+            // Otherwise, it will throw InvalidOperationException with message:
+            // Cannot apply a context that has been marshaled across AppDomains, that was not acquired through a Capture operation or that has already been the argument to a Set call.
+            executionContext = executionContext.CreateCopy();
+            ExecutionContext.Run(executionContext, static action => ((Action)action!).Invoke(), action);
+        }
+    }
+
     internal static TestFailedException? RunWithTimeoutAndCancellation(
         Action action, CancellationTokenSource cancellationTokenSource, TimeoutInfo? timeoutInfo, MethodInfo methodInfo,
         ExecutionContext? executionContext, string methodCanceledMessageFormat, string methodTimedOutMessageFormat,

src/Adapter/MSTest.TestAdapter/MSTest.TestAdapter.csproj

@@ -54,7 +54,8 @@
 
   <ItemGroup>
     <ProjectReference Include="$(RepoRoot)src\Adapter\MSTestAdapter.PlatformServices\MSTestAdapter.PlatformServices.csproj" />
-    <ProjectReference Include="$(RepoRoot)src\Platform\Microsoft.Testing.Extensions.VSTestBridge\Microsoft.Testing.Extensions.VSTestBridge.csproj" Condition=" '$(TargetFramework)' != '$(UwpMinimum)' " />
+    <ProjectReference Include="$(RepoRoot)src\Platform\Microsoft.Testing.Extensions.VSTestBridge\Microsoft.Testing.Extensions.VSTestBridge.csproj"
+                      Condition=" '$(TargetFramework)' != '$(UwpMinimum)' AND '$(TargetFramework)' != '$(ModernUwpMinimum)' " />
     <ProjectReference Include="$(RepoRoot)src\Analyzers\MSTest.Internal.Analyzers\MSTest.Internal.Analyzers.csproj">
       <OutputItemType>Analyzer</OutputItemType>
       <ReferenceOutputAssembly>false</ReferenceOutputAssembly>

src/Analyzers/MSTest.Analyzers.Package/MSTest.Analyzers.Package.csproj

@@ -22,8 +22,8 @@
   </PropertyGroup>
 
   <ItemGroup>
-    <ProjectReference Include="..\MSTest.Analyzers.CodeFixes\MSTest.Analyzers.CodeFixes.csproj" />
-    <ProjectReference Include="..\MSTest.Analyzers\MSTest.Analyzers.csproj" />
+    <ProjectReference Include="..\MSTest.Analyzers.CodeFixes\MSTest.Analyzers.CodeFixes.csproj" NoWarn="RT0002"  />
+    <ProjectReference Include="..\MSTest.Analyzers\MSTest.Analyzers.csproj" NoWarn="RT0002" />
   </ItemGroup>
 
   <ItemGroup>

src/Analyzers/MSTest.Analyzers/xlf/Resources.cs.xlf

@@ -19,7 +19,7 @@ The type declaring these methods should also respect the following rules:
 -The class shouldn't be 'static'
 -The class should be marked with '[TestClass]' (or a derived attribute)
 -the class should not be generic.</source>
-        <target state="translated">Aby byly metody s označením [AssemblyCleanup] platné, musí se řídit následujícím rozložením: 
+        <target state="translated">Aby byly metody s označením [AssemblyCleanup] platné, musí se řídit následujícím rozložením:
 – Nesmí být deklarované pro obecnou třídu.
 – Musí být public.
 – Musí být static.
@@ -29,7 +29,7 @@ The type declaring these methods should also respect the following rules:
 – Nesmí přijímat žádný parametr, nebo musí přijímat jediný parametr typu TestContext.
 – Návratový typ musí být void, Task nebo ValueTask.
 
-Typ deklarující tyto metody by měl také respektovat následující pravidla: 
+Typ deklarující tyto metody by měl také respektovat následující pravidla:
 – Typ by měl být třída.
 – Třída by měla být public nebo internal (pokud testovací projekt používá atribut [DiscoverInternals]).
 – Třída by neměla být static.
@@ -64,7 +64,7 @@ The type declaring these methods should also respect the following rules:
 -The class shouldn't be 'static'
 -The class should be marked with '[TestClass]' (or a derived attribute)
 -the class should not be generic.</source>
-        <target state="translated">Aby byly metody s označením [AssemblyInitialize] platné, musí se řídit následujícím rozložením: 
+        <target state="translated">Aby byly metody s označením [AssemblyInitialize] platné, musí se řídit následujícím rozložením:
 – Nesmí být deklarované pro obecnou třídu.
 – Musí být public.
 – Musí být static.
@@ -74,7 +74,7 @@ The type declaring these methods should also respect the following rules:
 – Musí mít jeden parametr typu TestContext.
 – Návratový typ musí být void, Task nebo ValueTask.
 
-Typ deklarující tyto metody by měl také respektovat následující pravidla: 
+Typ deklarující tyto metody by měl také respektovat následující pravidla:
 – Typ by měl být třída.
 – Třída by měla být public nebo internal (pokud testovací projekt používá atribut [DiscoverInternals]).
 – Třída by neměla být static.
@@ -181,7 +181,7 @@ The type declaring these methods should also respect the following rules:
 -The class shouldn't be 'static'
 -If the class is 'sealed', it should be marked with '[TestClass]' (or a derived attribute)
 -the class should not be generic.</source>
-        <target state="translated">Aby byly metody s označením [ClassCleanup] platné, musí se řídit následujícím rozložením: 
+        <target state="translated">Aby byly metody s označením [ClassCleanup] platné, musí se řídit následujícím rozložením:
 – Nesmí být deklarované pro obecnou třídu bez nastavení režimu InheritanceBehavior.
 – Musí být public.
 – Musí být static.
@@ -193,7 +193,7 @@ The type declaring these methods should also respect the following rules:
 – V případě třídy abstract by měl být zadán parametr atributu InheritanceBehavior.BeforeEachDerivedClass.
 – V případě třídy sealed by neměl být zadán parametr atributu InheritanceBehavior.BeforeEachDerivedClass.
 
-Typ deklarující tyto metody by měl také respektovat následující pravidla: 
+Typ deklarující tyto metody by měl také respektovat následující pravidla:
 – Typ by měl být třída.
 – Třída by měla být public nebo internal (pokud testovací projekt používá atribut [DiscoverInternals]).
 – Třída by neměla být static.
@@ -230,7 +230,7 @@ The type declaring these methods should also respect the following rules:
 -The class shouldn't be 'static'
 -If the class is 'sealed', it should be marked with '[TestClass]' (or a derived attribute)
 -the class should not be generic.</source>
-        <target state="translated">Aby byly metody s označením [ClassInitialize] platné, musí se řídit následujícím rozložením: 
+        <target state="translated">Aby byly metody s označením [ClassInitialize] platné, musí se řídit následujícím rozložením:
 – Nesmí být deklarované pro obecnou třídu bez nastavení režimu InheritanceBehavior.
 – Musí být public.
 – Musí být static.
@@ -242,7 +242,7 @@ The type declaring these methods should also respect the following rules:
 – V případě třídy abstract by měl být zadán parametr atributu InheritanceBehavior.BeforeEachDerivedClass.
 – V případě třídy sealed by neměl být zadán parametr atributu InheritanceBehavior.BeforeEachDerivedClass.
 
-Typ deklarující tyto metody by měl také respektovat následující pravidla: 
+Typ deklarující tyto metody by měl také respektovat následující pravidla:
 – Typ by měl být třída.
 – Třída by měla být public nebo internal (pokud testovací projekt používá atribut [DiscoverInternals]).
 – Třída by neměla být static.
@@ -594,7 +594,7 @@ The type declaring these methods should also respect the following rules:
 -The class should be 'public' or 'internal' (if the test project is using the '[DiscoverInternals]' attribute)
 -The class shouldn't be 'static'
 -If the class is 'sealed', it should be marked with '[TestClass]' (or a derived attribute).</source>
-        <target state="translated">Aby byly metody s označením [TestCleanup] platné, musí se řídit následujícím rozložením: 
+        <target state="translated">Aby byly metody s označením [TestCleanup] platné, musí se řídit následujícím rozložením:
 – Musí být public.
 – Nesmí být abstract.
 – Nesmí být async void.
@@ -604,7 +604,7 @@ The type declaring these methods should also respect the following rules:
 – Nesmí přijímat žádný parametr.
 – Návratový typ musí být void, Task nebo ValueTask.
 
-Typ deklarující tyto metody by měl také respektovat následující pravidla: 
+Typ deklarující tyto metody by měl také respektovat následující pravidla:
 – Typ by měl být třída.
 – Třída by měla být public nebo internal (pokud testovací projekt používá atribut [DiscoverInternals]).
 – Třída by neměla být static.
@@ -653,7 +653,7 @@ The type declaring these methods should also respect the following rules:
 -The class should be 'public' or 'internal' (if the test project is using the '[DiscoverInternals]' attribute)
 -The class shouldn't be 'static'
 -If the class is 'sealed', it should be marked with '[TestClass]' (or a derived attribute).</source>
-        <target state="translated">Aby byly metody s označením [TestInitialize] platné, musí se řídit následujícím rozložením: 
+        <target state="translated">Aby byly metody s označením [TestInitialize] platné, musí se řídit následujícím rozložením:
 – Musí být public.
 – Nesmí být abstract.
 – Nesmí být async void.
@@ -663,7 +663,7 @@ The type declaring these methods should also respect the following rules:
 – Nesmí přijímat žádný parametr.
 – Návratový typ musí být void, Task nebo ValueTask.
 
-Typ deklarující tyto metody by měl také respektovat následující pravidla: 
+Typ deklarující tyto metody by měl také respektovat následující pravidla:
 – Typ by měl být třída.
 – Třída by měla být public nebo internal (pokud testovací projekt používá atribut [DiscoverInternals]).
 – Třída by neměla být static.
@@ -821,4 +821,4 @@ Typ deklarující tyto metody by měl také respektovat následující pravidla:
       </trans-unit>
     </body>
   </file>
-</xliff>
+</xliff>

src/Package/MSTest/MSTest.csproj

@@ -1,4 +1,4 @@
-<Project>
+<Project>
 
   <!-- SDK top import -->
   <Import Project="Sdk.props" Sdk="MSBuild.Sdk.Extras" Condition=" '$(OS)' == 'Windows_NT' " />
@@ -44,10 +44,15 @@
   </Target>
 
   <ItemGroup>
-    <ProjectReference Include="$(RepoRoot)src\Adapter\MSTest.TestAdapter\MSTest.TestAdapter.csproj" />
-    <ProjectReference Include="$(RepoRoot)src\TestFramework\TestFramework.Extensions\TestFramework.Extensions.csproj" />
-    <ProjectReference Include="$(RepoRoot)src\Analyzers\MSTest.Analyzers.Package\MSTest.Analyzers.Package.csproj" />
-    <ProjectReference Include="$(RepoRoot)src\Platform\Microsoft.Testing.Extensions.TrxReport\Microsoft.Testing.Extensions.TrxReport.csproj" Condition=" '$(TargetFramework)' != '$(UwpMinimum)' " />
+    <ProjectReference Include="$(RepoRoot)src\Adapter\MSTest.TestAdapter\MSTest.TestAdapter.csproj"
+                      NoWarn="RT0002" />
+    <ProjectReference Include="$(RepoRoot)src\TestFramework\TestFramework.Extensions\TestFramework.Extensions.csproj"
+                      NoWarn="RT0002" />
+    <ProjectReference Include="$(RepoRoot)src\Analyzers\MSTest.Analyzers.Package\MSTest.Analyzers.Package.csproj"
+                      NoWarn="RT0002" />
+    <ProjectReference Include="$(RepoRoot)src\Platform\Microsoft.Testing.Extensions.TrxReport\Microsoft.Testing.Extensions.TrxReport.csproj"
+                      Condition=" '$(TargetFramework)' != '$(UwpMinimum)' "
+                      NoWarn="RT0002" />
 
     <!-- CodeCoverage 17.14 doesn't support netcoreapp3.1. -->
     <!-- We end up having System.Collections.Immutable 8.0.0 which is not compatible with netcoreapp3.1 -->

src/Platform/Microsoft.Testing.Extensions.CrashDump/Microsoft.Testing.Extensions.CrashDump.csproj

@@ -37,7 +37,6 @@ This package extends Microsoft Testing Platform to provide a crash dump function
 
   <ItemGroup>
     <ProjectReference Include="$(RepoRoot)src\Platform\Microsoft.Testing.Platform\Microsoft.Testing.Platform.csproj" />
-    <ProjectReference Include="$(RepoRoot)src\Platform\Microsoft.Testing.Extensions.TrxReport.Abstractions\Microsoft.Testing.Extensions.TrxReport.Abstractions.csproj" />
   </ItemGroup>
 
   <ItemGroup>

src/Platform/Microsoft.Testing.Extensions.VSTestBridge/Microsoft.Testing.Extensions.VSTestBridge.csproj

@@ -18,7 +18,7 @@
   <ItemGroup>
     <ProjectReference Include="$(RepoRoot)src\Platform\Microsoft.Testing.Platform\Microsoft.Testing.Platform.csproj" />
     <ProjectReference Include="$(RepoRoot)src\Platform\Microsoft.Testing.Extensions.TrxReport.Abstractions\Microsoft.Testing.Extensions.TrxReport.Abstractions.csproj" />
-    <ProjectReference Include="$(RepoRoot)src\Platform\Microsoft.Testing.Extensions.Telemetry\Microsoft.Testing.Extensions.Telemetry.csproj" />
+    <ProjectReference Include="$(RepoRoot)src\Platform\Microsoft.Testing.Extensions.Telemetry\Microsoft.Testing.Extensions.Telemetry.csproj" ReferenceOutputAssembly="false" />
   </ItemGroup>
 
   <!-- NuGet properties -->

src/Platform/Microsoft.Testing.Platform.MSBuild/Microsoft.Testing.Platform.MSBuild.csproj

@@ -18,7 +18,8 @@
   <ItemGroup>
     <ProjectReference Include="$(RepoRoot)\src\Platform\Microsoft.Testing.Platform\Microsoft.Testing.Platform.csproj" CopyToBuildOutput="true" />
     <!-- Deps only so we can include it in the lib of the NuGet package -->
-    <ProjectReference Include="$(RepoRoot)\src\Platform\Microsoft.Testing.Extensions.MSBuild\Microsoft.Testing.Extensions.MSBuild.csproj" PrivateAssets="All" />
+    <ProjectReference Include="$(RepoRoot)\src\Platform\Microsoft.Testing.Extensions.MSBuild\Microsoft.Testing.Extensions.MSBuild.csproj" PrivateAssets="All"
+                      NoWarn="RT0002" />
   </ItemGroup>
 
   <!-- NuGet properties -->

src/Platform/Microsoft.Testing.Platform/Messages/TestNodeProperties.cs

@@ -1,6 +1,8 @@
 // Copyright (c) Microsoft Corporation. All rights reserved.
 // Licensed under the MIT license. See LICENSE file in the project root for full license information.
 
+using Microsoft.Testing.Platform.TestHost;
+
 namespace Microsoft.Testing.Platform.Extensions.Messages;
 
 /// <summary>

test/Directory.Build.targets

@@ -36,6 +36,7 @@
     <ProjectReference Include="$(RepoRoot)src\Platform\Microsoft.Testing.Extensions.CrashDump\Microsoft.Testing.Extensions.CrashDump.csproj" />
     <ProjectReference Include="$(RepoRoot)src\Platform\Microsoft.Testing.Extensions.HangDump\Microsoft.Testing.Extensions.HangDump.csproj" />
     <ProjectReference Include="$(RepoRoot)src\Platform\Microsoft.Testing.Extensions.Retry\Microsoft.Testing.Extensions.Retry.csproj" />
+    <ProjectReference Include="$(RepoRoot)src\Platform\Microsoft.Testing.Extensions.Telemetry\Microsoft.Testing.Extensions.Telemetry.csproj" />
     <ProjectReference Include="$(RepoRoot)src\Platform\Microsoft.Testing.Extensions.TrxReport\Microsoft.Testing.Extensions.TrxReport.csproj" />
     <ProjectReference Include="$(RepoRoot)src\Platform\Microsoft.Testing.Extensions.AzureDevOpsReport\Microsoft.Testing.Extensions.AzureDevOpsReport.csproj" />
     <PackageReference Include="Microsoft.Testing.Platform.MSBuild" Condition="'$(UseMSTestFromSource)' == 'true'" />