diff options
| author | Armand Philippot <git@armandphilippot.com> | 2023-12-01 13:26:44 +0100 |
|---|---|---|
| committer | Armand Philippot <git@armandphilippot.com> | 2023-12-01 17:23:19 +0100 |
| commit | dfdbf6cac1fe3719dc71e130129d28e04ba4e225 (patch) | |
| tree | f865bdad53cef95bdfb10fc04174a0173ab36f15 /src/utils/hooks/use-thematics-list | |
| parent | 5b762b1b669454a89899c4bdf6008027d9615acf (diff) | |
refactor(pages): refine Thematic pages
* add a table of contents (however posts heading are
not included)
* rename posts list section title
* add a useThematic hook to refresh thematic contents
* add a useThematicLists hook to refresh thematics list
* add a `notIn` filter in thematics list fetcher to directly
remove unwanted thematics
* add Cypress tests
Diffstat (limited to 'src/utils/hooks/use-thematics-list')
| -rw-r--r-- | src/utils/hooks/use-thematics-list/index.ts | 1 | ||||
| -rw-r--r-- | src/utils/hooks/use-thematics-list/use-thematics-list.test.ts | 48 | ||||
| -rw-r--r-- | src/utils/hooks/use-thematics-list/use-thematics-list.ts | 50 |
3 files changed, 99 insertions, 0 deletions
diff --git a/src/utils/hooks/use-thematics-list/index.ts b/src/utils/hooks/use-thematics-list/index.ts new file mode 100644 index 0000000..a886017 --- /dev/null +++ b/src/utils/hooks/use-thematics-list/index.ts @@ -0,0 +1 @@ +export * from './use-thematics-list'; diff --git a/src/utils/hooks/use-thematics-list/use-thematics-list.test.ts b/src/utils/hooks/use-thematics-list/use-thematics-list.test.ts new file mode 100644 index 0000000..0e19c2d --- /dev/null +++ b/src/utils/hooks/use-thematics-list/use-thematics-list.test.ts @@ -0,0 +1,48 @@ +import { + afterEach, + beforeEach, + describe, + expect, + it, + jest, +} from '@jest/globals'; +import { renderHook, waitFor } from '@testing-library/react'; +import { useThematicsList } from './use-thematics-list'; + +describe('useThematicsList', () => { + beforeEach(() => { + /* Not sure why it is needed, but without it Jest was complaining with + * `Jest worker encountered 4 child process exceptions`... Maybe because of + * useSWR? */ + jest.useFakeTimers({ + doNotFake: ['queueMicrotask'], + }); + }); + + afterEach(() => { + jest.runOnlyPendingTimers(); + jest.useRealTimers(); + }); + + /* eslint-disable max-statements */ + it('fetch the requested thematics list', async () => { + const { result } = renderHook(() => useThematicsList()); + + // Inaccurate assertions count because of waitFor... + //expect.assertions(8); + expect.hasAssertions(); + + expect(result.current.thematics).toBeUndefined(); + expect(result.current.isError).toBe(false); + expect(result.current.isLoading).toBe(true); + expect(result.current.isValidating).toBe(true); + + jest.advanceTimersToNextTimer(); + + await waitFor(() => expect(result.current.thematics).toBeDefined()); + expect(result.current.isError).toBe(false); + expect(result.current.isLoading).toBe(false); + expect(result.current.isValidating).toBe(false); + }); + /* eslint-enable max-statements */ +}); diff --git a/src/utils/hooks/use-thematics-list/use-thematics-list.ts b/src/utils/hooks/use-thematics-list/use-thematics-list.ts new file mode 100644 index 0000000..f63815a --- /dev/null +++ b/src/utils/hooks/use-thematics-list/use-thematics-list.ts @@ -0,0 +1,50 @@ +import useSWR from 'swr'; +import { + type FetchThematicsListInput, + fetchThematicsList, +} from '../../../services/graphql'; +import type { + GraphQLConnection, + Maybe, + WPThematicPreview, +} from '../../../types'; + +export type UseThematicsListReturn< + T extends Maybe<GraphQLConnection<WPThematicPreview>>, +> = { + isError: boolean; + isLoading: boolean; + isValidating: boolean; + thematics: T extends undefined + ? Maybe<GraphQLConnection<WPThematicPreview>> + : GraphQLConnection<WPThematicPreview>; +}; + +export type UseThematicsListConfig< + T extends Maybe<GraphQLConnection<WPThematicPreview>>, +> = { + input?: FetchThematicsListInput; + fallback?: T; +}; + +export const useThematicsList = < + T extends Maybe<GraphQLConnection<WPThematicPreview>>, +>( + config?: UseThematicsListConfig<T> +): UseThematicsListReturn<T> => { + const { fallback, input } = config ?? {}; + const { data, error, isLoading, isValidating } = useSWR( + input ?? {}, + fetchThematicsList, + { fallbackData: fallback } + ); + + if (error) console.error(error); + + return { + isError: !!error, + isLoading, + isValidating, + thematics: data, + } as UseThematicsListReturn<T>; +}; |
