Conversation
|
The Workflow run is cancelling this PR. It has some failed jobs matching ^Pylint$,^Static checks$,^Build docs$,^Spell check docs$,^Backport packages$,^Checks: Helm tests$,^Test OpenAPI*. |
|
I think it's worth adding more unit tests here to make sure everything works. Unfortunately, we are currently migrating to new framework in which writing tests will be much easier to write. This means that we have to wait to merge this change until we have a new framework. |
|
Hello. We have a new test framework for Helm Chart. Can you add unit tests? Best regards, |
94b58ea to
7c39ee5
Compare
|
added. |
|
The PR should be OK to be merged with just subset of tests as it does not modify Core of Airflow. The committers might merge it or can add a label 'full tests needed' and re-run it to run all tests if they see it is needed! |
(cherry picked from commit 644ac1b)
(cherry picked from commit 644ac1b)
(cherry picked from commit 644ac1b)
(cherry picked from commit 644ac1b)
(cherry picked from commit 644ac1b)
closes: #11706
^ Add meaningful description above
Read the Pull Request Guidelines for more information.
In case of fundamental code change, Airflow Improvement Proposal (AIP) is needed.
In case of a new dependency, check compliance with the ASF 3rd Party License Policy.
In case of backwards incompatible changes please leave a note in UPDATING.md.