Files
cc-switch/tests/components/ProviderList.test.tsx
YoVinchen 1586451862 Feat/auto failover switch (#440)
* feat(failover): add auto-failover master switch with proxy integration

- Add persistent auto_failover_enabled setting in database
- Add get/set_auto_failover_enabled commands
- Provider router respects master switch state
- Proxy shutdown automatically disables failover
- Enabling failover auto-starts proxy server
- Optimistic updates for failover queue toggle

* feat(proxy): persist proxy takeover state across app restarts

- Add proxy_takeover_{app_type} settings for per-app state tracking
- Restore proxy takeover state automatically on app startup
- Preserve state on normal exit, clear on manual stop
- Add stop_with_restore_keep_state method for graceful shutdown

* fix(proxy): set takeover state for all apps in start_with_takeover

* fix(windows): hide console window when checking CLI versions

Add CREATE_NO_WINDOW flag to prevent command prompt from flashing
when detecting claude/codex/gemini CLI versions on Windows.

* refactor(failover): make auto-failover toggle per-app independent

- Change setting key from 'auto_failover_enabled' to 'auto_failover_enabled_{app_type}'
- Update provider_router to check per-app failover setting
- When failover disabled, use current provider only; when enabled, use queue order
- Add unit tests for failover enabled/disabled behavior

* feat(failover): auto-switch to higher priority provider on recovery

- After circuit breaker reset, check if recovered provider has higher priority
- Automatically switch back if queue_order is lower (higher priority)
- Stream health check now resets circuit breaker on success/degraded

* chore: remove unused start_proxy_with_takeover command

- Remove command registration from lib.rs
- Add comment clarifying failover queue is preserved on proxy stop

* feat(ui): integrate failover controls into provider cards

- Add failover toggle button to provider card actions
- Show priority badge (P1, P2, ...) for queued providers
- Highlight active provider with green border in failover mode
- Sync drag-drop order with failover queue
- Move per-app failover toggle to FailoverQueueManager
- Simplify SettingsPage failover section

* test(providers): add mocks for failover hooks in ProviderList tests

* refactor(failover): merge failover_queue table into providers

- Add in_failover_queue field to providers table
- Remove standalone failover_queue table and related indexes
- Simplify queue ordering by reusing sort_index field
- Remove reorder_failover_queue and set_failover_item_enabled commands
- Update frontend to use simplified FailoverQueueItem type

* fix(database): ensure in_failover_queue column exists for v2 databases

Add column check in create_tables to handle existing v2 databases
that were created before the failover queue refactor.

* fix(ui): differentiate active provider border color by proxy mode

- Use green border/gradient when proxy takeover is active
- Use blue border/gradient in normal mode (no proxy)
- Improves visual distinction between proxy and non-proxy states

* fix(database): clear provider health record when removing from failover queue

When a provider is removed from the failover queue, its health monitoring
is no longer needed. This change ensures the health record is also deleted
from the database to prevent stale data.

* fix(failover): improve cache cleanup for provider health and circuit breaker

- Use removeQueries instead of invalidateQueries when stopping proxy to
  completely clear health and circuit breaker caches
- Clear provider health and circuit breaker caches when removing from
  failover queue
- Refresh failover queue after drag-sort since queue order depends on
  sort_index
- Only show health badge when provider is in failover queue

* style: apply prettier formatting to App.tsx and ProviderList.tsx

* fix(proxy): handle missing health records and clear health on proxy stop

- Return default healthy state when provider health record not found
- Add clear_provider_health_for_app to clear health for specific app
- Clear app health records when stopping proxy takeover

* fix(proxy): track actual provider used in forwarding for accurate logging

Introduce ForwardResult and ForwardError structs to return the actual
provider that handled the request. This ensures usage statistics and
error logs reflect the correct provider after failover.
2025-12-23 12:37:36 +08:00

298 lines
8.6 KiB
TypeScript

import { render, screen, fireEvent } from "@testing-library/react";
import { describe, it, expect, vi, beforeEach } from "vitest";
import type { Provider } from "@/types";
import { ProviderList } from "@/components/providers/ProviderList";
const useDragSortMock = vi.fn();
const useSortableMock = vi.fn();
const providerCardRenderSpy = vi.fn();
vi.mock("@/hooks/useDragSort", () => ({
useDragSort: (...args: unknown[]) => useDragSortMock(...args),
}));
vi.mock("@/components/providers/ProviderCard", () => ({
ProviderCard: (props: any) => {
providerCardRenderSpy(props);
const {
provider,
onSwitch,
onEdit,
onDelete,
onDuplicate,
onConfigureUsage,
} = props;
return (
<div data-testid={`provider-card-${provider.id}`}>
<button
data-testid={`switch-${provider.id}`}
onClick={() => onSwitch(provider)}
>
switch
</button>
<button
data-testid={`edit-${provider.id}`}
onClick={() => onEdit(provider)}
>
edit
</button>
<button
data-testid={`duplicate-${provider.id}`}
onClick={() => onDuplicate(provider)}
>
duplicate
</button>
<button
data-testid={`usage-${provider.id}`}
onClick={() => onConfigureUsage(provider)}
>
usage
</button>
<button
data-testid={`delete-${provider.id}`}
onClick={() => onDelete(provider)}
>
delete
</button>
<span data-testid={`is-current-${provider.id}`}>
{props.isCurrent ? "current" : "inactive"}
</span>
<span data-testid={`drag-attr-${provider.id}`}>
{props.dragHandleProps?.attributes?.["data-dnd-id"] ?? "none"}
</span>
</div>
);
},
}));
vi.mock("@/components/UsageFooter", () => ({
default: () => <div data-testid="usage-footer" />,
}));
vi.mock("@dnd-kit/sortable", async () => {
const actual = await vi.importActual<any>("@dnd-kit/sortable");
return {
...actual,
useSortable: (...args: unknown[]) => useSortableMock(...args),
};
});
// Mock hooks that use QueryClient
vi.mock("@/hooks/useStreamCheck", () => ({
useStreamCheck: () => ({
checkProvider: vi.fn(),
isChecking: () => false,
}),
}));
vi.mock("@/lib/query/failover", () => ({
useAutoFailoverEnabled: () => ({ data: false }),
useFailoverQueue: () => ({ data: [] }),
useAddToFailoverQueue: () => ({ mutate: vi.fn() }),
useRemoveFromFailoverQueue: () => ({ mutate: vi.fn() }),
useReorderFailoverQueue: () => ({ mutate: vi.fn() }),
}));
function createProvider(overrides: Partial<Provider> = {}): Provider {
return {
id: overrides.id ?? "provider-1",
name: overrides.name ?? "Test Provider",
settingsConfig: overrides.settingsConfig ?? {},
category: overrides.category,
createdAt: overrides.createdAt,
sortIndex: overrides.sortIndex,
meta: overrides.meta,
websiteUrl: overrides.websiteUrl,
};
}
beforeEach(() => {
useDragSortMock.mockReset();
useSortableMock.mockReset();
providerCardRenderSpy.mockClear();
useSortableMock.mockImplementation(({ id }: { id: string }) => ({
setNodeRef: vi.fn(),
attributes: { "data-dnd-id": id },
listeners: { onPointerDown: vi.fn() },
transform: null,
transition: null,
isDragging: false,
}));
useDragSortMock.mockReturnValue({
sortedProviders: [],
sensors: [],
handleDragEnd: vi.fn(),
});
});
describe("ProviderList Component", () => {
it("should render skeleton placeholders when loading", () => {
const { container } = render(
<ProviderList
providers={{}}
currentProviderId=""
appId="claude"
onSwitch={vi.fn()}
onEdit={vi.fn()}
onDelete={vi.fn()}
onDuplicate={vi.fn()}
onOpenWebsite={vi.fn()}
isLoading
/>,
);
const placeholders = container.querySelectorAll(
".border-dashed.border-muted-foreground\\/40",
);
expect(placeholders).toHaveLength(3);
});
it("should show empty state and trigger create callback when no providers exist", () => {
const handleCreate = vi.fn();
useDragSortMock.mockReturnValueOnce({
sortedProviders: [],
sensors: [],
handleDragEnd: vi.fn(),
});
render(
<ProviderList
providers={{}}
currentProviderId=""
appId="claude"
onSwitch={vi.fn()}
onEdit={vi.fn()}
onDelete={vi.fn()}
onDuplicate={vi.fn()}
onOpenWebsite={vi.fn()}
onCreate={handleCreate}
/>,
);
const addButton = screen.getByRole("button", {
name: "provider.addProvider",
});
fireEvent.click(addButton);
expect(handleCreate).toHaveBeenCalledTimes(1);
});
it("should render in order returned by useDragSort and pass through action callbacks", () => {
const providerA = createProvider({ id: "a", name: "A" });
const providerB = createProvider({ id: "b", name: "B" });
const handleSwitch = vi.fn();
const handleEdit = vi.fn();
const handleDelete = vi.fn();
const handleDuplicate = vi.fn();
const handleUsage = vi.fn();
const handleOpenWebsite = vi.fn();
useDragSortMock.mockReturnValue({
sortedProviders: [providerB, providerA],
sensors: [],
handleDragEnd: vi.fn(),
});
render(
<ProviderList
providers={{ a: providerA, b: providerB }}
currentProviderId="b"
appId="claude"
onSwitch={handleSwitch}
onEdit={handleEdit}
onDelete={handleDelete}
onDuplicate={handleDuplicate}
onConfigureUsage={handleUsage}
onOpenWebsite={handleOpenWebsite}
/>,
);
// Verify sort order
expect(providerCardRenderSpy).toHaveBeenCalledTimes(2);
expect(providerCardRenderSpy.mock.calls[0][0].provider.id).toBe("b");
expect(providerCardRenderSpy.mock.calls[1][0].provider.id).toBe("a");
// Verify current provider marker
expect(providerCardRenderSpy.mock.calls[0][0].isCurrent).toBe(true);
// Drag attributes from useSortable
expect(
providerCardRenderSpy.mock.calls[0][0].dragHandleProps?.attributes[
"data-dnd-id"
],
).toBe("b");
expect(
providerCardRenderSpy.mock.calls[1][0].dragHandleProps?.attributes[
"data-dnd-id"
],
).toBe("a");
// Trigger action buttons
fireEvent.click(screen.getByTestId("switch-b"));
fireEvent.click(screen.getByTestId("edit-b"));
fireEvent.click(screen.getByTestId("duplicate-b"));
fireEvent.click(screen.getByTestId("usage-b"));
fireEvent.click(screen.getByTestId("delete-a"));
expect(handleSwitch).toHaveBeenCalledWith(providerB);
expect(handleEdit).toHaveBeenCalledWith(providerB);
expect(handleDuplicate).toHaveBeenCalledWith(providerB);
expect(handleUsage).toHaveBeenCalledWith(providerB);
expect(handleDelete).toHaveBeenCalledWith(providerA);
// Verify useDragSort call parameters
expect(useDragSortMock).toHaveBeenCalledWith(
{ a: providerA, b: providerB },
"claude",
);
});
it("filters providers with the search input", () => {
const providerAlpha = createProvider({ id: "alpha", name: "Alpha Labs" });
const providerBeta = createProvider({ id: "beta", name: "Beta Works" });
useDragSortMock.mockReturnValue({
sortedProviders: [providerAlpha, providerBeta],
sensors: [],
handleDragEnd: vi.fn(),
});
render(
<ProviderList
providers={{ alpha: providerAlpha, beta: providerBeta }}
currentProviderId=""
appId="claude"
onSwitch={vi.fn()}
onEdit={vi.fn()}
onDelete={vi.fn()}
onDuplicate={vi.fn()}
onOpenWebsite={vi.fn()}
/>,
);
fireEvent.keyDown(window, { key: "f", metaKey: true });
const searchInput = screen.getByPlaceholderText(
"Search name, notes, or URL...",
);
// Initially both providers are rendered
expect(screen.getByTestId("provider-card-alpha")).toBeInTheDocument();
expect(screen.getByTestId("provider-card-beta")).toBeInTheDocument();
fireEvent.change(searchInput, { target: { value: "beta" } });
expect(screen.queryByTestId("provider-card-alpha")).not.toBeInTheDocument();
expect(screen.getByTestId("provider-card-beta")).toBeInTheDocument();
fireEvent.change(searchInput, { target: { value: "gamma" } });
expect(screen.queryByTestId("provider-card-alpha")).not.toBeInTheDocument();
expect(screen.queryByTestId("provider-card-beta")).not.toBeInTheDocument();
expect(
screen.getByText("No providers match your search."),
).toBeInTheDocument();
});
});