Move deleteTask to factory pattern

frontend/src/app/tasks/hooks/useDeleteTask.ts

@@ -1,7 +1,7 @@
 import { useMutation, useQueryClient } from '@tanstack/react-query'
 import { Task } from '@/domain/Task'
 import { useAlert } from '@/ui/Alert/useAlert'
-import { deleteTask } from '@/infra/task/deleteTask'
+import { makeDeleteTask } from '@/infra/task/deleteTask'
 import { useClientFetch } from '@/infra/lib/useClientFetch'
 import { useGetCurrentUser } from '@/hooks/useGetCurrentUser/useGetCurrentUser'
 
@@ -10,8 +10,9 @@ export const useDeleteTask = () => {
   const queryClient = useQueryClient()
   const { id: userId } = useGetCurrentUser()
   const { showError, showSuccess } = useAlert()
+  const deleteTask = makeDeleteTask(apiClient)
 
-  const { mutate } = useMutation((taskId: number) => deleteTask(taskId, apiClient), {
+  const { mutate } = useMutation((taskId: number) => deleteTask(taskId), {
     onSuccess: (_, taskId) => {
       queryClient.setQueryData<Array<Task>>(['tasks', userId], (prevData) =>
         prevData!.filter((prevData) => prevData.id !== taskId)

frontend/src/infra/task/deleteTask.ts

@@ -1,17 +1,20 @@
 import { ApiClient } from '../lib/apiClient'
+import { Task } from '@/domain/Task'
+
+export const makeDeleteTask =
+  (apiClient: ApiClient) =>
+  async (taskId: Task['id']): Promise<string> => {
+    try {
+      const response = await apiClient(`/v1/timelog/tasks/${taskId}`, {
+        method: 'DELETE'
+      })
 
-export const deleteTask = async (taskId: number, apiClient: ApiClient): Promise<string> => {
-  return apiClient(`/v1/timelog/tasks/${taskId}`, {
-    method: 'DELETE'
-  })
-    .then((response) => {
       if (!response.ok) {
         throw Error(response.statusText)
       }
 
       return response.statusText
-    })
-    .catch((e) => {
-      throw new Error(e)
-    })
-}
+    } catch (e) {
+      throw e
+    }
+  }