rawwar commented on code in PR #47817:
URL: https://github.com/apache/airflow/pull/47817#discussion_r1997239663
##########
tests/api_fastapi/core_api/routes/public/test_dag_run.py:
##########
@@ -931,6 +932,13 @@ def test_patch_dag_run(self, test_client, dag_id, run_id,
patch_body, response_b
assert body["dag_run_id"] == run_id
assert body.get("state") == response_body.get("state")
assert body.get("note") == response_body.get("note")
+ user_id = (
+ session.query(DagRunNote)
+ .join(DagRun, DagRunNote.dag_run_id == DagRun.id)
Review Comment:
I see that the error is coming from DagRunNote's `__repr__` method:
```
def __repr__(self):
prefix = f"<{self.__class__.__name__}:
{self.dag_id}.{self.dagrun_id} {self.run_id}"
if self.map_index != -1:
prefix += f" map_index={self.map_index}"
return prefix + ">"
```
DagRunNote does not have any of dag_id, dagrun_id and run_id. Not sure why
we added these here.
CC: @pierrejeambrun
--
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]
