test: update tests in apollo-smith

Author: DaleSeo

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())),