jason810496 commented on code in PR #60314:
URL: https://github.com/apache/airflow/pull/60314#discussion_r2678614082
##########
airflow-core/tests/unit/api_fastapi/core_api/routes/ui/test_structure.py:
##########
@@ -729,3 +767,112 @@ def test_mapped_operator_in_task_group(self, dag_maker,
test_client, session):
)
assert mapped_in_group["is_mapped"] is True
assert mapped_in_group["operator"] == "PythonOperator"
+
Review Comment:
Some for some of the test method of test structure.
##########
airflow-core/tests/unit/api_fastapi/core_api/routes/ui/test_grid.py:
##########
@@ -933,3 +933,78 @@ def
test_nonlinear_structure_intermediate_with_both_directions(self, test_client
# Should return intermediate, its upstream path, and downstream path
# upstream: branch_a, start; downstream: merge, end
assert task_ids == ["branch_a", "end", "intermediate", "merge",
"start"]
+
+ # Tests for depth parameter
Review Comment:
Would it be better to add `pytest.mark.parametrize` to reuse the test method?
--
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]
