Fix formatting and add extra test

Author: jtuglu1

pyiceberg/expressions/parser.py

@@ -156,10 +156,8 @@ def _(result: ParseResults) -> Literal[L]:
 
 @between.set_parse_action
 def _(result: ParseResults) -> BooleanExpression:
-    return And(
-        GreaterThanOrEqual(result.column, result[2]),
-        LessThanOrEqual(result.column, result[4])
-    )
+    return And(GreaterThanOrEqual(result.column, result[2]), LessThanOrEqual(result.column, result[4]))
+
 
 @left_ref.set_parse_action
 def _(result: ParseResults) -> BooleanExpression:

tests/expressions/test_parser.py

@@ -39,7 +39,8 @@
     NotNull,
     NotStartsWith,
     Or,
-    StartsWith, Reference,
+    Reference,
+    StartsWith,
 )
 from pyiceberg.expressions.literals import DecimalLiteral, LongLiteral
 
@@ -239,10 +240,22 @@ def test_quoted_column_with_dots() -> None:
 def test_quoted_column_with_spaces() -> None:
     assert EqualTo("Foo Bar", "data") == parser.parse("\"Foo Bar\" = 'data'")
 
+
 def test_valid_between() -> None:
-    assert And(left=GreaterThanOrEqual(Reference(name="foo"), LongLiteral(1)), right=LessThanOrEqual(Reference(name="foo"), LongLiteral(3))) == parser.parse("foo between 1 and 3")
-    assert And(left=GreaterThanOrEqual(Reference(name="foo"), LongLiteral(1)), right=LessThanOrEqual(Reference(name="foo"), LongLiteral(1))) == parser.parse("foo between 1 and 1")
-    assert And(left=GreaterThanOrEqual(Reference(name="foo"), DecimalLiteral(Decimal(1.0))), right=LessThanOrEqual(Reference(name="foo"), DecimalLiteral(Decimal(4.0)))) == parser.parse("foo between 1.0 and 4.0")
+    assert And(
+        left=GreaterThanOrEqual(Reference(name="foo"), LongLiteral(1)),
+        right=LessThanOrEqual(Reference(name="foo"), LongLiteral(3)),
+    ) == parser.parse("foo between 1 and 3")
+    assert And(
+        left=GreaterThanOrEqual(Reference(name="foo"), LongLiteral(1)),
+        right=LessThanOrEqual(Reference(name="foo"), LongLiteral(1)),
+    ) == parser.parse("foo between 1 and 1")
+    assert And(
+        left=GreaterThanOrEqual(Reference(name="foo"), DecimalLiteral(Decimal(1.0))),
+        right=LessThanOrEqual(Reference(name="foo"), DecimalLiteral(Decimal(4.0))),
+    ) == parser.parse("foo between 1.0 and 4.0")
+    assert parser.parse("foo between 1 and 3") == parser.parse("1 <= foo and foo <= 3")
+
 
 def test_invalid_between() -> None:
     # boolean
@@ -253,4 +266,4 @@ def test_invalid_between() -> None:
     # string
     with pytest.raises(ParseException) as exc_info:
         parser.parse("foo between 'a' and 'b'")
-    assert "Expected number, found \"\'\"" in str(exc_info)
+    assert 'Expected number, found "\'"' in str(exc_info)