From afeb743ec380ac08a358efa5b8a87e6b1715c348 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Nicole=20Miko=C5=82ajczyk?= Date: Mon, 5 May 2025 21:43:59 +0200 Subject: [PATCH] pl-fe: remove unused code MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Signed-off-by: Nicole Mikołajczyk --- .../api/hooks/groups/use-group-media.test.ts | 44 ------------------- .../src/api/hooks/groups/use-group-media.ts | 16 ------- packages/pl-fe/src/entity-store/entities.ts | 3 -- 3 files changed, 63 deletions(-) delete mode 100644 packages/pl-fe/src/api/hooks/groups/use-group-media.test.ts delete mode 100644 packages/pl-fe/src/api/hooks/groups/use-group-media.ts diff --git a/packages/pl-fe/src/api/hooks/groups/use-group-media.test.ts b/packages/pl-fe/src/api/hooks/groups/use-group-media.test.ts deleted file mode 100644 index 3e27a9be5..000000000 --- a/packages/pl-fe/src/api/hooks/groups/use-group-media.test.ts +++ /dev/null @@ -1,44 +0,0 @@ -import { __stub } from 'pl-fe/api'; -import { buildStatus } from 'pl-fe/jest/factory'; -import { renderHook, waitFor } from 'pl-fe/jest/test-helpers'; - -import { useGroupMedia } from './use-group-media'; - -const status = buildStatus(); -const groupId = '1'; - -describe('useGroupMedia hook', () => { - describe('with a successful request', () => { - beforeEach(() => { - __stub((mock) => { - mock.onGet(`/api/v1/timelines/group/${groupId}?only_media=true`).reply(200, [status]); - }); - }); - - it('is successful', async () => { - const { result } = renderHook(() => useGroupMedia(groupId)); - - await waitFor(() => expect(result.current.isFetching).toBe(false)); - - expect(result.current.entities.length).toBe(1); - expect(result.current.entities[0].id).toBe(status.id); - }); - }); - - describe('with an unsuccessful query', () => { - beforeEach(() => { - __stub((mock) => { - mock.onGet(`/api/v1/timelines/group/${groupId}?only_media=true`).networkError(); - }); - }); - - it('is has error state', async() => { - const { result } = renderHook(() => useGroupMedia(groupId)); - - await waitFor(() => expect(result.current.isFetching).toBe(false)); - - expect(result.current.entities.length).toBe(0); - expect(result.current.isError).toBeTruthy(); - }); - }); -}); diff --git a/packages/pl-fe/src/api/hooks/groups/use-group-media.ts b/packages/pl-fe/src/api/hooks/groups/use-group-media.ts deleted file mode 100644 index d2b413483..000000000 --- a/packages/pl-fe/src/api/hooks/groups/use-group-media.ts +++ /dev/null @@ -1,16 +0,0 @@ -import { Entities } from 'pl-fe/entity-store/entities'; -import { useEntities } from 'pl-fe/entity-store/hooks/use-entities'; -import { useClient } from 'pl-fe/hooks/use-client'; -import { normalizeStatus } from 'pl-fe/normalizers/status'; - -const useGroupMedia = (groupId: string) => { - const client = useClient(); - - return useEntities( - [Entities.STATUSES, 'groupMedia', groupId], - () => client.timelines.groupTimeline(groupId, { only_media: true }), - { transform: normalizeStatus }) - ; -}; - -export { useGroupMedia }; diff --git a/packages/pl-fe/src/entity-store/entities.ts b/packages/pl-fe/src/entity-store/entities.ts index f4eee8d2b..236fc76a4 100644 --- a/packages/pl-fe/src/entity-store/entities.ts +++ b/packages/pl-fe/src/entity-store/entities.ts @@ -1,7 +1,6 @@ import type { GroupMember, GroupRelationship, Relationship } from 'pl-api'; import type { Account } from 'pl-fe/normalizers/account'; import type { Group } from 'pl-fe/normalizers/group'; -import type { Status } from 'pl-fe/normalizers/status'; enum Entities { ACCOUNTS = 'Accounts', @@ -9,7 +8,6 @@ enum Entities { GROUP_MEMBERSHIPS = 'GroupMemberships', GROUP_RELATIONSHIPS = 'GroupRelationships', RELATIONSHIPS = 'Relationships', - STATUSES = 'Statuses', } interface EntityTypes { @@ -18,7 +16,6 @@ interface EntityTypes { [Entities.GROUP_MEMBERSHIPS]: GroupMember; [Entities.GROUP_RELATIONSHIPS]: GroupRelationship; [Entities.RELATIONSHIPS]: Relationship; - [Entities.STATUSES]: Status; } export { Entities, type EntityTypes };