Complete implementation of 7 slices addressing E2E testing gaps: Slice 0+1: Wire Actions + ReasonPrompt - FocusView now uses useActions hook instead of direct act() calls - Added pendingAction state pattern for skip/defer/complete actions - ReasonPrompt integration with proper confirm/cancel flow - Tags support in DecisionEntry interface Slice 2: Drag Reorder UI - Installed @dnd-kit (core, sortable, utilities) - QueueView with DndContext, SortableContext, verticalListSortingStrategy - SortableQueueItem wrapper component using useSortable hook - pendingReorder state with ReasonPrompt for reorder reasons - Cmd+Up/Down keyboard shortcuts for accessibility - Fixed: Store item ID in PendingReorder to avoid stale queue reference Slice 3: System Tray Integration - tray.rs with TrayState, setup_tray, toggle_window_visibility - Menu with Show/Quit items - Left-click toggles window visibility - update_tray_badge command updates tooltip with item count - Frontend wiring in AppShell Slice 4: E2E Test Updates - Fixed test selectors for InboxView, Queue badge - Exposed inbox store for test seeding Slice 5: Staleness Visualization - Already implemented in computeStaleness() with tests Slice 6: Quick Wiring - onStartBatch callback wired to QueueView - SyncStatus rendered in nav area - SettingsView renders Settings component Slice 7: State Persistence - settings-store with hydrate/update methods - Tauri backend integration via read_settings/write_settings - AppShell hydrates settings on mount Bug fixes from code review: - close_bead now has error isolation (try/catch) so decision logging and queue advancement continue even if bead close fails - PendingReorder stores item ID to avoid stale queue reference E2E tests for all ACs (tests/e2e/followup-acs.spec.ts): - AC-F1: Drag reorder (4 tests) - AC-F2: ReasonPrompt integration (7 tests) - AC-F5: Staleness visualization (3 tests) - AC-F6: Batch mode (2 tests) - AC-F7: SyncStatus (2 tests) - ReasonPrompt behavior (3 tests) Tests: 388 frontend + 119 Rust + 32 E2E all passing Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
149 lines
4.2 KiB
TypeScript
149 lines
4.2 KiB
TypeScript
import { describe, it, expect, beforeEach, vi } from "vitest";
|
|
import { render, screen, waitFor } from "@testing-library/react";
|
|
import { QueryClient, QueryClientProvider } from "@tanstack/react-query";
|
|
import { DebugView } from "@/components/DebugView";
|
|
import { setMockResponse, resetMocks } from "../mocks/tauri-api";
|
|
|
|
function renderWithProviders(ui: React.ReactElement) {
|
|
const queryClient = new QueryClient({
|
|
defaultOptions: {
|
|
queries: {
|
|
retry: false,
|
|
},
|
|
},
|
|
});
|
|
|
|
return render(
|
|
<QueryClientProvider client={queryClient}>{ui}</QueryClientProvider>
|
|
);
|
|
}
|
|
|
|
describe("DebugView", () => {
|
|
beforeEach(() => {
|
|
resetMocks();
|
|
});
|
|
|
|
it("shows loading state initially", () => {
|
|
renderWithProviders(<DebugView />);
|
|
expect(screen.getByText(/loading/i)).toBeInTheDocument();
|
|
});
|
|
|
|
it("displays raw JSON data when loaded", async () => {
|
|
const mockStatus = {
|
|
last_sync: "2026-02-26T12:00:00Z",
|
|
is_healthy: true,
|
|
message: "Lore is healthy",
|
|
summary: {
|
|
open_issues: 5,
|
|
authored_mrs: 2,
|
|
reviewing_mrs: 3,
|
|
},
|
|
};
|
|
|
|
setMockResponse("get_lore_status", mockStatus);
|
|
renderWithProviders(<DebugView />);
|
|
|
|
await waitFor(() => {
|
|
expect(screen.queryByText(/loading/i)).not.toBeInTheDocument();
|
|
});
|
|
|
|
// Check that the debug heading is present
|
|
expect(screen.getByText(/lore debug/i)).toBeInTheDocument();
|
|
|
|
// Check that the JSON is displayed (look for key properties)
|
|
expect(screen.getByText(/is_healthy/)).toBeInTheDocument();
|
|
expect(screen.getByText(/open_issues/)).toBeInTheDocument();
|
|
expect(screen.getByText(/reviewing_mrs/)).toBeInTheDocument();
|
|
});
|
|
|
|
it("shows error state when fetch fails", async () => {
|
|
setMockResponse("get_lore_status", Promise.reject(new Error("Connection failed")));
|
|
renderWithProviders(<DebugView />);
|
|
|
|
await waitFor(() => {
|
|
expect(screen.queryByText(/loading/i)).not.toBeInTheDocument();
|
|
});
|
|
|
|
expect(screen.getByText(/error/i)).toBeInTheDocument();
|
|
});
|
|
|
|
it("shows health status indicator", async () => {
|
|
const mockStatus = {
|
|
last_sync: "2026-02-26T12:00:00Z",
|
|
is_healthy: true,
|
|
message: "Lore is healthy",
|
|
summary: {
|
|
open_issues: 5,
|
|
authored_mrs: 2,
|
|
reviewing_mrs: 3,
|
|
},
|
|
};
|
|
|
|
setMockResponse("get_lore_status", mockStatus);
|
|
renderWithProviders(<DebugView />);
|
|
|
|
await waitFor(() => {
|
|
expect(screen.queryByText(/loading/i)).not.toBeInTheDocument();
|
|
});
|
|
|
|
expect(screen.getByTestId("health-indicator")).toBeInTheDocument();
|
|
expect(screen.getByTestId("health-indicator")).toHaveClass("bg-green-500");
|
|
});
|
|
|
|
it("shows unhealthy indicator when lore is not healthy", async () => {
|
|
const mockStatus = {
|
|
last_sync: null,
|
|
is_healthy: false,
|
|
message: "lore not configured",
|
|
summary: null,
|
|
};
|
|
|
|
setMockResponse("get_lore_status", mockStatus);
|
|
renderWithProviders(<DebugView />);
|
|
|
|
await waitFor(() => {
|
|
expect(screen.queryByText(/loading/i)).not.toBeInTheDocument();
|
|
});
|
|
|
|
expect(screen.getByTestId("health-indicator")).toHaveClass("bg-red-500");
|
|
});
|
|
|
|
it("displays 'data since' timestamp when available", async () => {
|
|
const mockStatus = {
|
|
last_sync: "2026-02-26T12:00:00Z",
|
|
is_healthy: true,
|
|
message: "Lore is healthy",
|
|
summary: null,
|
|
};
|
|
|
|
setMockResponse("get_lore_status", mockStatus);
|
|
renderWithProviders(<DebugView />);
|
|
|
|
await waitFor(() => {
|
|
expect(screen.queryByText(/loading/i)).not.toBeInTheDocument();
|
|
});
|
|
|
|
// The timestamp appears in both the status section and raw JSON
|
|
const syncTimeElements = screen.getAllByText(/2026-02-26T12:00:00Z/);
|
|
expect(syncTimeElements.length).toBeGreaterThan(0);
|
|
});
|
|
|
|
it("shows 'all time' when last_sync is null", async () => {
|
|
const mockStatus = {
|
|
last_sync: null,
|
|
is_healthy: false,
|
|
message: "lore not configured",
|
|
summary: null,
|
|
};
|
|
|
|
setMockResponse("get_lore_status", mockStatus);
|
|
renderWithProviders(<DebugView />);
|
|
|
|
await waitFor(() => {
|
|
expect(screen.queryByText(/loading/i)).not.toBeInTheDocument();
|
|
});
|
|
|
|
expect(screen.getByText(/all time/i)).toBeInTheDocument();
|
|
});
|
|
});
|