fix(BA-4525): Update scope adapter/handler tests for GLOBAL removal

- Change GLOBAL scope adapter test to verify NotImplementedError is raised
- Fix search_scopes handler test to assert on element_type instead of scope_type

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

Author: fregataa

tests/unit/manager/api/rbac/test_scope_adapter.py

@@ -216,31 +216,17 @@ def test_build_querier_user_scope(self, adapter: ScopeAdapter) -> None:
 
         assert len(querier.conditions) == 1
 
-    def test_build_querier_global_scope(self, adapter: ScopeAdapter) -> None:
-        """Test building querier for global scope returns empty querier."""
-        limit = 10
-        offset = 0
+    def test_build_querier_global_scope_raises(self, adapter: ScopeAdapter) -> None:
+        """Test building querier for global scope raises NotImplementedError."""
         request = SearchScopesRequest(
-            filter=ScopeFilter(
-                name=StringFilter(
-                    i_contains="anything",
-                )
-            ),
-            order=[
-                ScopeOrder(
-                    field=ScopeOrderField.NAME,
-                    direction=OrderDirection.ASC,
-                )
-            ],
-            limit=limit,
-            offset=offset,
+            filter=None,
+            order=[],
+            limit=10,
+            offset=0,
         )
 
-        querier = adapter.build_querier(ScopeType.GLOBAL, request)
-
-        # Global scope ignores filters and orders
-        assert querier.conditions == []
-        assert querier.orders == []
+        with pytest.raises(NotImplementedError):
+            adapter.build_querier(ScopeType.GLOBAL, request)
 
 
 class TestScopeAdapterConvertToDTO:

tests/unit/manager/api/rbac/test_scope_handlers.py

@@ -257,7 +257,7 @@ async def test_search_scopes_calls_processor_with_action(
         mock_permission_controller.search_scopes.wait_for_complete.assert_called_once()
         call_args = mock_permission_controller.search_scopes.wait_for_complete.call_args
         action = call_args[0][0]
-        assert action.scope_type == self.TEST_SCOPE_TYPE
+        assert action.element_type == self.TEST_SCOPE_TYPE.to_element()
 
     async def test_search_scopes_rejects_non_superadmin(
         self,