Skip to content

feat: add AllInteractionsAreVerified#32

Merged
vbreuss merged 1 commit into
mainfrom
topic/add-allinteractionsareverified
Nov 24, 2025
Merged

feat: add AllInteractionsAreVerified#32
vbreuss merged 1 commit into
mainfrom
topic/add-allinteractionsareverified

Conversation

@vbreuss
Copy link
Copy Markdown
Member

@vbreuss vbreuss commented Nov 24, 2025

This PR adds a new AllInteractionsAreVerified assertion method for the Mockolate mocking library integration. This method verifies that all mock interactions have been properly verified in tests.

Key changes:

  • Introduced ThatMockVerify extension class with AllInteractionsAreVerified method
  • Refactored MyDescribableSubject into a shared helper class
  • Updated Mockolate dependency to version 0.36.0

Copilot AI review requested due to automatic review settings November 24, 2025 18:14
Copy link
Copy Markdown

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 adds a new AllInteractionsAreVerified assertion method for the Mockolate mocking library integration. This method verifies that all mock interactions have been properly verified in tests.

Key changes:

  • Introduced ThatMockVerify extension class with AllInteractionsAreVerified method
  • Refactored MyDescribableSubject into a shared helper class
  • Updated Mockolate dependency to version 0.36.0

Reviewed changes

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

Show a summary per file
File Description
ThatMockVerify.cs New partial class containing mock verification expectations
ThatMockVerify.AllInteractionsAreVerified.cs Implementation of the AllInteractionsAreVerified constraint
Helpers/MyDescribableSubject.cs Extracted helper class for formatting mock descriptions
ThatVerificationResult.cs Removed duplicated MyDescribableSubject class
ThatMockVerifyIs.cs Test interface definition for mock verification tests
ThatMockVerifyIs.AllInteractionsAreVerifiedTests.cs Comprehensive test coverage for the new verification method
aweXpect.Mockolate_*.txt Updated API surface for all target frameworks
Directory.Packages.props Updated Mockolate dependency version

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

Comment thread Source/aweXpect.Mockolate/Helpers/MyDescribableSubject.cs
@vbreuss vbreuss enabled auto-merge (squash) November 24, 2025 18:15
@sonarqubecloud
Copy link
Copy Markdown

@github-actions
Copy link
Copy Markdown

Test Results

  8 files  ± 0    8 suites  ±0   3s ⏱️ -1s
 77 tests + 4   76 ✅ + 4  1 💤 ±0  0 ❌ ±0 
515 runs  +28  514 ✅ +28  1 💤 ±0  0 ❌ ±0 

Results for commit 1c0d9a0. ± Comparison against base commit 7fb44af.

@vbreuss vbreuss merged commit 18ec4b8 into main Nov 24, 2025
12 checks passed
@vbreuss vbreuss deleted the topic/add-allinteractionsareverified branch November 24, 2025 18:18
@github-actions
Copy link
Copy Markdown

🚀 Benchmark Results

Details

BenchmarkDotNet v0.15.6, 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.100
[Host] : .NET 8.0.22 (8.0.22, 8.0.2225.52707), X64 RyuJIT x86-64-v3
DefaultJob : .NET 8.0.22 (8.0.22, 8.0.2225.52707), X64 RyuJIT x86-64-v3

Method Mean Error Allocated
Dummy_aweXpect 0.0 ns 0.0 ns -

@github-actions
Copy link
Copy Markdown

This is addressed in release v0.6.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.

Add verification that ThatAllInteractionsAreVerified

2 participants