scriptoriumadmin/litellm
1
0
Fork 0
mirror of https://github.com/BerriAI/litellm.git synced 2026-10-05 02:41:56 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions 49

chore(ui): repair failing vitest tests after shadcn migration (batch B)

Browse source 
Co-authored-by: yuneng-jiang <yuneng-berri@users.noreply.github.com>
This commit is contained in:
Cursor Agent 2026-04-24 15:27:41 +00:00
parent c84b7614e3
commit 10f6cad106
No known key found for this signature in database
4 changed files with 16 additions and 11 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
ui/litellm-dashboard/src/components/Settings/RouterSettings/Fallbacks/FallbackSelectionForm.test.tsx
View file

@ -214,7 +214,7 @@ describe("FallbackSelectionForm", () => {
);
expect(screen.getByRole("tab", { name: "gpt-4" })).toBeInTheDocument();
expect(screen.getAllByText("gpt-4").length).toBeGreaterThan(0);
expect(screen.getByText("gpt-3.5-turbo")).toBeInTheDocument();
expect(screen.getAllByText("gpt-3.5-turbo").length).toBeGreaterThan(0);
});
it("should not add group when add button clicked at maxGroups", () => {

8
ui/litellm-dashboard/src/components/TeamSSOSettings.test.tsx
View file

@ -171,7 +171,8 @@ describe("TeamSSOSettings", () => {
renderWithProviders(<TeamSSOSettings {...defaultProps} />);
expect(screen.getByTestId("spinner")).toBeInTheDocument();
// Loading state renders a shadcn Skeleton (div with animate-pulse).
expect(document.querySelector(".animate-pulse")).toBeInTheDocument();
});
it("should display error message when fetch fails", async () => {
@ -377,8 +378,9 @@ describe("TeamSSOSettings", () => {
await userEvent.click(screen.getByRole("button", { name: /Edit Settings/i }));
await waitFor(() => {
const numberInputs = screen.getAllByLabelText("number input");
// max_budget, tpm_limit, rpm_limit
// Edit mode renders 3 number <input type="number"> fields for
// max_budget, tpm_limit, rpm_limit (rendered as shadcn Inputs).
const numberInputs = document.querySelectorAll('input[type="number"]');
expect(numberInputs.length).toBe(3);
});
});

10
ui/litellm-dashboard/src/components/shared/advanced_date_picker.test.tsx
View file

@ -29,8 +29,8 @@ describe("AdvancedDatePicker", () => {
});
const openDropdown = (container: HTMLElement) => {
// Find the clickable div that contains the clock icon
const trigger = container.querySelector('[role="img"][aria-label="clock-circle"]')?.closest("div.cursor-pointer");
// Find the clickable div that contains the lucide Clock icon.
const trigger = container.querySelector(".lucide-clock")?.closest("div.cursor-pointer");
if (trigger) {
fireEvent.click(trigger);
}
@ -47,10 +47,10 @@ describe("AdvancedDatePicker", () => {
});
it("should display formatted date range", () => {
render(<AdvancedDatePicker value={defaultValue} onValueChange={mockOnValueChange} />);
const { container } = render(<AdvancedDatePicker value={defaultValue} onValueChange={mockOnValueChange} />);
// The component displays date range in the format "D MMM, HH:mm - D MMM, HH:mm"
// Just check that the clock icon is present
expect(screen.getByLabelText("clock-circle")).toBeInTheDocument();
// Just check that the (lucide) clock icon is present in the trigger.
expect(container.querySelector(".lucide-clock")).toBeInTheDocument();
});
it("should open dropdown when clicked", () => {

7
ui/litellm-dashboard/src/components/templates/key_edit_view.test.tsx
View file

@ -624,8 +624,11 @@ describe("KeyEditView", () => {
});
const orgFormItem = screen.getByText("Organization").closest(".ant-form-item");
const disabledSelect = orgFormItem?.querySelector(".ant-select-disabled");
expect(disabledSelect).toBeTruthy();
// OrganizationDropdown now renders a shadcn Select trigger
// (role="combobox") that gets the disabled attribute from Radix.
const orgCombobox = orgFormItem?.querySelector('[role="combobox"]') as HTMLButtonElement | null;
expect(orgCombobox).toBeTruthy();
expect(orgCombobox).toBeDisabled();
});
it("should not disable the organization dropdown for admin users", async () => {