From 6bbaf397149de1f2e53391e193dfcbee195e1b42 Mon Sep 17 00:00:00 2001
From: Jonathan Peppers <jonathan.peppers@microsoft.com>
Date: Wed, 21 Mar 2018 08:40:30 -0500
Subject: [PATCH] [tests] support for running tests on VSTS

To get our desired behavior on VSTS, we have to:
- Make sure our call to NUnit returns a failing exit code
- Set the "Continue on Error" option in VSTS
- Add a step at the end of our build definition in VSTS to fail the
build if any issues ocurred

To make this work, we have to change our `<Exec />` task usage to
`ContinueOnError="ErrorAndContinue"`.

Example builds with a failing test can be found here:
- https://devdiv.visualstudio.com/DevDiv/_build/index?buildId=1502302&tab=ms.vss-test-web.test-result-details
- https://jenkins.mono-project.com/job/xamarin-android-pr-builder/2739/
---
 build-tools/scripts/RunTests.targets | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/build-tools/scripts/RunTests.targets b/build-tools/scripts/RunTests.targets
index 674177f950c..6560631eef8 100644
--- a/build-tools/scripts/RunTests.targets
+++ b/build-tools/scripts/RunTests.targets
@@ -28,7 +28,7 @@
     <Exec
         Command="$(_NUnit) $(NUNIT_EXTRA) %(_TestAssembly.Identity) $(_Test) --result=&quot;TestResult-%(Filename).xml;format=nunit2&quot; --output=&quot;bin\Test$(Configuration)\TestOutput-%(Filename).txt&quot;"
         WorkingDirectory="$(_TopDir)"
-        ContinueOnError="True"
+        ContinueOnError="ErrorAndContinue"
     />
     <ItemGroup>
       <_RenameTestCasesGlob Include="$(_TopDir)\TestResult-*Tests.xml" />