GEOPY-1693: las-geoh5 import files driver doesn't support numeric well names

las_geoh5/import_las.py

@@ -309,6 +309,8 @@ def create_or_append_drillhole(
         translator = LASTranslator(NameOptions())
 
     name = translator.retrieve("well_name", lasfile)
+    if not isinstance(name, str):
+        name = str(name)
     if not name and logger is not None:
         logger.warning(
             "No well name provided for LAS file. "

tests/import_las_test.py

@@ -415,3 +415,35 @@ def test_warning_no_well_name(tmp_path: Path, caplog):
         )
 
         assert match in caplog.text
+
+
+def test_handle_numeric_well_name(tmp_path: Path):
+    with Workspace.create(tmp_path / "test.geoh5") as workspace:
+        dh_group = DrillholeGroup.create(workspace, name="dh_group")
+
+    lasfile = generate_lasfile(
+        "123",
+        {"UTMX": 0.0, "UTMY": 0.0, "ELEV": 10.0},
+        np.arange(0, 11, 1),
+        {"my_property": np.zeros(11)},
+    )
+    lasfile = write_lasfile(tmp_path, lasfile)
+
+    filepath = write_import_params_file(
+        tmp_path / "import_las_files.ui.json",
+        dh_group,
+        "my_property_group",
+        [lasfile],
+        (
+            "UTMX",
+            "UTMY",
+            "ELEV",
+        ),
+    )
+
+    module = importlib.import_module("las_geoh5.import_files.driver")
+    module.run(filepath)
+
+    with workspace.open():
+        dh_group = workspace.get_entity("dh_group")[0]
+        assert "123" in [k.name for k in dh_group.children]

tests/script_las_to_geoh5_test.py

@@ -19,6 +19,7 @@
 from .helpers import generate_lasfile, write_import_params_file, write_lasfile
 
 
+# pylint: disable=duplicate-code
 @pytest.fixture(scope="module", name="lasfile")
 def lasfile_fixture(tmp_path_factory) -> Path:
     input_dir = tmp_path_factory.mktemp("input")
@@ -75,6 +76,7 @@ def test_las_to_geoh5_without_output_name(
     """Test the las_to_geoh5 script."""
 
     workspace_file = input_workspace.h5file
+    assert isinstance(workspace_file, Path)
     modified_date = workspace_file.stat().st_mtime
 
     with patch("sys.argv", ["las_to_geoh5", str(params_filepath)]):
@@ -93,6 +95,7 @@ def test_las_to_geoh5_with_monitoring_folder(
     """Test the las_to_geoh5 script."""
 
     workspace_file = input_workspace.h5file
+    assert isinstance(workspace_file, Path)
     modified_date = workspace_file.stat().st_mtime
 
     monitoring_folder = tmp_path / "monitored here"
@@ -124,6 +127,7 @@ def test_las_to_geoh5_with_output_name(
     """Test the las_to_geoh5 script."""
 
     workspace_file = input_workspace.h5file
+    assert isinstance(workspace_file, Path)
     modified_date = workspace_file.stat().st_mtime
 
     working_dir = tmp_path / "working"
@@ -150,6 +154,7 @@ def test_las_to_geoh5_with_absolute_output_path(
     """Test the las_to_geoh5 script."""
 
     workspace_file = input_workspace.h5file
+    assert isinstance(workspace_file, Path)
     modified_date = workspace_file.stat().st_mtime
 
     output_dir = tmp_path / "output"