Support on_warning_callback with TestBehavior.BUILD and ExecutionMode.LOCAL

[corsettigyg]

Description

as of now, when we set TestBehavior.BUILD, we are not leveraging the method on_warning_callback that is available for Test nodes and Source Nodes. I have added the parsing to DbtBuildLocalOperator in order to fix it. I tested it locally and I got positive results

Related Issue(s)

#1569

Breaking Change?

Should not be

Checklist

• I have made corresponding changes to the documentation (if required)
• I have added tests that prove my fix is effective or that my feature works

[netlify]

✅ Deploy Preview for sunny-pastelito-5ecb04 canceled.

Name	Link
🔨 Latest commit	5867fd4
🔍 Latest deploy log	https://app.netlify.com/sites/sunny-pastelito-5ecb04/deploys/67c11983253c2f0008878dde

[corsettigyg]

it seems that the CI is failing due to a different error (#1567)

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 97.35%. Comparing base (0811e46) to head (5867fd4).
Report is 1 commits behind head on main.

Additional details and impacted files

@@           Coverage Diff           @@
##             main    #1571   +/-   ##
=======================================
  Coverage   97.34%   97.35%           
=======================================
  Files          80       80           
  Lines        4903     4920   +17     
=======================================
+ Hits         4773     4790   +17     
  Misses        130      130           

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

[tatiana]

@corsettigyg thanks a lot for improving this.

Ideally, we'd also have feature parity / fix this in the other executors, as an example:

• https://github.com/astronomer/astronomer-cosmos/blob/main/cosmos/operators/kubernetes.py#L120-L128 (which even has a generic class to handle warnings)
• https://github.com/astronomer/astronomer-cosmos/blob/main/cosmos/operators/docker.py#L96-L104
• https://github.com/astronomer/astronomer-cosmos/blob/main/cosmos/operators/gcp_cloud_run_job.py#L126-L134
• https://github.com/astronomer/astronomer-cosmos/blob/main/cosmos/operators/azure_container_instance.py#L104-L112

• astronomer-cosmos/cosmos/operators/aws_ecs.py

  Line 122 in 0811e46

  class DbtBuildAwsEcsOperator(DbtBuildMixin, DbtAwsEcsBaseOperator):

So their build operators also can receive on_warning_callback.

I understand these changes are outside the scope of the current PR, but could you please log a GitHub issue so we can keep track of them?

I'm happy for us to merge this PR once the checks pass.

[corsettigyg]

@tatiana Good catch ! I renamed the name of this PR to specify that it only accommodates the LocalOperator EM. There is an issue logged here that I have created.

Since the CI is all green, happy to contribute :)