test: update tests in apollo-smith

Author: DaleSeo

crates/apollo-parser/src/cst/generated/nodes.rs

@@ -1,11 +1,14 @@
 //! This is a generated file, please do not edit manually. Changes can be
 //! made in codegeneration that lives in `xtask` top-level dir.
 
-use crate::{
-    cst::{support, CstChildren, CstNode},
-    SyntaxKind::{self, *},
-    SyntaxNode, SyntaxToken, S,
-};
+use crate::cst::support;
+use crate::cst::CstChildren;
+use crate::cst::CstNode;
+use crate::SyntaxKind::*;
+use crate::SyntaxKind::{self};
+use crate::SyntaxNode;
+use crate::SyntaxToken;
+use crate::S;
 #[derive(Debug, Clone, PartialEq, Eq, Hash)]
 pub struct Name {
     pub(crate) syntax: SyntaxNode,

crates/apollo-parser/src/parser/grammar/variable.rs

@@ -26,11 +26,8 @@ pub(crate) fn variable_definitions(p: &mut Parser) {
     }
 
     // Continue parsing while we see descriptions or variables
-    loop {
-        match p.peek() {
-            Some(TokenKind::StringValue | T![$]) => variable_definition(p),
-            _ => break,
-        }
+    while let Some(TokenKind::StringValue | T![$]) = p.peek() {
+        variable_definition(p);
     }
 
     p.expect(T![')'], S![')']);

crates/apollo-smith/src/fragment.rs

@@ -25,6 +25,7 @@ pub struct FragmentDef {
 impl From<FragmentDef> for ast::Definition {
     fn from(x: FragmentDef) -> Self {
         ast::FragmentDefinition {
+            description: None,
             name: x.name.into(),
             type_condition: x.type_condition.name.into(),
             directives: Directive::to_ast(x.directives),

crates/apollo-smith/src/operation.rs

@@ -28,6 +28,7 @@ pub struct OperationDef {
 impl From<OperationDef> for ast::Definition {
     fn from(x: OperationDef) -> Self {
         ast::OperationDefinition {
+            description: None,
             operation_type: x.operation_type.into(),
             name: x.name.map(Into::into),
             directives: Directive::to_ast(x.directives),

crates/apollo-smith/src/variable.rs

@@ -27,6 +27,7 @@ pub struct VariableDef {
 impl From<VariableDef> for ast::VariableDefinition {
     fn from(x: VariableDef) -> Self {
         Self {
+            description: None,
             name: x.name.into(),
             ty: Node::new(x.ty.into()),
             default_value: x.default_value.map(|x| Node::new(x.into())),