| Select | : null} {columns.map((column, index) => ({column.title} | ))} @@ -89,6 +135,23 @@ vi.mock('antd', async () => { key={resolveRowKey(record, rowKey, rowIndex)} data-testid={`table-row-${resolveRowKey(record, rowKey, rowIndex)}`} > + {rowSelection ? ( ++ { + const rawKey = resolveRowKeyValue(record, rowKey, rowIndex) + const selectedKeys = rowSelection.selectedRowKeys ?? [] + const nextKeys = selectedKeys.map(String).includes(String(rawKey)) + ? selectedKeys.filter((value) => String(value) !== String(rawKey)) + : [...selectedKeys, rawKey] + rowSelection.onChange?.(nextKeys) + }} + /> + | + ) : null} {columns.map((column, columnIndex) => { const value = column.dataIndex ? record[column.dataIndex] : undefined const content = column.render ? column.render(value, record, rowIndex) : value @@ -115,6 +178,8 @@ vi.mock('@/services/users', () => ({ listUsers: (params: UserListParams) => listUsersMock(params), deleteUser: (id: number) => deleteUserMock(id), updateUserStatus: (id: number, payload: { status: UserStatus }) => updateUserStatusMock(id, payload), + batchUpdateStatus: (ids: number[], status: UserStatus) => batchUpdateStatusMock(ids, status), + batchDelete: (ids: number[]) => batchDeleteMock(ids), getUserRoles: (id: number) => getUserRolesMock(id), })) @@ -304,6 +369,8 @@ describe('UsersPage', () => { listUsersMock.mockReset() deleteUserMock.mockReset() updateUserStatusMock.mockReset() + batchUpdateStatusMock.mockReset() + batchDeleteMock.mockReset() getUserRolesMock.mockReset() listRolesMock.mockReset() @@ -339,6 +406,16 @@ describe('UsersPage', () => { )) }) + batchUpdateStatusMock.mockImplementation(async (ids: number[], status: UserStatus) => { + currentUsers = currentUsers.map((user) => ( + ids.includes(user.id) ? { ...user, status } : user + )) + }) + + batchDeleteMock.mockImplementation(async (ids: number[]) => { + currentUsers = currentUsers.filter((user) => !ids.includes(user.id)) + }) + getUserRolesMock.mockImplementation(async (id: number) => ( id === 5 ? [roles[0], roles[1]] : [roles[1]] )) @@ -355,6 +432,7 @@ describe('UsersPage', () => { )) vi.spyOn(message, 'success').mockImplementation(() => undefined as never) vi.spyOn(message, 'error').mockImplementation(() => undefined as never) + vi.spyOn(message, 'warning').mockImplementation(() => undefined as never) }) afterEach(() => { @@ -501,4 +579,30 @@ describe('UsersPage', () => { await waitFor(() => expect(screen.getByText('admin-root')).toBeInTheDocument()) expect(listUsersMock).toHaveBeenCalledTimes(2) }) + + it('opens a stronger batch-delete confirmation and only deletes after explicit modal confirmation', async () => { + const user = userEvent.setup() + + render(
|---|