You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@airflow.apache.org by GitBox <gi...@apache.org> on 2021/11/23 00:32:51 UTC

[GitHub] [airflow] rsg17 commented on pull request #19727: Add test case for views connections muldelete function

rsg17 commented on pull request #19727:
URL: https://github.com/apache/airflow/pull/19727#issuecomment-976038954


   > Hmm. This test will always succeed because the Connection objects are not actually saved to the database, and thus not actually “deleted” by the view (because they don’t exist from the view’s standpoint). To properly test the view, you need to save the objects in to the database, and test those are properly removed afterwards. You can see tests in `test_views_variable.py` for an example.
   
   Updated. Could you check again?


-- 
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-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org