Cover comparison bounds and Spark BETWEEN behavior

Author: LucaCappelletti94

tests/sqlparser_common.rs

@@ -21056,6 +21056,14 @@ fn parse_comparison_in_between_low_bound() {
             "SELECT a BETWEEN b BETWEEN 1 AND 2 AND c",
             "Expected: AND, found: BETWEEN",
         ),
+        (
+            "SELECT a BETWEEN (1 AND 2 FROM t",
+            "Expected: ), found: FROM",
+        ),
+        (
+            "SELECT a BETWEEN b = (1, 2 FROM t",
+            "Expected: ), found: FROM",
+        ),
     ] {
         assert_eq!(
             dialects.parse_sql_statements(sql),

tests/sqlparser_spark.rs

@@ -362,3 +362,22 @@ fn test_substring() {
 fn test_pipe_operator() {
     spark().verified_stmt("SELECT * FROM t |> WHERE x > 1 |> SELECT x AS y |> ORDER BY y");
 }
+
+// Spark is not part of all_dialects, so the between low bound comparison is pinned here.
+
+#[test]
+fn parse_comparison_in_between_low_bound() {
+    match spark().verified_expr("a BETWEEN b > 1 AND c") {
+        Expr::Between { low, .. } => assert!(
+            matches!(
+                &*low,
+                Expr::BinaryOp {
+                    op: BinaryOperator::Gt,
+                    ..
+                }
+            ),
+            "{low:?}"
+        ),
+        other => panic!("expected BETWEEN, got {other:?}"),
+    }
+}