Skip to content

coverage: improve mutation test coverage (3)#350

Merged
vbreuss merged 1 commit intomainfrom
topic/improve-stryker-covearge
Jan 6, 2026
Merged

coverage: improve mutation test coverage (3)#350
vbreuss merged 1 commit intomainfrom
topic/improve-stryker-covearge

Conversation

@vbreuss
Copy link
Contributor

@vbreuss vbreuss commented Jan 6, 2026

This PR improves mutation test coverage by adding new test cases that verify sorting behavior of unverified interactions and the reset functionality of the interaction index counter.

Key changes

  • Added three new test methods to verify interaction tracking edge cases
  • Removed an unused using directive for Mockolate.Exceptions

@vbreuss vbreuss self-assigned this Jan 6, 2026
@vbreuss vbreuss added the coverage Additional tests for edge cases or improvement of test coverage label Jan 6, 2026
Copilot AI review requested due to automatic review settings January 6, 2026 16:42
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 improves mutation test coverage by adding new test cases that verify sorting behavior of unverified interactions and the reset functionality of the interaction index counter.

Key changes:

  • Added three new test methods to verify interaction tracking edge cases
  • Removed an unused using directive for Mockolate.Exceptions

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.

File Description
Tests/Mockolate.Tests/Verify/MockVerifyTests.cs Added tests verifying that unverified interactions maintain proper sorted order and that verification state updates correctly across multiple calls
Tests/Mockolate.Tests/MockSetupsTests.cs Added test verifying that clearing all interactions properly resets the interaction index counter to zero
Source/Mockolate/Interactions/MockInteractions.cs Removed unused using Mockolate.Exceptions directive

@vbreuss
Copy link
Contributor Author

vbreuss commented Jan 6, 2026

See also #347 and #349

@vbreuss vbreuss enabled auto-merge (squash) January 6, 2026 16:43
@vbreuss vbreuss disabled auto-merge January 6, 2026 16:45
@vbreuss vbreuss merged commit 4010f9f into main Jan 6, 2026
8 checks passed
@vbreuss vbreuss deleted the topic/improve-stryker-covearge branch January 6, 2026 16:45
@sonarqubecloud
Copy link

sonarqubecloud bot commented Jan 6, 2026

@github-actions
Copy link

github-actions bot commented Jan 6, 2026

🚀 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.619 μs 0.0099 μs 0.0093 μs 0.2270 - 3.72 KB
Simple_Moq 179.728 μs 1.1418 μs 0.9534 μs 0.4883 - 14.55 KB
Simple_NSubstitute 6.019 μs 0.0242 μs 0.0226 μs 0.5569 0.0076 9.14 KB
Simple_FakeItEasy 6.838 μs 0.0339 μs 0.0317 μs 0.4959 - 8.11 KB

@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

coverage Additional tests for edge cases or improvement of test coverage state: released The issue is released

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants