You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
Repository navigation
Add fastAPI test coverage for upstream_map_indexes in execution API #50541
#49996 added support for returning upstream_map_indexes as response of the /run endpoint but there is no test coverage that checks if expected value is returned or not.
We should ideally cover it so that any regressions are caught.
Committer
I acknowledge that I am a maintainer/committer of the Apache Airflow project.
I looked into this and found that there’s already a parameterized test (test_ti_run) for the /execution/task-instances/{ti.id}/run endpoint. It patches _get_upstream_map_indexes and asserts that the upstream_map_indexes field is returned in the response with the expected structure across multiple scenarios (ints, lists, None, mixed types).
This seems to cover the gap mentioned in the issue — verifying that the field is returned correctly in the API response. Could you confirm if this is the intended scope, or if there’s anything additional expected here?
Activity
anasatzemoso commentedon Sep 18, 2025
Can I work on this issue ?
anasatzemoso commentedon Sep 19, 2025
I looked into this and found that there’s already a parameterized test
(test_ti_run)for the/execution/task-instances/{ti.id}/runendpoint. It patches_get_upstream_map_indexesand asserts that theupstream_map_indexesfield is returned in the response with the expected structure across multiple scenarios (ints, lists, None, mixed types).This seems to cover the gap mentioned in the issue — verifying that the field is returned correctly in the API response. Could you confirm if this is the intended scope, or if there’s anything additional expected here?
fix: add test for upstream_map_indexes not in HEAD API response
fix: add test for upstream_map_indexes not in HEAD API response
fix: add test for upstream_map_indexes not in HEAD API response
fix: add test for upstream_map_indexes not in HEAD API response (apac…
fix: add test for upstream_map_indexes not in HEAD API response (apac…
fix: add test for upstream_map_indexes not in HEAD API response (apac…
fix: add test for upstream_map_indexes not in HEAD API response (apac…
fix: add test for upstream_map_indexes not in HEAD API response (#62283)
fix: add test for upstream_map_indexes not in HEAD API response (#62283)