Address PR comments

2021-06-07 18:38:54 +01:00
parent f603b08d05
commit a56591fa12
1 changed files with 2 additions and 1 deletions
@@ -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