Sanitize geomcolname

lonboard/_geoarrow/_duckdb.py

@@ -115,7 +115,7 @@ def _from_geometry(
     # A poor-man's string interpolation check
     # We can't pass in SQL-templated strings for the column name
     re_match = r"[a-zA-Z][a-zA-Z0-9_]*"
-    assert re.match(
+    assert re.fullmatch(
         re_match,
         geom_col_name,
     ), f"Expected geometry column name to match regex: {re_match}"

tests/test_duckdb.py

@@ -240,3 +240,25 @@ def test_geometry_only_column_type_geometry():
 
     # Should create layer without erroring
     _layer = ScatterplotLayer.from_duckdb(query, con)
+
+
+def test_sanitize_column_name():
+    # For WKB parsing
+    pytest.importorskip("shapely")
+
+    con = duckdb.connect()
+    sql = f"""
+        INSTALL spatial;
+        LOAD spatial;
+        SELECT
+          * EXCLUDE geom,
+          geom as "non_alphanum_colname_()"
+        FROM ST_Read("{cities_gdal_path}");
+        """
+    rel = con.sql(sql)
+
+    with pytest.raises(
+        AssertionError,
+        match="Expected geometry column name to match regex:",
+    ):
+        viz(rel, con=con)