turn on gdal exceptions in some test_table_definitions_check

PDOK-16629

geopackage_validator/generate.py

@@ -172,10 +172,10 @@ def generate_table_definitions(
     return result
 
 
-def get_datasource_for_path(gpkg_path: str) -> DataSource:
+def get_datasource_for_path(gpkg_path: str, error_handler=None) -> DataSource:
     """Starts the geopackage validation."""
     utils.check_gdal_version()
-    return utils.open_dataset(gpkg_path)
+    return utils.open_dataset(gpkg_path, error_handler)
 
 
 def generate_definitions_for_path(

geopackage_validator/utils.py

@@ -42,7 +42,7 @@
 }
 
 
-def open_dataset(filename=None, error_handler=None) -> DataSource:
+def open_dataset(filename: str = None, error_handler: Callable = None) -> DataSource:
     if error_handler is not None:
         gdal.UseExceptions()
         gdal.PushErrorHandler(error_handler)

tests/validations/test_table_definitions_check.py

@@ -168,9 +168,14 @@ def test_legacy_table_definitions_check_table_changed():
     ]
 
 
+def gdal_error_handler_print_err(err_level, err_no, err_msg):
+    print(f"GDAL error: err_level: {err_level}\nerr_no: {err_no}\nerr_msg: {err_msg}")
+
+
 def test_table_definitions_check_changed_indexes_and_fks():
     datasource = get_datasource_for_path(
-        "tests/data/test_allcorrect_with_indexes_and_fks.gpkg"
+        "tests/data/test_allcorrect_with_indexes_and_fks.gpkg",
+        gdal_error_handler_print_err,
     )
     table_definitions = load_table_definitions(
         "tests/data/test_changed_indexes_and_fks_definition.yml"
@@ -194,7 +199,9 @@ def test_table_definitions_check_changed_indexes_and_fks():
 
 
 def test_table_definitions_check_fk_violation():
-    datasource = get_datasource_for_path("tests/data/test_foreign_key_violation.gpkg")
+    datasource = get_datasource_for_path(
+        "tests/data/test_foreign_key_violation.gpkg", gdal_error_handler_print_err
+    )
     table_definitions = load_table_definitions(
         "tests/data/test_allcorrect_with_indexes_and_fks_definition.yml"
     )