fix

Author: sbdchd

crates/squawk_parser/src/grammar.rs

@@ -114,14 +114,13 @@ fn opt_paren_select(p: &mut Parser<'_>) -> Option<CompletedMarker> {
         }
     }
     p.expect(R_PAREN);
-    let cm = m.complete(p, PAREN_SELECT);
-    let cm = if p.at_ts(COMPOUND_SELECT_FIRST) {
-        compound_select(p, cm)
+    if p.at_ts(COMPOUND_SELECT_FIRST) {
+        let cm = m.complete(p, PAREN_SELECT);
+        Some(compound_select(p, cm))
     } else {
-        cm
-    };
-    select_trailing_clauses(p);
-    Some(cm)
+        select_trailing_clauses(p);
+        Some(m.complete(p, PAREN_SELECT))
+    }
 }
 
 const SELECT_FIRST: TokenSet = TokenSet::new(&[SELECT_KW, TABLE_KW, WITH_KW, VALUES_KW]);

crates/squawk_parser/tests/data/ok/precedence.sql

@@ -3,7 +3,6 @@ SELECT (((SELECT 2)) + 3);
 SELECT (((SELECT 2)) UNION SELECT 2);
 
 
--- TODO!
 SELECT foo UNION SELECT bar ORDER BY baz;
 -- equal to:
 (SELECT foo UNION SELECT bar) ORDER BY baz;

crates/squawk_parser/tests/snapshots/tests__precedence_ok.snap

@@ -73,8 +73,6 @@ SOURCE_FILE
             R_PAREN ")"
   SEMICOLON ";"
   WHITESPACE "\n\n\n"
-  COMMENT "-- TODO!"
-  WHITESPACE "\n"
   COMPOUND_SELECT
     SELECT
       SELECT_CLAUSE
@@ -130,13 +128,13 @@ SOURCE_FILE
               NAME_REF
                 IDENT "bar"
     R_PAREN ")"
-  WHITESPACE " "
-  ORDER_BY_CLAUSE
-    ORDER_KW "ORDER"
-    WHITESPACE " "
-    BY_KW "BY"
     WHITESPACE " "
-    NAME_REF
-      IDENT "baz"
+    ORDER_BY_CLAUSE
+      ORDER_KW "ORDER"
+      WHITESPACE " "
+      BY_KW "BY"
+      WHITESPACE " "
+      NAME_REF
+        IDENT "baz"
   SEMICOLON ";"
   WHITESPACE "\n"