Fixed failing ruff tests

Author: Spaarsh

pyproject.toml

@@ -81,7 +81,6 @@ ignore = [
     "UP007",   # Disallowing Union is pedantic
     # TODO: Enable all of the following, but this PR is getting too large already
     "PLR0913",
-    "SIM102",
     "TRY003",
     "PLR2004",
     "PD901",

python/datafusion/functions.py

@@ -790,10 +790,7 @@ def regexp_count(
     """
     if flags is not None:
         flags = flags.expr
-    if start is not None:
-        start = start.expr
-    else:
-        start = Expr.expr
+    start = start.expr if start is not None else Expr.expr
     return Expr(f.regexp_count(string.expr, pattern.expr, start, flags))
 
 

python/tests/test_wrapper_coverage.py

@@ -28,7 +28,7 @@
     from enum import EnumMeta as EnumType
 
 
-def missing_exports(internal_obj, wrapped_obj) -> None:  # noqa: C901
+def missing_exports(internal_obj, wrapped_obj) -> None:
     """
     Identify if any of the rust exposted structs or functions do not have wrappers.
 
@@ -56,9 +56,8 @@ def missing_exports(internal_obj, wrapped_obj) -> None:  # noqa: C901
         # __kwdefaults__ and __doc__. As long as these are None on the internal
         # object, it's okay to skip them. However if they do exist on the internal
         # object they must also exist on the wrapped object.
-        if internal_attr is not None:
-            if wrapped_attr is None:
-                pytest.fail(f"Missing attribute: {internal_attr_name}")
+        if internal_attr is not None and wrapped_attr is None:
+            pytest.fail(f"Missing attribute: {internal_attr_name}")
 
         if internal_attr_name in ["__self__", "__class__"]:
             continue