potiuk commented on issue #12748: URL: https://github.com/apache/airflow/issues/12748#issuecomment-826571603
I think as of April after some fixes, the coverage is back on track: Particularly: https://github.com/apache/airflow/commit/1dfbb8d2031cb8a3e95e4bf91aa478857c5c3a85 helped with that because master merges started to succeed and "master merge" builds are the ones contributing "full coverage" information Example: * s3 is 94%: https://codecov.io/gh/apache/airflow/src/master/airflow/providers/amazon/aws/hooks/s3.py * overall coverage is 89.97% I looked at the coverage graphs (the sunburst chart is pretty usefule) and there are a few interesting things - maybe we would like to address them: * Kubernetes executor has surprisingly low 64% https://codecov.io/gh/apache/airflow/src/master/airflow/executors/kubernetes_executor.py - looks like some of the "flush/end" code is not tested. * Task command has a few missing parts: https://codecov.io/gh/apache/airflow/src/master/airflow/cli/commands/task_command.py (77%) * There are quite a number of untested views: https://codecov.io/gh/apache/airflow/src/master/airflow/www/views.py 80% * Kuberentes refresh_config has untested method https://codecov.io/gh/apache/airflow/src/master/airflow/kubernetes/refresh_config.py 75% -- 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: [email protected]
