refactor: migrate to TUnit by vbreuss · Pull Request #223 · Testably/Mockolate

vbreuss · 2025-11-21T08:19:46Z

This PR migrates the test framework from xUnit to TUnit by replacing test attributes and updating helper utilities.

Key Changes

Replaced [Fact] attributes with [Test] throughout test files
Replaced [Theory]/[InlineData] with [Test]/[Arguments] for parameterized tests
Changed global using from Xunit to TUnit in the test helpers
Added a new Record helper class to replace xUnit's Record.Exception functionality
Updated assertion syntax in delegate tests to use aweXpect instead of xUnit assertions
Uses pre-release Stryker version that supports MTP

Copilot

Pull Request Overview

This PR migrates the test framework from xUnit to TUnit by replacing test attributes and updating helper utilities.

Key Changes

Replaced [Fact] attributes with [Test] throughout test files
Replaced [Theory]/[InlineData] with [Test]/[Arguments] for parameterized tests
Changed global using from Xunit to TUnit in the test helpers
Added a new Record helper class to replace xUnit's Record.Exception functionality
Updated assertion syntax in delegate tests to use aweXpect instead of xUnit assertions

Reviewed Changes

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

Show a summary per file

File	Description
Tests/Mockolate.Tests/TestHelpers/Usings.cs	Updated global using from Xunit to TUnit
Tests/Mockolate.Tests/TestHelpers/Record.cs	Added Record helper class with Exception method for capturing exceptions
Tests/Mockolate.Tests/Verify/*.cs	Migrated test attributes from [Fact] to [Test] and [Theory]/[InlineData] to [Test]/[Arguments]
Tests/Mockolate.Tests/MockDelegates/DelegateTests.cs	Updated assertions from xUnit Assert to aweXpect syntax
Tests/Mockolate.Tests/Mock/.cs	Migrated all test attributes to TUnit equivalents across all mock test files

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

github-actions · 2025-11-21T08:22:11Z

Test Results

21 files ±0 21 suites ±0 21m 33s ⏱️ + 15m 54s
2 788 tests - 8 2 788 ✅ - 7 0 💤 - 1 0 ❌ ±0
18 788 runs - 1 18 788 ✅ ±0 0 💤 - 1 0 ❌ ±0

Results for commit aa90d42. ± Comparison against base commit bb36529.

♻️ This comment has been updated with latest results.

github-actions · 2025-11-21T08:22:53Z

🚀 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.103

[Host] : .NET 10.0.3 (10.0.3, 10.0.326.7603), 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.516 μs	0.0174 μs	0.0163 μs	0.2460	-	4.02 KB
Simple_Moq	175.503 μs	1.3583 μs	1.2041 μs	0.4883	-	14.55 KB
Simple_NSubstitute	5.640 μs	0.0165 μs	0.0154 μs	0.5493	0.0076	9.09 KB
Simple_FakeItEasy	6.470 μs	0.0138 μs	0.0129 μs	0.4959	-	8.11 KB
Simple_TUnitMocks	2.263 μs	0.0730 μs	0.0683 μs	0.4005	0.0076	6.55 KB

Copilot

Pull Request Overview

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

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copilot

Pull request overview

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

Copilot

Pull request overview

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

Copilot

Pull request overview

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

Copilot

Pull request overview

Copilot reviewed 148 out of 160 changed files in this pull request and generated 5 comments.

Copilot

Pull request overview

Copilot reviewed 150 out of 162 changed files in this pull request and generated 2 comments.

Comments suppressed due to low confidence (1)

Source/Mockolate/Mockolate.csproj:15

Mockolate is described as supporting .NET Framework 4.8; if the library uses init accessors / records, IsExternalInit is typically also required for net48. Restricting this package to only netstandard2.0 can break compilation for the .NET Framework target. Consider expanding the condition to include net48 (or conditioning on $(TargetFrameworkIdentifier) for .NET Framework/non-.NET targets that need the polyfill).

	<ItemGroup Condition="'$(TargetFramework)' == 'netstandard2.0'">
<PackageReference Include="IsExternalInit">
<PrivateAssets>all</PrivateAssets>
<IncludeAssets>runtime; build; native; contentfiles; analyzers; buildtransitive</IncludeAssets>
</PackageReference>

Copilot · 2026-03-01T05:57:59Z

@vbreuss I've opened a new pull request, #502, to work on those changes. Once the pull request is ready, I'll request review from you.

Copilot

Pull request overview

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

Copilot

Pull request overview

Copilot reviewed 152 out of 164 changed files in this pull request and generated 1 comment.

Copilot

Pull request overview

Copilot reviewed 153 out of 165 changed files in this pull request and generated 4 comments.

sonarqubecloud · 2026-03-01T07:29:52Z

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

I finally got the [CI-Build in Mockolate](Testably/Mockolate#223) to work with TUnit. The problem was, that the `--results-directory` is no custom CLI argument, but has to be specified before the `--` separator. *Even though the error message is very misleading :(*

This reverts commit 3e0dab6.

This PR reverts the migration from xUnit to TUnit (done in PR #223), because mutation testing does not yet work with Microsoft Testing Platform (MTP) mode that TUnit uses.

github-actions · 2026-03-02T08:30:38Z

This is addressed in release v1.5.1.

vbreuss self-assigned this Nov 21, 2025

CopilotAI review requested due to automatic review settings November 21, 2025 08:19

CopilotAI reviewed Nov 21, 2025
View reviewed changes

vbreuss force-pushed the topic/migrate-to-tunit branch from f73fab0 to 37f8f6cCompare November 21, 2025 08:31

CopilotAI review requested due to automatic review settings November 21, 2025 13:05

CopilotAI reviewed Nov 21, 2025
View reviewed changes

CopilotAI review requested due to automatic review settings December 13, 2025 14:03

vbreuss force-pushed the topic/migrate-to-tunit branch from fb8ec75 to 2ad57d9Compare December 13, 2025 14:03

CopilotAI reviewed Dec 13, 2025
View reviewed changes

vbreuss force-pushed the topic/migrate-to-tunit branch from fa2f047 to fc25794Compare December 28, 2025 14:35

CopilotAI review requested due to automatic review settings December 28, 2025 14:35

CopilotAI reviewed Dec 28, 2025
View reviewed changes

CopilotAI review requested due to automatic review settings December 28, 2025 15:07

CopilotAI reviewed Dec 28, 2025
View reviewed changes

refactor: migrate to TUnit
22a9fa3

vbreuss force-pushed the topic/migrate-to-tunit branch from ade82c5 to 22a9fa3Compare February 28, 2026 21:02

CopilotAI review requested due to automatic review settings February 28, 2026 21:11

CopilotAI reviewed Feb 28, 2026
View reviewed changes

vbreuss force-pushed the topic/migrate-to-tunit branch from 1267217 to 6657b44Compare February 28, 2026 21:14

CopilotAI review requested due to automatic review settings February 28, 2026 21:20

vbreuss force-pushed the topic/migrate-to-tunit branch from 6657b44 to 0f59ad9Compare February 28, 2026 21:20

CopilotAI reviewed Feb 28, 2026
View reviewed changes

Comment threadPipeline/Build.UnitTest.cs
Comment threadPipeline/Build.ApiChecks.cs Outdated

Adjust build pipeline
399022e

vbreuss force-pushed the topic/migrate-to-tunit branch from 0f59ad9 to 399022eCompare March 1, 2026 05:55

CopilotAI mentioned this pull request Mar 1, 2026
fix: use MTP-compatible --project flag in dotnet test commands #502
Closed

CopilotAI review requested due to automatic review settings March 1, 2026 06:29

CopilotAI reviewed Mar 1, 2026
View reviewed changes

Comment threadPipeline/Build.UnitTest.cs
Comment threadPipeline/Build.ApiChecks.cs
Comment threadTests/Directory.Build.props

vbreuss force-pushed the topic/migrate-to-tunit branch from 841d121 to c23949bCompare March 1, 2026 06:37

Use TUnit.Engine
ce65eab

CopilotAI review requested due to automatic review settings March 1, 2026 07:16

vbreuss force-pushed the topic/migrate-to-tunit branch from c23949b to ce65eabCompare March 1, 2026 07:16

CopilotAI reviewed Mar 1, 2026
View reviewed changes

Comment threadTests/Mockolate.Analyzers.Tests/MockabilityAnalyzerTests.cs

vbreuss added 2 commits March 1, 2026 08:20

Use Stryker 4.12.0
5722e4c

Increase timeout
c20fc22

CopilotAI review requested due to automatic review settings March 1, 2026 07:22

Use net10 for mutation tests
aa90d42

CopilotAI reviewed Mar 1, 2026
View reviewed changes

Comment threadPipeline/Build.MutationTests.cs
Comment threadTests/Mockolate.Tests/Verify/VerificationResultTests.AsyncTests.cs
Comment threadTests/Mockolate.Tests/Verify/VerificationResultTests.AsyncTests.cs
Comment threadTests/Mockolate.SourceGenerators.Tests/GeneralTests.cs

vbreuss merged commit 3e0dab6 into mainMar 1, 2026
12 checks passed

vbreuss deleted the topic/migrate-to-tunit branch March 1, 2026 07:40

vbreuss mentioned this pull request Mar 1, 2026
docs: fix position of --results-directory in documentation thomhurst/TUnit#5038
Merged
26 tasks

vbreuss added a commit that referenced this pull request Mar 2, 2026
Revert "refactor: migrate to TUnit (#223)"
61cf63b
This reverts commit 3e0dab6.

vbreuss mentioned this pull request Mar 2, 2026
revert: migrate to TUnit #505
Merged

vbreuss added a commit that referenced this pull request Mar 2, 2026
revert: migrate to TUnit (#505)
c3d698c
This PR reverts the migration from xUnit to TUnit (done in PR #223), because mutation testing does not yet work with Microsoft Testing Platform (MTP) mode that TUnit uses.

github-actionsBot added the state: released The issue is released label Mar 2, 2026

vbreuss mentioned this pull request Apr 30, 2026
refactor: migrate to TUnit #726
Draft

Uh oh!

Uh oh!

Conversation

vbreuss commented Nov 21, 2025• edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Key Changes

Uh oh!

CopilotAI left a comment

Choose a reason for hiding this comment

Pull Request Overview

Key Changes

Reviewed Changes

Uh oh!

github-actionsBot commented Nov 21, 2025• edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Test Results

Uh oh!

github-actionsBot commented Nov 21, 2025• edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

🚀 Benchmark Results

Uh oh!

CopilotAI left a comment

Choose a reason for hiding this comment

Pull Request Overview

Uh oh!

CopilotAI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

CopilotAI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

CopilotAI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

CopilotAI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

CopilotAI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Uh oh!

Uh oh!

CopilotAI commented Mar 1, 2026

Uh oh!

CopilotAI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Uh oh!

Uh oh!

Uh oh!

CopilotAI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Uh oh!

CopilotAI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

sonarqubecloudBot commented Mar 1, 2026

Quality Gate passed

Uh oh!

Uh oh!

github-actionsBot commented Mar 2, 2026

Uh oh!

vbreuss commented Nov 21, 2025•
edited
Loading

github-actionsBot commented Nov 21, 2025•
edited
Loading

github-actionsBot commented Nov 21, 2025•
edited
Loading