Skip to content

Commit

Permalink
fix: useTableSorting tests fails randomly, causing PR check to fail (#…
Browse files Browse the repository at this point in the history
…13631)

Co-authored-by: David Ovrelid <46874830+framitdavid@users.noreply.github.com>
  • Loading branch information
Jondyr and framitdavid committed Sep 25, 2024
1 parent 49d1da7 commit 6b0343b
Showing 1 changed file with 6 additions and 6 deletions.
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
import { useTableSorting } from './useTableSorting';
import { renderHook, waitFor } from '@testing-library/react';
import { act, renderHook } from '@testing-library/react';
import type { Rows } from '../components';

describe('useTableSorting', () => {
Expand Down Expand Up @@ -28,7 +28,7 @@ describe('useTableSorting', () => {

it('should sort rows in ascending order when a column is clicked', async () => {
const { result } = renderHook(() => useTableSorting(rows, { enable: true }));
await waitFor(() => result.current.handleSorting('creator'));
act(() => result.current.handleSorting('creator'));

const creatorsAscending: string[] = [];
result.current.sortedRows.forEach((row) => {
Expand All @@ -42,8 +42,8 @@ describe('useTableSorting', () => {

it('should sort rows in descending order when the same column is clicked again', async () => {
const { result } = renderHook(() => useTableSorting(rows, { enable: true }));
await waitFor(() => result.current.handleSorting('creator'));
await waitFor(() => result.current.handleSorting('creator'));
act(() => result.current.handleSorting('creator'));
act(() => result.current.handleSorting('creator'));

const creatorsDescending: string[] = [];
result.current.sortedRows.forEach((row) => {
Expand All @@ -57,8 +57,8 @@ describe('useTableSorting', () => {

it('should reset the sort direction to ascending when a different column is clicked', async () => {
const { result } = renderHook(() => useTableSorting(rows, { enable: true }));
await waitFor(() => result.current.handleSorting('creator'));
await waitFor(() => result.current.handleSorting('id'));
act(() => result.current.handleSorting('creator'));
act(() => result.current.handleSorting('id'));
expect(result.current.sortedRows).toEqual(rows);
});

Expand Down

0 comments on commit 6b0343b

Please sign in to comment.