From dd5296b87ea05d7b6cfad386b76b28ac02da8873 Mon Sep 17 00:00:00 2001
From: Navin <navb@microsoft.com>
Date: Tue, 28 Jan 2020 18:32:54 +0530
Subject: [PATCH] Calculating the right test outcome to set the context result.
 (#2749)

---
 .../TestResults/Legacy/LegacyTestRunDataPublisher.cs            | 2 +-
 src/Agent.Worker/TestResults/TestDataPublisher.cs               | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/src/Agent.Worker/TestResults/Legacy/LegacyTestRunDataPublisher.cs b/src/Agent.Worker/TestResults/Legacy/LegacyTestRunDataPublisher.cs
index 37b9b5d739..43e61ff6a0 100644
--- a/src/Agent.Worker/TestResults/Legacy/LegacyTestRunDataPublisher.cs
+++ b/src/Agent.Worker/TestResults/Legacy/LegacyTestRunDataPublisher.cs
@@ -363,7 +363,7 @@ private bool GetTestRunOutcome(TestRunData testRunData, TestRunSummary testRunSu
                     default: break;
                 }
 
-                if(!_calculateTestRunSummary)
+                if(!_calculateTestRunSummary && anyFailedTests)
                 {
                     return anyFailedTests;
                 }
diff --git a/src/Agent.Worker/TestResults/TestDataPublisher.cs b/src/Agent.Worker/TestResults/TestDataPublisher.cs
index b16fd32bc1..5678edf2c2 100644
--- a/src/Agent.Worker/TestResults/TestDataPublisher.cs
+++ b/src/Agent.Worker/TestResults/TestDataPublisher.cs
@@ -125,7 +125,7 @@ private bool GetTestRunOutcome(IExecutionContext executionContext, IList<TestRun
                         default: break;
                     }
 
-                    if(!_calculateTestRunSummary)
+                    if(!_calculateTestRunSummary && anyFailedTests)
                     {
                         return anyFailedTests;
                     }