proper skip for no matplotlib import

Author: TimMonko

tests/widgets/test_workflow_inspector.py

@@ -7,20 +7,18 @@
 
 import pytest
 
+from ndev_workflows.widgets._workflow_inspector import (
+    HAS_MATPLOTLIB,
+    WorkflowInspector,
+)
+
 
 class TestWorkflowInspector:
     """Core WorkflowInspector widget tests."""
 
     @pytest.fixture
     def inspector(self, make_napari_viewer, qtbot):
         """Create a WorkflowInspector widget."""
-        pytest.importorskip('matplotlib')
-        pytest.importorskip('networkx')
-
-        from ndev_workflows.widgets._workflow_inspector import (
-            WorkflowInspector,
-        )
-
         viewer = make_napari_viewer()
         widget = WorkflowInspector(viewer)
         qtbot.addWidget(widget)
@@ -96,6 +94,7 @@ def identity(x):
         assert inspector._use_live_mode is True
         assert inspector._loaded_workflow is None
 
+    @pytest.mark.skipif(not HAS_MATPLOTLIB, reason='requires matplotlib')
     def test_graph_updates_on_workflow_load(self, inspector, tmp_path):
         """Test that graph is drawn when workflow is loaded."""
         from ndev_workflows import Workflow, save_workflow
@@ -210,13 +209,13 @@ def test_pending_updates_property(self, make_napari_viewer):
         assert 'modified' not in manager.pending_updates
 
 
+@pytest.mark.skipif(not HAS_MATPLOTLIB, reason='requires matplotlib')
 class TestDraggableNodes:
-    """Test DraggableNodes interaction handling."""
+    """Test DraggableNodes interaction handling (requires matplotlib)."""
 
     @pytest.fixture
     def draggable_nodes(self):
         """Create DraggableNodes with real MplCanvas."""
-        pytest.importorskip('matplotlib')
         from unittest.mock import MagicMock
 
         from ndev_workflows.widgets._workflow_inspector import (
@@ -272,7 +271,6 @@ def test_update_node_status_changes_color(self, draggable_nodes):
 
     def test_select_node_with_viewer(self, make_napari_viewer):
         """Test clicking a node selects the layer in viewer."""
-        pytest.importorskip('matplotlib')
         import numpy as np
 
         from ndev_workflows.widgets._workflow_inspector import (