Skip to content

feat: add support for method body inlining in generated assertions for file-scoped classes #3645

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
thomhurst merged 9 commits into from
Nov 1, 2025
Merged

feat: add support for method body inlining in generated assertions for file-scoped classes #3645

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
9 commits
Select commit Hold shift + click to select a range
da9d83a
feat: add support for method body inlining in generated assertions fo…
thomhurst Nov 1, 2025
be1ed67
Refactor assertion methods for improved readability and performance
thomhurst Nov 1, 2025
08238b9
Refactor assertion extensions to use InlineMethodBody in GenerateAsse…
thomhurst Nov 1, 2025
e5e98ae
feat: add suppression attributes for TargetSite assertions and handle…
thomhurst Nov 1, 2025
78b3e49
feat: categorize suppression and diagnostic attributes for generated …
thomhurst Nov 1, 2025
eaeca9f
Refactor StringBuilder assertions to use Length property instead of c…
thomhurst Nov 1, 2025
253bfa7
Refactor code structure for improved readability and maintainability
thomhurst Nov 1, 2025
1e97fb0
Remove temporary debug code for file-scoped type handling in MethodAs…
thomhurst Nov 1, 2025
173744c
Refactor assertion extension class names for consistency
thomhurst Nov 1, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
1 change: 1 addition & 0 deletions .gitignore
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -426,3 +426,4 @@ nul

# Claude Code settings
.claude/settings.local.json
.claude/agents
12 changes: 6 additions & 6 deletions ...rator.Tests/ArrayAssertionGeneratorTests.GeneratesArrayAssertions.DotNet10_0.verified.txt
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -35,7 +35,7 @@ public sealed class _IsEmpty_Assertion<T> : Assertion<T[]>
return Task.FromResult(AssertionResult.Failed("Actual value is null"));
}

var result = value.IsEmpty<T>();
var result = value!.Length == 0;
return Task.FromResult(result
? AssertionResult.Passed
: AssertionResult.Failed($"found {value}"));
Expand Down Expand Up @@ -73,7 +73,7 @@ public sealed class _IsNotEmpty_Assertion<T> : Assertion<T[]>
return Task.FromResult(AssertionResult.Failed("Actual value is null"));
}

var result = value.IsNotEmpty<T>();
var result = value != null && value!.Length > 0;
return Task.FromResult(result
? AssertionResult.Passed
: AssertionResult.Failed($"found {value}"));
Expand Down Expand Up @@ -111,7 +111,7 @@ public sealed class _IsSingleElement_Assertion<T> : Assertion<T[]>
return Task.FromResult(AssertionResult.Failed("Actual value is null"));
}

var result = value.IsSingleElement<T>();
var result = value!.Length == 1;
return Task.FromResult(result
? AssertionResult.Passed
: AssertionResult.Failed($"found {value}"));
Expand Down Expand Up @@ -149,7 +149,7 @@ public sealed class _IsNotSingleElement_Assertion<T> : Assertion<T[]>
return Task.FromResult(AssertionResult.Failed("Actual value is null"));
}

var result = value.IsNotSingleElement<T>();
var result = value!.Length != 1;
return Task.FromResult(result
? AssertionResult.Passed
: AssertionResult.Failed($"found {value}"));
Expand Down Expand Up @@ -187,7 +187,7 @@ public sealed class IEnumerableT_IsSingleElement_Assertion<T> : Assertion<System
return Task.FromResult(AssertionResult.Failed("Actual value is null"));
}

var result = value.IsSingleElement<T>();
var result = value != null && value!.Skip(1).Take(1).Any() == false && value!.Any();
return Task.FromResult(result
? AssertionResult.Passed
: AssertionResult.Failed($"found {value}"));
Expand Down Expand Up @@ -225,7 +225,7 @@ public sealed class IEnumerableT_IsNotSingleElement_Assertion<T> : Assertion<Sys
return Task.FromResult(AssertionResult.Failed("Actual value is null"));
}

var result = value.IsNotSingleElement<T>();
var result = value == null || value!.Skip(1).Take(1).Any() || !value!.Any();
return Task.FromResult(result
? AssertionResult.Passed
: AssertionResult.Failed($"found {value}"));
Expand Down
12 changes: 6 additions & 6 deletions ...erator.Tests/ArrayAssertionGeneratorTests.GeneratesArrayAssertions.DotNet8_0.verified.txt
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -35,7 +35,7 @@ public sealed class _IsEmpty_Assertion<T> : Assertion<T[]>
return Task.FromResult(AssertionResult.Failed("Actual value is null"));
}

var result = value.IsEmpty<T>();
var result = value!.Length == 0;
return Task.FromResult(result
? AssertionResult.Passed
: AssertionResult.Failed($"found {value}"));
Expand Down Expand Up @@ -73,7 +73,7 @@ public sealed class _IsNotEmpty_Assertion<T> : Assertion<T[]>
return Task.FromResult(AssertionResult.Failed("Actual value is null"));
}

var result = value.IsNotEmpty<T>();
var result = value != null && value!.Length > 0;
return Task.FromResult(result
? AssertionResult.Passed
: AssertionResult.Failed($"found {value}"));
Expand Down Expand Up @@ -111,7 +111,7 @@ public sealed class _IsSingleElement_Assertion<T> : Assertion<T[]>
return Task.FromResult(AssertionResult.Failed("Actual value is null"));
}

var result = value.IsSingleElement<T>();
var result = value!.Length == 1;
return Task.FromResult(result
? AssertionResult.Passed
: AssertionResult.Failed($"found {value}"));
Expand Down Expand Up @@ -149,7 +149,7 @@ public sealed class _IsNotSingleElement_Assertion<T> : Assertion<T[]>
return Task.FromResult(AssertionResult.Failed("Actual value is null"));
}

var result = value.IsNotSingleElement<T>();
var result = value!.Length != 1;
return Task.FromResult(result
? AssertionResult.Passed
: AssertionResult.Failed($"found {value}"));
Expand Down Expand Up @@ -187,7 +187,7 @@ public sealed class IEnumerableT_IsSingleElement_Assertion<T> : Assertion<System
return Task.FromResult(AssertionResult.Failed("Actual value is null"));
}

var result = value.IsSingleElement<T>();
var result = value != null && value!.Skip(1).Take(1).Any() == false && value!.Any();
return Task.FromResult(result
? AssertionResult.Passed
: AssertionResult.Failed($"found {value}"));
Expand Down Expand Up @@ -225,7 +225,7 @@ public sealed class IEnumerableT_IsNotSingleElement_Assertion<T> : Assertion<Sys
return Task.FromResult(AssertionResult.Failed("Actual value is null"));
}

var result = value.IsNotSingleElement<T>();
var result = value == null || value!.Skip(1).Take(1).Any() || !value!.Any();
return Task.FromResult(result
? AssertionResult.Passed
: AssertionResult.Failed($"found {value}"));
Expand Down
12 changes: 6 additions & 6 deletions ...erator.Tests/ArrayAssertionGeneratorTests.GeneratesArrayAssertions.DotNet9_0.verified.txt
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -35,7 +35,7 @@ public sealed class _IsEmpty_Assertion<T> : Assertion<T[]>
return Task.FromResult(AssertionResult.Failed("Actual value is null"));
}

var result = value.IsEmpty<T>();
var result = value!.Length == 0;
return Task.FromResult(result
? AssertionResult.Passed
: AssertionResult.Failed($"found {value}"));
Expand Down Expand Up @@ -73,7 +73,7 @@ public sealed class _IsNotEmpty_Assertion<T> : Assertion<T[]>
return Task.FromResult(AssertionResult.Failed("Actual value is null"));
}

var result = value.IsNotEmpty<T>();
var result = value != null && value!.Length > 0;
return Task.FromResult(result
? AssertionResult.Passed
: AssertionResult.Failed($"found {value}"));
Expand Down Expand Up @@ -111,7 +111,7 @@ public sealed class _IsSingleElement_Assertion<T> : Assertion<T[]>
return Task.FromResult(AssertionResult.Failed("Actual value is null"));
}

var result = value.IsSingleElement<T>();
var result = value!.Length == 1;
return Task.FromResult(result
? AssertionResult.Passed
: AssertionResult.Failed($"found {value}"));
Expand Down Expand Up @@ -149,7 +149,7 @@ public sealed class _IsNotSingleElement_Assertion<T> : Assertion<T[]>
return Task.FromResult(AssertionResult.Failed("Actual value is null"));
}

var result = value.IsNotSingleElement<T>();
var result = value!.Length != 1;
return Task.FromResult(result
? AssertionResult.Passed
: AssertionResult.Failed($"found {value}"));
Expand Down Expand Up @@ -187,7 +187,7 @@ public sealed class IEnumerableT_IsSingleElement_Assertion<T> : Assertion<System
return Task.FromResult(AssertionResult.Failed("Actual value is null"));
}

var result = value.IsSingleElement<T>();
var result = value != null && value!.Skip(1).Take(1).Any() == false && value!.Any();
return Task.FromResult(result
? AssertionResult.Passed
: AssertionResult.Failed($"found {value}"));
Expand Down Expand Up @@ -225,7 +225,7 @@ public sealed class IEnumerableT_IsNotSingleElement_Assertion<T> : Assertion<Sys
return Task.FromResult(AssertionResult.Failed("Actual value is null"));
}

var result = value.IsNotSingleElement<T>();
var result = value == null || value!.Skip(1).Take(1).Any() || !value!.Any();
return Task.FromResult(result
? AssertionResult.Passed
: AssertionResult.Failed($"found {value}"));
Expand Down
12 changes: 6 additions & 6 deletions ...Generator.Tests/ArrayAssertionGeneratorTests.GeneratesArrayAssertions.Net4_7.verified.txt
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -35,7 +35,7 @@ public sealed class _IsEmpty_Assertion<T> : Assertion<T[]>
return Task.FromResult(AssertionResult.Failed("Actual value is null"));
}

var result = value.IsEmpty<T>();
var result = value!.Length == 0;
return Task.FromResult(result
? AssertionResult.Passed
: AssertionResult.Failed($"found {value}"));
Expand Down Expand Up @@ -73,7 +73,7 @@ public sealed class _IsNotEmpty_Assertion<T> : Assertion<T[]>
return Task.FromResult(AssertionResult.Failed("Actual value is null"));
}

var result = value.IsNotEmpty<T>();
var result = value != null && value!.Length > 0;
return Task.FromResult(result
? AssertionResult.Passed
: AssertionResult.Failed($"found {value}"));
Expand Down Expand Up @@ -111,7 +111,7 @@ public sealed class _IsSingleElement_Assertion<T> : Assertion<T[]>
return Task.FromResult(AssertionResult.Failed("Actual value is null"));
}

var result = value.IsSingleElement<T>();
var result = value!.Length == 1;
return Task.FromResult(result
? AssertionResult.Passed
: AssertionResult.Failed($"found {value}"));
Expand Down Expand Up @@ -149,7 +149,7 @@ public sealed class _IsNotSingleElement_Assertion<T> : Assertion<T[]>
return Task.FromResult(AssertionResult.Failed("Actual value is null"));
}

var result = value.IsNotSingleElement<T>();
var result = value!.Length != 1;
return Task.FromResult(result
? AssertionResult.Passed
: AssertionResult.Failed($"found {value}"));
Expand Down Expand Up @@ -187,7 +187,7 @@ public sealed class IEnumerableT_IsSingleElement_Assertion<T> : Assertion<System
return Task.FromResult(AssertionResult.Failed("Actual value is null"));
}

var result = value.IsSingleElement<T>();
var result = value != null && value!.Skip(1).Take(1).Any() == false && value!.Any();
return Task.FromResult(result
? AssertionResult.Passed
: AssertionResult.Failed($"found {value}"));
Expand Down Expand Up @@ -225,7 +225,7 @@ public sealed class IEnumerableT_IsNotSingleElement_Assertion<T> : Assertion<Sys
return Task.FromResult(AssertionResult.Failed("Actual value is null"));
}

var result = value.IsNotSingleElement<T>();
var result = value == null || value!.Skip(1).Take(1).Any() || !value!.Any();
return Task.FromResult(result
? AssertionResult.Passed
: AssertionResult.Failed($"found {value}"));
Expand Down
8 changes: 4 additions & 4 deletions ...r.Tests/BooleanAssertionGeneratorTests.GeneratesBooleanAssertions.DotNet10_0.verified.txt
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -29,7 +29,7 @@ public sealed class Bool_IsTrue_Assertion : Assertion<bool>
return Task.FromResult(AssertionResult.Failed($"threw {exception.GetType().FullName}"));
}

var result = value.IsTrue();
var result = value == true;
return Task.FromResult(result
? AssertionResult.Passed
: AssertionResult.Failed($"found {value}"));
Expand Down Expand Up @@ -61,7 +61,7 @@ public sealed class Bool_IsFalse_Assertion : Assertion<bool>
return Task.FromResult(AssertionResult.Failed($"threw {exception.GetType().FullName}"));
}

var result = value.IsFalse();
var result = value == false;
return Task.FromResult(result
? AssertionResult.Passed
: AssertionResult.Failed($"found {value}"));
Expand Down Expand Up @@ -98,7 +98,7 @@ public sealed class NullableBool_IsTrue_Assertion : Assertion<bool?>
return Task.FromResult(AssertionResult.Failed("Actual value is null"));
}

var result = value.IsTrue();
var result = value! == true;
return Task.FromResult(result
? AssertionResult.Passed
: AssertionResult.Failed($"found {value}"));
Expand Down Expand Up @@ -135,7 +135,7 @@ public sealed class NullableBool_IsFalse_Assertion : Assertion<bool?>
return Task.FromResult(AssertionResult.Failed("Actual value is null"));
}

var result = value.IsFalse();
var result = value! == false;
return Task.FromResult(result
? AssertionResult.Passed
: AssertionResult.Failed($"found {value}"));
Expand Down
8 changes: 4 additions & 4 deletions ...or.Tests/BooleanAssertionGeneratorTests.GeneratesBooleanAssertions.DotNet8_0.verified.txt
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -29,7 +29,7 @@ public sealed class Bool_IsTrue_Assertion : Assertion<bool>
return Task.FromResult(AssertionResult.Failed($"threw {exception.GetType().FullName}"));
}

var result = value.IsTrue();
var result = value == true;
return Task.FromResult(result
? AssertionResult.Passed
: AssertionResult.Failed($"found {value}"));
Expand Down Expand Up @@ -61,7 +61,7 @@ public sealed class Bool_IsFalse_Assertion : Assertion<bool>
return Task.FromResult(AssertionResult.Failed($"threw {exception.GetType().FullName}"));
}

var result = value.IsFalse();
var result = value == false;
return Task.FromResult(result
? AssertionResult.Passed
: AssertionResult.Failed($"found {value}"));
Expand Down Expand Up @@ -98,7 +98,7 @@ public sealed class NullableBool_IsTrue_Assertion : Assertion<bool?>
return Task.FromResult(AssertionResult.Failed("Actual value is null"));
}

var result = value.IsTrue();
var result = value! == true;
return Task.FromResult(result
? AssertionResult.Passed
: AssertionResult.Failed($"found {value}"));
Expand Down Expand Up @@ -135,7 +135,7 @@ public sealed class NullableBool_IsFalse_Assertion : Assertion<bool?>
return Task.FromResult(AssertionResult.Failed("Actual value is null"));
}

var result = value.IsFalse();
var result = value! == false;
return Task.FromResult(result
? AssertionResult.Passed
: AssertionResult.Failed($"found {value}"));
Expand Down
8 changes: 4 additions & 4 deletions ...or.Tests/BooleanAssertionGeneratorTests.GeneratesBooleanAssertions.DotNet9_0.verified.txt
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -29,7 +29,7 @@ public sealed class Bool_IsTrue_Assertion : Assertion<bool>
return Task.FromResult(AssertionResult.Failed($"threw {exception.GetType().FullName}"));
}

var result = value.IsTrue();
var result = value == true;
return Task.FromResult(result
? AssertionResult.Passed
: AssertionResult.Failed($"found {value}"));
Expand Down Expand Up @@ -61,7 +61,7 @@ public sealed class Bool_IsFalse_Assertion : Assertion<bool>
return Task.FromResult(AssertionResult.Failed($"threw {exception.GetType().FullName}"));
}

var result = value.IsFalse();
var result = value == false;
return Task.FromResult(result
? AssertionResult.Passed
: AssertionResult.Failed($"found {value}"));
Expand Down Expand Up @@ -98,7 +98,7 @@ public sealed class NullableBool_IsTrue_Assertion : Assertion<bool?>
return Task.FromResult(AssertionResult.Failed("Actual value is null"));
}

var result = value.IsTrue();
var result = value! == true;
return Task.FromResult(result
? AssertionResult.Passed
: AssertionResult.Failed($"found {value}"));
Expand Down Expand Up @@ -135,7 +135,7 @@ public sealed class NullableBool_IsFalse_Assertion : Assertion<bool?>
return Task.FromResult(AssertionResult.Failed("Actual value is null"));
}

var result = value.IsFalse();
var result = value! == false;
return Task.FromResult(result
? AssertionResult.Passed
: AssertionResult.Failed($"found {value}"));
Expand Down
8 changes: 4 additions & 4 deletions ...rator.Tests/BooleanAssertionGeneratorTests.GeneratesBooleanAssertions.Net4_7.verified.txt
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -29,7 +29,7 @@ public sealed class Bool_IsTrue_Assertion : Assertion<bool>
return Task.FromResult(AssertionResult.Failed($"threw {exception.GetType().FullName}"));
}

var result = value.IsTrue();
var result = value == true;
return Task.FromResult(result
? AssertionResult.Passed
: AssertionResult.Failed($"found {value}"));
Expand Down Expand Up @@ -61,7 +61,7 @@ public sealed class Bool_IsFalse_Assertion : Assertion<bool>
return Task.FromResult(AssertionResult.Failed($"threw {exception.GetType().FullName}"));
}

var result = value.IsFalse();
var result = value == false;
return Task.FromResult(result
? AssertionResult.Passed
: AssertionResult.Failed($"found {value}"));
Expand Down Expand Up @@ -98,7 +98,7 @@ public sealed class NullableBool_IsTrue_Assertion : Assertion<bool?>
return Task.FromResult(AssertionResult.Failed("Actual value is null"));
}

var result = value.IsTrue();
var result = value! == true;
return Task.FromResult(result
? AssertionResult.Passed
: AssertionResult.Failed($"found {value}"));
Expand Down Expand Up @@ -135,7 +135,7 @@ public sealed class NullableBool_IsFalse_Assertion : Assertion<bool?>
return Task.FromResult(AssertionResult.Failed("Actual value is null"));
}

var result = value.IsFalse();
var result = value! == false;
return Task.FromResult(result
? AssertionResult.Passed
: AssertionResult.Failed($"found {value}"));
Expand Down
Loading
Loading