fix of #48

Author: ea-rus

mindsdb_sql_parser/ast/select/operation.py

@@ -10,7 +10,7 @@ def __init__(self, op, args, *args_, **kwargs):
         self.op = ' '.join(op.lower().split())
         self.args = []
         for item in args:
-            if isinstance(item, ASTNode) and item.parentheses:
+            if self.op in ("in", "not in") and isinstance(item, ASTNode) and item.parentheses:
                 item.parentheses = False
                 item = Tuple([item])
             self.args.append(item)

tests/test_base_sql/test_select_structure.py

@@ -1363,4 +1363,27 @@ def test_lateral_join(self):
         )
         ast = parse_sql(query)
         assert str(ast) == str(expected_ast)
-        assert ast.to_tree() == expected_ast.to_tree()
+        assert ast.to_tree() == expected_ast.to_tree()
+
+    def test_many_conditions(self):
+        sql = """
+            select * from kb where
+               (content like 'white' and size='small') 
+            or (content = 'big')
+        """
+        ast = parse_sql(sql)
+        expected_ast = Select(
+            targets=[Star()],
+            from_table=Identifier(parts=['kb']),
+            where=BinaryOperation(op='or', args=[
+                    BinaryOperation(op='and', args=[
+                        BinaryOperation(op='like', args=[Identifier('content'), Constant('white')]),
+                        BinaryOperation(op='=', args=[Identifier('size'), Constant('small')]),
+                    ], parentheses=True),
+                    BinaryOperation(op='=', args=[Identifier('content'), Constant('big')], parentheses=True),
+                ]
+            )
+        )
+        assert ast.to_tree() == expected_ast.to_tree()
+        assert str(ast) == str(expected_ast)
+