Fix tests locally and in CI

When tests run locally from within VS, we use net472 so that TestDriven can run them quickly. In CI, we use net5, which requires Microsoft.TestPlatform.ObjectModel (see microsoft/vstest#2469 (comment))
devlooped · Oct 3, 2020 · 5717b45 · 5717b45
1 parent e71405f
commit 5717b45
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 2 deletions.
diff --git a/src/Directory.Packages.props b/src/Directory.Packages.props
@@ -15,6 +15,7 @@
     <PackageVersion Include="TypeNameFormatter.Sources" Version="1.0.1" />
 
     <PackageVersion Include="Microsoft.NET.Test.Sdk" Version="16.7.1" />
+    <PackageVersion Include="Microsoft.TestPlatform.ObjectModel" Version="16.7.1" />
     <PackageVersion Include="Moq" Version="4.14.5" />
     <PackageVersion Include="xunit" Version="2.4.1" />
     <PackageVersion Include="xunit.runner.console" Version="2.4.1" />

diff --git a/src/Stunts.UnitTests/Stunts.UnitTests.csproj b/src/Stunts.UnitTests/Stunts.UnitTests.csproj
@@ -1,11 +1,13 @@
-<Project Sdk="Microsoft.NET.Sdk">
+<Project Sdk="Microsoft.NET.Sdk">
 
   <PropertyGroup>
-    <TargetFramework>net472</TargetFramework>
+    <TargetFramework Condition="'$(BuildingInsideVisualStudio)' == 'true'">net472</TargetFramework>
+    <TargetFramework Condition="'$(BuildingInsideVisualStudio)' != 'true'">net5.0</TargetFramework>
   </PropertyGroup>
 
   <ItemGroup>
     <PackageReference Include="Microsoft.NET.Test.Sdk" />
+    <PackageReference Include="Microsoft.TestPlatform.ObjectModel" />
     <PackageReference Include="Moq" />
     <PackageReference Include="xunit" />
     <PackageReference Include="xunit.runner.console" />