This is an automated email from the ASF dual-hosted git repository.

potiuk pushed a commit to branch remove-legacy-ui
in repository https://gitbox.apache.org/repos/asf/airflow.git


The following commit(s) were added to refs/heads/remove-legacy-ui by this push:
     new 47a163cdd14 fixup! fixup! Fix compat tests
47a163cdd14 is described below

commit 47a163cdd1442143fba6d732e2f7dfb548aa6853
Author: Jarek Potiuk <[email protected]>
AuthorDate: Wed Feb 26 21:02:47 2025 +0100

    fixup! fixup! Fix compat tests
---
 .../tests/unit/google/common/auth_backend/test_google_openid.py     | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git 
a/providers/google/tests/unit/google/common/auth_backend/test_google_openid.py 
b/providers/google/tests/unit/google/common/auth_backend/test_google_openid.py
index 8563c300003..938e63bb9a0 100644
--- 
a/providers/google/tests/unit/google/common/auth_backend/test_google_openid.py
+++ 
b/providers/google/tests/unit/google/common/auth_backend/test_google_openid.py
@@ -23,7 +23,7 @@ from google.auth.exceptions import GoogleAuthError
 
 from airflow.providers.google.version_compat import AIRFLOW_V_3_0_PLUS
 
-if AIRFLOW_V_3_0_PLUS:
+if not AIRFLOW_V_3_0_PLUS:
     pytest.skip(
         
"``providers/google/tests/unit/google/common/auth_backend/test_google_openid.py``
 is only compatible with Airflow 2.X.",
         allow_module_level=True,
@@ -35,8 +35,6 @@ from tests_common.test_utils.db import clear_db_pools
 
 @pytest.fixture(scope="module")
 def google_openid_app():
-    from airflow.providers.fab.www.app import create_app
-
     def factory():
         with conf_vars(
             {
@@ -47,6 +45,8 @@ def google_openid_app():
                 ): 
"airflow.providers.fab.auth_manager.fab_auth_manager.FabAuthManager",
             }
         ):
+            from airflow.providers.fab.www.app import create_app
+
             _app = create_app(enable_plugins=False)
             _app.config["AUTH_ROLE_PUBLIC"] = None
             return _app

Reply via email to