test: add some missing tests

Author: bra-i-am

src/authz-module/components/AuthZTitle.test.tsx

@@ -59,4 +59,32 @@ describe('AuthZTitle', () => {
       expect(onClick).toHaveBeenCalled();
     });
   });
+
+  it('renders action buttons with icons', () => {
+    const mockIcon = () => <span data-testid="mock-icon">Icon</span>;
+    const onClick = jest.fn();
+    const actions = [
+      { label: 'Save', icon: mockIcon, onClick },
+    ];
+
+    render(<AuthZTitle {...defaultProps} actions={actions} />);
+
+    const button = screen.getByRole('button', { name: 'Icon Save' });
+    expect(button).toBeInTheDocument();
+    expect(screen.getByTestId('mock-icon')).toBeInTheDocument();
+  });
+
+  it('renders ReactNode actions alongside button actions', () => {
+    const onClick = jest.fn();
+    const customAction = <div data-testid="custom-action">Custom Action</div>;
+    const actions = [
+      { label: 'Save', onClick },
+      customAction,
+    ];
+
+    render(<AuthZTitle {...defaultProps} actions={actions} />);
+
+    expect(screen.getByRole('button', { name: 'Save' })).toBeInTheDocument();
+    expect(screen.getByTestId('custom-action')).toBeInTheDocument();
+  });
 });

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

@@ -2,7 +2,9 @@ import { ReactNode } from 'react';
 import { act, renderHook, waitFor } from '@testing-library/react';
 import { QueryClient, QueryClientProvider } from '@tanstack/react-query';
 import { getAuthenticatedHttpClient } from '@edx/frontend-platform/auth';
-import { useLibrary, useTeamMembers } from './hooks';
+import {
+  useLibrary, useTeamMembers, useAddTeamMember, useTeamRoles,
+} from './hooks';
 
 jest.mock('@edx/frontend-platform/auth', () => ({
   getAuthenticatedHttpClient: jest.fn(),
@@ -30,6 +32,24 @@ const mockLibrary = {
   slug: 'test-library',
 };
 
+const mockTeamRoles = [
+  {
+    id: 'admin',
+    name: 'Administrator',
+    description: 'Full access to the library',
+  },
+  {
+    id: 'author',
+    name: 'Author',
+    description: 'Can create and edit content',
+  },
+  {
+    id: 'collaborator',
+    name: 'Collaborator',
+    description: 'Can view and comment on content',
+  },
+];
+
 const createWrapper = () => {
   const queryClient = new QueryClient({
     defaultOptions: {
@@ -113,13 +133,119 @@ describe('useLibrary', () => {
 
     const wrapper = createWrapper();
     try {
-      act(()=>{
+      act(() => {
         renderHook(() => useLibrary('lib123'), { wrapper });
-      })
+      });
     } catch (e) {
       expect(e).toEqual(new Error('Not found'));
     }
 
     expect(getAuthenticatedHttpClient).toHaveBeenCalled();
   });
 });
+
+describe('useAddTeamMember', () => {
+  beforeEach(() => {
+    jest.clearAllMocks();
+  });
+
+  it('successfully adds team members', async () => {
+    const mockResponse = {
+      completed: [
+        {
+          user: 'jdoe',
+          status: 'role_added',
+        },
+        {
+          user: '[email protected]',
+          status: 'already_has_role',
+        },
+      ],
+      errors: [],
+    };
+
+    getAuthenticatedHttpClient.mockReturnValue({
+      put: jest.fn().mockResolvedValue({ data: mockResponse }),
+    });
+
+    const { result } = renderHook(() => useAddTeamMember(), {
+      wrapper: createWrapper(),
+    });
+
+    const addTeamMemberData = {
+      scope: 'lib:123',
+      users: ['jdoe'],
+      role: 'author',
+    };
+
+    await act(async () => {
+      result.current.mutate({ data: addTeamMemberData });
+    });
+
+    await waitFor(() => expect(result.current.isSuccess).toBe(true));
+
+    expect(getAuthenticatedHttpClient).toHaveBeenCalled();
+    expect(result.current.data).toEqual(mockResponse);
+  });
+
+  it('handles error when adding team members fails', async () => {
+    getAuthenticatedHttpClient.mockReturnValue({
+      put: jest.fn().mockRejectedValue(new Error('Failed to add members')),
+    });
+
+    const { result } = renderHook(() => useAddTeamMember(), {
+      wrapper: createWrapper(),
+    });
+
+    const addTeamMemberData = {
+      scope: 'lib:123',
+      users: ['jdoe'],
+      role: 'author',
+    };
+
+    await act(async () => {
+      result.current.mutate({ data: addTeamMemberData });
+    });
+
+    await waitFor(() => expect(result.current.isError).toBe(true));
+
+    expect(getAuthenticatedHttpClient).toHaveBeenCalled();
+    expect(result.current.error).toEqual(new Error('Failed to add members'));
+  });
+});
+
+describe('useTeamRoles', () => {
+  beforeEach(() => {
+    jest.clearAllMocks();
+  });
+
+  it('returns team roles when API call succeeds', async () => {
+    getAuthenticatedHttpClient.mockReturnValue({
+      get: jest.fn().mockResolvedValue({ data: mockTeamRoles }),
+    });
+
+    const { result } = renderHook(() => useTeamRoles('lib:123'), {
+      wrapper: createWrapper(),
+    });
+
+    await waitFor(() => expect(result.current.data).toEqual(mockTeamRoles));
+
+    expect(getAuthenticatedHttpClient).toHaveBeenCalled();
+  });
+
+  it('throws on error when API call fails', () => {
+    getAuthenticatedHttpClient.mockReturnValue({
+      get: jest.fn().mockRejectedValue(new Error('Failed to fetch roles')),
+    });
+
+    try {
+      act(() => {
+        renderHook(() => useTeamRoles('lib:123'), { wrapper: createWrapper() });
+      });
+    } catch (e) {
+      expect(e).toEqual(new Error('Failed to fetch roles'));
+    }
+
+    expect(getAuthenticatedHttpClient).toHaveBeenCalled();
+  });
+});