scali commented on PR #52634: URL: https://github.com/apache/airflow/pull/52634#issuecomment-3323528347
> > > Hi @scali, can you add some test coverage as well? If you look for `webserverConfigConfigMapName` in `helm-test`, that should give you examples to follow. > > > > > > Hi @jedcunningham, I've added some unit tests for ConfigMap of ApiServer > > I think you remove the change and kept only the tests I don't know what happened (maybe the fact I've maid my changes in the `main` branch of my fork), but in the meantime, I saw that the chart/templates/_helpers.yaml file that I had modified at the beginning [had been modified afterwards](https://github.com/apache/airflow/commit/09771cbf90cfd84546a977129980d2f31c691195#diff-dc12e5cc8016c85ad9e662af7c8b8cfb49d91965225e94aa12146dc1f371fc86) in this [commit](https://github.com/apache/airflow/commit/09771cbf90cfd84546a977129980d2f31c691195) -- 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]
