Skip to content

feat: re-enable mutation testing#328

Merged
vbreuss merged 2 commits intomainfrom
topic/re-enable-stryker
Jan 5, 2026
Merged

feat: re-enable mutation testing#328
vbreuss merged 2 commits intomainfrom
topic/re-enable-stryker

Conversation

@vbreuss
Copy link
Contributor

@vbreuss vbreuss commented Dec 18, 2025

This PR re-enables mutation testing for the project by updating the Stryker.NET version and toggling the mutation test flag. The changes address a previously blocking issue that has been resolved upstream.

Key Changes

  • Re-enabled mutation testing by setting DisableMutationTests to false
  • Updated Stryker.NET tool version from 4.7.0 to 4.10.0

This update depends on the implementation of stryker-mutator/stryker-net#3327 (comment) and stryker-mutator/stryker-net#3353.

After stryker-mutator/stryker-net#3327 (comment) got implemented, re-enable stryker mutation testing
Copilot AI review requested due to automatic review settings December 18, 2025 19:10
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR re-enables mutation testing for the project by updating the Stryker.NET version and toggling the mutation test flag. The changes address a previously blocking issue that has been resolved upstream.

Key Changes:

  • Re-enabled mutation testing by setting DisableMutationTests to false
  • Updated Stryker.NET tool version from 4.7.0 to 4.9.0

@github-actions
Copy link

github-actions bot commented Dec 18, 2025

Test Results

    14 files  ±  0      14 suites  ±0   5m 22s ⏱️ +36s
 1 802 tests + 21   1 801 ✅ + 21  1 💤 ±0  0 ❌ ±0 
12 031 runs  +107  12 030 ✅ +107  1 💤 ±0  0 ❌ ±0 

Results for commit 893f6c0. ± Comparison against base commit 8e21298.

This pull request removes 2 and adds 23 tests. Note that renamed tests count towards both.
Mockolate.SourceGenerators.Tests.MockGeneratorTests ‑ WhenUsingMockCreateFromOtherNamespace_ShouldNotBeIncluded
Mockolate.SourceGenerators.Tests.Sources.ForMockTests+ImplementClassTests ‑ Methods_ShouldSupportRefAndOutParameters
Mockolate.SourceGenerators.Tests.GeneralTests ‑ MockOfGenericTypes_ShouldHandleSpecialGenericParameters
Mockolate.SourceGenerators.Tests.GeneralTests ‑ MultipleInterfacesWithSameName_ShouldAddSuffixToSetupAndVerifyMethods
Mockolate.SourceGenerators.Tests.MockGeneratorTests ‑ WhenUsingCustomMockGeneratorAttribute_ShouldNotBeIncluded
Mockolate.SourceGenerators.Tests.MockGeneratorTests ‑ WhenUsingIncorrectMockGeneratorAttribute_ShouldNotBeIncluded
Mockolate.SourceGenerators.Tests.Sources.ForMockTests ‑ CustomDelegates_ShouldSupportSpanAndReadOnlySpanParameters
Mockolate.SourceGenerators.Tests.Sources.ForMockTests ‑ ShouldHandleComplexInheritanceWithSealedAndInternalMembers
Mockolate.SourceGenerators.Tests.Sources.ForMockTests ‑ ShouldNotIncludeSealedOverrideSpecialMethods
Mockolate.SourceGenerators.Tests.Sources.ForMockTests ‑ ShouldNotIncludeSealedOverrideSpecialMethodsWithNonNullableParameters
Mockolate.SourceGenerators.Tests.Sources.ForMockTests+ImplementClassTests ‑ Methods_Generic_ShouldApplyAllConstraints(constraint: "class, allows ref struct")
Mockolate.SourceGenerators.Tests.Sources.ForMockTests+ImplementClassTests ‑ Methods_Generic_ShouldApplyAllConstraints(constraint: "class, notnull")
…

♻️ This comment has been updated with latest results.

@github-actions
Copy link

github-actions bot commented Dec 18, 2025

🚀 Benchmark Results

Details

BenchmarkDotNet v0.15.8, Linux Ubuntu 24.04.3 LTS (Noble Numbat)
AMD EPYC 7763 2.45GHz, 1 CPU, 4 logical and 2 physical cores
.NET SDK 10.0.101
[Host] : .NET 10.0.1 (10.0.1, 10.0.125.57005), X64 RyuJIT x86-64-v3

Job=InProcess Toolchain=InProcessEmitToolchain IterationCount=15
LaunchCount=1 WarmupCount=10

Method Mean Error StdDev Gen0 Gen1 Allocated
Simple_Mockolate 1.525 μs 0.0096 μs 0.0090 μs 0.2270 - 3.72 KB
Simple_Moq 182.873 μs 0.9094 μs 0.8062 μs 0.4883 - 14.77 KB
Simple_NSubstitute 5.704 μs 0.0727 μs 0.0680 μs 0.5493 0.0076 9.09 KB
Simple_FakeItEasy 6.613 μs 0.0252 μs 0.0223 μs 0.4959 - 8.11 KB

@sonarqubecloud
Copy link

sonarqubecloud bot commented Jan 5, 2026

@vbreuss vbreuss merged commit 15c4614 into main Jan 5, 2026
12 checks passed
@vbreuss vbreuss deleted the topic/re-enable-stryker branch January 5, 2026 06:54
@github-actions
Copy link

github-actions bot commented Jan 6, 2026

This is addressed in release v0.49.0.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

state: released The issue is released

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants