fix: add importorskip for permission tests

Author: bencap

tests/lib/permissions/conftest.py

@@ -1,15 +1,17 @@
 """Shared fixtures and helpers for permissions tests."""
 
 from dataclasses import dataclass
-from typing import Optional, Union
+from typing import TYPE_CHECKING, Optional, Union
 from unittest.mock import Mock
 
 import pytest
 
-from mavedb.lib.permissions.actions import Action
 from mavedb.models.enums.contribution_role import ContributionRole
 from mavedb.models.enums.user_role import UserRole
 
+if TYPE_CHECKING:
+    from mavedb.lib.permissions.actions import Action
+
 
 @dataclass
 class PermissionTest:
@@ -33,7 +35,7 @@ class PermissionTest:
     entity_type: str
     entity_state: Optional[str]
     user_type: str
-    action: Action
+    action: "Action"
     should_be_permitted: Union[bool, str]
     expected_code: Optional[int] = None
     description: Optional[str] = None

tests/lib/permissions/test_collection.py

@@ -1,10 +1,14 @@
+# ruff: noqa: E402
+
 """Tests for Collection permissions module."""
 
+import pytest
+
+pytest.importorskip("fastapi", reason="Skipping permissions tests; FastAPI is required but not installed.")
+
 from typing import Callable, List
 from unittest import mock
 
-import pytest
-
 from mavedb.lib.permissions.actions import Action
 from mavedb.lib.permissions.collection import (
     _handle_add_badge_action,

tests/lib/permissions/test_core.py

@@ -1,9 +1,13 @@
-"""Tests for core permissions functionality."""
+# ruff: noqa: E402
 
-from unittest.mock import Mock, patch
+"""Tests for core permissions functionality."""
 
 import pytest
 
+pytest.importorskip("fastapi", reason="Skipping permissions tests; FastAPI is required but not installed.")
+
+from unittest.mock import Mock, patch
+
 from mavedb.lib.permissions import (
     assert_permission,
     collection,

tests/lib/permissions/test_experiment.py

@@ -1,10 +1,14 @@
+# ruff: noqa: E402
+
 """Tests for Experiment permissions module."""
 
+import pytest
+
+pytest.importorskip("fastapi", reason="Skipping permissions tests; FastAPI is required but not installed.")
+
 from typing import Callable, List
 from unittest import mock
 
-import pytest
-
 from mavedb.lib.permissions.actions import Action
 from mavedb.lib.permissions.experiment import (
     _handle_add_score_set_action,

tests/lib/permissions/test_experiment_set.py

@@ -1,10 +1,14 @@
+# ruff: noqa: E402
+
 """Tests for ExperimentSet permissions module."""
 
+import pytest
+
+pytest.importorskip("fastapi", reason="Skipping permissions tests; FastAPI is required but not installed.")
+
 from typing import Callable, List
 from unittest import mock
 
-import pytest
-
 from mavedb.lib.permissions.actions import Action
 from mavedb.lib.permissions.experiment_set import (
     _handle_add_experiment_action,

tests/lib/permissions/test_models.py

@@ -1,5 +1,11 @@
+# ruff: noqa: E402
+
 """Tests for permissions models module."""
 
+import pytest
+
+pytest.importorskip("fastapi", reason="Skipping permissions tests; FastAPI is required but not installed.")
+
 from mavedb.lib.permissions.models import PermissionResponse
 
 

tests/lib/permissions/test_score_calibration.py

@@ -1,10 +1,14 @@
+# ruff: noqa: E402
+
 """Tests for ScoreCalibration permissions module."""
 
+import pytest
+
+pytest.importorskip("fastapi", reason="Skipping permissions tests; FastAPI is required but not installed.")
+
 from typing import Callable, List
 from unittest import mock
 
-import pytest
-
 from mavedb.lib.permissions.actions import Action
 from mavedb.lib.permissions.score_calibration import (
     _handle_change_rank_action,

tests/lib/permissions/test_score_set.py

@@ -1,10 +1,14 @@
+# ruff: noqa: E402
+
 """Tests for ScoreSet permissions module."""
 
+import pytest
+
+pytest.importorskip("fastapi", reason="Skipping permissions tests; FastAPI is required but not installed.")
+
 from typing import Callable, List
 from unittest import mock
 
-import pytest
-
 from mavedb.lib.permissions.actions import Action
 from mavedb.lib.permissions.score_set import (
     _handle_delete_action,

tests/lib/permissions/test_user.py

@@ -1,10 +1,14 @@
+# ruff: noqa: E402
+
 """Tests for User permissions module."""
 
+import pytest
+
+pytest.importorskip("fastapi", reason="Skipping permissions tests; FastAPI is required but not installed.")
+
 from typing import Callable, List
 from unittest import mock
 
-import pytest
-
 from mavedb.lib.permissions.actions import Action
 from mavedb.lib.permissions.user import (
     _handle_add_role_action,

tests/lib/permissions/test_utils.py

@@ -1,9 +1,13 @@
-"""Tests for permissions utils module."""
+# ruff: noqa: E402
 
-from unittest.mock import Mock
+"""Tests for permissions utils module."""
 
 import pytest
 
+pytest.importorskip("fastapi", reason="Skipping permissions tests; FastAPI is required but not installed.")
+
+from unittest.mock import Mock
+
 from mavedb.lib.permissions.utils import deny_action_for_entity, roles_permitted
 from mavedb.models.enums.contribution_role import ContributionRole
 from mavedb.models.enums.user_role import UserRole