Description
The decision was made that it's the responsibility of the test framework package to tell VS to stop autoinserting this boilerplate in test csprojs:
<ItemGroup>
<Service Include="{82a7f48d-3b50-4b1e-b82e-3ada8210c358}" />
</ItemGroup>
See:
microsoft/vstest#394 (comment)
https://github.com/Microsoft/vstest/pull/440/files#r111104404
Any project that references NUnit and does not reference Microsoft.NET.Test.Sdk
will benefit from this.
This is a super easy fix. We add the following \build\NUnit.props
to the nupkg and we're done:
<Project>
<ItemGroup>
<ProjectCapability Include="TestContainer" />
</ItemGroup>
</Project>
From my perspective it's much nicer to do this in the framework package than the adapter package. The only projects where I use the adapter package are where I need to use dotnet test
, and one day, I'll be using our future .NET Core console runner. (dotnet nunit
anyone? =D)
Let me know if you agree and I'll add the file for 3.10.