fix wrong return type

Author: signedav

modelbaker/dbconnector/db_connector.py

@@ -280,11 +280,11 @@ def get_attrili_attrdb_mapping_by_owner(self, owners: list[str]) -> dict:
         """Used for ili2db version 3"""
         return {}
 
-    def get_models(self) -> dict:
+    def get_models(self) -> list[dict]:
         """
         Returns the models in use, the ili-file content and the direct parents of the model.
         """
-        return {}
+        return []
 
     def ili_version(self) -> str:
         """
@@ -399,12 +399,12 @@ def edit_basket(self, basket_config: dict) -> tuple[bool, str]:
         """
         return False, None
 
-    def get_tid_handling(self) -> dict:
+    def get_tid_handling(self) -> bool:
         """
         Returns `True` if a tid handling is enabled according to the settings table (when the database has been created with `--createTidCol`).
         If t_ili_tids are used only because of a stable id definition in the model (with `OID as` in the topic or the class definition), this parameter is not set and this function will return `{}`.
         """
-        return {}
+        return False
 
     def get_ili2db_settings(self) -> dict:
         """

modelbaker/dbconnector/gpkg_connector.py

@@ -711,9 +711,9 @@ def get_attrili_attrdb_mapping_by_owner(self, owners: list[str]) -> dict:
         )
         return cursor
 
-    def get_models(self) -> dict:
+    def get_models(self) -> list[dict]:
         if not self._table_exists("T_ILI2DB_TRAFO"):
-            return {}
+            return []
 
         # Get MODELS
         cursor = self.conn.cursor()
@@ -1105,7 +1105,7 @@ def edit_basket(self, basket_config: dict) -> tuple[bool, str]:
             'Could not edit basket for topic "{}" and dataset "{}"'
         ).format(basket_config["topic"], basket_config["datasetname"])
 
-    def get_tid_handling(self) -> dict:
+    def get_tid_handling(self) -> bool:
         if self._table_exists(GPKG_SETTINGS_TABLE):
             cursor = self.conn.cursor()
             cursor.execute(

modelbaker/dbconnector/mssql_connector.py

@@ -763,9 +763,9 @@ def get_iliname_dbname_mapping(self, sqlnames: list[str] = []) -> dict:
 
         return result
 
-    def get_models(self) -> dict:
+    def get_models(self) -> list[dict]:
         if not self._table_exists("t_ili2db_trafo"):
-            return {}
+            return []
 
         # Get MODELS
         if self.schema:
@@ -808,7 +808,7 @@ def get_models(self) -> dict:
                         list_result.append(result)
                         result = dict()
             return list_result
-        return {}
+        return []
 
     def get_classili_classdb_mapping(self, models_info, extended_classes):
         result = {}
@@ -1184,7 +1184,7 @@ def edit_basket(self, basket_config: dict) -> tuple[bool, str]:
             'Could not edit basket for topic "{}" and dataset "{}"'
         ).format(basket_config["topic"], basket_config["datasetname"])
 
-    def get_tid_handling(self) -> dict:
+    def get_tid_handling(self) -> bool:
         if self.schema and self._table_exists(SETTINGS_TABLE):
             cur = self.conn.cursor()
             cur.execute(

modelbaker/dbconnector/pg_connector.py

@@ -915,9 +915,9 @@ def get_attrili_attrdb_mapping_by_owner(self, owners: list[str]) -> dict:
 
         return {}
 
-    def get_models(self) -> dict:
+    def get_models(self) -> list[dict]:
         if not self._table_exists("t_ili2db_trafo"):
-            return {}
+            return []
 
         # Get MODELS
         if self.schema:
@@ -964,7 +964,7 @@ def get_models(self) -> dict:
                         result = dict()
 
             return list_result
-        return {}
+        return []
 
     def ili_version(self) -> str:
         cur = self.conn.cursor()
@@ -1282,7 +1282,7 @@ def edit_basket(self, basket_config: dict) -> tuple[bool, str]:
             'Could not edit basket for topic "{}" and dataset "{}"'
         ).format(basket_config["topic"], basket_config["datasetname"])
 
-    def get_tid_handling(self) -> dict:
+    def get_tid_handling(self) -> bool:
         if self.schema and self._table_exists(PG_SETTINGS_TABLE):
             cur = self.conn.cursor()
             cur.execute(