Skip to content

Conversation

@vbreuss
Copy link
Contributor

@vbreuss vbreuss commented Nov 21, 2025

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

@vbreuss vbreuss self-assigned this Nov 21, 2025
Copilot AI review requested due to automatic review settings November 21, 2025 08:19
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 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
Copy link

github-actions bot commented Nov 21, 2025

Test Results

0 files   -     14  0 suites   - 14   0s ⏱️ - 6m 17s
0 tests  -  1 791  0 ✅  -  1 790  0 💤  - 1  0 ❌ ±0 
0 runs   - 11 966  0 ✅  - 11 965  0 💤  - 1  0 ❌ ±0 

Results for commit ade82c5. ± Comparison against base commit 5069c06.

♻️ This comment has been updated with latest results.

@github-actions
Copy link

github-actions bot commented Nov 21, 2025

🚀 Benchmark Results

Details

BenchmarkDotNet v0.14.0, Ubuntu 24.04.3 LTS (Noble Numbat)
AMD EPYC 7763, 1 CPU, 4 logical and 2 physical cores
.NET SDK 10.0.101
[Host] : .NET 10.0.1 (10.0.125.57005), X64 RyuJIT AVX2

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

Method Mean Error StdDev Gen0 Gen1 Allocated
Simple_Mockolate 1.467 μs 0.0173 μs 0.0161 μs 0.2232 - 3.66 KB
Simple_Moq 178.353 μs 1.3406 μs 1.1884 μs 0.4883 - 14.55 KB
Simple_NSubstitute 5.733 μs 0.0135 μs 0.0120 μs 0.5569 0.0076 9.14 KB
Simple_FakeItEasy 6.339 μs 0.0261 μs 0.0232 μs 0.4959 - 8.11 KB

@vbreuss vbreuss force-pushed the topic/migrate-to-tunit branch from f73fab0 to 37f8f6c Compare November 21, 2025 08:31
Copilot AI review requested due to automatic review settings November 21, 2025 13:05
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

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 AI review requested due to automatic review settings December 13, 2025 14:03
@vbreuss vbreuss force-pushed the topic/migrate-to-tunit branch from fb8ec75 to 2ad57d9 Compare December 13, 2025 14:03
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

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

@vbreuss vbreuss force-pushed the topic/migrate-to-tunit branch from fa2f047 to fc25794 Compare December 28, 2025 14:35
Copilot AI review requested due to automatic review settings December 28, 2025 14:35
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

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

Copilot AI review requested due to automatic review settings December 28, 2025 15:07
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

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

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants