From a56591fa126862e58ec6ab5b384282d2acaea538 Mon Sep 17 00:00:00 2001 From: jonawals Date: Mon, 7 Jun 2021 18:38:54 +0100 Subject: [PATCH] Address PR comments --- .../Runtime/Code/Source/TestImpactRuntime.cpp | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/Code/Tools/TestImpactFramework/Runtime/Code/Source/TestImpactRuntime.cpp b/Code/Tools/TestImpactFramework/Runtime/Code/Source/TestImpactRuntime.cpp index ef4bf5cd03..784a46ffa0 100644 --- a/Code/Tools/TestImpactFramework/Runtime/Code/Source/TestImpactRuntime.cpp +++ b/Code/Tools/TestImpactFramework/Runtime/Code/Source/TestImpactRuntime.cpp @@ -291,7 +291,8 @@ namespace TestImpact "Test target '%s' completed its test run successfully but produced no coverage data", job.GetTestTarget()->GetName().c_str())); } - else if (!job.GetTestCoverge().has_value()) + + if (!job.GetTestCoverge().has_value()) { // When a test run completes with failing tests but produces no coverage artifact that's typically a sign of the // test aborting due to an unhandled exception, in which case ignore it and let it be picked up in the failure report