rawwar commented on code in PR #48524:
URL: https://github.com/apache/airflow/pull/48524#discussion_r2020802414
##########
airflow-core/tests/unit/api_fastapi/core_api/routes/public/test_dag_run.py:
##########
@@ -985,6 +1010,17 @@ def test_patch_dag_run_with_update_mask(
assert response.status_code == expected_status_code
for key, value in response_body.items():
assert response_json.get(key) == value
+ dr_note = (
+ session.query(DagRunNote)
+ .join(DagRun, DagRunNote.dag_run_id == DagRun.id)
+ .filter(DagRun.run_id == DAG1_RUN1_ID)
+ .one_or_none()
+ )
+ if note_data is None:
+ assert dr_note is None
+ else:
+ assert dr_note.user_id == note_data.get("user_id")
+ assert dr_note.content == note_data.get("content")
Review Comment:
Thanks! I too noticed it after I pushed earlier commit. Along with this, I
also had to remove the unused import(DagRunNote). I just pushed it sometime
ago.
--
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]
