fix pattern matching order and adjust tests

Author: ewgenius

datafusion/sql/src/unparser/expr.rs

@@ -261,32 +261,20 @@ impl Unparser<'_> {
                     uses_odbc_syntax: false,
                 }))
             }
-            Expr::Like(Like {
+            Expr::SimilarTo(Like {
                 negated,
                 expr,
                 pattern,
                 escape_char,
-                case_insensitive,
-            }) => {
-                if *case_insensitive {
-                    Ok(ast::Expr::ILike {
-                        negated: *negated,
-                        expr: Box::new(self.expr_to_sql_inner(expr)?),
-                        pattern: Box::new(self.expr_to_sql_inner(pattern)?),
-                        escape_char: escape_char.map(|c| c.to_string()),
-                        any: false,
-                    })
-                } else {
-                    Ok(ast::Expr::Like {
-                        negated: *negated,
-                        expr: Box::new(self.expr_to_sql_inner(expr)?),
-                        pattern: Box::new(self.expr_to_sql_inner(pattern)?),
-                        escape_char: escape_char.map(|c| c.to_string()),
-                        any: false,
-                    })
-                }
-            }
-            Expr::SimilarTo(Like {
+                case_insensitive: _,
+            }) => Ok(ast::Expr::Like {
+                negated: *negated,
+                expr: Box::new(self.expr_to_sql_inner(expr)?),
+                pattern: Box::new(self.expr_to_sql_inner(pattern)?),
+                escape_char: escape_char.map(|c| c.to_string()),
+                any: false,
+            }),
+            Expr::Like(Like {
                 negated,
                 expr,
                 pattern,
@@ -1833,24 +1821,24 @@ mod tests {
                 r#"a NOT LIKE 'foo' ESCAPE 'o'"#,
             ),
             (
-                Expr::SimilarTo(Like {
-                    negated: false,
+                Expr::Like(Like {
+                    negated: true,
                     expr: Box::new(col("a")),
                     pattern: Box::new(lit("foo")),
                     escape_char: Some('o'),
-                    case_insensitive: false,
+                    case_insensitive: true,
                 }),
-                r#"a LIKE 'foo' ESCAPE 'o'"#,
+                r#"a NOT ILIKE 'foo' ESCAPE 'o'"#,
             ),
             (
-                Expr::Like(Like {
-                    negated: true,
+                Expr::SimilarTo(Like {
+                    negated: false,
                     expr: Box::new(col("a")),
                     pattern: Box::new(lit("foo")),
                     escape_char: Some('o'),
-                    case_insensitive: true,
+                    case_insensitive: false,
                 }),
-                r#"a NOT ILIKE 'foo' ESCAPE 'o'"#,
+                r#"a LIKE 'foo' ESCAPE 'o'"#,
             ),
             (
                 Expr::SimilarTo(Like {
@@ -1860,7 +1848,7 @@ mod tests {
                     escape_char: Some('o'),
                     case_insensitive: true,
                 }),
-                r#"a ILIKE 'foo' ESCAPE 'o'"#,
+                r#"a LIKE 'foo' ESCAPE 'o'"#,
             ),
             (
                 Expr::Literal(ScalarValue::Date64(Some(0))),