test: addint ut to improve coverage

Author: jacobo-dominguez-wgu

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

@@ -13,6 +13,10 @@ jest.mock('@edx/frontend-platform/auth', () => ({
   configure: jest.fn(),
 }));
 
+jest.mock('@edx/frontend-platform/logging', () => ({
+  logError: jest.fn(),
+}));
+
 const mockUser = {
   username: 'johndoe',
   email: '[email protected]',
@@ -273,6 +277,76 @@ describe('AuditUserPage', () => {
     });
   });
 
+  it('shows error toast when role revocation succeeds but returns errors', async () => {
+    (getAuthenticatedHttpClient as jest.Mock).mockReturnValue({
+      get: jest
+        .fn()
+        .mockResolvedValueOnce({ data: mockUser })
+        .mockResolvedValueOnce({ data: mockAssignments }),
+      delete: jest.fn().mockResolvedValue({
+        data: {
+          errors: ['Failed to revoke user role'],
+          completed: [],
+        },
+      }),
+    });
+
+    renderWithRouter();
+
+    await waitFor(() => {
+      expect(screen.getByRole('button', { name: /delete role action/i })).toBeInTheDocument();
+    });
+
+    const user = userEvent.setup();
+    const deleteButton = screen.getByRole('button', { name: /delete role action/i });
+    await user.click(deleteButton);
+
+    await waitFor(() => {
+      expect(screen.getByRole('dialog')).toBeInTheDocument();
+      expect(screen.getByRole('button', { name: /remove/i })).toBeInTheDocument();
+    });
+
+    const removeButton = screen.getByRole('button', { name: /remove/i });
+    await user.click(removeButton);
+
+    await waitFor(() => {
+      expect(screen.getByText(/something went wrong/i)).toBeInTheDocument();
+    });
+  });
+
+  it('shows error toast with retry when role revocation fails', async () => {
+    (getAuthenticatedHttpClient as jest.Mock).mockReturnValue({
+      get: jest
+        .fn()
+        .mockResolvedValueOnce({ data: mockUser })
+        .mockResolvedValueOnce({ data: mockAssignments }),
+      delete: jest.fn().mockRejectedValue(new Error('Network error')),
+    });
+
+    renderWithRouter();
+
+    await waitFor(() => {
+      expect(screen.getByRole('button', { name: /delete role action/i })).toBeInTheDocument();
+    });
+
+    const user = userEvent.setup();
+    const deleteButton = screen.getByRole('button', { name: /delete role action/i });
+    await user.click(deleteButton);
+
+    await waitFor(() => {
+      expect(screen.getByRole('dialog')).toBeInTheDocument();
+      expect(screen.getByRole('button', { name: /remove/i })).toBeInTheDocument();
+    });
+
+    const removeButton = screen.getByRole('button', { name: /remove/i });
+    await user.click(removeButton);
+
+    await waitFor(() => {
+      expect(screen.getByText(/something went wrong on our end/i)).toBeInTheDocument();
+      expect(screen.getByText(/try again later/i)).toBeInTheDocument();
+    });
+  });
+
   it('shows the extra warning when rolesCount is 1', async () => {
     (getAuthenticatedHttpClient as jest.Mock).mockReturnValue({
       get: jest

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

@@ -195,7 +195,7 @@ const AuditUserPage = () => {
           title: '',
         }}
         navLinks={navLinks}
-        activeLabel={username || ''}
+        activeLabel={user?.username || ''}
         pageTitle={user?.username || ''}
         pageSubtitle={user?.email || ''}
         actions={