Update system tests path in google providers#60043
Update system tests path in google providers#60043Prab-27 wants to merge 2 commits intoapache:mainfrom
Conversation
ca7abe7 to
6e32788
Compare
|
@kacpermuda , Is my approach correct? |
|
I don't think there is a correct approach here, if it fixes the problem then it's fine. Are you able to run system tests with breeze after that change? |
|
Yeah, I should the tests first and then get back to you |
|
Thanks @kacpermuda , I was wrong in file path — it should start with I should have tested the changes locally before updating all the files |
|
I have updated path - And I believe this is unrelated error and paths are updated correctly and I need to update through all Could you please share your thoughts on this ? |
|
@potiuk , When I tried to commit - Ruff failed checks - I'm working on fixing this it but from this(#60676 (comment)) should I revert this |
|
Actually - there is nothing wrong with the original imports- they are correct. |
Yes, I agree! I got the error at that time and spent time fixing it, but I should have checked it after a few days. Thanks a lot !!! I realized I was going in the wrong direction. |
Provider system tests failed due to a path import error. This PR fixes all system test paths
Related discussion : in slack here
^ 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.