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

jedcunningham pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/airflow.git


The following commit(s) were added to refs/heads/main by this push:
     new c092d44f170 Disable test because of flakyness (#45947)
c092d44f170 is described below

commit c092d44f17002bfe1ba720365e34960872387571
Author: Vincent <97131062+vincb...@users.noreply.github.com>
AuthorDate: Wed Jan 22 21:38:14 2025 -0500

    Disable test because of flakyness (#45947)
---
 airflow/auth/managers/simple/views/auth.py    | 4 ----
 tests/auth/managers/simple/views/test_auth.py | 3 +++
 2 files changed, 3 insertions(+), 4 deletions(-)

diff --git a/airflow/auth/managers/simple/views/auth.py 
b/airflow/auth/managers/simple/views/auth.py
index 64c697ecbcc..faa12b92b05 100644
--- a/airflow/auth/managers/simple/views/auth.py
+++ b/airflow/auth/managers/simple/views/auth.py
@@ -18,7 +18,6 @@ from __future__ import annotations
 
 from urllib.parse import parse_qsl, urlencode, urlsplit, urlunsplit
 
-import pytest
 from flask import redirect, request, session, url_for
 from flask_appbuilder import expose
 
@@ -64,9 +63,6 @@ class SimpleAuthManagerAuthenticationViews(AirflowBaseView):
         session.clear()
         return redirect(url_for("SimpleAuthManagerAuthenticationViews.login"))
 
-    @pytest.mark.skip(
-        "This test will be deleted soon, meanwhile disabling it because it is 
flaky. See: https://github.com/apache/airflow/issues/45818";
-    )
     @csrf.exempt
     @expose("/login_submit", methods=("GET", "POST"))
     def login_submit(self):
diff --git a/tests/auth/managers/simple/views/test_auth.py 
b/tests/auth/managers/simple/views/test_auth.py
index 633dcbd0f21..25adb0a7eac 100644
--- a/tests/auth/managers/simple/views/test_auth.py
+++ b/tests/auth/managers/simple/views/test_auth.py
@@ -55,6 +55,9 @@ class TestSimpleAuthManagerAuthenticationViews:
             assert response.location == "/login"
             assert session.get("user") is None
 
+    @pytest.mark.skip(
+        "This test will be deleted soon, meanwhile disabling it because it is 
flaky. See: https://github.com/apache/airflow/issues/45818";
+    )
     @pytest.mark.parametrize(
         "username, password, is_successful, query_params, expected_redirect",
         [

Reply via email to