jroachgolf84 opened a new pull request, #66881: URL: https://github.com/apache/airflow/pull/66881
## Description Per #66877: > The Airflow 3 task-group path asks the Execution API for a map of task states, then counts a Dag run as matching when every observed truthy state is in `allowed_states`. This means a task-group state map containing only `None` values, or a mix of successful tasks and `None` values, is counted as a successful group. To handle this, the `if state` logic was removed from the generation of the object being evaluated with `all()`. Instead, `None` values remained in the object, and were evaluated with `all()` (which then returns `False`, the intended behavior). A check for the list of matching states was also added. ## Testing To validate these changes, a number of parameterizations were added to `providers/standard/tests/unit/standard/utils/test_sensor_helper.py::test_get_count_by_matched_states`. These tests can be run using the command below: ```bash breeze testing providers-tests providers/standard/tests/unit/standard/utils/test_sensor_helper.py ``` * closes: #66877 ##### Was generative AI tooling used to co-author this PR? No, generative AI tooling was NOT used to co-author this PR. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
