Re: [PR] Improve test coverage for the rest api modules [airflow]

2023-10-31 Thread via GitHub
ephraimbuddy commented on PR #35219: URL: https://github.com/apache/airflow/pull/35219#issuecomment-1787466333 Hi @potiuk, @vincbeck I added additional commits after your approvals, you can take another look, especially on test_task_instance_endpoint.py -- This is an automated message fro

Re: [PR] Improve test coverage for the rest api modules [airflow]

2023-10-31 Thread via GitHub
potiuk merged PR #35219: URL: https://github.com/apache/airflow/pull/35219 -- 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: commits-unsubscr...@airflow.a

Re: [PR] Improve test coverage for the rest api modules [airflow]

2023-10-31 Thread via GitHub
Jorricks commented on PR #35219: URL: https://github.com/apache/airflow/pull/35219#issuecomment-1787954418 > except MultipleResultsFound: Yes I vaguely remember this and thinking as well that it should never be possible that it would get two results back. If it did, it would be a pro