Make AST constructors parametrized (#307)

Author: sungam3r

.github/labeler.yml

@@ -1,6 +1,6 @@
 test:
-  - src/GraphQLParser.ApiTests/**/*
   - src/GraphQLParser.Tests/**/*
+  - src/GraphQLParser.ApiTests/**/*.cs
 
 CI:
   - .github/workflows/**/*

src/GraphQLParser.ApiTests/GraphQLParser.approved.txt

@@ -7,10 +7,23 @@ namespace GraphQLParser.AST;
 /// </summary>
 public abstract class ASTListNode<TNode> : ASTNode, IReadOnlyList<TNode>
 {
+    internal ASTListNode()
+    {
+        Items = null!;
+    }
+
+    /// <summary>
+    /// Creates a new instance of <see cref="ASTListNode{TNode}"/>.
+    /// </summary>
+    protected ASTListNode(List<TNode> items)
+    {
+        Items = items;
+    }
+
     /// <summary>
     /// A list of nested AST nodes.
     /// </summary>
-    public List<TNode> Items { get; set; } = null!;
+    public List<TNode> Items { get; set; }
 
     /// <summary>
     /// Get the number of AST nodes in the list.

src/GraphQLParser/AST/ASTListNode.cs

@@ -5,6 +5,18 @@ namespace GraphQLParser.AST;
 /// </summary>
 public class GraphQLArgumentsDefinition : ASTListNode<GraphQLInputValueDefinition>
 {
+    internal GraphQLArgumentsDefinition()
+    {
+    }
+
+    /// <summary>
+    /// Creates a new instance of <see cref="GraphQLArgumentsDefinition"/>.
+    /// </summary>
+    public GraphQLArgumentsDefinition(List<GraphQLInputValueDefinition> items)
+        : base(items)
+    {
+    }
+
     /// <inheritdoc/>
     public override ASTNodeKind Kind => ASTNodeKind.ArgumentsDefinition;
 }

src/GraphQLParser/AST/Definitions/GraphQLArgumentsDefinition.cs

@@ -8,6 +8,20 @@ namespace GraphQLParser.AST;
 [DebuggerDisplay("GraphQLDirectiveDefinition: {Name}")]
 public class GraphQLDirectiveDefinition : GraphQLTypeDefinition, IHasArgumentsDefinitionNode
 {
+    internal GraphQLDirectiveDefinition()
+    {
+        Locations = null!;
+    }
+
+    /// <summary>
+    /// Creates a new instance of <see cref="GraphQLDirectiveDefinition"/>.
+    /// </summary>
+    public GraphQLDirectiveDefinition(GraphQLName name, GraphQLDirectiveLocations locations)
+        : base(name)
+    {
+        Locations = locations;
+    }
+
     /// <inheritdoc/>
     public override ASTNodeKind Kind => ASTNodeKind.DirectiveDefinition;
 
@@ -29,7 +43,7 @@ public class GraphQLDirectiveDefinition : GraphQLTypeDefinition, IHasArgumentsDe
     /// <summary>
     /// Returns a node with a list of locations representing the valid locations this directive may be placed.
     /// </summary>
-    public GraphQLDirectiveLocations Locations { get; set; } = null!;
+    public GraphQLDirectiveLocations Locations { get; set; }
 }
 
 internal sealed class GraphQLDirectiveDefinitionWithLocation : GraphQLDirectiveDefinition

src/GraphQLParser/AST/Definitions/GraphQLDirectiveDefinition.cs

@@ -8,6 +8,18 @@ namespace GraphQLParser.AST;
 [DebuggerDisplay("GraphQLEnumTypeDefinition: {Name}")]
 public class GraphQLEnumTypeDefinition : GraphQLTypeDefinition, IHasDirectivesNode
 {
+    internal GraphQLEnumTypeDefinition()
+    {
+    }
+
+    /// <summary>
+    /// Creates a new instance of <see cref="GraphQLEnumTypeDefinition"/>.
+    /// </summary>
+    public GraphQLEnumTypeDefinition(GraphQLName name)
+        : base(name)
+    {
+    }
+
     /// <inheritdoc/>
     public override ASTNodeKind Kind => ASTNodeKind.EnumTypeDefinition;
 

src/GraphQLParser/AST/Definitions/GraphQLEnumTypeDefinition.cs

@@ -8,6 +8,20 @@ namespace GraphQLParser.AST;
 [DebuggerDisplay("GraphQLEnumValueDefinition: {EnumValue}")]
 public class GraphQLEnumValueDefinition : GraphQLTypeDefinition, IHasDirectivesNode
 {
+    internal GraphQLEnumValueDefinition()
+    {
+        EnumValue = null!;
+    }
+
+    /// <summary>
+    /// Creates a new instance of <see cref="GraphQLEnumValueDefinition"/>.
+    /// </summary>
+    public GraphQLEnumValueDefinition(GraphQLName name, GraphQLEnumValue enumValue)
+        : base(name)
+    {
+        EnumValue = enumValue;
+    }
+
     /// <inheritdoc/>
     public override ASTNodeKind Kind => ASTNodeKind.EnumValueDefinition;
 
@@ -16,7 +30,7 @@ public class GraphQLEnumValueDefinition : GraphQLTypeDefinition, IHasDirectivesN
     /// <see cref="GraphQLTypeDefinition.Name"/> property holds almost
     /// the same data and should be set as well.
     /// </summary>
-    public GraphQLEnumValue EnumValue { get; set; } = null!;
+    public GraphQLEnumValue EnumValue { get; set; }
 
     /// <inheritdoc/>
     public GraphQLDirectives? Directives { get; set; }

src/GraphQLParser/AST/Definitions/GraphQLEnumValueDefinition.cs

@@ -5,6 +5,18 @@ namespace GraphQLParser.AST;
 /// </summary>
 public class GraphQLEnumValuesDefinition : ASTListNode<GraphQLEnumValueDefinition>
 {
+    internal GraphQLEnumValuesDefinition()
+    {
+    }
+
+    /// <summary>
+    /// Creates a new instance of <see cref="GraphQLEnumValuesDefinition"/>.
+    /// </summary>
+    public GraphQLEnumValuesDefinition(List<GraphQLEnumValueDefinition> items)
+        : base(items)
+    {
+    }
+
     /// <inheritdoc/>
     public override ASTNodeKind Kind => ASTNodeKind.EnumValuesDefinition;
 }

src/GraphQLParser/AST/Definitions/GraphQLEnumValuesDefinition.cs

@@ -7,11 +7,24 @@ namespace GraphQLParser.AST;
 /// </summary>
 public abstract class GraphQLExecutableDefinition : ASTNode, IHasSelectionSetNode, IHasDirectivesNode
 {
+    internal GraphQLExecutableDefinition()
+    {
+        SelectionSet = null!;
+    }
+
+    /// <summary>
+    /// Creates a new instance of <see cref="GraphQLExecutableDefinition"/>.
+    /// </summary>
+    protected GraphQLExecutableDefinition(GraphQLSelectionSet selectionSet)
+    {
+        SelectionSet = selectionSet;
+    }
+
     /// <inheritdoc/>
     public GraphQLDirectives? Directives { get; set; }
 
     /// <inheritdoc/>
 #pragma warning disable CS8767 // Nullability of reference types in return type doesn't match implicitly implemented member (possibly because of nullability attributes).
-    public GraphQLSelectionSet SelectionSet { get; set; } = null!;
+    public GraphQLSelectionSet SelectionSet { get; set; }
 #pragma warning restore CS8767
 }

src/GraphQLParser/AST/Definitions/GraphQLExecutableDefinition.cs

@@ -8,6 +8,20 @@ namespace GraphQLParser.AST;
 [DebuggerDisplay("GraphQLFieldDefinition: {Name}")]
 public class GraphQLFieldDefinition : GraphQLTypeDefinition, IHasDirectivesNode, IHasArgumentsDefinitionNode
 {
+    internal GraphQLFieldDefinition()
+    {
+        Type = null!;
+    }
+
+    /// <summary>
+    /// Creates a new instance of <see cref="GraphQLFieldDefinition"/>.
+    /// </summary>
+    public GraphQLFieldDefinition(GraphQLName name, GraphQLType type)
+        : base(name)
+    {
+        Type = type;
+    }
+
     /// <inheritdoc/>
     public override ASTNodeKind Kind => ASTNodeKind.FieldDefinition;
 
@@ -19,7 +33,7 @@ public class GraphQLFieldDefinition : GraphQLTypeDefinition, IHasDirectivesNode,
     /// <summary>
     /// Nested <see cref="GraphQLType"/> AST node with field type.
     /// </summary>
-    public GraphQLType Type { get; set; } = null!;
+    public GraphQLType Type { get; set; }
 
     /// <inheritdoc/>
     public GraphQLDirectives? Directives { get; set; }