scriptoriumadmin/litellm
1
0
Fork 0
mirror of https://github.com/BerriAI/litellm.git synced 2026-09-20 00:11:50 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions 134

chore(ui): remove orphaned useDeleteProject hook and its test

Browse source 
Co-Authored-By: Devin AI <158243242+devin-ai-integration[bot]@users.noreply.github.com>
This commit is contained in:
mateo 2026-09-17 20:08:08 +00:00
parent decbb96382
commit 5f3fd28f40
3 changed files with 0 additions and 142 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

10
ui/litellm-dashboard/eslint-suppressions.json
View file

@ -437,16 +437,6 @@
"count": 1
}
},
"src/app/(dashboard)/hooks/projects/useDeleteProject.test.ts": {
"react/display-name": {
"count": 1
}
},
"src/app/(dashboard)/hooks/projects/useDeleteProject.ts": {
"no-restricted-syntax": {
"count": 1
}
},
"src/app/(dashboard)/hooks/projects/useProjectDetails.test.ts": {
"react/display-name": {
"count": 1

86
ui/litellm-dashboard/src/app/(dashboard)/hooks/projects/useDeleteProject.test.ts
View file

@ -1,86 +0,0 @@
import { describe, it, expect, vi, beforeEach } from "vitest";
import { renderHook, waitFor } from "@testing-library/react";
import { QueryClient, QueryClientProvider } from "@tanstack/react-query";
import React, { ReactNode } from "react";
import { useDeleteProject } from "./useDeleteProject";
import { projectKeys } from "./useProjects";
vi.mock("@/components/networking", () => ({
getProxyBaseUrl: vi.fn(() => ""),
getGlobalLitellmHeaderName: vi.fn(() => "Authorization"),
deriveErrorMessage: vi.fn((data: any) => data?.error || "Error"),
handleError: vi.fn(),
}));
const mockUseAuthorized = vi.fn();
vi.mock("@/app/(dashboard)/hooks/useAuthorized", () => ({
default: () => mockUseAuthorized(),
}));
function makeWrapper(queryClient: QueryClient) {
return ({ children }: { children: ReactNode }) =>
React.createElement(QueryClientProvider, { client: queryClient }, children);
}
describe("useDeleteProject", () => {
let queryClient: QueryClient;
beforeEach(() => {
queryClient = new QueryClient({
defaultOptions: { queries: { retry: false }, mutations: { retry: false } },
});
vi.clearAllMocks();
global.fetch = vi.fn();
mockUseAuthorized.mockReturnValue({ accessToken: "test-token", userRole: "Admin" });
});
it("should render", () => {
const { result } = renderHook(() => useDeleteProject(), {
wrapper: makeWrapper(queryClient),
});
expect(result.current.mutate).toBeDefined();
});
it("should send DELETE to /project/delete with the given project IDs", async () => {
(global.fetch as any).mockResolvedValue({ ok: true, json: async () => ({}) });
const { result } = renderHook(() => useDeleteProject(), {
wrapper: makeWrapper(queryClient),
});
await result.current.mutateAsync(["proj-1", "proj-2"]);
const [url, init] = (global.fetch as any).mock.calls[0];
expect(url).toContain("/project/delete");
expect(init.method).toBe("DELETE");
expect(JSON.parse(init.body)).toEqual({ project_ids: ["proj-1", "proj-2"] });
});
it("should invalidate project queries on success", async () => {
(global.fetch as any).mockResolvedValue({ ok: true, json: async () => ({}) });
const invalidateSpy = vi.spyOn(queryClient, "invalidateQueries");
const { result } = renderHook(() => useDeleteProject(), {
wrapper: makeWrapper(queryClient),
});
await result.current.mutateAsync(["proj-1"]);
expect(invalidateSpy).toHaveBeenCalledWith({ queryKey: projectKeys.all });
});
it("should set isError when the request fails", async () => {
(global.fetch as any).mockResolvedValue({
ok: false,
json: async () => ({ error: "Not found" }),
});
const { result } = renderHook(() => useDeleteProject(), {
wrapper: makeWrapper(queryClient),
});
result.current.mutateAsync(["proj-1"]).catch(() => {});
await waitFor(() => expect(result.current.isError).toBe(true));
});
it("should throw when accessToken is missing", async () => {
mockUseAuthorized.mockReturnValue({ accessToken: null, userRole: "Admin" });
const { result } = renderHook(() => useDeleteProject(), {
wrapper: makeWrapper(queryClient),
});
await expect(result.current.mutateAsync(["proj-1"])).rejects.toThrow("Access token is required");
expect(global.fetch).not.toHaveBeenCalled();
});
});

46
ui/litellm-dashboard/src/app/(dashboard)/hooks/projects/useDeleteProject.ts
View file

@ -1,46 +0,0 @@
import { useMutation, useQueryClient } from "@tanstack/react-query";
import { getProxyBaseUrl, getGlobalLitellmHeaderName, deriveErrorMessage, handleError } from "@/components/networking";
import useAuthorized from "@/app/(dashboard)/hooks/useAuthorized";
import { projectKeys } from "./useProjects";
// ── Fetch function ───────────────────────────────────────────────────────────
const deleteProjects = async (accessToken: string, projectIds: string[]): Promise<void> => {
const baseUrl = getProxyBaseUrl();
const url = `${baseUrl}/project/delete`;
const response = await fetch(url, {
method: "DELETE",
headers: {
[getGlobalLitellmHeaderName()]: `Bearer ${accessToken}`,
"Content-Type": "application/json",
},
body: JSON.stringify({ project_ids: projectIds }),
});
if (!response.ok) {
const errorData = await response.json();
const errorMessage = deriveErrorMessage(errorData);
handleError(errorMessage);
throw new Error(errorMessage);
}
};
// ── Hook ─────────────────────────────────────────────────────────────────────
export const useDeleteProject = () => {
const { accessToken } = useAuthorized();
const queryClient = useQueryClient();
return useMutation<void, Error, string[]>({
mutationFn: async (projectIds) => {
if (!accessToken) {
throw new Error("Access token is required");
}
return deleteProjects(accessToken, projectIds);
},
onSuccess: () => {
queryClient.invalidateQueries({ queryKey: projectKeys.all });
},
});
};