fix: preserve empty boolean property fallback

Keep the behavior introduced on main while resolving the property helper conflict.

Author: daviddallakyan2005

pyiceberg/utils/properties.py

@@ -58,7 +58,7 @@ def property_as_bool(
     property_name: str,
     default: bool,
 ) -> bool:
-    if (value := properties.get(property_name)) is not None:
+    if (value := properties.get(property_name)) not in (None, ""):
         try:
             return strtobool(str(value))
         except ValueError as e:

tests/utils/test_properties.py

@@ -118,13 +118,7 @@ def test_property_as_float_with_empty_string() -> None:
 
 
 def test_property_as_bool_with_empty_string() -> None:
-    properties = {
-        "some_bool_prop": "",
-    }
-
-    with pytest.raises(ValueError) as exc:
-        property_as_bool(properties, "some_bool_prop", default=True)
-    assert "Could not parse table property some_bool_prop to a boolean: " in str(exc.value)
+    assert property_as_bool({"some_bool_prop": ""}, "some_bool_prop", default=True) is True
 
 
 def test_get_first_property_value() -> None: