claneys commented on code in PR #37365:
URL: https://github.com/apache/airflow/pull/37365#discussion_r1505587309


##########
helm_tests/airflow_core/test_triggerer.py:
##########


Review Comment:
   It is possible, this would tight worker and triggerer db tests together. I 
began to wrote test in it before put them here. 



##########
helm_tests/airflow_core/test_worker.py:
##########


Review Comment:
   About the tested key ? No this is on purpose as the postresql trigger use 
connectionFromEnv and mysql one connectionStringFromEnv this test is meant to 
avoid confusion, wrong default used or copy/paste errors.



-- 
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]

Reply via email to