[Fusion] Replaced filtering with assertion in MergeArgumentDefinitions (#8195)

glen-84 · web-flow · commit 363907a25073 · 2025-03-28T15:55:22.000+02:00
diff --git a/src/HotChocolate/Fusion-vnext/src/Fusion.Composition/SourceSchemaMerger.cs b/src/HotChocolate/Fusion-vnext/src/Fusion.Composition/SourceSchemaMerger.cs
@@ -151,8 +151,7 @@ private static MutableInputFieldDefinition MergeArguments(
         ImmutableArray<FieldArgumentInfo> argumentGroup,
         MutableSchemaDefinition mergedSchema)
     {
-        // Remove all arguments marked with @require.
-        argumentGroup = [.. argumentGroup.Where(i => !i.Argument.HasRequireDirective())];
+        Assert(!argumentGroup.Any(i => i.Argument.HasRequireDirective()));
 
         var mergedArgument = argumentGroup.Select(i => i.Argument).FirstOrDefault();
 

Original file line number	Diff line number	Diff line change
`@@ -151,8 +151,7 @@ private static MutableInputFieldDefinition MergeArguments(`
`151`	`151`	`ImmutableArray<FieldArgumentInfo> argumentGroup,`
`152`	`152`	`MutableSchemaDefinition mergedSchema)`
`153`	`153`	`{`
`154`		`- // Remove all arguments marked with @require.`
`155`		`- argumentGroup = [.. argumentGroup.Where(i => !i.Argument.HasRequireDirective())];`
	`154`	`+ Assert(!argumentGroup.Any(i => i.Argument.HasRequireDirective()));`
`156`	`155`
`157`	`156`	`var mergedArgument = argumentGroup.Select(i => i.Argument).FirstOrDefault();`
`158`	`157`