debug file order in tests

Author: Cody Baker

nwbinspector/nwbinspector.py

@@ -19,6 +19,7 @@
 import pynwb
 import yaml
 from tqdm import tqdm
+from natsort import natsorted
 
 from . import available_checks
 from .inspector_tools import (
@@ -379,7 +380,7 @@ def inspect_all(
                 yield InspectorMessage(
                     message=(
                         f"The identifier '{identifier}' is used across the .nwb files: "
-                        f"{[str(x) for x in nwbfiles_with_identifier]}\n"
+                        f"{natsorted([x.name for x in nwbfiles_with_identifier])}. "
                         "The identifier of any NWBFile should be a completely unique value - "
                         "we recommend using uuid4 to achieve this."
                     ),
@@ -388,7 +389,7 @@ def inspect_all(
                     object_type="NWBFile",
                     object_name="root",
                     location="/",
-                    file_path=str(Path(nwbfiles_with_identifier[0]).parent),
+                    file_path=str(nwbfiles_with_identifier[0].parent),
                 )
 
     nwbfiles_iterable = nwbfiles

tests/test_inspector.py

@@ -11,6 +11,7 @@
 from pynwb.file import TimeIntervals
 from pynwb.behavior import SpatialSeries, Position
 from hdmf.common import DynamicTable
+from natsort import natsorted
 
 from nwbinspector import (
     Importance,
@@ -617,12 +618,11 @@ def tearDownClass(cls):
         rmtree(cls.tempdir)
 
     def test_check_unique_identifiers_fail(self):
-        print(list(inspect_all(path=self.tempdir, select=["check_data_orientation"])))
         assert list(inspect_all(path=self.tempdir, select=["check_data_orientation"])) == [
             InspectorMessage(
                 message=(
                     "The identifier 'not a unique identifier!' is used across the .nwb files: "
-                    f"{[str(x) for x in self.non_unique_id_nwbfile_paths]}\n"
+                    f"{natsorted([Path(x).name for x in self.non_unique_id_nwbfile_paths])}. "
                     "The identifier of any NWBFile should be a completely unique value - "
                     "we recommend using uuid4 to achieve this."
                 ),
@@ -631,6 +631,6 @@ def test_check_unique_identifiers_fail(self):
                 object_type="NWBFile",
                 object_name="root",
                 location="/",
-                file_path=str(Path(self.non_unique_id_nwbfile_paths[0]).parent),
+                file_path=str(self.tempdir),
             )
         ]