fix nested string; add nested test

gluesql · Jun 6, 2022 · 043896b · 043896b
1 parent 52bbf44
commit 043896b
Showing 1 changed file with 6 additions and 1 deletion.
diff --git a/core/src/ast/expr.rs b/core/src/ast/expr.rs
@@ -114,7 +114,7 @@ impl ToSql for Expr {
             Expr::Extract { field, expr } => {
                 format!(r#"EXTRACT({field} FROM "{}")"#, expr.to_sql())
             }
-            Expr::Nested(expr) => format!("todo:Nested({})", expr.to_sql()),
+            Expr::Nested(expr) => format!("({})", expr.to_sql()),
             Expr::Literal(s) => s.to_sql(),
             Expr::TypedString { data_type, value } => format!("{data_type}(\"{value}\")"),
             Expr::Case {
@@ -302,6 +302,11 @@ mod tests {
             .to_sql()
         );
 
+        assert_eq!(
+            "(id)",
+            Expr::Nested(Box::new(Expr::Identifier("id".to_owned()))).to_sql(),
+        );
+
         assert_eq!(
             "id BETWEEN low AND high",
             Expr::Between {