Fix Tinkerpop integration tests execution envrionment variable name#52826
Conversation
|
cc: @ahmadtfarhan |
|
Yeah. We changed it recently .. and not fully. I also prefer to keep consistency with "provider" name rather than with "tested" thingie (even though technically this is integration with |
…iable name (#52826) (cherry picked from commit 53cb9f0) Co-authored-by: GPK <[email protected]>
…iable name (apache#52826) (cherry picked from commit 53cb9f0) Co-authored-by: GPK <[email protected]>
Totally agree! I just never had the time to do the change. |
…iable name (#52826) (#52830) (cherry picked from commit 53cb9f0) Co-authored-by: GPK <[email protected]>
We have fixed integration test discovery to discover only provided integration, here #52462 , I have missed updating env vars casuing the tests skipping 😞
This fixes them now.
^ 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.rstor{issue_number}.significant.rst, in airflow-core/newsfragments.