mirrors/archon
1
0
Fork 0
mirror of https://github.com/coleam00/Archon.git synced 2025-12-24 02:39:17 -05:00
Code Issues Packages Projects Releases Wiki Activity

Handle fetch aborts in API client

Browse Source
This commit is contained in:
Rasmus Widing
2025-09-18 14:37:56 +03:00
parent ddbf05862e
commit d53012abfc
2 changed files with 42 additions and 20 deletions
Download Patch File Download Diff File Expand all files Collapse all files

58
archon-ui-main/src/features/projects/tasks/services/tests/taskService.test.ts
View File

@@ -51,10 +51,13 @@ describe("taskService", () => {
const result = await taskService.createTask(mockTaskData);
// Verify the API was called correctly
expect(callAPIWithETag).toHaveBeenCalledWith("/api/tasks", {
method: "POST",
body: JSON.stringify(mockTaskData),
});
expect(callAPIWithETag).toHaveBeenCalledWith(
"/api/tasks",
expect.objectContaining({
method: "POST",
body: JSON.stringify(mockTaskData),
}),
);
// Verify the task is properly unwrapped
expect(result).toEqual(mockTask);
@@ -103,10 +106,13 @@ describe("taskService", () => {
const result = await taskService.updateTask(taskId, mockUpdates);
// Verify the API was called correctly
expect(callAPIWithETag).toHaveBeenCalledWith(`/api/tasks/${taskId}`, {
method: "PUT",
body: JSON.stringify(mockUpdates),
});
expect(callAPIWithETag).toHaveBeenCalledWith(
`/api/tasks/${taskId}`,
expect.objectContaining({
method: "PUT",
body: JSON.stringify(mockUpdates),
}),
);
// Verify the task is properly unwrapped
expect(result).toEqual(mockUpdatedTask);
@@ -130,10 +136,13 @@ describe("taskService", () => {
const result = await taskService.updateTask(taskId, partialUpdate);
expect(callAPIWithETag).toHaveBeenCalledWith(`/api/tasks/${taskId}`, {
method: "PUT",
body: JSON.stringify(partialUpdate),
});
expect(callAPIWithETag).toHaveBeenCalledWith(
`/api/tasks/${taskId}`,
expect.objectContaining({
method: "PUT",
body: JSON.stringify(partialUpdate),
}),
);
expect(result.description).toBe(partialUpdate.description);
});
@@ -175,10 +184,13 @@ describe("taskService", () => {
const result = await taskService.updateTaskStatus(taskId, newStatus);
// Verify the API was called correctly
expect(callAPIWithETag).toHaveBeenCalledWith(`/api/tasks/${taskId}`, {
method: "PUT",
body: JSON.stringify({ status: newStatus }),
});
expect(callAPIWithETag).toHaveBeenCalledWith(
`/api/tasks/${taskId}`,
expect.objectContaining({
method: "PUT",
body: JSON.stringify({ status: newStatus }),
}),
);
// Verify the task is properly unwrapped
expect(result).toEqual(mockUpdatedTask);
@@ -203,9 +215,12 @@ describe("taskService", () => {
await taskService.deleteTask(taskId);
expect(callAPIWithETag).toHaveBeenCalledWith(`/api/tasks/${taskId}`, {
method: "DELETE",
});
expect(callAPIWithETag).toHaveBeenCalledWith(
`/api/tasks/${taskId}`,
expect.objectContaining({
method: "DELETE",
}),
);
});
it("should handle API errors properly", async () => {
@@ -251,7 +266,10 @@ describe("taskService", () => {
const result = await taskService.getTasksByProject(projectId);
expect(callAPIWithETag).toHaveBeenCalledWith(`/api/projects/${projectId}/tasks`, { signal: undefined });
expect(callAPIWithETag).toHaveBeenCalledWith(
`/api/projects/${projectId}/tasks`,
expect.objectContaining({ signal: undefined })
);
expect(result).toEqual(mockTasks);
expect(result).toHaveLength(2);
});

4
archon-ui-main/src/features/shared/apiWithEtag.ts
View File

@@ -111,6 +111,10 @@ export async function callAPIWithETag<T = unknown>(endpoint: string, options: Re
throw error;
}
if (error instanceof Error && error.name === "AbortError") {
throw error;
}
throw new APIServiceError(
`Failed to call API ${endpoint}: ${error instanceof Error ? error.message : "Unknown error"}`,
"NETWORK_ERROR",