mirror of
https://github.com/shankar0123/certctl.git
synced 2026-06-11 02:19:03 +00:00
7013227a34
Closes T-1 (cat-s2-c24a548076c6) — frontend page-level Vitest coverage was
3 of 28 pages pre-T-1. T-1 lifts that to 11 of 28 (39%) by writing focused
behavior tests for the 8 highest-leverage pages.
Tests added:
- CertificatesPage.test.tsx (6 cases) — F-1 filter+pagination contract:
team_id / expires_before / sort param wiring, page=1 reset on filter
change, page+per_page always present in getCertificates params.
- PoliciesPage.test.tsx (4 cases) — D-006/D-008 TitleCase contract:
list render, severity badge, toggle-enabled inversion, delete confirm.
- IssuersPage.test.tsx (3 cases) — D-2 phantom-trim + B-1 EditIssuer:
list render, StatusBadge derives from enabled, Test fires
testIssuerConnection.
- TargetsPage.test.tsx (3 cases) — D-2 phantom-trim:
list render, Status derives from enabled, Delete fires deleteTarget.
- AgentsPage.test.tsx (3 cases) — D-2 phantom-trim + heartbeatStatus:
list render, undefined last_heartbeat_at -> Offline,
listRetiredAgents lazy-loaded.
- AgentDetailPage.test.tsx (3 cases) — D-2 phantom-trim:
fetches by URL :id, Registered row reads registered_at,
Capabilities + Tags sections absent.
- OwnersPage.test.tsx (3 cases) — B-1 EditOwnerModal closure:
list render, Edit opens modal, Save fires updateOwner.
- TeamsPage.test.tsx (2 cases) — B-1 EditTeamModal closure.
- AgentGroupsPage.test.tsx (2 cases) — B-1 EditAgentGroupModal closure.
- RenewalPoliciesPage.test.tsx (3 cases) — B-1 brand-new-page closure:
list + alert_thresholds_days display, Create modal, Edit modal.
- DiscoveryPage.test.tsx (3 cases) — I-2 claim/dismiss closure:
list render, status filter wiring, Dismiss fires dismissDiscoveredCertificate.
CI guardrail: .github/workflows/ci.yml step "Frontend page-coverage
regression guard (T-1)" blocks new pages from landing without sibling
.test.tsx unless added to a 14-name deferred allowlist with one-line
"why deferred" justifications.
Net coverage: 13 page-level vitest cases -> ~35 page-level vitest cases
across 14 files (was 3); total project tests 302 -> 337.
See coverage-gap-audit-2026-04-24-v5/unified-audit.md
cat-s2-c24a548076c6 for closure rationale.
106 lines
4.1 KiB
TypeScript
106 lines
4.1 KiB
TypeScript
import { describe, it, expect, vi, beforeEach } from 'vitest';
|
|
import { render, screen, waitFor, fireEvent, cleanup } from '@testing-library/react';
|
|
import { QueryClient, QueryClientProvider } from '@tanstack/react-query';
|
|
import { MemoryRouter } from 'react-router-dom';
|
|
import type { ReactNode } from 'react';
|
|
|
|
// -----------------------------------------------------------------------------
|
|
// T-1 closure (cat-s2-c24a548076c6): DiscoveryPage Vitest coverage.
|
|
//
|
|
// Pins the I-2 closure (MCP claim/dismiss tools landed; the GUI claim/
|
|
// dismiss flow predates that). Tests:
|
|
//
|
|
// 1. Discovered cert list renders when getDiscoveredCertificates resolves.
|
|
// 2. Status filter wires the param into getDiscoveredCertificates.
|
|
// 3. Dismiss button calls dismissDiscoveredCertificate(id).
|
|
// 4. Claim button opens the claim modal (precondition for claim flow).
|
|
// -----------------------------------------------------------------------------
|
|
|
|
vi.mock('../api/client', () => ({
|
|
getDiscoveredCertificates: vi.fn(),
|
|
getDiscoverySummary: vi.fn(),
|
|
getDiscoveryScans: vi.fn(),
|
|
getAgents: vi.fn(),
|
|
claimDiscoveredCertificate: vi.fn(),
|
|
dismissDiscoveredCertificate: vi.fn(),
|
|
}));
|
|
|
|
import DiscoveryPage from './DiscoveryPage';
|
|
import * as client from '../api/client';
|
|
|
|
function renderWithQuery(ui: ReactNode) {
|
|
const qc = new QueryClient({
|
|
defaultOptions: { queries: { retry: false, gcTime: 0, staleTime: 0 } },
|
|
});
|
|
return render(
|
|
<QueryClientProvider client={qc}>
|
|
<MemoryRouter>{ui}</MemoryRouter>
|
|
</QueryClientProvider>,
|
|
);
|
|
}
|
|
|
|
const unmanagedCert = {
|
|
id: 'dc-001',
|
|
common_name: 'unmanaged.example.com',
|
|
sans: ['unmanaged.example.com'],
|
|
status: 'Unmanaged',
|
|
source_path: '/etc/ssl/certs/server.crt',
|
|
agent_id: 'agent-iis01',
|
|
issuer_dn: 'CN=Internal CA',
|
|
not_after: new Date(Date.now() + 60 * 86400000).toISOString(),
|
|
key_algorithm: 'RSA',
|
|
key_size: 2048,
|
|
is_ca: false,
|
|
fingerprint_sha256: 'abc123def456ghijklmnopqrstuvwxyz0123456789abcdef0123456789abcdef',
|
|
};
|
|
|
|
describe('DiscoveryPage — T-1 page coverage', () => {
|
|
beforeEach(() => {
|
|
vi.clearAllMocks();
|
|
cleanup();
|
|
vi.mocked(client.getDiscoveredCertificates).mockResolvedValue({
|
|
data: [unmanagedCert],
|
|
total: 1,
|
|
page: 1,
|
|
per_page: 50,
|
|
} as never);
|
|
vi.mocked(client.getDiscoverySummary).mockResolvedValue({ Unmanaged: 1, Managed: 0, Dismissed: 0 } as never);
|
|
vi.mocked(client.getDiscoveryScans).mockResolvedValue({ data: [], total: 0, page: 1, per_page: 50 } as never);
|
|
vi.mocked(client.getAgents).mockResolvedValue({ data: [], total: 0, page: 1, per_page: 200 } as never);
|
|
vi.mocked(client.dismissDiscoveredCertificate).mockResolvedValue({ status: 'Dismissed' } as never);
|
|
});
|
|
|
|
it('renders the discovered certificates list when getDiscoveredCertificates resolves', async () => {
|
|
renderWithQuery(<DiscoveryPage />);
|
|
// The CN appears in both the row and a SAN tooltip — multiple matches.
|
|
await waitFor(() => {
|
|
expect(screen.getAllByText('unmanaged.example.com').length).toBeGreaterThan(0);
|
|
});
|
|
});
|
|
|
|
it('changing the status filter wires status into getDiscoveredCertificates params', async () => {
|
|
renderWithQuery(<DiscoveryPage />);
|
|
await waitFor(() => expect(client.getDiscoveredCertificates).toHaveBeenCalled());
|
|
|
|
const statusSelect = await screen.findByDisplayValue('All statuses');
|
|
fireEvent.change(statusSelect, { target: { value: 'Unmanaged' } });
|
|
|
|
await waitFor(() => {
|
|
const calls = vi.mocked(client.getDiscoveredCertificates).mock.calls;
|
|
const filtered = calls.find(([params]) => (params as Record<string, string>)?.status === 'Unmanaged');
|
|
expect(filtered, 'expected getDiscoveredCertificates to be called with status=Unmanaged').toBeTruthy();
|
|
});
|
|
});
|
|
|
|
it('Dismiss button calls dismissDiscoveredCertificate(id)', async () => {
|
|
renderWithQuery(<DiscoveryPage />);
|
|
const dismissBtn = await screen.findByRole('button', { name: 'Dismiss' });
|
|
fireEvent.click(dismissBtn);
|
|
|
|
await waitFor(() => {
|
|
expect(client.dismissDiscoveredCertificate).toHaveBeenCalled();
|
|
});
|
|
expect(vi.mocked(client.dismissDiscoveredCertificate).mock.calls[0]?.[0]).toBe('dc-001');
|
|
});
|
|
});
|