This is an automated email from the ASF dual-hosted git repository.

potiuk pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/airflow.git


The following commit(s) were added to refs/heads/main by this push:
     new 98d8b6e1f39 Fix test_get_task_states_with_task_group_id_and_task_id to 
actually cover its name (#72294)
98d8b6e1f39 is described below

commit 98d8b6e1f39b9c39617863c0568dfa94be7227e2
Author: Jyun-An Chen <[email protected]>
AuthorDate: Wed Sep 9 09:06:33 2026 +0800

    Fix test_get_task_states_with_task_group_id_and_task_id to actually cover 
its name (#72294)
    
    The test only ever passed task_group_id in the request, never task_ids,
    so it exercised the same code path as test_get_task_states_group_id_basic
    and left the task_group_id + task_ids union branch in
    get_task_instance_states untested despite the test's name.
---
 .../unit/api_fastapi/execution_api/versions/head/test_task_instances.py | 2 ++
 1 file changed, 2 insertions(+)

diff --git 
a/airflow-core/tests/unit/api_fastapi/execution_api/versions/head/test_task_instances.py
 
b/airflow-core/tests/unit/api_fastapi/execution_api/versions/head/test_task_instances.py
index 80392fd17e2..571057c12df 100644
--- 
a/airflow-core/tests/unit/api_fastapi/execution_api/versions/head/test_task_instances.py
+++ 
b/airflow-core/tests/unit/api_fastapi/execution_api/versions/head/test_task_instances.py
@@ -3942,6 +3942,7 @@ class TestGetTaskStates:
             params={
                 "dag_id": 
"test_get_task_group_states_with_multiple_task_tasks",
                 "task_group_id": "group1",
+                "task_ids": ["task2"],
             },
         )
         assert response.status_code == 200
@@ -3949,6 +3950,7 @@ class TestGetTaskStates:
             "task_states": {
                 "test": {
                     "group1.task1": "success",
+                    "task2": "failed",
                 },
             },
         }

Reply via email to