ephraimbuddy commented on code in PR #44221:
URL: https://github.com/apache/airflow/pull/44221#discussion_r1864890763
##########
tests_common/test_utils/db.py:
##########
@@ -73,7 +73,12 @@ def initial_db_init():
from airflow.www.extensions.init_appbuilder import init_appbuilder
from airflow.www.extensions.init_auth_manager import get_auth_manager
+ from tests_common.test_utils.compat import AIRFLOW_V_3_0_PLUS
+
db.resetdb()
+ if AIRFLOW_V_3_0_PLUS:
+ db.downgrade(to_revision="5f2621c13b39")
+ db.upgradedb(to_revision="head")
Review Comment:
I can’t see how to limit it to only the test that runs migration. My
thinking is that we can’t limit it due to running tests in parallel and that
doing it specifically for the /utils/db tests could affect other tests running.
Also, the code only runs when we have —with-db-init flag on tests, which i
think happens when tests are initialized and not on every one of the tests
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
