added input validation testing

Author: radiant-tangent

tests/test_api_tasks.py

@@ -467,6 +467,12 @@ async def test_move_task(
     assert len(requests_mock.calls) == 3
     assert response is True
 
+    with pytest.raises(
+        ValueError,
+        match="Either `project_id`, `section_id`, or `parent_id` must be provided.",
+    ):
+        response = await todoist_api_async.move_task(task_id)
+
 
 @pytest.mark.asyncio
 async def test_delete_task(

todoist_api_python/api.py

@@ -496,13 +496,9 @@ def move_task(
         :return: True if the task was moved successfully,
                  False otherwise (possibly raise `HTTPError` instead).
         :raises requests.exceptions.HTTPError: If the API request fails.
-        :raises ValueError: When `task_id` is not provided.
         :raises ValueError: If neither `project_id`, `section_id`,
                 nor `parent_id` is provided.
         """
-        if task_id is None:
-            raise ValueError("`task_id` must be provided.")
-
         if project_id is None and section_id is None and parent_id is None:
             raise ValueError(
                 "Either `project_id`, `section_id`, or `parent_id` must be provided."

todoist_api_python/api_async.py

@@ -365,7 +365,6 @@ async def move_task(
         :return: True if the task was moved successfully,
                  False otherwise (possibly raise `HTTPError` instead).
         :raises requests.exceptions.HTTPError: If the API request fails.
-        :raises ValueError: When `task_id` is not provided.
         :raises ValueError: If neither `project_id`, `section_id`,
                 nor `parent_id` is provided.
         """