Improve Code Coverage for Airflow

@potiuk

Description

Airflow code coverage is currently at an impressive 84% but we would like to improve the code coverage and get to at least 95%.
This is a call to help improve the code coverage by adding the missing tests.

You can use the code coverage tool to find modules to improve. Here is the link to the documentation: https://github.com/apache/airflow/blob/main/contributing-docs/testing/unit_tests.rst#code-coverage

Below is how you can discover modules to work on, follow the above link for up-to-date instruction:

Initiate a breeze shell.
Execute one of the commands below based on the desired coverage area:
Core: python scripts/cov/core_coverage.py
REST API: python scripts/cov/restapi_coverage.py
CLI: python scripts/cov/cli_coverage.py
Webserver: python scripts/cov/www_coverage.py
After execution, the coverage report will be available at: http://localhost:28000/dev/coverage/index.html.
Choose any of the modules and improve it

Note
In order to see the coverage report, you must start webserver first in breeze environment via airflow webserver. Once you enter breeze, you can start tmux (terminal multiplexer) and split the terminal (by pressing ctrl-B " for example) to continue testing and run the webserver in one terminal and run tests in the second one (you can switch between the terminals with ctrl-B <arrow>).

PRs should be per module or related codebase

UPDATE (added by @potiuk )

Also you might want to have a look at #35442 - we had a "project structure" test that was supposed to detect modules that do not have corresponding tests and fail our build in this case. This test has been however broken for a long time and we have many missing modules on the list.

Add missing tests for the DAG endpoint (#35158)

Improve test coverage for the rest api modules

@ephraimbuddy

Hey @ephraimbuddy - I added a comment in the issue description.

Also you might want to have a look at #35442 - we had a "project structure" test that was supposed to detect modules that do not have corresponding tests and fail our build in this case. This test has been however broken for a long time and we have many missing modules on the list.

Add missing tests for the DAG endpoint (#35158)

@ephraimbuddy

@ephraimbuddy I am trying to generate the coverage report on my local and following the instructions but can't see anything load on the UI. Can you please let me know if I am doing something wrong?

As far as I see, the webserver logs are fine:

INFO:     172.18.0.1:63228 - "GET / HTTP/1.1" 200 OK
INFO:     172.18.0.1:63228 - "GET /webapp/static/assets/index-IuX-ZHgj.js HTTP/1.1" 200 OK
INFO:     172.18.0.1:63228 - "GET /webapp/static/pin_32.png HTTP/1.1" 200 OK
INFO:     172.18.0.1:61072 - "GET /dev/coverage/index.html HTTP/1.1" 200 OK
INFO:     172.18.0.1:61072 - "GET /webapp/static/assets/index-IuX-ZHgj.js HTTP/1.1" 200 OK
INFO:     172.18.0.1:61072 - "GET /webapp/static/pin_32.png HTTP/1.1" 200 OK

@kunaljubce

Can you try and fix the view @kunaljubce. The old UI where we had the view is gone, and there's a new one now

Add missing tests for the DAG endpoint (#35158)

@kunaljubce

Hi @kunaljubce, I have a PR #51035 to fix the coverage report UI issue, maybe you can take a look.

Add missing tests for the DAG endpoint (#35158)

Navigation Menu

Repository navigation

Improve Code Coverage for Airflow #35127

Description

Metadata

Assignees

Labels

Type

Projects

Milestone

Relationships

Development

Improve Code Coverage for Airflow #35127

Description

Description

Activity

potiuk commented on Nov 5, 2023

kunaljubce commented on Mar 8, 2025

ephraimbuddy commented on Mar 10, 2025

aaron-y-chen commented on May 25, 2025

Metadata

Metadata

Assignees

Labels

Type

Projects

Milestone

Relationships

Development

Issue actions