From 652fb6b1804502cd5712d2f33b1a9b284217c9be Mon Sep 17 00:00:00 2001 From: Felipe Cardoso Date: Fri, 7 Nov 2025 07:55:05 +0100 Subject: [PATCH] Remove obsolete tests and unused imports in organization members and management components - Deleted `OrganizationMembersPage` tests as they are no longer relevant. - Removed unused `updated_at` property from test data in multiple components. - Cleaned up redundant `useAuth` imports from `OrganizationMembersContent` and `OrganizationManagementContent`. --- .../OrganizationManagementContent.tsx | 1 - .../OrganizationMembersContent.tsx | 1 - .../organizations/[id]/members/page.test.tsx | 72 ------------------- .../OrganizationActionMenu.test.tsx | 1 - .../OrganizationFormDialog.test.tsx | 1 - .../OrganizationListTable.test.tsx | 2 - 6 files changed, 78 deletions(-) delete mode 100644 frontend/tests/app/admin/organizations/[id]/members/page.test.tsx diff --git a/frontend/src/components/admin/organizations/OrganizationManagementContent.tsx b/frontend/src/components/admin/organizations/OrganizationManagementContent.tsx index 978772f..61ed7ce 100644 --- a/frontend/src/components/admin/organizations/OrganizationManagementContent.tsx +++ b/frontend/src/components/admin/organizations/OrganizationManagementContent.tsx @@ -9,7 +9,6 @@ import { useState, useCallback } from 'react'; import { useSearchParams, useRouter } from 'next/navigation'; import { Plus } from 'lucide-react'; import { Button } from '@/components/ui/button'; -import { useAuth } from '@/lib/auth/AuthContext'; import { useAdminOrganizations, type Organization, diff --git a/frontend/src/components/admin/organizations/OrganizationMembersContent.tsx b/frontend/src/components/admin/organizations/OrganizationMembersContent.tsx index c76ede9..592cb9b 100644 --- a/frontend/src/components/admin/organizations/OrganizationMembersContent.tsx +++ b/frontend/src/components/admin/organizations/OrganizationMembersContent.tsx @@ -9,7 +9,6 @@ import { useState, useCallback } from 'react'; import { useSearchParams, useRouter } from 'next/navigation'; import { UserPlus } from 'lucide-react'; import { Button } from '@/components/ui/button'; -import { useAuth } from '@/lib/auth/AuthContext'; import { useOrganizationMembers, useGetOrganization, diff --git a/frontend/tests/app/admin/organizations/[id]/members/page.test.tsx b/frontend/tests/app/admin/organizations/[id]/members/page.test.tsx deleted file mode 100644 index 660ef2b..0000000 --- a/frontend/tests/app/admin/organizations/[id]/members/page.test.tsx +++ /dev/null @@ -1,72 +0,0 @@ -/** - * Tests for Organization Members Page - */ - -import React from 'react'; -import { render, screen } from '@testing-library/react'; -import OrganizationMembersPage from '@/app/admin/organizations/[id]/members/page'; - -// Mock Next.js navigation -jest.mock('next/navigation', () => ({ - useSearchParams: jest.fn(() => new URLSearchParams()), - useRouter: jest.fn(() => ({ - push: jest.fn(), - replace: jest.fn(), - })), -})); - -// Mock AuthContext -jest.mock('@/lib/auth/AuthContext', () => ({ - useAuth: jest.fn(() => ({ - user: { id: '1', email: 'admin@test.com', is_superuser: true }, - })), -})); - -// Mock hooks -jest.mock('@/lib/api/hooks/useAdmin', () => ({ - useOrganizationMembers: jest.fn(() => ({ - data: { data: [], pagination: { total: 0, page: 1, page_size: 20, total_pages: 1, has_next: false, has_prev: false } }, - isLoading: false, - })), - useGetOrganization: jest.fn(() => ({ - data: { id: 'org-1', name: 'Test Org', slug: 'test-org', description: '', is_active: true, created_at: '2025-01-01', updated_at: '2025-01-01', member_count: 0 }, - isLoading: false, - })), -})); - -// Mock child components -jest.mock('@/components/admin/organizations/OrganizationMembersContent', () => ({ - OrganizationMembersContent: ({ organizationId }: any) => ( -
Organization Members Content for {organizationId}
- ), -})); - -describe('OrganizationMembersPage', () => { - const mockParams = { id: 'org-123' }; - - it('renders organization members page', () => { - render(); - - expect(screen.getByTestId('organization-members-content')).toBeInTheDocument(); - }); - - it('passes organization ID to content component', () => { - render(); - - expect(screen.getByText(/org-123/)).toBeInTheDocument(); - }); - - it('renders back button link', () => { - const { container } = render(); - - const backLink = container.querySelector('a[href="/admin/organizations"]'); - expect(backLink).toBeInTheDocument(); - }); - - it('renders container with proper spacing', () => { - const { container } = render(); - - const mainContainer = container.querySelector('.container'); - expect(mainContainer).toBeInTheDocument(); - }); -}); diff --git a/frontend/tests/components/admin/organizations/OrganizationActionMenu.test.tsx b/frontend/tests/components/admin/organizations/OrganizationActionMenu.test.tsx index 1196ce8..8852de4 100644 --- a/frontend/tests/components/admin/organizations/OrganizationActionMenu.test.tsx +++ b/frontend/tests/components/admin/organizations/OrganizationActionMenu.test.tsx @@ -35,7 +35,6 @@ describe('OrganizationActionMenu', () => { description: 'Leading provider', is_active: true, created_at: '2025-01-01T00:00:00Z', - updated_at: '2025-01-01T00:00:00Z', member_count: 10, }; diff --git a/frontend/tests/components/admin/organizations/OrganizationFormDialog.test.tsx b/frontend/tests/components/admin/organizations/OrganizationFormDialog.test.tsx index 1cc0b8f..a31c0e1 100644 --- a/frontend/tests/components/admin/organizations/OrganizationFormDialog.test.tsx +++ b/frontend/tests/components/admin/organizations/OrganizationFormDialog.test.tsx @@ -42,7 +42,6 @@ describe('OrganizationFormDialog', () => { description: 'Test description', is_active: true, created_at: '2025-01-01', - updated_at: '2025-01-01', member_count: 5, }; diff --git a/frontend/tests/components/admin/organizations/OrganizationListTable.test.tsx b/frontend/tests/components/admin/organizations/OrganizationListTable.test.tsx index 8ce52c7..213b836 100644 --- a/frontend/tests/components/admin/organizations/OrganizationListTable.test.tsx +++ b/frontend/tests/components/admin/organizations/OrganizationListTable.test.tsx @@ -25,7 +25,6 @@ describe('OrganizationListTable', () => { description: 'Leading provider of innovative solutions', is_active: true, created_at: '2025-01-01T00:00:00Z', - updated_at: '2025-01-01T00:00:00Z', member_count: 15, }, { @@ -34,7 +33,6 @@ describe('OrganizationListTable', () => { description: null, is_active: false, created_at: '2025-01-15T00:00:00Z', - updated_at: '2025-01-15T00:00:00Z', member_count: 3, }, ];