From b09f64b51bacb3d82dd0c49c652a1e5445368de9 Mon Sep 17 00:00:00 2001 From: "renovate[bot]" <29139614+renovate[bot]@users.noreply.github.com> Date: Fri, 13 Nov 2026 20:54:17 +0000 Subject: [PATCH 1/3] chore(deps): update dependency csharpier.msbuild to 1.4.0 --- Directory.Packages.props | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/Directory.Packages.props b/Directory.Packages.props index f1a289a..f745ca8 100644 --- a/Directory.Packages.props +++ b/Directory.Packages.props @@ -4,7 +4,7 @@ true - + From 1305470d3d8d854fc89a4885e52b0ebfa9c2864f Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Martin=20St=C3=BChmer?= Date: Fri, 13 Nov 2026 21:54:55 +0100 Subject: [PATCH 2/3] style: apply csharpier 1.4.0 formatting --- .../IsolatedAssemblyRunner.cs | 6 +-- .../Analyzers/TestSurfaceAnalysis.cs | 18 ++++----- .../Equivalence/EquivalenceClassifier.cs | 17 ++++----- .../TestSurfaceManifestGenerator.cs | 5 +-- .../Mutations/WellKnownTypeCache.cs | 8 ++-- .../Analyzers/MSTestSurfaceAnalyzerTests.cs | 8 ++-- .../NUnitTestSurfaceAnalyzerTests.cs | 5 +-- .../TUnitTestSurfaceAnalyzerTests.cs | 5 +-- .../Analyzers/TestMarkerRecognitionTests.cs | 3 +- .../XunitV2TestSurfaceAnalyzerTests.cs | 5 +-- .../XunitV3TestSurfaceAnalyzerTests.cs | 5 +-- .../TestSurfaceAnalysisDiscoveryTests.cs | 7 ++-- .../Mutations/MutantCompilerTests.cs | 31 ++++++--------- .../ArithmeticAssignmentMutatorTests.cs | 10 ++--- .../ArithmeticOperatorMutatorTests.cs | 10 ++--- .../Operators/BooleanLiteralMutatorTests.cs | 11 ++---- .../Operators/CaseConversionMutatorTests.cs | 8 ++-- .../CollectionInitializerMutatorTests.cs | 5 +-- .../ConditionalExpressionMutatorTests.cs | 5 +-- .../Operators/CultureInfoMutatorTests.cs | 5 +-- .../IncrementDecrementMutatorTests.cs | 3 +- .../Operators/IntegralTypeCheckTests.cs | 5 +-- .../Operators/LinqMethodMutatorTests.cs | 8 ++-- .../Operators/MathMethodMutatorTests.cs | 8 ++-- .../Operators/NullCoalescingMutatorTests.cs | 10 ++--- .../Operators/NullableLiteralMutatorTests.cs | 38 +++++++------------ .../Operators/NumericLiteralMutatorTests.cs | 10 ++--- .../Operators/ParenthesizationMutatorTests.cs | 9 ++--- .../Operators/RegexLookaroundMutatorTests.cs | 7 ++-- .../Operators/RegexOptionsMutatorTests.cs | 12 ++---- ...StringComparerComparisonAxisParityTests.cs | 5 +-- .../Operators/StringComparerMutatorTests.cs | 10 ++--- .../Operators/StringComparisonMutatorTests.cs | 16 +++----- .../Operators/StringLiteralMutatorTests.cs | 10 ++--- .../Operators/StringMethodMutatorTests.cs | 8 ++-- .../Operators/UnaryOperatorMutatorTests.cs | 3 +- .../ReachabilityClosureMemberTests.cs | 5 +-- .../Reachability/ReachableSymbolSetTests.cs | 5 +-- 38 files changed, 142 insertions(+), 207 deletions(-) diff --git a/src/NetEvolve.FrameShift.Execution/IsolatedAssemblyRunner.cs b/src/NetEvolve.FrameShift.Execution/IsolatedAssemblyRunner.cs index 45d050a..4dc9633 100644 --- a/src/NetEvolve.FrameShift.Execution/IsolatedAssemblyRunner.cs +++ b/src/NetEvolve.FrameShift.Execution/IsolatedAssemblyRunner.cs @@ -74,14 +74,12 @@ string methodName using var assemblyStream = new MemoryStream(assemblyBytes); var assembly = context.LoadFromStream(assemblyStream); - var type = - assembly.GetType(typeFullName, throwOnError: false) + var type = assembly.GetType(typeFullName, throwOnError: false) ?? throw new TypeLoadException($"The mutant assembly does not declare a type named '{typeFullName}'."); // Public only, deliberately: every mainstream test framework requires a public test method, so // reaching into non-public members here would recognise a shape none of them actually run. - var method = - type.GetMethod(methodName, BindingFlags.Public | BindingFlags.Instance | BindingFlags.Static) + var method = type.GetMethod(methodName, BindingFlags.Public | BindingFlags.Instance | BindingFlags.Static) ?? throw new MissingMethodException(typeFullName, methodName); var instance = method.IsStatic ? null : Activator.CreateInstance(type); diff --git a/src/NetEvolve.FrameShift/Analyzers/TestSurfaceAnalysis.cs b/src/NetEvolve.FrameShift/Analyzers/TestSurfaceAnalysis.cs index 582163c..208c058 100644 --- a/src/NetEvolve.FrameShift/Analyzers/TestSurfaceAnalysis.cs +++ b/src/NetEvolve.FrameShift/Analyzers/TestSurfaceAnalysis.cs @@ -136,14 +136,13 @@ internal static ImmutableArray GetTestMethods( CancellationToken cancellationToken ) { - var perCompilation = TestMethodDiscoveryCache.GetValue( - compilation, - static _ => new ConcurrentDictionary>() - ); + var perCompilation = TestMethodDiscoveryCache.GetValue(compilation, static _ => new ConcurrentDictionary< + Type, + ImmutableArray + >()); - return perCompilation.GetOrAdd( - probe.GetType(), - _ => TestMethodDiscovery.FindTestMethods(compilation, recognizer, cancellationToken) + return perCompilation.GetOrAdd(probe.GetType(), _ => + TestMethodDiscovery.FindTestMethods(compilation, recognizer, cancellationToken) ); } @@ -169,9 +168,8 @@ CancellationToken cancellationToken static _ => new ConcurrentDictionary>() ); - return perCompilation.GetOrAdd( - probe.GetType(), - _ => TestSurfaceCollector.FindTestsWithoutProductionReference(compilation, recognizer, cancellationToken) + return perCompilation.GetOrAdd(probe.GetType(), _ => + TestSurfaceCollector.FindTestsWithoutProductionReference(compilation, recognizer, cancellationToken) ); } diff --git a/src/NetEvolve.FrameShift/Equivalence/EquivalenceClassifier.cs b/src/NetEvolve.FrameShift/Equivalence/EquivalenceClassifier.cs index 9e96fa1..d454218 100644 --- a/src/NetEvolve.FrameShift/Equivalence/EquivalenceClassifier.cs +++ b/src/NetEvolve.FrameShift/Equivalence/EquivalenceClassifier.cs @@ -613,13 +613,13 @@ out var mutatedTokens } return ClassifyQuantifierCanonicalization( - originalPattern, - originalTokens, - mutatedPattern, - mutatedTokens, - CanonicalizeExactOne, - RegexExactOneQuantifierReason - ) + originalPattern, + originalTokens, + mutatedPattern, + mutatedTokens, + CanonicalizeExactOne, + RegexExactOneQuantifierReason + ) ?? ClassifyQuantifierCanonicalization( originalPattern, originalTokens, @@ -1042,8 +1042,7 @@ CancellationToken cancellationToken } var scope = mutation.Original.FirstAncestorOrSelf(); - var diagnostics = - unreachableCodeDiagnosticsCache?.GetDiagnostics(semanticModel, scope?.Span, cancellationToken) + var diagnostics = unreachableCodeDiagnosticsCache?.GetDiagnostics(semanticModel, scope?.Span, cancellationToken) ?? semanticModel.GetDiagnostics(scope?.Span, cancellationToken); var mutationSpan = mutation.Location.SourceSpan; diff --git a/src/NetEvolve.FrameShift/Generation/TestSurfaceManifestGenerator.cs b/src/NetEvolve.FrameShift/Generation/TestSurfaceManifestGenerator.cs index 5308bf2..3e3d89f 100644 --- a/src/NetEvolve.FrameShift/Generation/TestSurfaceManifestGenerator.cs +++ b/src/NetEvolve.FrameShift/Generation/TestSurfaceManifestGenerator.cs @@ -87,9 +87,8 @@ public void Initialize(IncrementalGeneratorInitializationContext context) { var provider = context.CompilationProvider.Combine(context.AnalyzerConfigOptionsProvider); - context.RegisterSourceOutput( - provider, - static (productionContext, source) => Execute(productionContext, source.Left, source.Right) + context.RegisterSourceOutput(provider, static (productionContext, source) => + Execute(productionContext, source.Left, source.Right) ); } diff --git a/src/NetEvolve.FrameShift/Mutations/WellKnownTypeCache.cs b/src/NetEvolve.FrameShift/Mutations/WellKnownTypeCache.cs index 255a134..a3a1f4a 100644 --- a/src/NetEvolve.FrameShift/Mutations/WellKnownTypeCache.cs +++ b/src/NetEvolve.FrameShift/Mutations/WellKnownTypeCache.cs @@ -59,10 +59,10 @@ internal static class WellKnownTypeCache throw new ArgumentNullException(nameof(metadataName)); } - var perCompilation = _cache.GetValue( - compilation, - static _ => new ConcurrentDictionary(StringComparer.Ordinal) - ); + var perCompilation = _cache.GetValue(compilation, static _ => new ConcurrentDictionary< + string, + INamedTypeSymbol? + >(StringComparer.Ordinal)); return perCompilation.GetOrAdd(metadataName, name => compilation.GetTypeByMetadataName(name)); } diff --git a/tests/NetEvolve.FrameShift.Tests.Integration/Analyzers/MSTestSurfaceAnalyzerTests.cs b/tests/NetEvolve.FrameShift.Tests.Integration/Analyzers/MSTestSurfaceAnalyzerTests.cs index e3393fb..321d713 100644 --- a/tests/NetEvolve.FrameShift.Tests.Integration/Analyzers/MSTestSurfaceAnalyzerTests.cs +++ b/tests/NetEvolve.FrameShift.Tests.Integration/Analyzers/MSTestSurfaceAnalyzerTests.cs @@ -350,8 +350,7 @@ private static CSharpCompilation CreateTest(Compilation production) => /// The MSTest probe did not recognise the compilation. private static string CreateManifest(Compilation test) { - var recognizer = - MSTestTestFrameworkProbe.Instance.TryCreateRecognizer(test) + var recognizer = MSTestTestFrameworkProbe.Instance.TryCreateRecognizer(test) ?? throw new InvalidOperationException( "The MSTest probe did not recognise a compilation built against the real MSTest assemblies." ); @@ -360,9 +359,8 @@ private static string CreateManifest(Compilation test) } private static MethodDeclarationSyntax FindMethod(Compilation compilation, string name) => - SyntaxNodeLocator.FindFirst( - compilation.SyntaxTrees.First(), - method => string.Equals(method.Identifier.ValueText, name, StringComparison.Ordinal) + SyntaxNodeLocator.FindFirst(compilation.SyntaxTrees.First(), method => + string.Equals(method.Identifier.ValueText, name, StringComparison.Ordinal) ); private static Task> RunAsync( diff --git a/tests/NetEvolve.FrameShift.Tests.Integration/Analyzers/NUnitTestSurfaceAnalyzerTests.cs b/tests/NetEvolve.FrameShift.Tests.Integration/Analyzers/NUnitTestSurfaceAnalyzerTests.cs index 665b6a5..c4c916e 100644 --- a/tests/NetEvolve.FrameShift.Tests.Integration/Analyzers/NUnitTestSurfaceAnalyzerTests.cs +++ b/tests/NetEvolve.FrameShift.Tests.Integration/Analyzers/NUnitTestSurfaceAnalyzerTests.cs @@ -444,9 +444,8 @@ private static string CreateManifest(Compilation test) } private static MethodDeclarationSyntax FindMethod(Compilation compilation, string name) => - SyntaxNodeLocator.FindFirst( - compilation.SyntaxTrees.First(), - method => string.Equals(method.Identifier.ValueText, name, StringComparison.Ordinal) + SyntaxNodeLocator.FindFirst(compilation.SyntaxTrees.First(), method => + string.Equals(method.Identifier.ValueText, name, StringComparison.Ordinal) ); private static Task> RunAsync( diff --git a/tests/NetEvolve.FrameShift.Tests.Integration/Analyzers/TUnitTestSurfaceAnalyzerTests.cs b/tests/NetEvolve.FrameShift.Tests.Integration/Analyzers/TUnitTestSurfaceAnalyzerTests.cs index 644f963..b2a5696 100644 --- a/tests/NetEvolve.FrameShift.Tests.Integration/Analyzers/TUnitTestSurfaceAnalyzerTests.cs +++ b/tests/NetEvolve.FrameShift.Tests.Integration/Analyzers/TUnitTestSurfaceAnalyzerTests.cs @@ -895,9 +895,8 @@ private static string CreateManifest(Compilation test) } private static MethodDeclarationSyntax FindMethod(Compilation compilation, string name) => - SyntaxNodeLocator.FindFirst( - compilation.SyntaxTrees.First(), - method => string.Equals(method.Identifier.ValueText, name, StringComparison.Ordinal) + SyntaxNodeLocator.FindFirst(compilation.SyntaxTrees.First(), method => + string.Equals(method.Identifier.ValueText, name, StringComparison.Ordinal) ); private static Task> RunAsync( diff --git a/tests/NetEvolve.FrameShift.Tests.Integration/Analyzers/TestMarkerRecognitionTests.cs b/tests/NetEvolve.FrameShift.Tests.Integration/Analyzers/TestMarkerRecognitionTests.cs index 9b8711a..bbb7c1e 100644 --- a/tests/NetEvolve.FrameShift.Tests.Integration/Analyzers/TestMarkerRecognitionTests.cs +++ b/tests/NetEvolve.FrameShift.Tests.Integration/Analyzers/TestMarkerRecognitionTests.cs @@ -1116,8 +1116,7 @@ private static CSharpCompilation CreateTest(string productionSource, string test /// private static string CreateManifest(Compilation test, ITestFrameworkProbe probe) { - var recognizer = - probe.TryCreateRecognizer(test) + var recognizer = probe.TryCreateRecognizer(test) ?? throw new InvalidOperationException( $"The probe of '{probe.FrameworkName}' did not recognise the marker fixture." ); diff --git a/tests/NetEvolve.FrameShift.Tests.Integration/Analyzers/XunitV2TestSurfaceAnalyzerTests.cs b/tests/NetEvolve.FrameShift.Tests.Integration/Analyzers/XunitV2TestSurfaceAnalyzerTests.cs index ee42745..c9222a9 100644 --- a/tests/NetEvolve.FrameShift.Tests.Integration/Analyzers/XunitV2TestSurfaceAnalyzerTests.cs +++ b/tests/NetEvolve.FrameShift.Tests.Integration/Analyzers/XunitV2TestSurfaceAnalyzerTests.cs @@ -480,9 +480,8 @@ private static string CreateManifest(Compilation test) } private static MethodDeclarationSyntax FindMethod(Compilation compilation, string name) => - SyntaxNodeLocator.FindFirst( - compilation.SyntaxTrees.First(), - method => string.Equals(method.Identifier.ValueText, name, StringComparison.Ordinal) + SyntaxNodeLocator.FindFirst(compilation.SyntaxTrees.First(), method => + string.Equals(method.Identifier.ValueText, name, StringComparison.Ordinal) ); private static Task> RunAsync( diff --git a/tests/NetEvolve.FrameShift.Tests.Integration/Analyzers/XunitV3TestSurfaceAnalyzerTests.cs b/tests/NetEvolve.FrameShift.Tests.Integration/Analyzers/XunitV3TestSurfaceAnalyzerTests.cs index 82cc32d..cbda330 100644 --- a/tests/NetEvolve.FrameShift.Tests.Integration/Analyzers/XunitV3TestSurfaceAnalyzerTests.cs +++ b/tests/NetEvolve.FrameShift.Tests.Integration/Analyzers/XunitV3TestSurfaceAnalyzerTests.cs @@ -473,9 +473,8 @@ private static string CreateManifest(Compilation test) } private static MethodDeclarationSyntax FindMethod(Compilation compilation, string name) => - SyntaxNodeLocator.FindFirst( - compilation.SyntaxTrees.First(), - method => string.Equals(method.Identifier.ValueText, name, StringComparison.Ordinal) + SyntaxNodeLocator.FindFirst(compilation.SyntaxTrees.First(), method => + string.Equals(method.Identifier.ValueText, name, StringComparison.Ordinal) ); private static Task> RunAsync( diff --git a/tests/NetEvolve.FrameShift.Tests.Unit/Analyzers/TestSurfaceAnalysisDiscoveryTests.cs b/tests/NetEvolve.FrameShift.Tests.Unit/Analyzers/TestSurfaceAnalysisDiscoveryTests.cs index 105004b..05971f2 100644 --- a/tests/NetEvolve.FrameShift.Tests.Unit/Analyzers/TestSurfaceAnalysisDiscoveryTests.cs +++ b/tests/NetEvolve.FrameShift.Tests.Unit/Analyzers/TestSurfaceAnalysisDiscoveryTests.cs @@ -115,9 +115,10 @@ ImmutableArray testMethods ); #pragma warning restore CS0618 - var method = - typeof(TestSurfaceAnalysis).GetMethod("FindAwakeFrameworks", BindingFlags.NonPublic | BindingFlags.Static) - ?? throw new InvalidOperationException("TestSurfaceAnalysis.FindAwakeFrameworks was not found."); + var method = typeof(TestSurfaceAnalysis).GetMethod( + "FindAwakeFrameworks", + BindingFlags.NonPublic | BindingFlags.Static + ) ?? throw new InvalidOperationException("TestSurfaceAnalysis.FindAwakeFrameworks was not found."); var result = method.Invoke(null, [context, probe, recognizer, testMethods, FrameShiftOptions.Default]); diff --git a/tests/NetEvolve.FrameShift.Tests.Unit/Mutations/MutantCompilerTests.cs b/tests/NetEvolve.FrameShift.Tests.Unit/Mutations/MutantCompilerTests.cs index f4ceb1a..c46b97a 100644 --- a/tests/NetEvolve.FrameShift.Tests.Unit/Mutations/MutantCompilerTests.cs +++ b/tests/NetEvolve.FrameShift.Tests.Unit/Mutations/MutantCompilerTests.cs @@ -271,16 +271,13 @@ public async Task Verify_ConcurrentlyForDistinctMutations_ReturnsViableForEveryO var mutations = CreateOperatorSwapMutations(tree); var results = new ConcurrentBag(); - _ = Parallel.ForEach( - Enumerable.Range(0, 32), - _ => + _ = Parallel.ForEach(Enumerable.Range(0, 32), _ => + { + foreach (var mutation in mutations) { - foreach (var mutation in mutations) - { - results.Add(compiler.Verify(mutation, tree, CancellationToken.None)); - } + results.Add(compiler.Verify(mutation, tree, CancellationToken.None)); } - ); + }); using (Assert.Multiple()) { @@ -300,14 +297,11 @@ public async Task Verify_ConcurrentlyForTheSameMutations_ReturnsConsistentVerdic var broken = CreateBrokenMutation(tree); var results = new ConcurrentBag<(string OperatorId, MutantViability Viability)>(); - _ = Parallel.ForEach( - Enumerable.Range(0, 32), - _ => - { - results.Add((ViableOperatorId, compiler.Verify(viable, tree, CancellationToken.None))); - results.Add((BrokenOperatorId, compiler.Verify(broken, tree, CancellationToken.None))); - } - ); + _ = Parallel.ForEach(Enumerable.Range(0, 32), _ => + { + results.Add((ViableOperatorId, compiler.Verify(viable, tree, CancellationToken.None))); + results.Add((BrokenOperatorId, compiler.Verify(broken, tree, CancellationToken.None))); + }); using (Assert.Multiple()) { @@ -789,9 +783,8 @@ private static Mutation CreateUnparsableMutation(SyntaxTree tree) /// The mutation. private static Mutation CreateBrokenMutation(SyntaxTree tree, string operatorId) { - var original = SyntaxNodeLocator.FindFirst( - tree, - node => string.Equals(node.Token.ValueText, "7", StringComparison.Ordinal) + var original = SyntaxNodeLocator.FindFirst(tree, node => + string.Equals(node.Token.ValueText, "7", StringComparison.Ordinal) ); var replacement = SyntaxFactory.LiteralExpression( SyntaxKind.StringLiteralExpression, diff --git a/tests/NetEvolve.FrameShift.Tests.Unit/Mutations/Operators/ArithmeticAssignmentMutatorTests.cs b/tests/NetEvolve.FrameShift.Tests.Unit/Mutations/Operators/ArithmeticAssignmentMutatorTests.cs index 6316c05..ee3076e 100644 --- a/tests/NetEvolve.FrameShift.Tests.Unit/Mutations/Operators/ArithmeticAssignmentMutatorTests.cs +++ b/tests/NetEvolve.FrameShift.Tests.Unit/Mutations/Operators/ArithmeticAssignmentMutatorTests.cs @@ -1017,8 +1017,7 @@ public async Task Mapper_SyntaxKindIsNoCompoundAssignment_ThrowsArgumentOutOfRan public async Task HasCounterpart_OperandCountDiffers_ReturnsFalse() { var (compilation, _, _) = CompilationFactory.CreateWithModel(UnaryAndBinaryMinusSource); - var money = - compilation.GetTypeByMetadataName("Fixtures.Money") + var money = compilation.GetTypeByMetadataName("Fixtures.Money") ?? throw new InvalidOperationException("The fixture does not declare 'Fixtures.Money'."); var unaryMinus = money.GetMembers("op_UnaryNegation").OfType().Single(); var binaryMinus = money.GetMembers("op_Subtraction").OfType().Single(); @@ -1124,9 +1123,10 @@ private static ImmutableArray AllCounterparts(string originalName) => /// private static ArgumentOutOfRangeException InvokeMapper(string mapperName, SyntaxKind expressionKind) { - var mapper = - typeof(ArithmeticAssignmentMutator).GetMethod(mapperName, BindingFlags.NonPublic | BindingFlags.Static) - ?? throw new InvalidOperationException($"The mapping method '{mapperName}' no longer exists."); + var mapper = typeof(ArithmeticAssignmentMutator).GetMethod( + mapperName, + BindingFlags.NonPublic | BindingFlags.Static + ) ?? throw new InvalidOperationException($"The mapping method '{mapperName}' no longer exists."); try { diff --git a/tests/NetEvolve.FrameShift.Tests.Unit/Mutations/Operators/ArithmeticOperatorMutatorTests.cs b/tests/NetEvolve.FrameShift.Tests.Unit/Mutations/Operators/ArithmeticOperatorMutatorTests.cs index 315d7ad..1f56c1f 100644 --- a/tests/NetEvolve.FrameShift.Tests.Unit/Mutations/Operators/ArithmeticOperatorMutatorTests.cs +++ b/tests/NetEvolve.FrameShift.Tests.Unit/Mutations/Operators/ArithmeticOperatorMutatorTests.cs @@ -882,8 +882,7 @@ public async Task Mapper_SyntaxKindIsNotArithmetic_ThrowsArgumentOutOfRangeExcep public async Task HasCounterpart_OperandCountDiffers_ReturnsFalse() { var (compilation, _, _) = CompilationFactory.CreateWithModel(UnaryAndBinaryMinusSource); - var money = - compilation.GetTypeByMetadataName("Fixtures.Money") + var money = compilation.GetTypeByMetadataName("Fixtures.Money") ?? throw new InvalidOperationException("The fixture does not declare 'Fixtures.Money'."); var unaryMinus = money.GetMembers("op_UnaryNegation").OfType().Single(); var binaryMinus = money.GetMembers("op_Subtraction").OfType().Single(); @@ -978,9 +977,10 @@ private static ImmutableArray AllCounterparts(string originalName) => /// private static ArgumentOutOfRangeException InvokeMapper(string mapperName, SyntaxKind expressionKind) { - var mapper = - typeof(ArithmeticOperatorMutator).GetMethod(mapperName, BindingFlags.NonPublic | BindingFlags.Static) - ?? throw new InvalidOperationException($"The mapping method '{mapperName}' no longer exists."); + var mapper = typeof(ArithmeticOperatorMutator).GetMethod( + mapperName, + BindingFlags.NonPublic | BindingFlags.Static + ) ?? throw new InvalidOperationException($"The mapping method '{mapperName}' no longer exists."); try { diff --git a/tests/NetEvolve.FrameShift.Tests.Unit/Mutations/Operators/BooleanLiteralMutatorTests.cs b/tests/NetEvolve.FrameShift.Tests.Unit/Mutations/Operators/BooleanLiteralMutatorTests.cs index fbd8f72..8d52580 100644 --- a/tests/NetEvolve.FrameShift.Tests.Unit/Mutations/Operators/BooleanLiteralMutatorTests.cs +++ b/tests/NetEvolve.FrameShift.Tests.Unit/Mutations/Operators/BooleanLiteralMutatorTests.cs @@ -288,15 +288,12 @@ private static (SyntaxTree Tree, Mutation[] Mutations) Run(string source, Func mutation.ApplyTo(tree).ToString(); private static SyntaxNode FindBooleanLiteral(SyntaxTree tree) => - SyntaxNodeLocator.FindFirst( - tree, - static literal => - literal.IsKind(SyntaxKind.TrueLiteralExpression) || literal.IsKind(SyntaxKind.FalseLiteralExpression) + SyntaxNodeLocator.FindFirst(tree, static literal => + literal.IsKind(SyntaxKind.TrueLiteralExpression) || literal.IsKind(SyntaxKind.FalseLiteralExpression) ); private static SyntaxNode FindNumericLiteral(SyntaxTree tree) => - SyntaxNodeLocator.FindFirst( - tree, - static literal => literal.IsKind(SyntaxKind.NumericLiteralExpression) + SyntaxNodeLocator.FindFirst(tree, static literal => + literal.IsKind(SyntaxKind.NumericLiteralExpression) ); } diff --git a/tests/NetEvolve.FrameShift.Tests.Unit/Mutations/Operators/CaseConversionMutatorTests.cs b/tests/NetEvolve.FrameShift.Tests.Unit/Mutations/Operators/CaseConversionMutatorTests.cs index d370b29..eb481e7 100644 --- a/tests/NetEvolve.FrameShift.Tests.Unit/Mutations/Operators/CaseConversionMutatorTests.cs +++ b/tests/NetEvolve.FrameShift.Tests.Unit/Mutations/Operators/CaseConversionMutatorTests.cs @@ -807,11 +807,9 @@ private static Mutation Pick(ImmutableArray mutations, string source, } private static InvocationExpressionSyntax FindCall(SyntaxTree tree, string methodName) => - SyntaxNodeLocator.FindFirst( - tree, - invocation => - invocation.Expression is MemberAccessExpressionSyntax access - && string.Equals(access.Name.Identifier.ValueText, methodName, StringComparison.Ordinal) + SyntaxNodeLocator.FindFirst(tree, invocation => + invocation.Expression is MemberAccessExpressionSyntax access + && string.Equals(access.Name.Identifier.ValueText, methodName, StringComparison.Ordinal) ); private static string[] DisplayNames(ImmutableArray mutations) => diff --git a/tests/NetEvolve.FrameShift.Tests.Unit/Mutations/Operators/CollectionInitializerMutatorTests.cs b/tests/NetEvolve.FrameShift.Tests.Unit/Mutations/Operators/CollectionInitializerMutatorTests.cs index 6b58d22..95b12b1 100644 --- a/tests/NetEvolve.FrameShift.Tests.Unit/Mutations/Operators/CollectionInitializerMutatorTests.cs +++ b/tests/NetEvolve.FrameShift.Tests.Unit/Mutations/Operators/CollectionInitializerMutatorTests.cs @@ -443,9 +443,8 @@ public async Task CreateMutations_UnsupportedSyntaxKind_ReturnsEmpty() { var (_, semanticModel, tree) = CompilationFactory.CreateWithModel(ArrayInitializerSource); var mutator = new CollectionInitializerMutator(); - var literal = SyntaxNodeLocator.FindFirst( - tree, - static candidate => candidate.IsKind(SyntaxKind.NumericLiteralExpression) + var literal = SyntaxNodeLocator.FindFirst(tree, static candidate => + candidate.IsKind(SyntaxKind.NumericLiteralExpression) ); Mutation[] mutations = [.. mutator.CreateMutations(literal, semanticModel, CancellationToken.None)]; diff --git a/tests/NetEvolve.FrameShift.Tests.Unit/Mutations/Operators/ConditionalExpressionMutatorTests.cs b/tests/NetEvolve.FrameShift.Tests.Unit/Mutations/Operators/ConditionalExpressionMutatorTests.cs index 5abd2ed..581de22 100644 --- a/tests/NetEvolve.FrameShift.Tests.Unit/Mutations/Operators/ConditionalExpressionMutatorTests.cs +++ b/tests/NetEvolve.FrameShift.Tests.Unit/Mutations/Operators/ConditionalExpressionMutatorTests.cs @@ -857,8 +857,7 @@ private static SyntaxNode FindConditional(SyntaxTree tree) => SyntaxNodeLocator.FindFirst(tree); private static SyntaxNode FindNumericLiteral(SyntaxTree tree) => - SyntaxNodeLocator.FindFirst( - tree, - static literal => literal.IsKind(SyntaxKind.NumericLiteralExpression) + SyntaxNodeLocator.FindFirst(tree, static literal => + literal.IsKind(SyntaxKind.NumericLiteralExpression) ); } diff --git a/tests/NetEvolve.FrameShift.Tests.Unit/Mutations/Operators/CultureInfoMutatorTests.cs b/tests/NetEvolve.FrameShift.Tests.Unit/Mutations/Operators/CultureInfoMutatorTests.cs index 6f14d5b..955fa95 100644 --- a/tests/NetEvolve.FrameShift.Tests.Unit/Mutations/Operators/CultureInfoMutatorTests.cs +++ b/tests/NetEvolve.FrameShift.Tests.Unit/Mutations/Operators/CultureInfoMutatorTests.cs @@ -761,9 +761,8 @@ private static string Ids(IEnumerable mutations) => string.Join(", ", mutations.Select(mutation => mutation.OperatorId)); private static SyntaxNode FindCultureAccess(SyntaxTree tree) => - SyntaxNodeLocator.FindFirst( - tree, - static access => IsCultureMemberName(access.Name.Identifier.ValueText) + SyntaxNodeLocator.FindFirst(tree, static access => + IsCultureMemberName(access.Name.Identifier.ValueText) ); private static bool IsCultureMemberName(string name) => diff --git a/tests/NetEvolve.FrameShift.Tests.Unit/Mutations/Operators/IncrementDecrementMutatorTests.cs b/tests/NetEvolve.FrameShift.Tests.Unit/Mutations/Operators/IncrementDecrementMutatorTests.cs index 77f372f..43512bb 100644 --- a/tests/NetEvolve.FrameShift.Tests.Unit/Mutations/Operators/IncrementDecrementMutatorTests.cs +++ b/tests/NetEvolve.FrameShift.Tests.Unit/Mutations/Operators/IncrementDecrementMutatorTests.cs @@ -552,8 +552,7 @@ public async Task CreateMutations_MemberWithTheCounterpartNameThatIsNoOperator_R public async Task HasCounterpart_MemberIsNoOperator_ReturnsFalse() { var (compilation, _, _) = CompilationFactory.CreateWithModel(BorrowedOperatorNameSource); - var ticks = - compilation.GetTypeByMetadataName("Fixtures.Ticks") + var ticks = compilation.GetTypeByMetadataName("Fixtures.Ticks") ?? throw new InvalidOperationException("The fixture does not declare 'Fixtures.Ticks'."); var increment = ticks.GetMembers("op_Increment").OfType().Single(); diff --git a/tests/NetEvolve.FrameShift.Tests.Unit/Mutations/Operators/IntegralTypeCheckTests.cs b/tests/NetEvolve.FrameShift.Tests.Unit/Mutations/Operators/IntegralTypeCheckTests.cs index 89d212e..8f2ed9d 100644 --- a/tests/NetEvolve.FrameShift.Tests.Unit/Mutations/Operators/IntegralTypeCheckTests.cs +++ b/tests/NetEvolve.FrameShift.Tests.Unit/Mutations/Operators/IntegralTypeCheckTests.cs @@ -129,9 +129,8 @@ public async Task IsIntegral_NullType_ReturnsFalse() => private static ITypeSymbol GetParameterType(string parameterName) { var (_, semanticModel, tree) = CompilationFactory.CreateWithModel(Source); - var parameter = SyntaxNodeLocator.FindFirst( - tree, - candidate => string.Equals(candidate.Identifier.ValueText, parameterName, StringComparison.Ordinal) + var parameter = SyntaxNodeLocator.FindFirst(tree, candidate => + string.Equals(candidate.Identifier.ValueText, parameterName, StringComparison.Ordinal) ); var symbol = semanticModel.GetDeclaredSymbol(parameter) as IParameterSymbol; diff --git a/tests/NetEvolve.FrameShift.Tests.Unit/Mutations/Operators/LinqMethodMutatorTests.cs b/tests/NetEvolve.FrameShift.Tests.Unit/Mutations/Operators/LinqMethodMutatorTests.cs index 9f86bf6..f7061b0 100644 --- a/tests/NetEvolve.FrameShift.Tests.Unit/Mutations/Operators/LinqMethodMutatorTests.cs +++ b/tests/NetEvolve.FrameShift.Tests.Unit/Mutations/Operators/LinqMethodMutatorTests.cs @@ -396,11 +396,9 @@ private static string CreateSource(string call) => QueryTemplate.Replace(CallPlaceholder, call, StringComparison.Ordinal); private static InvocationExpressionSyntax FindCall(SyntaxTree tree, string methodName) => - SyntaxNodeLocator.FindFirst( - tree, - invocation => - invocation.Expression is MemberAccessExpressionSyntax access - && string.Equals(access.Name.Identifier.ValueText, methodName, StringComparison.Ordinal) + SyntaxNodeLocator.FindFirst(tree, invocation => + invocation.Expression is MemberAccessExpressionSyntax access + && string.Equals(access.Name.Identifier.ValueText, methodName, StringComparison.Ordinal) ); private static string Join(IEnumerable values) => string.Join(", ", values); diff --git a/tests/NetEvolve.FrameShift.Tests.Unit/Mutations/Operators/MathMethodMutatorTests.cs b/tests/NetEvolve.FrameShift.Tests.Unit/Mutations/Operators/MathMethodMutatorTests.cs index 49bd657..19d6f78 100644 --- a/tests/NetEvolve.FrameShift.Tests.Unit/Mutations/Operators/MathMethodMutatorTests.cs +++ b/tests/NetEvolve.FrameShift.Tests.Unit/Mutations/Operators/MathMethodMutatorTests.cs @@ -509,11 +509,9 @@ private static string CreateBinarySource(string type, string methodName) => .Replace(MethodPlaceholder, methodName, StringComparison.Ordinal); private static InvocationExpressionSyntax FindCall(SyntaxTree tree, string methodName) => - SyntaxNodeLocator.FindFirst( - tree, - invocation => - invocation.Expression is MemberAccessExpressionSyntax access - && string.Equals(access.Name.Identifier.ValueText, methodName, StringComparison.Ordinal) + SyntaxNodeLocator.FindFirst(tree, invocation => + invocation.Expression is MemberAccessExpressionSyntax access + && string.Equals(access.Name.Identifier.ValueText, methodName, StringComparison.Ordinal) ); private static string[] DisplayNames(ImmutableArray mutations) => diff --git a/tests/NetEvolve.FrameShift.Tests.Unit/Mutations/Operators/NullCoalescingMutatorTests.cs b/tests/NetEvolve.FrameShift.Tests.Unit/Mutations/Operators/NullCoalescingMutatorTests.cs index 363ee3a..9a3484e 100644 --- a/tests/NetEvolve.FrameShift.Tests.Unit/Mutations/Operators/NullCoalescingMutatorTests.cs +++ b/tests/NetEvolve.FrameShift.Tests.Unit/Mutations/Operators/NullCoalescingMutatorTests.cs @@ -317,15 +317,13 @@ private static (SemanticModel SemanticModel, SyntaxTree Tree, Mutation[] Mutatio private static string Rewrite(SyntaxTree tree, Mutation mutation) => mutation.ApplyTo(tree).ToString(); private static SyntaxNode FindCoalesce(SyntaxTree tree) => - SyntaxNodeLocator.FindFirst( - tree, - static binary => binary.IsKind(SyntaxKind.CoalesceExpression) + SyntaxNodeLocator.FindFirst(tree, static binary => + binary.IsKind(SyntaxKind.CoalesceExpression) ); private static SyntaxNode FindAddition(SyntaxTree tree) => - SyntaxNodeLocator.FindFirst( - tree, - static binary => binary.IsKind(SyntaxKind.AddExpression) + SyntaxNodeLocator.FindFirst(tree, static binary => + binary.IsKind(SyntaxKind.AddExpression) ); private static SyntaxNode FindAssignment(SyntaxTree tree) => diff --git a/tests/NetEvolve.FrameShift.Tests.Unit/Mutations/Operators/NullableLiteralMutatorTests.cs b/tests/NetEvolve.FrameShift.Tests.Unit/Mutations/Operators/NullableLiteralMutatorTests.cs index 9da15c5..1b7f7a9 100644 --- a/tests/NetEvolve.FrameShift.Tests.Unit/Mutations/Operators/NullableLiteralMutatorTests.cs +++ b/tests/NetEvolve.FrameShift.Tests.Unit/Mutations/Operators/NullableLiteralMutatorTests.cs @@ -674,44 +674,34 @@ private static (SyntaxTree Tree, Mutation[] Mutations) Run(string source, Func mutation.ApplyTo(tree).ToString(); private static SyntaxNode FindBooleanOrNullLiteral(SyntaxTree tree) => - SyntaxNodeLocator.FindFirst( - tree, - static literal => - literal.IsKind(SyntaxKind.TrueLiteralExpression) - || literal.IsKind(SyntaxKind.FalseLiteralExpression) - || literal.IsKind(SyntaxKind.NullLiteralExpression) + SyntaxNodeLocator.FindFirst(tree, static literal => + literal.IsKind(SyntaxKind.TrueLiteralExpression) + || literal.IsKind(SyntaxKind.FalseLiteralExpression) + || literal.IsKind(SyntaxKind.NullLiteralExpression) ); private static SyntaxNode FindNumericOrNullLiteral(SyntaxTree tree) => - SyntaxNodeLocator.FindFirst( - tree, - static literal => - literal.IsKind(SyntaxKind.NumericLiteralExpression) || literal.IsKind(SyntaxKind.NullLiteralExpression) + SyntaxNodeLocator.FindFirst(tree, static literal => + literal.IsKind(SyntaxKind.NumericLiteralExpression) || literal.IsKind(SyntaxKind.NullLiteralExpression) ); private static SyntaxNode FindUnaryMinusExpression(SyntaxTree tree) => - SyntaxNodeLocator.FindFirst( - tree, - static unary => unary.IsKind(SyntaxKind.UnaryMinusExpression) + SyntaxNodeLocator.FindFirst(tree, static unary => + unary.IsKind(SyntaxKind.UnaryMinusExpression) ); private static SyntaxNode FindCharOrNullLiteral(SyntaxTree tree) => - SyntaxNodeLocator.FindFirst( - tree, - static literal => - literal.IsKind(SyntaxKind.CharacterLiteralExpression) - || literal.IsKind(SyntaxKind.NullLiteralExpression) + SyntaxNodeLocator.FindFirst(tree, static literal => + literal.IsKind(SyntaxKind.CharacterLiteralExpression) || literal.IsKind(SyntaxKind.NullLiteralExpression) ); private static SyntaxNode FindDefaultLiteral(SyntaxTree tree) => - SyntaxNodeLocator.FindFirst( - tree, - static literal => literal.IsKind(SyntaxKind.DefaultLiteralExpression) + SyntaxNodeLocator.FindFirst(tree, static literal => + literal.IsKind(SyntaxKind.DefaultLiteralExpression) ); private static SyntaxNode FindStringLiteral(SyntaxTree tree) => - SyntaxNodeLocator.FindFirst( - tree, - static literal => literal.IsKind(SyntaxKind.StringLiteralExpression) + SyntaxNodeLocator.FindFirst(tree, static literal => + literal.IsKind(SyntaxKind.StringLiteralExpression) ); } diff --git a/tests/NetEvolve.FrameShift.Tests.Unit/Mutations/Operators/NumericLiteralMutatorTests.cs b/tests/NetEvolve.FrameShift.Tests.Unit/Mutations/Operators/NumericLiteralMutatorTests.cs index d90319b..980d157 100644 --- a/tests/NetEvolve.FrameShift.Tests.Unit/Mutations/Operators/NumericLiteralMutatorTests.cs +++ b/tests/NetEvolve.FrameShift.Tests.Unit/Mutations/Operators/NumericLiteralMutatorTests.cs @@ -948,9 +948,8 @@ private static (SyntaxTree Tree, Mutation[] Mutations) Run(string source, Func mutation.ApplyTo(tree).ToString(); private static SyntaxNode FindNumericLiteral(SyntaxTree tree) => - SyntaxNodeLocator.FindFirst( - tree, - static literal => literal.IsKind(SyntaxKind.NumericLiteralExpression) + SyntaxNodeLocator.FindFirst(tree, static literal => + literal.IsKind(SyntaxKind.NumericLiteralExpression) ); /// @@ -966,8 +965,7 @@ private static LiteralExpressionSyntax FindDirectiveLiteral(SyntaxTree tree) => .First(static literal => literal.IsKind(SyntaxKind.NumericLiteralExpression)); private static SyntaxNode FindStringLiteral(SyntaxTree tree) => - SyntaxNodeLocator.FindFirst( - tree, - static literal => literal.IsKind(SyntaxKind.StringLiteralExpression) + SyntaxNodeLocator.FindFirst(tree, static literal => + literal.IsKind(SyntaxKind.StringLiteralExpression) ); } diff --git a/tests/NetEvolve.FrameShift.Tests.Unit/Mutations/Operators/ParenthesizationMutatorTests.cs b/tests/NetEvolve.FrameShift.Tests.Unit/Mutations/Operators/ParenthesizationMutatorTests.cs index 143a6c8..94dbdbc 100644 --- a/tests/NetEvolve.FrameShift.Tests.Unit/Mutations/Operators/ParenthesizationMutatorTests.cs +++ b/tests/NetEvolve.FrameShift.Tests.Unit/Mutations/Operators/ParenthesizationMutatorTests.cs @@ -407,11 +407,10 @@ string source /// The guard no longer exists. private static bool InvokeIsGenuineArithmeticOperand(ExpressionSyntax expression, SemanticModel semanticModel) { - var guard = - typeof(ParenthesizationMutator).GetMethod( - "IsGenuineArithmeticOperand", - BindingFlags.NonPublic | BindingFlags.Static - ) ?? throw new InvalidOperationException("The operand guard no longer exists."); + var guard = typeof(ParenthesizationMutator).GetMethod( + "IsGenuineArithmeticOperand", + BindingFlags.NonPublic | BindingFlags.Static + ) ?? throw new InvalidOperationException("The operand guard no longer exists."); return (bool)guard.Invoke(null, [expression, semanticModel, CancellationToken.None])!; } diff --git a/tests/NetEvolve.FrameShift.Tests.Unit/Mutations/Operators/RegexLookaroundMutatorTests.cs b/tests/NetEvolve.FrameShift.Tests.Unit/Mutations/Operators/RegexLookaroundMutatorTests.cs index 7bd98f5..1282ab9 100644 --- a/tests/NetEvolve.FrameShift.Tests.Unit/Mutations/Operators/RegexLookaroundMutatorTests.cs +++ b/tests/NetEvolve.FrameShift.Tests.Unit/Mutations/Operators/RegexLookaroundMutatorTests.cs @@ -337,9 +337,10 @@ public async Task TryCreateRewrite_TokenTextTheTokenizerNeverProduces_ReturnsNul /// The rewrite factory no longer exists. private static object? InvokeTryCreateRewrite(string pattern, RegexToken token) { - var method = - typeof(RegexLookaroundMutator).GetMethod("TryCreateRewrite", BindingFlags.NonPublic | BindingFlags.Static) - ?? throw new InvalidOperationException("The rewrite factory 'TryCreateRewrite' no longer exists."); + var method = typeof(RegexLookaroundMutator).GetMethod( + "TryCreateRewrite", + BindingFlags.NonPublic | BindingFlags.Static + ) ?? throw new InvalidOperationException("The rewrite factory 'TryCreateRewrite' no longer exists."); return method.Invoke(null, [pattern, token]); } diff --git a/tests/NetEvolve.FrameShift.Tests.Unit/Mutations/Operators/RegexOptionsMutatorTests.cs b/tests/NetEvolve.FrameShift.Tests.Unit/Mutations/Operators/RegexOptionsMutatorTests.cs index 1a48922..785d7ec 100644 --- a/tests/NetEvolve.FrameShift.Tests.Unit/Mutations/Operators/RegexOptionsMutatorTests.cs +++ b/tests/NetEvolve.FrameShift.Tests.Unit/Mutations/Operators/RegexOptionsMutatorTests.cs @@ -725,14 +725,10 @@ public async Task CreateMutations_FullyQualifiedFlag_KeepsTheQualifier() [Test] public async Task CreateMutations_TypeQualifierOfAQualifiedFlag_ReturnsEmpty() { - var (_, mutations) = Mutate( - QualifiedSource, - static tree => - SyntaxNodeLocator.FindFirst( - tree, - static memberAccess => - string.Equals(memberAccess.Name.Identifier.ValueText, "RegexOptions", StringComparison.Ordinal) - ) + var (_, mutations) = Mutate(QualifiedSource, static tree => + SyntaxNodeLocator.FindFirst(tree, static memberAccess => + string.Equals(memberAccess.Name.Identifier.ValueText, "RegexOptions", StringComparison.Ordinal) + ) ); _ = await Assert.That(mutations).IsEmpty(); diff --git a/tests/NetEvolve.FrameShift.Tests.Unit/Mutations/Operators/StringComparerComparisonAxisParityTests.cs b/tests/NetEvolve.FrameShift.Tests.Unit/Mutations/Operators/StringComparerComparisonAxisParityTests.cs index 69de3b6..d9e8a5e 100644 --- a/tests/NetEvolve.FrameShift.Tests.Unit/Mutations/Operators/StringComparerComparisonAxisParityTests.cs +++ b/tests/NetEvolve.FrameShift.Tests.Unit/Mutations/Operators/StringComparerComparisonAxisParityTests.cs @@ -77,9 +77,8 @@ private static string AxesOf(MutationOperatorBase mutator, string template, stri { var fixture = template.Replace("MEMBER", source, StringComparison.Ordinal); var (_, semanticModel, tree) = CompilationFactory.CreateWithModel(fixture); - var node = SyntaxNodeLocator.FindFirst( - tree, - access => _memberNames.Contains(access.Name.Identifier.ValueText, StringComparer.Ordinal) + var node = SyntaxNodeLocator.FindFirst(tree, access => + _memberNames.Contains(access.Name.Identifier.ValueText, StringComparer.Ordinal) ); var mutations = mutator.CreateMutations(node, semanticModel, CancellationToken.None).ToArray(); diff --git a/tests/NetEvolve.FrameShift.Tests.Unit/Mutations/Operators/StringComparerMutatorTests.cs b/tests/NetEvolve.FrameShift.Tests.Unit/Mutations/Operators/StringComparerMutatorTests.cs index 6072a07..8db7525 100644 --- a/tests/NetEvolve.FrameShift.Tests.Unit/Mutations/Operators/StringComparerMutatorTests.cs +++ b/tests/NetEvolve.FrameShift.Tests.Unit/Mutations/Operators/StringComparerMutatorTests.cs @@ -692,14 +692,12 @@ private static bool IsComparerAccess(MemberAccessExpressionSyntax access) => _memberNames.Contains(access.Name.Identifier.ValueText, StringComparer.Ordinal); private static SyntaxNode FindTypeQualifier(SyntaxTree tree) => - SyntaxNodeLocator.FindFirst( - tree, - static access => string.Equals(access.Name.Identifier.ValueText, "StringComparer", StringComparison.Ordinal) + SyntaxNodeLocator.FindFirst(tree, static access => + string.Equals(access.Name.Identifier.ValueText, "StringComparer", StringComparison.Ordinal) ); private static SyntaxNode FindImportedMember(SyntaxTree tree) => - SyntaxNodeLocator.FindFirst( - tree, - static name => string.Equals(name.Identifier.ValueText, "Ordinal", StringComparison.Ordinal) + SyntaxNodeLocator.FindFirst(tree, static name => + string.Equals(name.Identifier.ValueText, "Ordinal", StringComparison.Ordinal) ); } diff --git a/tests/NetEvolve.FrameShift.Tests.Unit/Mutations/Operators/StringComparisonMutatorTests.cs b/tests/NetEvolve.FrameShift.Tests.Unit/Mutations/Operators/StringComparisonMutatorTests.cs index 50e4b45..3868613 100644 --- a/tests/NetEvolve.FrameShift.Tests.Unit/Mutations/Operators/StringComparisonMutatorTests.cs +++ b/tests/NetEvolve.FrameShift.Tests.Unit/Mutations/Operators/StringComparisonMutatorTests.cs @@ -707,21 +707,17 @@ private static string TargetOf(Mutation mutation) => ((MemberAccessExpressionSyntax)mutation.Replacement).Name.Identifier.ValueText; private static SyntaxNode FindComparisonAccess(SyntaxTree tree) => - SyntaxNodeLocator.FindFirst( - tree, - static access => _memberNames.Contains(access.Name.Identifier.ValueText, StringComparer.Ordinal) + SyntaxNodeLocator.FindFirst(tree, static access => + _memberNames.Contains(access.Name.Identifier.ValueText, StringComparer.Ordinal) ); private static SyntaxNode FindTypeQualifier(SyntaxTree tree) => - SyntaxNodeLocator.FindFirst( - tree, - static access => - string.Equals(access.Name.Identifier.ValueText, "StringComparison", StringComparison.Ordinal) + SyntaxNodeLocator.FindFirst(tree, static access => + string.Equals(access.Name.Identifier.ValueText, "StringComparison", StringComparison.Ordinal) ); private static SyntaxNode FindImportedMember(SyntaxTree tree) => - SyntaxNodeLocator.FindFirst( - tree, - static name => string.Equals(name.Identifier.ValueText, "Ordinal", StringComparison.Ordinal) + SyntaxNodeLocator.FindFirst(tree, static name => + string.Equals(name.Identifier.ValueText, "Ordinal", StringComparison.Ordinal) ); } diff --git a/tests/NetEvolve.FrameShift.Tests.Unit/Mutations/Operators/StringLiteralMutatorTests.cs b/tests/NetEvolve.FrameShift.Tests.Unit/Mutations/Operators/StringLiteralMutatorTests.cs index 83dd435..1045c07 100644 --- a/tests/NetEvolve.FrameShift.Tests.Unit/Mutations/Operators/StringLiteralMutatorTests.cs +++ b/tests/NetEvolve.FrameShift.Tests.Unit/Mutations/Operators/StringLiteralMutatorTests.cs @@ -349,14 +349,12 @@ private static Mutation[] Mutate(StringLiteralMutator mutator, SyntaxNode node, private static string Rewrite(SyntaxTree tree, Mutation mutation) => mutation.ApplyTo(tree).ToString(); private static SyntaxNode FindStringLiteral(SyntaxTree tree) => - SyntaxNodeLocator.FindFirst( - tree, - static literal => literal.IsKind(SyntaxKind.StringLiteralExpression) + SyntaxNodeLocator.FindFirst(tree, static literal => + literal.IsKind(SyntaxKind.StringLiteralExpression) ); private static SyntaxNode FindNumericLiteral(SyntaxTree tree) => - SyntaxNodeLocator.FindFirst( - tree, - static literal => literal.IsKind(SyntaxKind.NumericLiteralExpression) + SyntaxNodeLocator.FindFirst(tree, static literal => + literal.IsKind(SyntaxKind.NumericLiteralExpression) ); } diff --git a/tests/NetEvolve.FrameShift.Tests.Unit/Mutations/Operators/StringMethodMutatorTests.cs b/tests/NetEvolve.FrameShift.Tests.Unit/Mutations/Operators/StringMethodMutatorTests.cs index 6a083da..f3cbdcc 100644 --- a/tests/NetEvolve.FrameShift.Tests.Unit/Mutations/Operators/StringMethodMutatorTests.cs +++ b/tests/NetEvolve.FrameShift.Tests.Unit/Mutations/Operators/StringMethodMutatorTests.cs @@ -515,11 +515,9 @@ private static Mutation Pick(ImmutableArray mutations, string source, } private static InvocationExpressionSyntax FindCall(SyntaxTree tree, string methodName) => - SyntaxNodeLocator.FindFirst( - tree, - invocation => - invocation.Expression is MemberAccessExpressionSyntax access - && string.Equals(access.Name.Identifier.ValueText, methodName, StringComparison.Ordinal) + SyntaxNodeLocator.FindFirst(tree, invocation => + invocation.Expression is MemberAccessExpressionSyntax access + && string.Equals(access.Name.Identifier.ValueText, methodName, StringComparison.Ordinal) ); private static string[] DisplayNames(ImmutableArray mutations) => diff --git a/tests/NetEvolve.FrameShift.Tests.Unit/Mutations/Operators/UnaryOperatorMutatorTests.cs b/tests/NetEvolve.FrameShift.Tests.Unit/Mutations/Operators/UnaryOperatorMutatorTests.cs index 96cb63b..8d50447 100644 --- a/tests/NetEvolve.FrameShift.Tests.Unit/Mutations/Operators/UnaryOperatorMutatorTests.cs +++ b/tests/NetEvolve.FrameShift.Tests.Unit/Mutations/Operators/UnaryOperatorMutatorTests.cs @@ -417,8 +417,7 @@ public async Task CreateMutations_MemberWithTheCounterpartNameThatIsNoOperator_P public async Task HasCounterpart_MemberIsNoOperator_ReturnsFalse() { var (compilation, _, _) = CompilationFactory.CreateWithModel(BorrowedOperatorNameSource); - var money = - compilation.GetTypeByMetadataName("Fixtures.Money") + var money = compilation.GetTypeByMetadataName("Fixtures.Money") ?? throw new InvalidOperationException("The fixture does not declare 'Fixtures.Money'."); var negate = money.GetMembers("op_UnaryNegation").OfType().Single(); diff --git a/tests/NetEvolve.FrameShift.Tests.Unit/Reachability/ReachabilityClosureMemberTests.cs b/tests/NetEvolve.FrameShift.Tests.Unit/Reachability/ReachabilityClosureMemberTests.cs index c3d1472..48c7254 100644 --- a/tests/NetEvolve.FrameShift.Tests.Unit/Reachability/ReachabilityClosureMemberTests.cs +++ b/tests/NetEvolve.FrameShift.Tests.Unit/Reachability/ReachabilityClosureMemberTests.cs @@ -597,9 +597,8 @@ private static TestSurfaceManifest Manifest(params ISymbol[] seeds) } private static LocalFunctionStatementSyntax LocalFunction(SyntaxTree tree, string name) => - SyntaxNodeLocator.FindFirst( - tree, - localFunction => string.Equals(localFunction.Identifier.ValueText, name, StringComparison.Ordinal) + SyntaxNodeLocator.FindFirst(tree, localFunction => + string.Equals(localFunction.Identifier.ValueText, name, StringComparison.Ordinal) ); private static IMethodSymbol Method(Compilation compilation, string typeName, string methodName) => diff --git a/tests/NetEvolve.FrameShift.Tests.Unit/Reachability/ReachableSymbolSetTests.cs b/tests/NetEvolve.FrameShift.Tests.Unit/Reachability/ReachableSymbolSetTests.cs index 08f31fa..63428d9 100644 --- a/tests/NetEvolve.FrameShift.Tests.Unit/Reachability/ReachableSymbolSetTests.cs +++ b/tests/NetEvolve.FrameShift.Tests.Unit/Reachability/ReachableSymbolSetTests.cs @@ -292,9 +292,8 @@ private static IMethodSymbol ReducedExtensionInvocation(SemanticModel semanticMo private static ISymbol LocalFunction(SemanticModel semanticModel, SyntaxTree tree, string name) { - var declaration = SyntaxNodeLocator.FindFirst( - tree, - node => string.Equals(node.Identifier.ValueText, name, StringComparison.Ordinal) + var declaration = SyntaxNodeLocator.FindFirst(tree, node => + string.Equals(node.Identifier.ValueText, name, StringComparison.Ordinal) ); return semanticModel.GetDeclaredSymbol(declaration)!; From 349e5667aa7431c90308ca110db251519ccb6913 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Martin=20St=C3=BChmer?= Date: Fri, 13 Nov 2026 22:10:26 +0100 Subject: [PATCH 3/3] fix(tests): use explicit empty set to satisfy MA0002 after csharpier 1.4.0 bump --- .../Reachability/ReachabilityClosureBehavioralTests.cs | 5 ++++- .../Reachability/ReachabilityClosureMemberTests.cs | 5 ++++- .../Reachability/ReachabilityClosurePolymorphismTests.cs | 5 ++++- .../Reachability/ReachabilityClosureTests.cs | 5 ++++- 4 files changed, 16 insertions(+), 4 deletions(-) diff --git a/tests/NetEvolve.FrameShift.Tests.Unit/Reachability/ReachabilityClosureBehavioralTests.cs b/tests/NetEvolve.FrameShift.Tests.Unit/Reachability/ReachabilityClosureBehavioralTests.cs index 0b7fc3b..ecae68e 100644 --- a/tests/NetEvolve.FrameShift.Tests.Unit/Reachability/ReachabilityClosureBehavioralTests.cs +++ b/tests/NetEvolve.FrameShift.Tests.Unit/Reachability/ReachabilityClosureBehavioralTests.cs @@ -294,7 +294,10 @@ public async Task ComputeBehavioral_CancelledToken_ThrowsOperationCanceledExcept } private static TestSurfaceManifest PlainManifest(params string[] referencedMemberIds) => - new TestSurfaceManifest([], ImmutableHashSet.Create(StringComparer.Ordinal, referencedMemberIds)); + new TestSurfaceManifest( + ImmutableHashSet.Empty, + ImmutableHashSet.Create(StringComparer.Ordinal, referencedMemberIds) + ); private static TestSurfaceManifest BehavioralManifest(params string[] behavioralReferencedMemberIds) => new TestSurfaceManifest( diff --git a/tests/NetEvolve.FrameShift.Tests.Unit/Reachability/ReachabilityClosureMemberTests.cs b/tests/NetEvolve.FrameShift.Tests.Unit/Reachability/ReachabilityClosureMemberTests.cs index 48c7254..90ffeff 100644 --- a/tests/NetEvolve.FrameShift.Tests.Unit/Reachability/ReachabilityClosureMemberTests.cs +++ b/tests/NetEvolve.FrameShift.Tests.Unit/Reachability/ReachabilityClosureMemberTests.cs @@ -593,7 +593,10 @@ private static TestSurfaceManifest Manifest(params ISymbol[] seeds) { string[] referencedMemberIds = [.. seeds.Select(DeclarationId)]; - return new TestSurfaceManifest([], ImmutableHashSet.Create(StringComparer.Ordinal, referencedMemberIds)); + return new TestSurfaceManifest( + ImmutableHashSet.Empty, + ImmutableHashSet.Create(StringComparer.Ordinal, referencedMemberIds) + ); } private static LocalFunctionStatementSyntax LocalFunction(SyntaxTree tree, string name) => diff --git a/tests/NetEvolve.FrameShift.Tests.Unit/Reachability/ReachabilityClosurePolymorphismTests.cs b/tests/NetEvolve.FrameShift.Tests.Unit/Reachability/ReachabilityClosurePolymorphismTests.cs index 8714ade..6d68032 100644 --- a/tests/NetEvolve.FrameShift.Tests.Unit/Reachability/ReachabilityClosurePolymorphismTests.cs +++ b/tests/NetEvolve.FrameShift.Tests.Unit/Reachability/ReachabilityClosurePolymorphismTests.cs @@ -987,7 +987,10 @@ public async Task ComputeFromReferences_AbstractMemberReachedByTwoTests_Attribut } private static TestSurfaceManifest Manifest(params string[] referencedMemberIds) => - new TestSurfaceManifest([], ImmutableHashSet.Create(StringComparer.Ordinal, referencedMemberIds)); + new TestSurfaceManifest( + ImmutableHashSet.Empty, + ImmutableHashSet.Create(StringComparer.Ordinal, referencedMemberIds) + ); private static ImmutableDictionary> References( params (string TestId, string[] ReferencedMemberIds)[] tests diff --git a/tests/NetEvolve.FrameShift.Tests.Unit/Reachability/ReachabilityClosureTests.cs b/tests/NetEvolve.FrameShift.Tests.Unit/Reachability/ReachabilityClosureTests.cs index 25ba43d..dc96a42 100644 --- a/tests/NetEvolve.FrameShift.Tests.Unit/Reachability/ReachabilityClosureTests.cs +++ b/tests/NetEvolve.FrameShift.Tests.Unit/Reachability/ReachabilityClosureTests.cs @@ -709,7 +709,10 @@ public async Task GetEnclosingTestIds_LocalFunction_MatchesTheAttributionOfItsEn } private static TestSurfaceManifest Manifest(params string[] referencedMemberIds) => - new TestSurfaceManifest([], ImmutableHashSet.Create(StringComparer.Ordinal, referencedMemberIds)); + new TestSurfaceManifest( + ImmutableHashSet.Empty, + ImmutableHashSet.Create(StringComparer.Ordinal, referencedMemberIds) + ); private static ImmutableDictionary> References( params (string TestId, string[] ReferencedMemberIds)[] tests