Fix Assert.Throws to validate parameter names (#7007)

stephentoub · web-flow · commit 57004d9c37d4 · 2025-11-04T02:47:21.000Z
diff --git a/src/Libraries/Microsoft.Extensions.AI.Abstractions/Contents/HostedFileContent.cs b/src/Libraries/Microsoft.Extensions.AI.Abstractions/Contents/HostedFileContent.cs
@@ -25,7 +25,7 @@ public sealed class HostedFileContent : AIContent
     /// <exception cref="ArgumentException"><paramref name="fileId"/> is empty or composed entirely of whitespace.</exception>
     public HostedFileContent(string fileId)
     {
-        FileId = fileId;
+        FileId = Throw.IfNullOrWhitespace(fileId);
     }
 
     /// <summary>
diff --git a/test/Libraries/Microsoft.Extensions.AI.Abstractions.Tests/Contents/HostedFileContentTests.cs b/test/Libraries/Microsoft.Extensions.AI.Abstractions.Tests/Contents/HostedFileContentTests.cs
@@ -12,9 +12,9 @@ public class HostedFileContentTests
     [Fact]
     public void Constructor_InvalidInput_Throws()
     {
-        Assert.Throws<ArgumentNullException>(() => new HostedFileContent(null!));
-        Assert.Throws<ArgumentException>(() => new HostedFileContent(string.Empty));
-        Assert.Throws<ArgumentException>(() => new HostedFileContent(" "));
+        Assert.Throws<ArgumentNullException>("fileId", () => new HostedFileContent(null!));
+        Assert.Throws<ArgumentException>("fileId", () => new HostedFileContent(string.Empty));
+        Assert.Throws<ArgumentException>("fileId", () => new HostedFileContent(" "));
     }
 
     [Fact]
@@ -36,9 +36,9 @@ public void Constructor_PropsRoundtrip()
         c.FileId = "id456";
         Assert.Equal("id456", c.FileId);
 
-        Assert.Throws<ArgumentNullException>(() => c.FileId = null!);
-        Assert.Throws<ArgumentException>(() => c.FileId = string.Empty);
-        Assert.Throws<ArgumentException>(() => c.FileId = " ");
+        Assert.Throws<ArgumentNullException>("value", () => c.FileId = null!);
+        Assert.Throws<ArgumentException>("value", () => c.FileId = string.Empty);
+        Assert.Throws<ArgumentException>("value", () => c.FileId = " ");
 
         Assert.Null(c.RawRepresentation);
         object raw = new();
@@ -89,7 +89,7 @@ public void MediaType_Roundtrips()
     public void MediaType_InvalidValue_Throws(string invalidMediaType)
     {
         HostedFileContent c = new("id123");
-        Assert.Throws<ArgumentException>(() => c.MediaType = invalidMediaType);
+        Assert.Throws<ArgumentException>("value", () => c.MediaType = invalidMediaType);
     }
 
     [Theory]
diff --git a/test/Libraries/Microsoft.Extensions.AI.Abstractions.Tests/Contents/HostedVectorStoreContentTests.cs b/test/Libraries/Microsoft.Extensions.AI.Abstractions.Tests/Contents/HostedVectorStoreContentTests.cs
@@ -12,9 +12,9 @@ public class HostedVectorStoreContentTests
     [Fact]
     public void Constructor_InvalidInput_Throws()
     {
-        Assert.Throws<ArgumentNullException>(() => new HostedVectorStoreContent(null!));
-        Assert.Throws<ArgumentException>(() => new HostedVectorStoreContent(string.Empty));
-        Assert.Throws<ArgumentException>(() => new HostedVectorStoreContent(" "));
+        Assert.Throws<ArgumentNullException>("vectorStoreId", () => new HostedVectorStoreContent(null!));
+        Assert.Throws<ArgumentException>("vectorStoreId", () => new HostedVectorStoreContent(string.Empty));
+        Assert.Throws<ArgumentException>("vectorStoreId", () => new HostedVectorStoreContent(" "));
     }
 
     [Fact]
@@ -35,9 +35,9 @@ public void Constructor_PropsRoundtrip()
         c.VectorStoreId = "id456";
         Assert.Equal("id456", c.VectorStoreId);
 
-        Assert.Throws<ArgumentNullException>(() => c.VectorStoreId = null!);
-        Assert.Throws<ArgumentException>(() => c.VectorStoreId = string.Empty);
-        Assert.Throws<ArgumentException>(() => c.VectorStoreId = " ");
+        Assert.Throws<ArgumentNullException>("value", () => c.VectorStoreId = null!);
+        Assert.Throws<ArgumentException>("value", () => c.VectorStoreId = string.Empty);
+        Assert.Throws<ArgumentException>("value", () => c.VectorStoreId = " ");
         Assert.Equal("id456", c.VectorStoreId);
 
         Assert.Null(c.RawRepresentation);
diff --git a/test/Libraries/Microsoft.Extensions.AI.Abstractions.Tests/Utilities/AIJsonSchemaTransformCacheTests.cs b/test/Libraries/Microsoft.Extensions.AI.Abstractions.Tests/Utilities/AIJsonSchemaTransformCacheTests.cs
@@ -12,13 +12,13 @@ public static class AIJsonSchemaTransformCacheTests
     [Fact]
     public static void NullOptions_ThrowsArgumentNullException()
     {
-        Assert.Throws<ArgumentNullException>(() => new AIJsonSchemaTransformCache(transformOptions: null!));
+        Assert.Throws<ArgumentNullException>("transformOptions", () => new AIJsonSchemaTransformCache(transformOptions: null!));
     }
 
     [Fact]
     public static void EmptyOptions_ThrowsArgumentException()
     {
-        Assert.Throws<ArgumentException>(() => new AIJsonSchemaTransformCache(transformOptions: new()));
+        Assert.Throws<ArgumentException>("transformOptions", () => new AIJsonSchemaTransformCache(transformOptions: new()));
     }
 
     [Fact]
@@ -33,14 +33,14 @@ public static void TransformOptions_ReturnsExpectedValue()
     public static void NullFunction_ThrowsArgumentNullException()
     {
         AIJsonSchemaTransformCache cache = new(new() { ConvertBooleanSchemas = true });
-        Assert.Throws<ArgumentNullException>(() => cache.GetOrCreateTransformedSchema(function: null!));
+        Assert.Throws<ArgumentNullException>("function", () => cache.GetOrCreateTransformedSchema(function: null!));
     }
 
     [Fact]
     public static void NullResponseFormat_ThrowsArgumentNullException()
     {
         AIJsonSchemaTransformCache cache = new(new() { ConvertBooleanSchemas = true });
-        Assert.Throws<ArgumentNullException>(() => cache.GetOrCreateTransformedSchema(responseFormat: null!));
+        Assert.Throws<ArgumentNullException>("responseFormat", () => cache.GetOrCreateTransformedSchema(responseFormat: null!));
     }
 
     [Fact]
diff --git a/test/Libraries/Microsoft.Extensions.AI.Abstractions.Tests/Utilities/AIJsonUtilitiesTests.cs b/test/Libraries/Microsoft.Extensions.AI.Abstractions.Tests/Utilities/AIJsonUtilitiesTests.cs
@@ -1074,8 +1074,8 @@ public static void AddAIContentType_NonAIContent_ThrowsArgumentException()
     public static void AddAIContentType_BuiltInAIContent_ThrowsArgumentException()
     {
         JsonSerializerOptions options = new();
-        Assert.Throws<ArgumentException>(() => options.AddAIContentType<AIContent>("discriminator"));
-        Assert.Throws<ArgumentException>(() => options.AddAIContentType<TextContent>("discriminator"));
+        Assert.Throws<ArgumentException>("contentType", () => options.AddAIContentType<AIContent>("discriminator"));
+        Assert.Throws<ArgumentException>("contentType", () => options.AddAIContentType<TextContent>("discriminator"));
     }
 
     [Fact]
@@ -1398,8 +1398,8 @@ public static void TransformJsonSchema_ValidateWithTestData(ITestData testData)
     public static void TransformJsonSchema_InvalidOptions_ThrowsArgumentException()
     {
         JsonElement schema = JsonDocument.Parse("{}").RootElement;
-        Assert.Throws<ArgumentNullException>(() => AIJsonUtilities.TransformSchema(schema, transformOptions: null!));
-        Assert.Throws<ArgumentException>(() => AIJsonUtilities.TransformSchema(schema, transformOptions: new()));
+        Assert.Throws<ArgumentNullException>("transformOptions", () => AIJsonUtilities.TransformSchema(schema, transformOptions: null!));
+        Assert.Throws<ArgumentException>("transformOptions", () => AIJsonUtilities.TransformSchema(schema, transformOptions: new()));
     }
 
     [Theory]
@@ -1412,7 +1412,7 @@ public static void TransformJsonSchema_InvalidInput_ThrowsArgumentException(stri
     {
         JsonElement schema = JsonDocument.Parse(invalidSchema).RootElement;
         AIJsonSchemaTransformOptions transformOptions = new() { ConvertBooleanSchemas = true };
-        Assert.Throws<ArgumentException>(() => AIJsonUtilities.TransformSchema(schema, transformOptions));
+        Assert.Throws<ArgumentException>("schema", () => AIJsonUtilities.TransformSchema(schema, transformOptions));
     }
 
     private class DerivedAIContent : AIContent
diff --git a/test/Libraries/Microsoft.Extensions.AI.Evaluation.NLP.Tests/NGramTests.cs b/test/Libraries/Microsoft.Extensions.AI.Evaluation.NLP.Tests/NGramTests.cs
@@ -21,7 +21,7 @@ public void Constructor_ValuesAndLength()
     [Fact]
     public void Constructor_ThrowsOnEmpty()
     {
-        Assert.Throws<ArgumentOutOfRangeException>(() => new NGram<int>(Array.Empty<int>()));
+        Assert.Throws<ArgumentOutOfRangeException>("values", () => new NGram<int>(Array.Empty<int>()));
     }
 
     [Fact]
@@ -61,7 +61,7 @@ public void NGramBuilder_Create_Works()
     [Fact]
     public void CreateNGrams()
     {
-        Assert.Throws<ArgumentOutOfRangeException>(() => new int[0].CreateNGrams(-1).ToList());
+        Assert.Throws<ArgumentOutOfRangeException>("n", () => new int[0].CreateNGrams(-1).ToList());
 
         ReadOnlySpan<int> data = [1, 2, 3];
 
@@ -81,11 +81,11 @@ public void CreateNGrams()
     [Fact]
     public void CreateAllNGrams()
     {
-        Assert.Throws<ArgumentOutOfRangeException>(() => new int[0].CreateAllNGrams(-1).ToList());
+        Assert.Throws<ArgumentOutOfRangeException>("minN", () => new int[0].CreateAllNGrams(-1).ToList());
 
-        Assert.Throws<ArgumentOutOfRangeException>(() => new int[0].CreateAllNGrams(0).ToList());
+        Assert.Throws<ArgumentOutOfRangeException>("minN", () => new int[0].CreateAllNGrams(0).ToList());
 
-        Assert.Throws<ArgumentOutOfRangeException>(() => new int[0].CreateAllNGrams(1, 0).ToList());
+        Assert.Throws<ArgumentOutOfRangeException>("maxN", () => new int[0].CreateAllNGrams(1, 0).ToList());
 
         ReadOnlySpan<int> arr = [1, 2, 3];
 
diff --git a/test/Libraries/Microsoft.Extensions.AI.Integration.Tests/ToolReductionTests.cs b/test/Libraries/Microsoft.Extensions.AI.Integration.Tests/ToolReductionTests.cs
@@ -16,7 +16,7 @@ public class ToolReductionTests
     public void EmbeddingToolReductionStrategy_Constructor_ThrowsWhenToolLimitIsLessThanOrEqualToZero()
     {
         using var gen = new DeterministicTestEmbeddingGenerator();
-        Assert.Throws<ArgumentOutOfRangeException>(() => new EmbeddingToolReductionStrategy(gen, toolLimit: 0));
+        Assert.Throws<ArgumentOutOfRangeException>("toolLimit", () => new EmbeddingToolReductionStrategy(gen, toolLimit: 0));
     }
 
     [Fact]
diff --git a/test/Libraries/Microsoft.Extensions.AI.Tests/ChatReduction/MessageCountingChatReducerTests.cs b/test/Libraries/Microsoft.Extensions.AI.Tests/ChatReduction/MessageCountingChatReducerTests.cs
@@ -18,7 +18,7 @@ public class MessageCountingChatReducerTests
     [InlineData(-10)]
     public void Constructor_ThrowsOnInvalidTargetCount(int targetCount)
     {
-        Assert.Throws<ArgumentOutOfRangeException>(() => new MessageCountingChatReducer(targetCount));
+        Assert.Throws<ArgumentOutOfRangeException>(nameof(targetCount), () => new MessageCountingChatReducer(targetCount));
     }
 
     [Fact]
@@ -126,8 +126,8 @@ public async Task ReduceAsync_IgnoresFunctionCallsAndResults()
         [
             new ChatMessage(ChatRole.User, "What's the weather?"),
             new ChatMessage(ChatRole.Assistant, [new FunctionCallContent("call1", "get_weather", new Dictionary<string, object?> { ["location"] = "Seattle" })]),
-            new ChatMessage(ChatRole.Tool, [new FunctionResultContent("call1", "Sunny, 72�F")]),
-            new ChatMessage(ChatRole.Assistant, "The weather in Seattle is sunny and 72�F."),
+            new ChatMessage(ChatRole.Tool, [new FunctionResultContent("call1", "Sunny, 72°F")]),
+            new ChatMessage(ChatRole.Assistant, "The weather in Seattle is sunny and 72°F."),
             new ChatMessage(ChatRole.User, "Thanks!"),
             new ChatMessage(ChatRole.Assistant, "You're welcome!"),
         ];
diff --git a/test/Libraries/Microsoft.Extensions.AI.Tests/ChatReduction/SummarizingChatReducerTests.cs b/test/Libraries/Microsoft.Extensions.AI.Tests/ChatReduction/SummarizingChatReducerTests.cs
@@ -17,7 +17,7 @@ public class SummarizingChatReducerTests
     [Fact]
     public void Constructor_ThrowsOnNullChatClient()
     {
-        Assert.Throws<ArgumentNullException>(() => new SummarizingChatReducer(null!, targetCount: 5, threshold: 2));
+        Assert.Throws<ArgumentNullException>("chatClient", () => new SummarizingChatReducer(null!, targetCount: 5, threshold: 2));
     }
 
     [Theory]
@@ -27,7 +27,7 @@ public void Constructor_ThrowsOnNullChatClient()
     public void Constructor_ThrowsOnInvalidTargetCount(int targetCount)
     {
         using var chatClient = new TestChatClient();
-        Assert.Throws<ArgumentOutOfRangeException>(() => new SummarizingChatReducer(chatClient, targetCount, threshold: 2));
+        Assert.Throws<ArgumentOutOfRangeException>(nameof(targetCount), () => new SummarizingChatReducer(chatClient, targetCount, threshold: 2));
     }
 
     [Theory]
@@ -36,7 +36,7 @@ public void Constructor_ThrowsOnInvalidTargetCount(int targetCount)
     public void Constructor_ThrowsOnInvalidThresholdCount(int thresholdCount)
     {
         using var chatClient = new TestChatClient();
-        Assert.Throws<ArgumentOutOfRangeException>(() => new SummarizingChatReducer(chatClient, targetCount: 5, thresholdCount));
+        Assert.Throws<ArgumentOutOfRangeException>("threshold", () => new SummarizingChatReducer(chatClient, targetCount: 5, thresholdCount));
     }
 
     [Fact]

Original file line number	Diff line number	Diff line change
`@@ -25,7 +25,7 @@ public sealed class HostedFileContent : AIContent`
`25`	`25`	`/// <exception cref="ArgumentException"><paramref name="fileId"/> is empty or composed entirely of whitespace.</exception>`
`26`	`26`	`public HostedFileContent(string fileId)`
`27`	`27`	`{`
`28`		`- FileId = fileId;`
	`28`	`+ FileId = Throw.IfNullOrWhitespace(fileId);`
`29`	`29`	`}`
`30`	`30`
`31`	`31`	`/// <summary>`
Original file line number	Diff line number	Diff line change
`@@ -12,13 +12,13 @@ public static class AIJsonSchemaTransformCacheTests`
`12`	`12`	`[Fact]`
`13`	`13`	`public static void NullOptions_ThrowsArgumentNullException()`
`14`	`14`	`{`
`15`		`- Assert.Throws<ArgumentNullException>(() => new AIJsonSchemaTransformCache(transformOptions: null!));`
	`15`	`+ Assert.Throws<ArgumentNullException>("transformOptions", () => new AIJsonSchemaTransformCache(transformOptions: null!));`
`16`	`16`	`}`
`17`	`17`
`18`	`18`	`[Fact]`
`19`	`19`	`public static void EmptyOptions_ThrowsArgumentException()`
`20`	`20`	`{`
`21`		`- Assert.Throws<ArgumentException>(() => new AIJsonSchemaTransformCache(transformOptions: new()));`
	`21`	`+ Assert.Throws<ArgumentException>("transformOptions", () => new AIJsonSchemaTransformCache(transformOptions: new()));`
`22`	`22`	`}`
`23`	`23`
`24`	`24`	`[Fact]`
`@@ -33,14 +33,14 @@ public static void TransformOptions_ReturnsExpectedValue()`
`33`	`33`	`public static void NullFunction_ThrowsArgumentNullException()`
`34`	`34`	`{`
`35`	`35`	`AIJsonSchemaTransformCache cache = new(new() { ConvertBooleanSchemas = true });`
`36`		`- Assert.Throws<ArgumentNullException>(() => cache.GetOrCreateTransformedSchema(function: null!));`
	`36`	`+ Assert.Throws<ArgumentNullException>("function", () => cache.GetOrCreateTransformedSchema(function: null!));`
`37`	`37`	`}`
`38`	`38`
`39`	`39`	`[Fact]`
`40`	`40`	`public static void NullResponseFormat_ThrowsArgumentNullException()`
`41`	`41`	`{`
`42`	`42`	`AIJsonSchemaTransformCache cache = new(new() { ConvertBooleanSchemas = true });`
`43`		`- Assert.Throws<ArgumentNullException>(() => cache.GetOrCreateTransformedSchema(responseFormat: null!));`
	`43`	`+ Assert.Throws<ArgumentNullException>("responseFormat", () => cache.GetOrCreateTransformedSchema(responseFormat: null!));`
`44`	`44`	`}`
`45`	`45`
`46`	`46`	`[Fact]`
Original file line number	Diff line number	Diff line change
`@@ -1074,8 +1074,8 @@ public static void AddAIContentType_NonAIContent_ThrowsArgumentException()`
`1074`	`1074`	`public static void AddAIContentType_BuiltInAIContent_ThrowsArgumentException()`
`1075`	`1075`	`{`
`1076`	`1076`	`JsonSerializerOptions options = new();`
`1077`		`- Assert.Throws<ArgumentException>(() => options.AddAIContentType<AIContent>("discriminator"));`
`1078`		`- Assert.Throws<ArgumentException>(() => options.AddAIContentType<TextContent>("discriminator"));`
	`1077`	`+ Assert.Throws<ArgumentException>("contentType", () => options.AddAIContentType<AIContent>("discriminator"));`
	`1078`	`+ Assert.Throws<ArgumentException>("contentType", () => options.AddAIContentType<TextContent>("discriminator"));`
`1079`	`1079`	`}`
`1080`	`1080`
`1081`	`1081`	`[Fact]`
`@@ -1398,8 +1398,8 @@ public static void TransformJsonSchema_ValidateWithTestData(ITestData testData)`
`1398`	`1398`	`public static void TransformJsonSchema_InvalidOptions_ThrowsArgumentException()`
`1399`	`1399`	`{`
`1400`	`1400`	`JsonElement schema = JsonDocument.Parse("{}").RootElement;`
`1401`		`- Assert.Throws<ArgumentNullException>(() => AIJsonUtilities.TransformSchema(schema, transformOptions: null!));`
`1402`		`- Assert.Throws<ArgumentException>(() => AIJsonUtilities.TransformSchema(schema, transformOptions: new()));`
	`1401`	`+ Assert.Throws<ArgumentNullException>("transformOptions", () => AIJsonUtilities.TransformSchema(schema, transformOptions: null!));`
	`1402`	`+ Assert.Throws<ArgumentException>("transformOptions", () => AIJsonUtilities.TransformSchema(schema, transformOptions: new()));`
`1403`	`1403`	`}`
`1404`	`1404`
`1405`	`1405`	`[Theory]`
`@@ -1412,7 +1412,7 @@ public static void TransformJsonSchema_InvalidInput_ThrowsArgumentException(stri`
`1412`	`1412`	`{`
`1413`	`1413`	`JsonElement schema = JsonDocument.Parse(invalidSchema).RootElement;`
`1414`	`1414`	`AIJsonSchemaTransformOptions transformOptions = new() { ConvertBooleanSchemas = true };`
`1415`		`- Assert.Throws<ArgumentException>(() => AIJsonUtilities.TransformSchema(schema, transformOptions));`
	`1415`	`+ Assert.Throws<ArgumentException>("schema", () => AIJsonUtilities.TransformSchema(schema, transformOptions));`
`1416`	`1416`	`}`
`1417`	`1417`
`1418`	`1418`	`private class DerivedAIContent : AIContent`
Original file line number	Diff line number	Diff line change
`@@ -21,7 +21,7 @@ public void Constructor_ValuesAndLength()`
`21`	`21`	`[Fact]`
`22`	`22`	`public void Constructor_ThrowsOnEmpty()`
`23`	`23`	`{`
`24`		`- Assert.Throws<ArgumentOutOfRangeException>(() => new NGram<int>(Array.Empty<int>()));`
	`24`	`+ Assert.Throws<ArgumentOutOfRangeException>("values", () => new NGram<int>(Array.Empty<int>()));`
`25`	`25`	`}`
`26`	`26`
`27`	`27`	`[Fact]`
`@@ -61,7 +61,7 @@ public void NGramBuilder_Create_Works()`
`61`	`61`	`[Fact]`
`62`	`62`	`public void CreateNGrams()`
`63`	`63`	`{`
`64`		`- Assert.Throws<ArgumentOutOfRangeException>(() => new int[0].CreateNGrams(-1).ToList());`
	`64`	`+ Assert.Throws<ArgumentOutOfRangeException>("n", () => new int[0].CreateNGrams(-1).ToList());`
`65`	`65`
`66`	`66`	`ReadOnlySpan<int> data = [1, 2, 3];`
`67`	`67`
`@@ -81,11 +81,11 @@ public void CreateNGrams()`
`81`	`81`	`[Fact]`
`82`	`82`	`public void CreateAllNGrams()`
`83`	`83`	`{`
`84`		`- Assert.Throws<ArgumentOutOfRangeException>(() => new int[0].CreateAllNGrams(-1).ToList());`
	`84`	`+ Assert.Throws<ArgumentOutOfRangeException>("minN", () => new int[0].CreateAllNGrams(-1).ToList());`
`85`	`85`
`86`		`- Assert.Throws<ArgumentOutOfRangeException>(() => new int[0].CreateAllNGrams(0).ToList());`
	`86`	`+ Assert.Throws<ArgumentOutOfRangeException>("minN", () => new int[0].CreateAllNGrams(0).ToList());`
`87`	`87`
`88`		`- Assert.Throws<ArgumentOutOfRangeException>(() => new int[0].CreateAllNGrams(1, 0).ToList());`
	`88`	`+ Assert.Throws<ArgumentOutOfRangeException>("maxN", () => new int[0].CreateAllNGrams(1, 0).ToList());`
`89`	`89`
`90`	`90`	`ReadOnlySpan<int> arr = [1, 2, 3];`
`91`	`91`
Original file line number	Diff line number	Diff line change
`@@ -16,7 +16,7 @@ public class ToolReductionTests`
`16`	`16`	`public void EmbeddingToolReductionStrategy_Constructor_ThrowsWhenToolLimitIsLessThanOrEqualToZero()`
`17`	`17`	`{`
`18`	`18`	`using var gen = new DeterministicTestEmbeddingGenerator();`
`19`		`- Assert.Throws<ArgumentOutOfRangeException>(() => new EmbeddingToolReductionStrategy(gen, toolLimit: 0));`
	`19`	`+ Assert.Throws<ArgumentOutOfRangeException>("toolLimit", () => new EmbeddingToolReductionStrategy(gen, toolLimit: 0));`
`20`	`20`	`}`
`21`	`21`
`22`	`22`	`[Fact]`
Original file line number	Diff line number	Diff line change
`@@ -18,7 +18,7 @@ public class MessageCountingChatReducerTests`
`18`	`18`	`[InlineData(-10)]`
`19`	`19`	`public void Constructor_ThrowsOnInvalidTargetCount(int targetCount)`
`20`	`20`	`{`
`21`		`- Assert.Throws<ArgumentOutOfRangeException>(() => new MessageCountingChatReducer(targetCount));`
	`21`	`+ Assert.Throws<ArgumentOutOfRangeException>(nameof(targetCount), () => new MessageCountingChatReducer(targetCount));`
`22`	`22`	`}`
`23`	`23`
`24`	`24`	`[Fact]`
`@@ -126,8 +126,8 @@ public async Task ReduceAsync_IgnoresFunctionCallsAndResults()`
`126`	`126`	`[`
`127`	`127`	`new ChatMessage(ChatRole.User, "What's the weather?"),`
`128`	`128`	`new ChatMessage(ChatRole.Assistant, [new FunctionCallContent("call1", "get_weather", new Dictionary<string, object?> { ["location"] = "Seattle" })]),`
`129`		`- new ChatMessage(ChatRole.Tool, [new FunctionResultContent("call1", "Sunny, 72°F")]),`
`130`		`- new ChatMessage(ChatRole.Assistant, "The weather in Seattle is sunny and 72°F."),`
	`129`	`+ new ChatMessage(ChatRole.Tool, [new FunctionResultContent("call1", "Sunny, 72°F")]),`
	`130`	`+ new ChatMessage(ChatRole.Assistant, "The weather in Seattle is sunny and 72°F."),`
`131`	`131`	`new ChatMessage(ChatRole.User, "Thanks!"),`
`132`	`132`	`new ChatMessage(ChatRole.Assistant, "You're welcome!"),`
`133`	`133`	`];`
Original file line number	Diff line number	Diff line change
`@@ -17,7 +17,7 @@ public class SummarizingChatReducerTests`
`17`	`17`	`[Fact]`
`18`	`18`	`public void Constructor_ThrowsOnNullChatClient()`
`19`	`19`	`{`
`20`		`- Assert.Throws<ArgumentNullException>(() => new SummarizingChatReducer(null!, targetCount: 5, threshold: 2));`
	`20`	`+ Assert.Throws<ArgumentNullException>("chatClient", () => new SummarizingChatReducer(null!, targetCount: 5, threshold: 2));`
`21`	`21`	`}`
`22`	`22`
`23`	`23`	`[Theory]`
`@@ -27,7 +27,7 @@ public void Constructor_ThrowsOnNullChatClient()`
`27`	`27`	`public void Constructor_ThrowsOnInvalidTargetCount(int targetCount)`
`28`	`28`	`{`
`29`	`29`	`using var chatClient = new TestChatClient();`
`30`		`- Assert.Throws<ArgumentOutOfRangeException>(() => new SummarizingChatReducer(chatClient, targetCount, threshold: 2));`
	`30`	`+ Assert.Throws<ArgumentOutOfRangeException>(nameof(targetCount), () => new SummarizingChatReducer(chatClient, targetCount, threshold: 2));`
`31`	`31`	`}`
`32`	`32`
`33`	`33`	`[Theory]`
`@@ -36,7 +36,7 @@ public void Constructor_ThrowsOnInvalidTargetCount(int targetCount)`
`36`	`36`	`public void Constructor_ThrowsOnInvalidThresholdCount(int thresholdCount)`
`37`	`37`	`{`
`38`	`38`	`using var chatClient = new TestChatClient();`
`39`		`- Assert.Throws<ArgumentOutOfRangeException>(() => new SummarizingChatReducer(chatClient, targetCount: 5, thresholdCount));`
	`39`	`+ Assert.Throws<ArgumentOutOfRangeException>("threshold", () => new SummarizingChatReducer(chatClient, targetCount: 5, thresholdCount));`
`40`	`40`	`}`
`41`	`41`
`42`	`42`	`[Fact]`