Fix the comparison checks

The array comparison happens in the same way as normal comparison. Now
that we have support for the different types, the comparison should be
done correctly.

arangodantic/operators.py

@@ -45,13 +45,3 @@ class Operators(str, Enum):
     Operators.ALL: "all",
     Operators.NONE: "none",
 }
-
-# List of array comparison operators.
-array_comparison_operators = {
-    Operators.ALL_IN,
-    Operators.NONE_IN,
-    Operators.ANY_IN,
-    Operators.ANY,
-    Operators.ALL,
-    Operators.NONE,
-}

arangodantic/tests/test_model.py

@@ -234,7 +234,7 @@ async def test_find_with_comparisons(identity_collection):
             assert i.name in {"b", "c"}
 
     cursor = await (
-        Identity.find({"data.aliases": {Operators.ALL_IN: ["foo"]}}, count=True)
+        Identity.find({"data.aliases": {Operators.ALL_IN: ["foo", "bar"]}}, count=True)
     )
     async with cursor:
         assert len(cursor) == 1

arangodantic/utils.py

@@ -2,7 +2,7 @@
 from typing import Any, Dict, Iterable, List, Optional, Tuple
 
 from arangodantic.directions import DIRECTIONS
-from arangodantic.operators import array_comparison_operators, comparison_operators
+from arangodantic.operators import comparison_operators
 
 FilterTypes = Optional[Dict[str, Any]]
 SortTypes = Optional[Iterable[Tuple[str, str]]]
@@ -66,16 +66,9 @@ def build_filters(
                 bind_vars[value_bind_var] = value
 
                 # The actual comparison
-                if operator in array_comparison_operators:
-                    # These operators are checked against the instance.field instead
-                    # of the other way around, e.g. ["foo"] ALL IN i.data.aliases
-                    filter_list.append(
-                        f"@{value_bind_var} {operator} {instance_name}.{field_str}"
-                    )
-                else:
-                    filter_list.append(
-                        f"{instance_name}.{field_str} {operator} @{value_bind_var}"
-                    )
+                filter_list.append(
+                    f"{instance_name}.{field_str} {operator} @{value_bind_var}"
+                )
 
     return filter_list, bind_vars