Fix bug in test (previously hidden due to missing assert)

aio-libs · Jun 25, 2024 · 63b339d · 63b339d
1 parent 1c51f6a
commit 63b339d
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/aiodocker/utils.py b/aiodocker/utils.py
@@ -211,7 +211,7 @@ def clean_networks(
     return result
 
 
-def clean_filters(filters: Optional[Mapping] = None) -> str:
+def clean_filters(filters: Optional[Mapping[str, Any] | Sequence[str]] = None) -> str:
     """
     Checks the values inside `filters`
     https://docs.docker.com/engine/api/v1.29/#operation/ServiceList

diff --git a/tests/test_utils.py b/tests/test_utils.py
@@ -64,6 +64,6 @@ def test_clean_filters() -> None:
     result = {"a": ["1", "2", "3", "4"], "b": ["string"]}
     assert utils.clean_filters(filters=filters) == json.dumps(result)
 
-    filters = {}
-    result = {"a": ["1", "2", "3", "4"], "b": ["string"]}
-    assert utils.clean_filters(filters=filters) == json.dumps(result)
+    filters2: Sequence[str] = ()
+    result2: Sequence[str] = []
+    assert utils.clean_filters(filters=filters2) == json.dumps(result2)