Merge pull request #341 from NeurodataWithoutBorders/fix_images_pynwb_2_0_0

[Bug]: Correctly skip Images checks for PyNWB<2.1.0

Author: CodyCBakerPhD

src/nwbinspector/checks/images.py

@@ -1,16 +1,14 @@
 """Checks specific to the Images neurodata type."""
-from ..register_checks import register_check, Importance, InspectorMessage
-
-try:  # The Images neurodata type was unavailable prior to PyNWB v.2.1.0
-    from pynwb.base import Images
-
-    HAVE_IMAGES = True
-except ImportError:
-    HAVE_IMAGES = False
+from packaging.version import Version
 
 from pynwb.image import IndexSeries
 
-if HAVE_IMAGES:
+from ..register_checks import register_check, Importance, InspectorMessage
+from ..utils import get_package_version
+
+# The Images neurodata type was unavailable prior to PyNWB v.2.1.0
+if get_package_version(name="pynwb") >= Version("2.1.0"):
+    from pynwb.base import Images
 
     @register_check(importance=Importance.BEST_PRACTICE_VIOLATION, neurodata_type=Images)
     def check_order_of_images_unique(images: Images):

tests/unit_tests/test_images.py

@@ -3,21 +3,20 @@
 import numpy as np
 from pynwb import TimeSeries
 from pynwb.image import GrayscaleImage, IndexSeries
+from packaging.version import Version
 
 from nwbinspector import InspectorMessage, Importance
 from nwbinspector.checks.images import (
     check_order_of_images_unique,
     check_order_of_images_len,
     check_index_series_points_to_image,
 )
+from nwbinspector.utils import get_package_version
 
-try:
-    from pynwb.base import Images, ImageReferences
-
-    HAVE_IMAGES = True
-except ImportError:
-    HAVE_IMAGES = False
+HAVE_IMAGES = get_package_version(name="pynwb") >= Version("2.1.0")
 skip_reason = "You must have PyNWB>=v2.1.0 to run these tests!"
+if HAVE_IMAGES:
+    from pynwb.base import Images, ImageReferences
 
 
 @pytest.mark.skipif(not HAVE_IMAGES, reason=skip_reason)