Added test for dlis raw functions

test/dlis/test_dlis_functions.py

@@ -0,0 +1,17 @@
+from pathlib import Path
+
+from dlisio import dlis
+
+from wellbelog.belodlis.functions import unpack_physical_dlis, open_dlis_file
+
+folder_path = Path(__file__).parent.parent / 'test_files'
+file_path = folder_path / '1PIR1AL_conv_ccl_canhoneio.dlis'
+
+
+def test_open_dlis_file():
+    physical_file = open_dlis_file(file_path)
+    assert isinstance(physical_file, dlis.PhysicalFile)
+    logical_files = unpack_physical_dlis(physical_file)
+    assert isinstance(logical_files, list)
+    assert len(logical_files) > 0
+    assert isinstance(logical_files[0], dlis.LogicalFile)

test/dlis/test_dlis_reading.py

@@ -14,6 +14,10 @@ def test_search_files():
     assert len(dlis_files) == 1
     assert dlis_files[0].name == '1PIR1AL_conv_ccl_canhoneio.dlis'
 
+    # Testing the search_files method with a wrong path
+    dlis_files = dlis_processor.search_files(folder_path / 'wrong_path')
+    assert not len(dlis_files)
+
 
 def test_read_dlis_file():
     dlis_processor = DlisReader()

wellbelog/belodlis/reader.py

@@ -1,5 +1,6 @@
 import json
 import pathlib
+from typing import Optional
 
 from dlisio.dlis import Frame
 
@@ -33,7 +34,7 @@ def load_raw(self, path_to_file: str, unpack=False) -> PhysicalFileModel:
             return unpack_physical_dlis(file)
         return file
 
-    def search_files(self, path: str) -> list[pathlib.Path]:
+    def search_files(self, path: str) -> Optional[list[pathlib.Path]]:
         """
         Search for DLIS files in the given path and returns a list with the file paths.
 
@@ -47,7 +48,7 @@ def search_files(self, path: str) -> list[pathlib.Path]:
             return dlis_files
         except Exception as e:
             self.logger.error(f'Error while searching for DLIS files: {e}')
-            return dlis_files
+            return None
 
     def process_physical_file(self, path: str, folder_name: str = None) -> PhysicalFileModel:
         """