More mypy induced changes

pccommon/pccommon/config/__init__.py

@@ -24,4 +24,5 @@ def get_all_render_configs() -> Dict[str, DefaultRenderConfig]:
         for id, coll in get_apis_config()
         .get_collection_config_table()
         .get_all_configs()
+        if id is not None
     }

pccommon/pccommon/logging.py

@@ -61,7 +61,7 @@ def filter(self, record: logging.LogRecord) -> bool:
 # Prevent successful health check pings from being logged
 class HealthCheckFilter(logging.Filter):
     def filter(self, record: logging.LogRecord) -> bool:
-        if len(record.args) != 5:
+        if record.args is not None and len(record.args) != 5:
             return True
 
         args = cast(Tuple[str, str, str, str, int], record.args)

pccommon/pccommon/tables.py

@@ -222,7 +222,7 @@ def get_all(self) -> Iterable[Tuple[Optional[str], Optional[str], M]]:
                 yield (
                     partition_key,
                     row_key,
-                    self._parse_model(entity, partition_key, row_key),
+                    self._parse_model(entity, partition_key, row_key),  # type: ignore
                 )
 
 

pccommon/setup.py

@@ -21,14 +21,8 @@
 ]
 
 extra_reqs = {
-    "test": [
-        "pytest",
-        "pytest-asyncio",
-    ],
-    "dev": [
-        "pytest",
-        "pytest-asyncio",
-    ],
+    "test": ["pytest", "pytest-asyncio", "types-redis"],
+    "dev": ["pytest", "pytest-asyncio", "types-redis"],
 }
 
 setup(