STY: Apply ruff/flake8-bugbear rule B009

B009 Do not call `getattr` with a constant attribute value.
     It is not any safer than normal property access.

Author: DimitriPapadopoulos

nibabel/tests/conftest.py

@@ -6,7 +6,7 @@
 # Generate dynamic fixtures
 def pytest_generate_tests(metafunc):
     if 'supported_dtype' in metafunc.fixturenames:
-        if metafunc.cls is None or not getattr(metafunc.cls, 'image_class'):
+        if metafunc.cls is None or not metafunc.cls.image_class:
             raise pytest.UsageError(
                 'Attempting to use supported_dtype fixture outside an image test case'
             )

nibabel/viewers.py

@@ -447,7 +447,7 @@ def _set_position(self, x, y, z, notify=True):
     # Matplotlib handlers ####################################################
     def _in_axis(self, event):
         """Return axis index if within one of our axes, else None"""
-        if getattr(event, 'inaxes') is None:
+        if event.inaxes is None:
             return None
         for ii, ax in enumerate(self._axes):
             if event.inaxes is ax: