Multiple writes to xunit.abstraction in our build

[jaredpar]

This issue tracks removing the work around for double writes of xunit.abstractions.dll in our build.

This problem comes about due to a collection of packages that we reference. In order to leverage Test Explorer / LUT testing we need to have these packages referenced:

• xunit.runner.desktop
• xunit

Both of these include xunit.abstractions.dll in different forms: content and reference respectively. This creates a double write violation in our build and fails the correctness leg.

As a work around we use a target, RemoveDuplicateContent, to remove the DLL included via the
Content list. This issue tracks resolving the core problem with the xunit team and ideally removing this hack.

xunit/xunit#1651

[MattGertz]

Mondo escrow triage: test-only, moving out.

[jaredpar]

Fixed a while back.

[tmat]

Actually, this is still a problem. The workaround is in Imports.targets and removing it adds double-writes back.