chore(performance): Parallel mutation in solution mode (#2893)

* stopwatch * Add parallel foreach * Mutate projects in parallel * Format * Fix unit test --------- Co-authored-by: Richard Werkman <richard.werkman@infosupport.com>
stryker-mutator · Mar 29, 2024 · 0ab0ab3 · 0ab0ab3
1 parent d93b08c
commit 0ab0ab3
Show file tree

Hide file tree

Showing 3 changed files with 10 additions and 7 deletions.
diff --git a/src/Stryker.Core/Stryker.Core/Initialisation/ProjectOrchestrator.cs b/src/Stryker.Core/Stryker.Core/Initialisation/ProjectOrchestrator.cs
@@ -2,6 +2,7 @@
 using System.Collections.Generic;
 using System.IO;
 using System.Linq;
+using System.Threading.Tasks;
 using Microsoft.Extensions.Logging;
 using Mono.Cecil;
 using Stryker.Core.Baseline.Providers;
@@ -52,7 +53,7 @@ public IEnumerable<IMutationTestProcess> MutateProjects(StrykerOptions options,
             if (!projectInfos.Any())
             {
                 _logger.LogWarning("No project to mutate. Stryker will exit prematurely.");
-                yield break;
+                return Enumerable.Empty<IMutationTestProcess>();
             }
 
             _initializationProcess.BuildProjects(options, projectInfos);
@@ -63,10 +64,12 @@ public IEnumerable<IMutationTestProcess> MutateProjects(StrykerOptions options,
             InitializeDashboardProjectInformation(options, projectInfos.First());
             var inputs = _initializationProcess.GetMutationTestInputs(options, projectInfos, _runner);
 
-            foreach (var mutationTestInput in inputs)
+            var mutationTestProcesses = new List<IMutationTestProcess>();
+            Parallel.ForEach(inputs, mutationTestInput =>
             {
-                yield return _projectMutator.MutateProject(options, mutationTestInput, reporters);
-            }
+                mutationTestProcesses.Add(_projectMutator.MutateProject(options, mutationTestInput, reporters));
+            });
+            return mutationTestProcesses;
         }
 
         private void InitializeDashboardProjectInformation(StrykerOptions options, SourceProjectInfo projectInfo)

diff --git a/src/Stryker.Core/Stryker.Core/MutationTest/CsharpMutationProcess.cs b/src/Stryker.Core/Stryker.Core/MutationTest/CsharpMutationProcess.cs
@@ -48,7 +48,7 @@ public CsharpMutationProcess(
         /// This constructor is used by the <see cref="MutationTestProcess"/> initialization logic.
         /// </summary>
         /// <param name="options"></param>
-        public CsharpMutationProcess(StrykerOptions options) : this( null, options)
+        public CsharpMutationProcess(StrykerOptions options) : this(null, options)
         { }
 
         public void Mutate(MutationTestInput input)
@@ -59,7 +59,7 @@ public void Mutate(MutationTestInput input)
             var semanticModels = compilingProcess.GetSemanticModels(projectInfo.GetAllFiles().Cast<CsharpFileLeaf>().Select(x => x.SyntaxTree));
 
             // Mutate source files
-            foreach (var file in projectInfo.GetAllFiles().Cast<CsharpFileLeaf>())
+            foreach(var file in projectInfo.GetAllFiles().Cast<CsharpFileLeaf>())
             {
                 _logger.LogDebug($"Mutating {file.FullPath}");
                 // Mutate the syntax tree

diff --git a/src/Stryker.Core/Stryker.Core/TestRunners/VsTest/VsTestRunnerPool.cs b/src/Stryker.Core/Stryker.Core/TestRunners/VsTest/VsTestRunnerPool.cs
@@ -239,7 +239,7 @@ private CoverageRunResult BuildCoverageRunResultFromCoverageInfo(string property
                 leakedMutants = string.IsNullOrEmpty(propertyPairValue)
                     ? Enumerable.Empty<int>()
                     : propertyPairValue.Split(',').Select(int.Parse);
-                _logger.LogWarning(
+                _logger.LogDebug(
                     $"VsTestRunner: Some mutations were executed outside any test (mutation ids: {propertyPairValue}).");
             }
             else