[GitHub] [airflow] potiuk edited a comment on issue #11657: Better unit tests for Helm Chart

[GitBox <gi...@apache.org>]

potiuk edited a comment on issue #11657:
URL: https://github.com/apache/airflow/issues/11657#issuecomment-712105738


   I am absolutely for this. Comparing to the above proposal, it's heaven and earth.
   
   I very much dislike the current yaml based testing, they're difficult to maintain, refactor, automate, run from the IDE, debug, duplicate to write new tests etc. (unless I do not now how - maybe I just need some training).
   
   I've added a few yaml-based tests in the past and fixed a few failing ones when I added new features (like kerberos sidecar). And it was a pain. For me this is the main reason why I have not added tests to my changes to the helm chart - because I disliked the idea of being close to that part of the code.
   
   Sorry for the rant, it's not really, really bad with those yaml tests, but it is pretty close. I certainly feel subconscious " let's not go anywhere near those tests" when I think about them :). And looking at the proposal above, it is so much better. 
   
   


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

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