From f111685c5886f3e77edfd3621c98d8ac1b9bcce4 Mon Sep 17 00:00:00 2001 From: Armand Philippot Date: Fri, 24 Nov 2023 20:00:08 +0100 Subject: refactor(services, types): reorganize GraphQL fetchers and data types The Typescript mapped types was useful for autocompletion in fetchers but their are harder to maintain. I think it's better to keep each query close to its fetcher to have a better understanding of the fetched data. So I: * colocate queries with their own fetcher * colocate mutations with their own mutator * remove Typescript mapped types for queries and mutations * move data convertors inside graphql services * rename most of data types and fetchers --- src/utils/hooks/use-posts-list/use-posts-list.test.ts | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) (limited to 'src/utils/hooks/use-posts-list/use-posts-list.test.ts') diff --git a/src/utils/hooks/use-posts-list/use-posts-list.test.ts b/src/utils/hooks/use-posts-list/use-posts-list.test.ts index 1d11111..ff69de2 100644 --- a/src/utils/hooks/use-posts-list/use-posts-list.test.ts +++ b/src/utils/hooks/use-posts-list/use-posts-list.test.ts @@ -1,13 +1,13 @@ import { describe, expect, it } from '@jest/globals'; import { act, renderHook } from '@testing-library/react'; -import { getArticles } from '../../../services/graphql'; +import { fetchPostsList } from '../../../services/graphql'; import { usePostsList } from './use-posts-list'; describe('usePostsList', () => { it('can return the first new result index when loading more posts', async () => { const perPage = 5; const { result } = renderHook(() => - usePostsList({ fetcher: getArticles, perPage }) + usePostsList({ fetcher: fetchPostsList, perPage }) ); expect.assertions(2); -- cgit v1.2.3