Merge pull request #40521 from 333fred/verify-not-missing

Assert that ParsingTests.N nodes are not missing, fix up tests that didn't assume this.

src/Compilers/CSharp/Test/Syntax/Parsing/AsyncParsingTests.cs

@@ -649,9 +649,9 @@ class C
                             N(SyntaxKind.TypeArgumentList);
                             {
                                 N(SyntaxKind.LessThanToken);
-                                N(SyntaxKind.IdentifierName);
+                                M(SyntaxKind.IdentifierName);
                                 {
-                                    N(SyntaxKind.IdentifierToken);
+                                    M(SyntaxKind.IdentifierToken);
                                 }
                                 M(SyntaxKind.GreaterThanToken);
                             }
@@ -696,9 +696,9 @@ class C
                                 N(SyntaxKind.TypeArgumentList);
                                 {
                                     N(SyntaxKind.LessThanToken);
-                                    N(SyntaxKind.IdentifierName);
+                                    M(SyntaxKind.IdentifierName);
                                     {
-                                        N(SyntaxKind.IdentifierToken);
+                                        M(SyntaxKind.IdentifierToken);
                                     }
                                     M(SyntaxKind.GreaterThanToken);
                                 }
@@ -745,9 +745,9 @@ class C
                                 N(SyntaxKind.TypeArgumentList);
                                 {
                                     N(SyntaxKind.LessThanToken);
-                                    N(SyntaxKind.IdentifierName);
+                                    M(SyntaxKind.IdentifierName);
                                     {
-                                        N(SyntaxKind.IdentifierToken);
+                                        M(SyntaxKind.IdentifierToken);
                                     }
                                     M(SyntaxKind.GreaterThanToken);
                                 }

src/Compilers/CSharp/Test/Syntax/Parsing/AwaitParsingTests.cs

@@ -1854,7 +1854,7 @@ public void BadStaticAwaitInvocationExpressionStatementInSyncContext()
                             N(SyntaxKind.VariableDeclarator);
                             {
                                 N(SyntaxKind.IdentifierToken);
-                                N(SyntaxKind.BracketedArgumentList);
+                                M(SyntaxKind.BracketedArgumentList);
                                 {
                                     M(SyntaxKind.OpenBracketToken);
                                     M(SyntaxKind.CloseBracketToken);

src/Compilers/CSharp/Test/Syntax/Parsing/CrefParsingTests.cs

@@ -1475,9 +1475,9 @@ public void InvalidIdentifier()
         {
             UsingNode("2");
 
-            N(SyntaxKind.NameMemberCref);
+            M(SyntaxKind.NameMemberCref);
             {
-                N(SyntaxKind.IdentifierName);
+                M(SyntaxKind.IdentifierName);
                 {
                     M(SyntaxKind.IdentifierToken);
                 }

src/Compilers/CSharp/Test/Syntax/Parsing/DeclarationParsingTests.cs

@@ -6260,7 +6260,7 @@ class C
                 N(SyntaxKind.ClassDeclaration);
                 {
                     N(SyntaxKind.ClassKeyword);
-                    N(SyntaxKind.IdentifierToken);
+                    N(SyntaxKind.IdentifierToken, "C");
                     N(SyntaxKind.OpenBraceToken);
                     N(SyntaxKind.PropertyDeclaration);
                     {
@@ -6268,7 +6268,7 @@ class C
                         {
                             N(SyntaxKind.IntKeyword);
                         }
-                        N(SyntaxKind.IdentifierToken);
+                        N(SyntaxKind.IdentifierToken, "A");
                         N(SyntaxKind.AccessorList);
                         {
                             N(SyntaxKind.OpenBraceToken);
@@ -6284,17 +6284,21 @@ class C
                                         N(SyntaxKind.SimpleMemberAccessExpression);
                                         {
                                             N(SyntaxKind.ThisExpression);
-                                            N(SyntaxKind.ThisKeyword);
+                                            {
+                                                N(SyntaxKind.ThisKeyword);
+                                            }
                                             N(SyntaxKind.DotToken);
-                                            N(SyntaxKind.IdentifierName);
-                                            N(SyntaxKind.IdentifierToken);
+                                            M(SyntaxKind.IdentifierName);
+                                            {
+                                                M(SyntaxKind.IdentifierToken);
+                                            }
                                         }
-                                        N(SyntaxKind.SemicolonToken);
+                                        M(SyntaxKind.SemicolonToken);
                                     }
-                                    N(SyntaxKind.CloseBraceToken);
+                                    M(SyntaxKind.CloseBraceToken);
                                 }
                             }
-                            N(SyntaxKind.CloseBraceToken);
+                            M(SyntaxKind.CloseBraceToken);
                         }
                     }
                     N(SyntaxKind.FieldDeclaration);
@@ -6308,7 +6312,7 @@ class C
                             }
                             N(SyntaxKind.VariableDeclarator);
                             {
-                                N(SyntaxKind.IdentifierToken);
+                                N(SyntaxKind.IdentifierToken, "B");
                             }
                         }
                         N(SyntaxKind.SemicolonToken);
@@ -6317,6 +6321,7 @@ class C
                 }
                 N(SyntaxKind.EndOfFileToken);
             }
+            EOF();
         }
 
         [Fact]

src/Compilers/CSharp/Test/Syntax/Parsing/LambdaParameterParsingTests.cs

@@ -406,7 +406,7 @@ class C {
                                                         }
                                                         N(SyntaxKind.IdentifierToken); // parameter name
                                                     }
-                                                    N(SyntaxKind.CloseParenToken);
+                                                    M(SyntaxKind.CloseParenToken);
                                                 }
                                                 M(SyntaxKind.EqualsGreaterThanToken);
                                                 M(SyntaxKind.IdentifierName); // lambda body

src/Compilers/CSharp/Test/Syntax/Parsing/NullableParsingTests.cs

@@ -557,9 +557,9 @@ public void NullCoalescingOperator_NullableType_Invalid()
                         }
                     }
                     N(SyntaxKind.QuestionQuestionToken);
-                    N(SyntaxKind.IdentifierName);
+                    M(SyntaxKind.IdentifierName);
                     {
-                        N(SyntaxKind.IdentifierToken);
+                        M(SyntaxKind.IdentifierToken);
                     }
                 }
                 N(SyntaxKind.QuestionToken);
@@ -567,10 +567,10 @@ public void NullCoalescingOperator_NullableType_Invalid()
                 {
                     N(SyntaxKind.IdentifierToken, "y");
                 }
-                N(SyntaxKind.ColonToken);
-                N(SyntaxKind.IdentifierName);
+                M(SyntaxKind.ColonToken);
+                M(SyntaxKind.IdentifierName);
                 {
-                    N(SyntaxKind.IdentifierToken);
+                    M(SyntaxKind.IdentifierToken);
                 }
             }
             EOF();
@@ -695,12 +695,12 @@ public void DeclarationPattern_NullableType()
                                 N(SyntaxKind.IdentifierToken, "t");
                             }
                             N(SyntaxKind.ColonToken);
-                            N(SyntaxKind.IdentifierName);
+                            M(SyntaxKind.IdentifierName);
                             {
-                                N(SyntaxKind.IdentifierToken);
+                                M(SyntaxKind.IdentifierToken);
                             }
                         }
-                        N(SyntaxKind.ColonToken);
+                        M(SyntaxKind.ColonToken);
                     }
                     N(SyntaxKind.BreakStatement);
                     {
@@ -752,11 +752,11 @@ public void DeclarationPattern_NullableArray()
                                 N(SyntaxKind.BracketedArgumentList);
                                 {
                                     N(SyntaxKind.OpenBracketToken);
-                                    N(SyntaxKind.Argument);
+                                    M(SyntaxKind.Argument);
                                     {
-                                        N(SyntaxKind.IdentifierName);
+                                        M(SyntaxKind.IdentifierName);
                                         {
-                                            N(SyntaxKind.IdentifierToken);
+                                            M(SyntaxKind.IdentifierToken);
                                         }
                                     }
                                     N(SyntaxKind.CloseBracketToken);
@@ -768,12 +768,12 @@ public void DeclarationPattern_NullableArray()
                                 N(SyntaxKind.IdentifierToken, "t");
                             }
                             N(SyntaxKind.ColonToken);
-                            N(SyntaxKind.IdentifierName);
+                            M(SyntaxKind.IdentifierName);
                             {
-                                N(SyntaxKind.IdentifierToken);
+                                M(SyntaxKind.IdentifierToken);
                             }
                         }
-                        N(SyntaxKind.ColonToken);
+                        M(SyntaxKind.ColonToken);
                     }
                     N(SyntaxKind.BreakStatement);
                     {

src/Compilers/CSharp/Test/Syntax/Parsing/ParserErrorMessageTests.cs

@@ -5772,7 +5772,7 @@ class TestClass { }";
                                 N(SyntaxKind.IdentifierToken, "One");
                             }
                         }
-                        N(SyntaxKind.CommaToken, ""); // missing
+                        M(SyntaxKind.CommaToken);
                         N(SyntaxKind.Attribute);
                         {
                             N(SyntaxKind.IdentifierName);

src/Compilers/CSharp/Test/Syntax/Parsing/ParsingTests.cs

@@ -187,6 +187,7 @@ protected SyntaxNodeOrToken N(SyntaxKind kind, string? value = null)
             {
                 Assert.True(_treeEnumerator!.MoveNext());
                 Assert.Equal(kind, _treeEnumerator.Current.Kind());
+                Assert.False(_treeEnumerator.Current.IsMissing);
 
                 if (value != null)
                 {

src/Compilers/CSharp/Test/Syntax/Parsing/SuppressNullableWarningExpressionParsingTests.cs

@@ -188,7 +188,7 @@ public void TestEquals()
             {
                 N(SyntaxKind.IdentifierName);
                 {
-                    N(SyntaxKind.IdentifierToken);
+                    N(SyntaxKind.IdentifierToken, "o");
                 }
                 N(SyntaxKind.EqualsToken);
                 N(SyntaxKind.SimpleAssignmentExpression);
@@ -197,21 +197,22 @@ public void TestEquals()
                     {
                         N(SyntaxKind.IdentifierName);
                         {
-                            N(SyntaxKind.IdentifierToken);
+                            N(SyntaxKind.IdentifierToken, "o");
                         }
                         N(SyntaxKind.ExclamationEqualsToken);
-                        N(SyntaxKind.IdentifierName);
-                        {
-                            N(SyntaxKind.IdentifierToken);
-                        }
-                        N(SyntaxKind.EqualsToken);
-                        N(SyntaxKind.NullLiteralExpression);
+                        M(SyntaxKind.IdentifierName);
                         {
-                            N(SyntaxKind.NullKeyword);
+                            M(SyntaxKind.IdentifierToken);
                         }
                     }
+                    N(SyntaxKind.EqualsToken);
+                    N(SyntaxKind.NullLiteralExpression);
+                    {
+                        N(SyntaxKind.NullKeyword);
+                    }
                 }
             }
+            EOF();
         }
 
         [Fact]