-
Notifications
You must be signed in to change notification settings - Fork 14.3k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Enable "airflow tasks test" to run deferrable operator #37542
Enable "airflow tasks test" to run deferrable operator #37542
Conversation
ab1885f
to
93beff5
Compare
f9f0910
to
f281aef
Compare
We should probably rename the two underscore-leading names, otherwise lgtm. |
Do you mean |
Yes |
I renamed |
1960930
to
853f537
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good, the potential name change suggestion is not binding.
…ator" This reverts commit e6cb2153435616d1189526505a1855e596df4797.
…l mark it as private)
853f537
to
7b55586
Compare
Why
Currently,
airflow tasks test [DAG] [TASK]
cannot test tasks in deferrable mode; it just ends whenTaskDeferred
is raised.What
This PR adds the functionality to run inline triggerer when if
TaskDeferred
is raised when runningairflow tasks test [DAG] [TASK]
^ Add meaningful description above
Read the Pull Request Guidelines for more information.
In case of fundamental code changes, an 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 a newsfragment file, named
{pr_number}.significant.rst
or{issue_number}.significant.rst
, in newsfragments.