[IMPORTANT] dotnet-test should return 1 if test execution fails for any tfm

[natemcmaster]

Description

dotnet-test returns exit code 0 if the last TFM succeeded, even if one of the TFMs fails to execute.

Steps to reproduce

dotnet-restore and dotnet-test this

<Project Sdk="Microsoft.NET.Sdk" ToolsVersion="15.0">

  <PropertyGroup>
    <TargetFrameworks>netcoreapp1.0;net451</TargetFrameworks>
  </PropertyGroup>

  <ItemGroup>
    <Compile Include="**\*.cs" />
    <EmbeddedResource Include="**\*.resx" />
  </ItemGroup>

  <ItemGroup>
    <PackageReference Include="Microsoft.NETCore.App" Version="1.0.1" Condition="'$(TargetFramework)'=='netcoreapp1.0'" />
    <PackageReference Include="Microsoft.NET.Test.Sdk" Version="15.0.0-preview-20161123-03" />
    <PackageReference Include="xunit" Version="2.2.0-beta4-build3444" />
    <PackageReference Include="xunit.runner.visualstudio" Version="2.2.0-beta4-build1194" />
  </ItemGroup>

</Project>

Expected behavior

If any TFM fails, return exit code != 0.

Actual behavior

netcoreapp1.0 ❌ fails because of #191
net451 ✅
Return code 0

Environment

Using dotnet-cli 1.0.0-preview4-004233

[codito]

Fix available at dotnet/cli#5054

[codito]

dotnet/cli#5054 is merged now.