move default_args into kwargs

[david-mag]

Description

When using default argument for on_error_callback functions, it is not executed, because its in a sub-dictionary in kwargs.

[david-mag]

Small update to fix a bug where when using on_failure_callback in the default_args, callbacks for dbt tests that fail won't be executed.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 96.24%. Comparing base (110fb07) to head (238164c).
Report is 16 commits behind head on main.

Additional details and impacted files

@@           Coverage Diff           @@
##             main    #1382   +/-   ##
=======================================
  Coverage   96.24%   96.24%           
=======================================
  Files          67       67           
  Lines        4051     4053    +2     
=======================================
+ Hits         3899     3901    +2     
  Misses        152      152           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[tatiana]

Hi @david-mag thanks for your contribution, please, could you add a test to cover this use-case?

[david-mag]

added a test case for this

[pankajastro]

Hey @david-mag Just for my understanding could you please explain what is on_error_callback and share some DAG example how you are using it?

[david-mag]

Oh, my mistake ... I mean on_failure_callback 😳
https://airflow.apache.org/docs/apache-airflow/stable/administration-and-deployment/logging-monitoring/callbacks.html