From 677c50a7cc58e9caf24cd5ab117adeb9900431d3 Mon Sep 17 00:00:00 2001
From: Aaron Godfrey <me@aarongodfrey.dev>
Date: Mon, 4 Dec 2023 11:37:09 -0800
Subject: [PATCH] Exclude Todoist sub-tasks for the todo platform (#104914)

---
 homeassistant/components/todoist/todo.py | 3 +++
 tests/components/todoist/conftest.py     | 3 ++-
 tests/components/todoist/test_todo.py    | 8 ++++++++
 3 files changed, 13 insertions(+), 1 deletion(-)

diff --git a/homeassistant/components/todoist/todo.py b/homeassistant/components/todoist/todo.py
index 64e83b8cc6e..6231a6878ae 100644
--- a/homeassistant/components/todoist/todo.py
+++ b/homeassistant/components/todoist/todo.py
@@ -85,6 +85,9 @@ class TodoistTodoListEntity(CoordinatorEntity[TodoistCoordinator], TodoListEntit
             for task in self.coordinator.data:
                 if task.project_id != self._project_id:
                     continue
+                if task.parent_id is not None:
+                    # Filter out sub-tasks until they are supported by the UI.
+                    continue
                 if task.is_completed:
                     status = TodoItemStatus.COMPLETED
                 else:
diff --git a/tests/components/todoist/conftest.py b/tests/components/todoist/conftest.py
index 4e4d41b6914..42251b0ea18 100644
--- a/tests/components/todoist/conftest.py
+++ b/tests/components/todoist/conftest.py
@@ -46,6 +46,7 @@ def make_api_task(
     due: Due | None = None,
     project_id: str | None = None,
     description: str | None = None,
+    parent_id: str | None = None,
 ) -> Task:
     """Mock a todoist Task instance."""
     return Task(
@@ -61,7 +62,7 @@ def make_api_task(
         id=id or "1",
         labels=["Label1"],
         order=1,
-        parent_id=None,
+        parent_id=parent_id,
         priority=1,
         project_id=project_id or PROJECT_ID,
         section_id=None,
diff --git a/tests/components/todoist/test_todo.py b/tests/components/todoist/test_todo.py
index aa00e2c2ff4..1e94b52149c 100644
--- a/tests/components/todoist/test_todo.py
+++ b/tests/components/todoist/test_todo.py
@@ -51,6 +51,14 @@ def set_time_zone(hass: HomeAssistant) -> None:
             ],
             "0",
         ),
+        (
+            [
+                make_api_task(
+                    id="12345", content="sub-task", is_completed=False, parent_id="1"
+                )
+            ],
+            "0",
+        ),
     ],
 )
 async def test_todo_item_state(