potiuk commented on PR #39513: URL: https://github.com/apache/airflow/pull/39513#issuecomment-2104130905
cc: @ashb @kaxil -> per our discussion - there are still >~10 tests to fix + review of those changes I've already applied to make Provider's test suite compatible with 2.9.1. This is what I told you will be the "difficculty" with making the provider's test suite of ours work with past versions of Airflow. For 2.9 it's not a lot (but sill an effort to make and it will make our test code slightly more complicated because it will have to handle all the compatibility issues. What you see in this PR in "tests" folder are the changes needed so far. This is just after ~ month of refactors and chnages after we split-off main for 2.10. I guess for 2.8 and 2.7 there will be quite a bit more places where our tests make use of some internals of airflow - for assertions, mocking, imports and the like. Let's get the 2.9.1 change merged first and I will attempt to do 2.8 and 2.7 after to see how far we can go to not clutter our Provider's test suite too much with compatiblity code. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: commits-unsubscr...@airflow.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
