test: improve test coverage

Author: bra-i-am

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

@@ -4,6 +4,7 @@ import { QueryClient, QueryClientProvider } from '@tanstack/react-query';
 import { getAuthenticatedHttpClient } from '@edx/frontend-platform/auth';
 import {
   useLibrary, usePermissionsByRole, useTeamMembers, useAssignTeamMembersRole, useRevokeUserRoles,
+  useValidateUsers,
 } from './hooks';
 
 jest.mock('@edx/frontend-platform/auth', () => ({
@@ -241,6 +242,54 @@ describe('usePermissionsByRole', () => {
   });
 });
 
+describe('useValidateUsers', () => {
+  beforeEach(() => {
+    jest.clearAllMocks();
+  });
+
+  it('successfully validates users', async () => {
+    const mockResponse = {
+      validUsers: ['jdoe'],
+      invalidUsers: ['unknown_user'],
+    };
+
+    getAuthenticatedHttpClient.mockReturnValue({
+      post: jest.fn().mockResolvedValue({ data: mockResponse }),
+    });
+
+    const { result } = renderHook(() => useValidateUsers(), {
+      wrapper: createWrapper(),
+    });
+
+    await act(async () => {
+      result.current.mutate({ data: { users: ['jdoe', 'unknown_user'] } });
+    });
+
+    await waitFor(() => expect(result.current.isSuccess).toBe(true));
+
+    expect(getAuthenticatedHttpClient).toHaveBeenCalled();
+    expect(result.current.data).toEqual(mockResponse);
+  });
+
+  it('handles error when validation fails', async () => {
+    getAuthenticatedHttpClient.mockReturnValue({
+      post: jest.fn().mockRejectedValue(new Error('Validation failed')),
+    });
+
+    const { result } = renderHook(() => useValidateUsers(), {
+      wrapper: createWrapper(),
+    });
+
+    await act(async () => {
+      result.current.mutate({ data: { users: ['jdoe'] } });
+    });
+
+    await waitFor(() => expect(result.current.isError).toBe(true));
+
+    expect(result.current.error).toEqual(new Error('Validation failed'));
+  });
+});
+
 describe('useRevokeUserRoles', () => {
   beforeEach(() => {
     jest.clearAllMocks();

src/authz-module/libraries-manager/LibrariesTeamManager.test.tsx

@@ -3,11 +3,18 @@ import userEvent from '@testing-library/user-event';
 import { renderWrapper } from '@src/setupTest';
 import { initializeMockApp } from '@edx/frontend-platform/testing';
 import { useLibrary, useUpdateLibrary } from '@src/authz-module/data/hooks';
+import { useNavigate, useLocation } from 'react-router-dom';
 import { useLibraryAuthZ } from './context';
 import LibrariesTeamManager from './LibrariesTeamManager';
 import { ToastManagerProvider } from './ToastManagerContext';
 import { CONTENT_LIBRARY_PERMISSIONS } from './constants';
 
+jest.mock('react-router-dom', () => ({
+  ...jest.requireActual('react-router-dom'),
+  useNavigate: jest.fn(),
+  useLocation: jest.fn().mockReturnValue({ hash: '' }),
+}));
+
 jest.mock('./context', () => {
   const actual = jest.requireActual('./context');
   return {
@@ -53,6 +60,7 @@ describe('LibrariesTeamManager', () => {
     allowPublicRead: false,
   };
   const mutate = jest.fn();
+  const mockNavigate = jest.fn();
   const libraryAuthZContext = {
     libraryId: libraryData.id,
     libraryName: libraryData.title,
@@ -90,6 +98,8 @@ describe('LibrariesTeamManager', () => {
       mutate,
       isPending: false,
     });
+    (useNavigate as jest.Mock).mockReturnValue(mockNavigate);
+    (useLocation as jest.Mock).mockReturnValue({ hash: '' });
   });
 
   it('renders tabs and layout content correctly', () => {
@@ -136,7 +146,7 @@ describe('LibrariesTeamManager', () => {
     const permissionsTab = await screen.findByRole('tab', { name: /permissions/i });
     await user.click(permissionsTab);
 
-    const tablePermissionMatrix = await screen.getByRole('table');
+    const tablePermissionMatrix = screen.getByRole('table');
     const matrixScope = within(tablePermissionMatrix);
 
     expect(matrixScope.getByText('Library')).toBeInTheDocument();
@@ -152,4 +162,11 @@ describe('LibrariesTeamManager', () => {
     // TODO: Update expected URL when dedicated Manage Access page is created
     expect(navLink).toHaveAttribute('href', '/authz/libraries/lib-001');
   });
+
+  it('navigates to assign role wizard when "Assign Role" button is clicked', async () => {
+    const user = userEvent.setup();
+    renderTeamManager();
+    await user.click(screen.getByRole('button', { name: 'Assign Role' }));
+    expect(mockNavigate).toHaveBeenCalledWith('/authz/assign-role?scope=lib-001');
+  });
 });