tweaks

Author: aljazerzen

src/parser.rs

@@ -195,7 +195,7 @@ impl std::error::Error for ParserError {}
 // By default, allow expressions up to this deep before erroring
 const DEFAULT_REMAINING_DEPTH: usize = 50;
 
-#[derive(Default, Clone)]
+#[derive(Debug, Default, Clone, PartialEq, Eq)]
 pub struct ParserOptions {
     pub trailing_commas: bool,
 }

tests/sqlparser_common.rs

@@ -6896,18 +6896,20 @@ fn parse_trailing_comma() {
         }),
     };
 
-    // We shouldn't verify, because trailing commas will not get generated back
-    trailing_commas
-        .parse_sql_statements("SELECT album_id, name, FROM track")
-        .unwrap();
-
-    trailing_commas
-        .parse_sql_statements("SELECT * FROM track ORDER BY milliseconds,")
-        .unwrap();
-
-    trailing_commas
-        .parse_sql_statements("SELECT DISTINCT ON (album_id,) name FROM track")
-        .unwrap();
+    trailing_commas.one_statement_parses_to(
+        "SELECT album_id, name, FROM track",
+        "SELECT album_id, name FROM track",
+    );
+
+    trailing_commas.one_statement_parses_to(
+        "SELECT * FROM track ORDER BY milliseconds,",
+        "SELECT * FROM track ORDER BY milliseconds",
+    );
+
+    trailing_commas.one_statement_parses_to(
+        "SELECT DISTINCT ON (album_id,) name FROM track",
+        "SELECT DISTINCT ON (album_id) name FROM track",
+    );
 
     trailing_commas.verified_stmt("SELECT album_id, name FROM track");