refactor: update types import

Author: dcoa

src/authz-module/audit-user/index.tsx

@@ -22,7 +22,7 @@ import {
 } from '@src/authz-module/components/TableCells';
 import { useQuerySettings } from '@src/authz-module/hooks/useQuerySettings';
 import { useRevokeUserRoles, useUserAssignedRoles } from '@src/authz-module/data/hooks';
-import { RoleToDelete } from 'types';
+import { RoleToDelete } from '@src/types';
 import { useToastManager } from '@src/components/ToastManager/ToastManagerContext';
 import UserPermissions from '@src/authz-module/components/UserPermissions';
 import OrgFilter from '@src/authz-module/components/TableControlBar/OrgFilter';

src/authz-module/components/RenderPermissionColumn.tsx

@@ -1,5 +1,5 @@
 import { Icon } from '@openedx/paragon';
-import { PermissionMetadata, ResourceMetadata } from 'types';
+import { PermissionMetadata, ResourceMetadata } from '@src/types';
 import ResourceTooltip from './ResourceTooltip';
 
 export type PermissionItem = ResourceMetadata & {

src/authz-module/constants.ts

@@ -1,4 +1,4 @@
-import { PermissionMetadata, ResourceMetadata, Role } from 'types';
+import { PermissionMetadata, ResourceMetadata, Role } from '@src/types';
 import {
   School, LibraryBooks, Article, Group, LocalOffer,
   BookOpen,

src/authz-module/data/hooks.test.tsx

@@ -87,42 +87,6 @@ const mockOrgs = {
   ],
 };
 
-const mockScopes = {
-  count: 2,
-  next: null,
-  previous: null,
-  results: [
-    {
-      externalKey: 'course-v1:OpenedX+DemoX+DemoCourse',
-      displayName: 'Open edX Demo Course',
-      org: {
-        id: 1,
-        created: '2026-04-02T19:30:36.779095Z',
-        modified: '2026-04-02T19:30:36.779095Z',
-        name: 'OpenedX',
-        shortName: 'OpenedX',
-        description: '',
-        logo: null,
-        active: true,
-      },
-    },
-    {
-      externalKey: 'lib:WGU:CSPROB',
-      displayName: 'Computer Science Problems',
-      org: {
-        id: 2,
-        created: '2026-04-02T19:31:21.196446Z',
-        modified: '2026-04-02T19:31:21.196446Z',
-        name: 'WGU',
-        shortName: 'WGU',
-        description: '',
-        logo: null,
-        active: true,
-      },
-    },
-  ],
-};
-
 const mockQuerySettings: QuerySettings = {
   roles: null,
   scopes: null,
@@ -558,26 +522,33 @@ describe('useScopes', () => {
     await waitFor(() => expect(result.current.isError).toBe(true));
     expect(result.current.error).toBeDefined();
   });
+
+  it('handles search parameter', async () => {
+    mockHttpClient().mockReturnValue({
+      get: jest.fn().mockResolvedValue({ data: makeScopesResponse() }),
+    });
+
+    const { result } = renderHook(() => useScopes({ search: 'library' }), { wrapper: createWrapper() });
+
+    await waitFor(() => expect(result.current.isSuccess).toBe(true));
+    expect(result.current.data?.pages[0].results).toHaveLength(1);
+  });
 });
 
 describe('useOrgs', () => {
   beforeEach(() => {
     jest.clearAllMocks();
   });
 
-  const mockOrgsResult = [{
-    id: 1, name: 'Org One', shortName: 'org1', description: '', logo: null, active: true,
-  }];
-
   it('returns organizations on success', async () => {
     mockHttpClient().mockReturnValue({
-      get: jest.fn().mockResolvedValue({ data: { results: mockOrgsResult } }),
+      get: jest.fn().mockResolvedValue({ data: mockOrgs }),
     });
 
     const { result } = renderHook(() => useOrgs(), { wrapper: createWrapper() });
 
     await waitFor(() => expect(result.current.isSuccess).toBe(true));
-    expect(result.current.data?.results).toEqual(mockOrgsResult);
+    expect(result.current.data?.results).toEqual(mockOrgs.results);
   });
 
   it('handles error when API fails', async () => {
@@ -789,16 +760,7 @@ describe('useAllRoleAssignments', () => {
 
   it('fetches and returns role assignments', async () => {
     const { result } = renderHook(
-      () => useAllRoleAssignments({
-        roles: null,
-        scopes: null,
-        organizations: null,
-        search: null,
-        order: null,
-        sortBy: null,
-        pageSize: 10,
-        pageIndex: 0,
-      }),
+      () => useAllRoleAssignments(mockQuerySettings),
       { wrapper: createWrapper() },
     );
     await waitFor(() => {
@@ -817,16 +779,7 @@ describe('useAllRoleAssignments', () => {
       })),
     });
     const { result } = renderHook(
-      () => useAllRoleAssignments({
-        roles: null,
-        scopes: null,
-        organizations: null,
-        search: null,
-        order: null,
-        sortBy: null,
-        pageSize: 10,
-        pageIndex: 0,
-      }),
+      () => useAllRoleAssignments(mockQuerySettings),
       { wrapper: createWrapper() },
     );
     await waitFor(() => {
@@ -836,70 +789,6 @@ describe('useAllRoleAssignments', () => {
   });
 });
 
-describe('useOrgs', () => {
-  beforeEach(() => {
-    mockHttpClient().mockReturnValue({
-      get: jest.fn(() => Promise.resolve({ data: mockOrgs })),
-    });
-  });
-
-  it('fetches and returns organizations', async () => {
-    const { result } = renderHook(() => useOrgs(), { wrapper: createWrapper() });
-    await waitFor(() => {
-      expect(result.current.data?.results).toHaveLength(2);
-      expect(result.current.data?.results[0].name).toBe('Organization 1');
-      expect(result.current.data?.count).toBe(2);
-    });
-  });
-
-  it('handles empty results', async () => {
-    mockHttpClient().mockReturnValueOnce({
-      get: jest.fn(() => Promise.resolve({
-        data: {
-          count: 0, next: null, previous: null, results: [],
-        },
-      })),
-    });
-    const { result } = renderHook(() => useOrgs(), { wrapper: createWrapper() });
-    await waitFor(() => {
-      expect(result.current.data?.results).toEqual([]);
-      expect(result.current.data?.count).toBe(0);
-    });
-  });
-});
-
-describe('useScopes', () => {
-  beforeEach(() => {
-    mockHttpClient().mockReturnValue({
-      get: jest.fn(() => Promise.resolve({ data: mockScopes })),
-    });
-  });
-
-  it('fetches and returns scopes', async () => {
-    const { result } = renderHook(() => useScopes(), { wrapper: createWrapper() });
-    await waitFor(() => {
-      expect(result.current.data?.pages[0].results).toHaveLength(2);
-      expect(result.current.data?.pages[0].results[0].displayName).toBe('Open edX Demo Course');
-      expect(result.current.data?.pages[0].count).toBe(2);
-    });
-  });
-
-  it('handles empty results', async () => {
-    mockHttpClient().mockReturnValueOnce({
-      get: jest.fn(() => Promise.resolve({
-        data: {
-          count: 0, next: null, previous: null, results: [],
-        },
-      })),
-    });
-    const { result } = renderHook(() => useScopes(), { wrapper: createWrapper() });
-    await waitFor(() => {
-      expect(result.current.data?.pages[0].results).toEqual([]);
-      expect(result.current.data?.pages[0].count).toBe(0);
-    });
-  });
-});
-
 describe('useUserAssignedRoles', () => {
   beforeEach(() => {
     jest.clearAllMocks();
@@ -1026,74 +915,3 @@ describe('useUserAssignedRoles', () => {
     expect(mockGet).toHaveBeenCalledTimes(2);
   });
 });
-
-describe('useScopes', () => {
-  const mockScopesData = {
-    count: 2,
-    next: null,
-    previous: null,
-    results: [
-      {
-        displayName: 'Test Library 1',
-        scope: 'lib:test-library-1',
-      },
-      {
-        displayName: 'Test Course 1',
-        scope: 'course:test-course-1',
-      },
-    ],
-  };
-
-  beforeEach(() => {
-    jest.clearAllMocks();
-  });
-
-  it('returns scopes when API call succeeds', async () => {
-    mockHttpClient().mockReturnValue({
-      get: jest.fn().mockResolvedValue({ data: mockScopesData }),
-    });
-
-    const { result } = renderHook(() => useScopes(), {
-      wrapper: createWrapper(),
-    });
-
-    await waitFor(() => expect(result.current.isSuccess).toBe(true));
-
-    expect(getAuthenticatedHttpClient).toHaveBeenCalled();
-    expect(result.current.data?.pages[0]).toEqual(mockScopesData);
-    expect(result.current.data?.pages[0].results).toHaveLength(2);
-  });
-
-  it('handles search parameter', async () => {
-    mockHttpClient().mockReturnValue({
-      get: jest.fn().mockResolvedValue({
-        data: {
-          count: 1, next: null, previous: null, results: [mockScopesData.results[0]],
-        },
-      }),
-    });
-
-    const { result } = renderHook(() => useScopes({ search: 'library' }), {
-      wrapper: createWrapper(),
-    });
-
-    await waitFor(() => expect(result.current.isSuccess).toBe(true));
-    expect(result.current.data?.pages[0].results).toHaveLength(1);
-  });
-
-  it('handles error when API call fails', async () => {
-    mockHttpClient().mockReturnValue({
-      get: jest.fn().mockRejectedValue(new Error('API failure')),
-    });
-
-    const { result } = renderHook(() => useScopes(), {
-      wrapper: createWrapper(),
-    });
-
-    await waitFor(() => expect(result.current.isError).toBe(true));
-
-    expect(getAuthenticatedHttpClient).toHaveBeenCalled();
-    expect(result.current.error).toBeDefined();
-    expect(result.current.data).toBeUndefined();
-  });
-});

src/authz-module/role-assignation-wizard/AssignRoleWizard.tsx

@@ -6,7 +6,7 @@ import {
   Stepper, Button, StatefulButton, Icon,
 } from '@openedx/paragon';
 import { SpinnerSimple } from '@openedx/paragon/icons';
-import { RoleMetadata } from 'types';
+import { RoleMetadata } from '@src/types';
 import { useToastManager } from '@src/components/ToastManager/ToastManagerContext';
 import SelectUsersAndRoleStep from './components/SelectUsersAndRoleStep';
 import DefineApplicationScopeStep from './components/DefineApplicationScopeStep';

src/authz-module/role-assignation-wizard/components/OrgSection.tsx

@@ -2,7 +2,7 @@ import { useState } from 'react';
 import { Icon } from '@openedx/paragon';
 import { ExpandLess, ExpandMore } from '@openedx/paragon/icons';
 import { useIntl } from '@edx/frontend-platform/i18n';
-import { Scope } from 'types';
+import { Scope } from '@src/types';
 import messages from '../messages';
 import ScopeCheckboxItem from './ScopeCheckboxItem';
 

src/authz-module/role-assignation-wizard/components/ScopeCheckboxItem.tsx

@@ -1,5 +1,5 @@
 import { Form } from '@openedx/paragon';
-import { Scope } from 'types';
+import { Scope } from '@src/types';
 
 interface ScopeCheckboxItemProps {
   scope: Scope;

src/authz-module/role-assignation-wizard/components/ScopeList.tsx

@@ -1,7 +1,7 @@
 import { useEffect, useRef } from 'react';
 import { Spinner } from '@openedx/paragon';
 import { useIntl } from '@edx/frontend-platform/i18n';
-import { Org, Scope } from 'types';
+import { Org, Scope } from '@src/types';
 import OrgSection from './OrgSection';
 import ScopeCheckboxItem from './ScopeCheckboxItem';
 import messages from '../messages';

src/authz-module/role-assignation-wizard/components/SelectUsersAndRoleStep.tsx

@@ -5,7 +5,7 @@ import {
   Tooltip,
 } from '@openedx/paragon';
 import { getConfig } from '@edx/frontend-platform';
-import { RoleMetadata } from 'types';
+import { RoleMetadata } from '@src/types';
 import HighlightedUsersInput from './HighlightedUsersInput';
 import messages from '../messages';
 

src/authz-module/roles-permissions/course/constants.ts

@@ -1,4 +1,4 @@
-import { PermissionMetadata, ResourceMetadata, RoleMetadata } from 'types';
+import { PermissionMetadata, ResourceMetadata, RoleMetadata } from '@src/types';
 import {
   LibraryBooks, Article, Group, LocalOffer,
   BookOpen,