correct mscodecoverage not collected when RunInParallel

Author: tonyhallett

FineCodeCoverageTests/TestContainerDiscovery_Tests.cs

@@ -141,6 +141,21 @@ public void Should_Not_ReloadCoverage_When_TestExecutionFinished_And_Reloading_W
             AssertShouldNotReloadCoverage();
         }
 
+        [Test]
+        public void Should_ReloadCoverage_When_TestExecutionFinished_If_RunInParallel_But_MsCodeCoverage()
+        {
+            var (operation,_,__) = SetUpForProceedPath();
+            SetUpOptions(mockAppOptions =>
+            {
+                mockAppOptions.Setup(o => o.Enabled).Returns(true);
+                mockAppOptions.Setup(o => o.RunInParallel).Returns(true);
+                mockAppOptions.Setup(o => o.MsCodeCoverage).Returns(true);
+            });
+            RaiseTestExecutionFinished(operation);
+
+            mocker.Verify<IFCCEngine>(engine => engine.ReloadCoverage(It.IsAny<Func<Task<List<ICoverageProject>>>>()));
+        }
+
         [Test]
         public async Task Should_ReloadCoverage_When_TestExecutionStarting_And_Settings_RunInParallel_Is_True()
         {

SharedProject/Impl/TestContainerDiscovery/TestContainerDiscoverer.cs

@@ -117,7 +117,7 @@ private void CombinedLog(string message)
         private void TestExecutionFinished(IOperation operation)
         {
             var settings = appOptionsProvider.Get();
-            if (!settings.Enabled || settings.RunInParallel)
+            if (!settings.Enabled || (settings.RunInParallel && !settings.MsCodeCoverage))
             {
                 return;
             }