From 7075738cf63d99d3e96426d430bd9cb6fdb2c6a9 Mon Sep 17 00:00:00 2001
From: Aryan Iyappan <aryaniyappan2006@gmail.com>
Date: Fri, 28 Jun 2024 15:05:48 +0530
Subject: [PATCH] update existing tests

---
 client/src/components/home-page/Todo.tsx          |  2 ++
 .../src/components/home-page/TodoController.tsx   |  1 -
 server/tests/unit/todos/test_repositories.py      | 15 +++++++++++++--
 3 files changed, 15 insertions(+), 3 deletions(-)

diff --git a/client/src/components/home-page/Todo.tsx b/client/src/components/home-page/Todo.tsx
index f8edf8b..038ec18 100644
--- a/client/src/components/home-page/Todo.tsx
+++ b/client/src/components/home-page/Todo.tsx
@@ -62,6 +62,7 @@ export default function Todo({ todo, connectionId }: Props) {
         </p>
         <div className="flex gap-2 grow justify-end opacity-0 group-hover:opacity-100 transition-opacity">
           <Button
+            title={data.completed ? "mark as incomplete" : "mark as complete"}
             size={"icon"}
             variant={"ghost"}
             disabled={isToggleCompletedMutationInFlight}
@@ -78,6 +79,7 @@ export default function Todo({ todo, connectionId }: Props) {
             )}
           </Button>
           <Button
+            title="delete todo"
             size={"icon"}
             variant={"ghost"}
             disabled={isDeleteMutationInFlight}
diff --git a/client/src/components/home-page/TodoController.tsx b/client/src/components/home-page/TodoController.tsx
index 1b08fe0..9e4ca23 100644
--- a/client/src/components/home-page/TodoController.tsx
+++ b/client/src/components/home-page/TodoController.tsx
@@ -70,7 +70,6 @@ export default function TodoController({ rootQuery }: Props) {
   const onSubmit: SubmitHandler<z.infer<typeof createTodoSchema>> = async (
     input
   ) => {
-    console.log(data.todos.__id);
     form.reset();
     commitMutation({
       variables: { content: input.content, connections: [data.todos.__id] },
diff --git a/server/tests/unit/todos/test_repositories.py b/server/tests/unit/todos/test_repositories.py
index 7a82fa1..f9ac8d5 100644
--- a/server/tests/unit/todos/test_repositories.py
+++ b/server/tests/unit/todos/test_repositories.py
@@ -47,6 +47,17 @@ async def test_get_all_todos(todo_repo: TodoRepo) -> None:
 
 
 async def test_delete_todo(todo: Todo, todo_repo: TodoRepo) -> None:
-    """Ensure we can delete a todo by ID."""
-    await todo_repo.delete(todo_id=todo.id)
+    """Ensure we can delete a todo."""
+    await todo_repo.delete(todo=todo)
     assert await todo_repo.get(todo_id=todo.id) is None
+
+
+async def test_update_todo(todo: Todo, todo_repo: TodoRepo) -> None:
+    """Ensure we can update a todo."""
+    # mark todo as complete
+    await todo_repo.update(todo=todo, completed=True)
+    assert todo.completed
+
+    # mark todo as incomplete
+    await todo_repo.update(todo=todo, completed=False)
+    assert not todo.completed