Fix GitHub Actions widget test failures

- Add conditional skip for widget tests when dependencies unavailable
- Update CI workflow to install widget dependencies
- Widget tests now gracefully skip in environments without IPython/ipywidgets

Resolves pytest failures in GitHub Actions CI environment.

Author: jeremymanning

.github/workflows/tests.yml

@@ -31,7 +31,7 @@ jobs:
     - name: Install dependencies
       run: |
         python -m pip install --upgrade pip
-        pip install -e ".[dev,test,kubernetes]"
+        pip install -e ".[dev,test,kubernetes,widget]"
 
     - name: Lint with black
       run: |
@@ -70,7 +70,7 @@ jobs:
     - name: Install dependencies
       run: |
         python -m pip install --upgrade pip
-        pip install -e ".[dev,test,kubernetes]"
+        pip install -e ".[dev,test,kubernetes,widget]"
         
     - name: Run integration tests
       run: |

tests/test_widget_fixes.py

@@ -4,15 +4,26 @@
 
 import importlib
 
+import pytest
+
 import clustrix.notebook_magic
 from clustrix.config import ClusterConfig
 
+# Check if widget dependencies are available
+try:
+    import IPython
+    import ipywidgets
+    WIDGET_DEPS_AVAILABLE = True
+except ImportError:
+    WIDGET_DEPS_AVAILABLE = False
+
 # Reload to ensure fresh state
 importlib.reload(clustrix.notebook_magic)
 
 # Import after reload to get the refreshed module
 DEFAULT_CONFIGS = clustrix.notebook_magic.DEFAULT_CONFIGS
-ClusterConfigWidget = clustrix.notebook_magic.ClusterConfigWidget
+if WIDGET_DEPS_AVAILABLE:
+    ClusterConfigWidget = clustrix.notebook_magic.ClusterConfigWidget
 
 
 class TestWidgetConfigurationFixes:
@@ -57,6 +68,7 @@ def test_cloud_provider_field_mapping(self):
         assert "hf_hardware" in hf_config
         assert "hf_sdk" in hf_config
 
+    @pytest.mark.skipif(not WIDGET_DEPS_AVAILABLE, reason="Widget dependencies not available")
     def test_widget_safe_value_setting(self):
         """Test that widget safely handles values not in dropdown options."""
         widget = ClusterConfigWidget(auto_display=False)
@@ -87,6 +99,7 @@ def test_widget_safe_value_setting(self):
         assert widget.azure_region.value in widget.azure_region.options
         assert widget.azure_instance_type.value in widget.azure_instance_type.options
 
+    @pytest.mark.skipif(not WIDGET_DEPS_AVAILABLE, reason="Widget dependencies not available")
     def test_widget_save_load_cycle(self):
         """Test that widget can save and load configurations correctly."""
         widget = ClusterConfigWidget(auto_display=False)
@@ -178,6 +191,7 @@ def test_cloud_provider_fields_in_config(self):
         assert hf_config.hf_username == "test-user"
         assert hf_config.hf_sdk == "gradio"
 
+    @pytest.mark.skipif(not WIDGET_DEPS_AVAILABLE, reason="Widget dependencies not available")
     def test_widget_dropdown_population(self):
         """Test that widget properly populates dropdown options."""
         widget = ClusterConfigWidget(auto_display=False)
@@ -214,6 +228,7 @@ def test_no_name_description_in_default_configs(self):
                 "description" not in test_config
             ), f"Config '{config_name}' should not have 'description' field initially"
 
+    @pytest.mark.skipif(not WIDGET_DEPS_AVAILABLE, reason="Widget dependencies not available")
     def test_widget_cluster_type_change_updates_options(self):
         """Test that changing cluster type updates dropdown options."""
         widget = ClusterConfigWidget(auto_display=False)