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-thematic | |
| 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-thematic')
| -rw-r--r-- | src/utils/hooks/use-thematic/index.ts | 1 | ||||
| -rw-r--r-- | src/utils/hooks/use-thematic/use-thematic.test.ts | 56 | ||||
| -rw-r--r-- | src/utils/hooks/use-thematic/use-thematic.ts | 31 |
3 files changed, 88 insertions, 0 deletions
diff --git a/src/utils/hooks/use-thematic/index.ts b/src/utils/hooks/use-thematic/index.ts new file mode 100644 index 0000000..319f4b5 --- /dev/null +++ b/src/utils/hooks/use-thematic/index.ts @@ -0,0 +1 @@ +export * from './use-thematic'; diff --git a/src/utils/hooks/use-thematic/use-thematic.test.ts b/src/utils/hooks/use-thematic/use-thematic.test.ts new file mode 100644 index 0000000..43d0a57 --- /dev/null +++ b/src/utils/hooks/use-thematic/use-thematic.test.ts @@ -0,0 +1,56 @@ +import { + afterEach, + beforeEach, + describe, + expect, + it, + jest, +} from '@jest/globals'; +import { renderHook, waitFor } from '@testing-library/react'; +import { wpThematicsFixture } from '../../../../tests/fixtures'; +import { ROUTES } from '../../constants'; +import { useThematic } from './use-thematic'; + +describe('useThematic', () => { + 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 thematic', async () => { + const { result } = renderHook(() => + useThematic(wpThematicsFixture[0].slug) + ); + + // Inaccurate assertions count because of waitFor... + //expect.assertions(8); + expect.hasAssertions(); + + expect(result.current.thematic).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.thematic?.slug).toBe( + `${ROUTES.THEMATICS}/${wpThematicsFixture[0].slug}` + ) + ); + 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-thematic/use-thematic.ts b/src/utils/hooks/use-thematic/use-thematic.ts new file mode 100644 index 0000000..68127d2 --- /dev/null +++ b/src/utils/hooks/use-thematic/use-thematic.ts @@ -0,0 +1,31 @@ +import useSWR from 'swr'; +import { + convertWPThematicToThematic, + fetchThematic, +} from '../../../services/graphql'; +import type { Maybe, Thematic, WPThematic } from '../../../types'; + +export type UseThematicReturn<T extends Maybe<WPThematic>> = { + isError: boolean; + isLoading: boolean; + isValidating: boolean; + thematic: T extends undefined ? Maybe<Thematic> : Thematic; +}; + +export const useThematic = <T extends Maybe<WPThematic>>( + slug: string, + fallback?: T +): UseThematicReturn<T> => { + const { data, error, isLoading, isValidating } = useSWR(slug, fetchThematic, { + fallbackData: fallback, + }); + + if (error) console.error(error); + + return { + isError: !!error, + isLoading, + isValidating, + thematic: data ? convertWPThematicToThematic(data) : undefined, + } as UseThematicReturn<T>; +}; |
