This is an automated email from the ASF dual-hosted git repository.
potiuk 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 3f38c4cffef remove pytest db_test marker where unnecessary (#52171)
3f38c4cffef is described below
commit 3f38c4cffef9949c16de0b8dc89cfd9371a24be0
Author: Ankit Chaurasia <[email protected]>
AuthorDate: Wed Jun 25 01:20:32 2025 +0545
remove pytest db_test marker where unnecessary (#52171)
---
.../tests/unit/fab/auth_manager/api/auth/backend/test_basic_auth.py | 1 -
.../fab/tests/unit/fab/auth_manager/api/auth/backend/test_session.py | 1 -
.../fab/tests/unit/fab/auth_manager/api_fastapi/routes/test_login.py | 3 ---
.../tests/unit/fab/auth_manager/api_fastapi/services/test_login.py | 1 -
providers/fab/tests/unit/fab/auth_manager/cli_commands/test_utils.py | 2 --
providers/fab/tests/unit/fab/auth_manager/test_fab_auth_manager.py | 5 -----
6 files changed, 13 deletions(-)
diff --git
a/providers/fab/tests/unit/fab/auth_manager/api/auth/backend/test_basic_auth.py
b/providers/fab/tests/unit/fab/auth_manager/api/auth/backend/test_basic_auth.py
index 845e601a947..b48b33a4fc2 100644
---
a/providers/fab/tests/unit/fab/auth_manager/api/auth/backend/test_basic_auth.py
+++
b/providers/fab/tests/unit/fab/auth_manager/api/auth/backend/test_basic_auth.py
@@ -59,7 +59,6 @@ def function_decorated():
mock_call()
[email protected]_test
class TestBasicAuth:
def setup_method(self) -> None:
mock_call.reset_mock()
diff --git
a/providers/fab/tests/unit/fab/auth_manager/api/auth/backend/test_session.py
b/providers/fab/tests/unit/fab/auth_manager/api/auth/backend/test_session.py
index 296edd11b7b..2d8b1fec909 100644
--- a/providers/fab/tests/unit/fab/auth_manager/api/auth/backend/test_session.py
+++ b/providers/fab/tests/unit/fab/auth_manager/api/auth/backend/test_session.py
@@ -38,7 +38,6 @@ def function_decorated():
mock_call()
[email protected]_test
class TestSessionAuth:
def setup_method(self) -> None:
mock_call.reset_mock()
diff --git
a/providers/fab/tests/unit/fab/auth_manager/api_fastapi/routes/test_login.py
b/providers/fab/tests/unit/fab/auth_manager/api_fastapi/routes/test_login.py
index b596e76f5d9..aaf53d6af20 100644
--- a/providers/fab/tests/unit/fab/auth_manager/api_fastapi/routes/test_login.py
+++ b/providers/fab/tests/unit/fab/auth_manager/api_fastapi/routes/test_login.py
@@ -19,12 +19,9 @@ from __future__ import annotations
from unittest.mock import patch
-import pytest
-
from airflow.providers.fab.auth_manager.api_fastapi.datamodels.login import
LoginBody, LoginResponse
[email protected]_test
class TestLogin:
dummy_login_body = LoginBody(username="dummy", password="dummy")
dummy_token = LoginResponse(access_token="DUMMY_TOKEN")
diff --git
a/providers/fab/tests/unit/fab/auth_manager/api_fastapi/services/test_login.py
b/providers/fab/tests/unit/fab/auth_manager/api_fastapi/services/test_login.py
index 5014177fe0e..89dfd3e1ca4 100644
---
a/providers/fab/tests/unit/fab/auth_manager/api_fastapi/services/test_login.py
+++
b/providers/fab/tests/unit/fab/auth_manager/api_fastapi/services/test_login.py
@@ -29,7 +29,6 @@ if TYPE_CHECKING:
from airflow.providers.fab.auth_manager.api_fastapi.datamodels.login
import LoginResponse
[email protected]_test
@patch("airflow.providers.fab.auth_manager.api_fastapi.services.login.get_auth_manager")
class TestLogin:
def setup_method(
diff --git
a/providers/fab/tests/unit/fab/auth_manager/cli_commands/test_utils.py
b/providers/fab/tests/unit/fab/auth_manager/cli_commands/test_utils.py
index 4cb7485cf1c..262ecd04777 100644
--- a/providers/fab/tests/unit/fab/auth_manager/cli_commands/test_utils.py
+++ b/providers/fab/tests/unit/fab/auth_manager/cli_commands/test_utils.py
@@ -32,8 +32,6 @@ from tests_common.test_utils.config import conf_vars
with ignore_provider_compatibility_error("2.9.0+", __file__):
from airflow.providers.fab.auth_manager.cli_commands.utils import
get_application_builder
-pytestmark = pytest.mark.db_test
-
@pytest.fixture
def flask_app():
diff --git a/providers/fab/tests/unit/fab/auth_manager/test_fab_auth_manager.py
b/providers/fab/tests/unit/fab/auth_manager/test_fab_auth_manager.py
index b4d99b69284..69f3cedf26d 100644
--- a/providers/fab/tests/unit/fab/auth_manager/test_fab_auth_manager.py
+++ b/providers/fab/tests/unit/fab/auth_manager/test_fab_auth_manager.py
@@ -154,7 +154,6 @@ class TestFabAuthManager:
result = auth_manager_with_appbuilder.serialize_user(user)
assert result == {"sub": str(user.id)}
- @pytest.mark.db_test
@mock.patch.object(FabAuthManager, "get_user")
def test_is_logged_in(self, mock_get_user, auth_manager_with_appbuilder):
user = Mock()
@@ -163,7 +162,6 @@ class TestFabAuthManager:
assert auth_manager_with_appbuilder.is_logged_in() is False
- @pytest.mark.db_test
@mock.patch.object(FabAuthManager, "get_user")
def test_is_logged_in_with_inactive_user(self, mock_get_user,
auth_manager_with_appbuilder):
user = Mock()
@@ -690,11 +688,9 @@ class TestFabAuthManager:
delete_user(flask_app, "username")
- @pytest.mark.db_test
def test_security_manager_return_fab_security_manager_override(self,
auth_manager_with_appbuilder):
assert isinstance(auth_manager_with_appbuilder.security_manager,
FabAirflowSecurityManagerOverride)
- @pytest.mark.db_test
def test_security_manager_return_custom_provided(self, flask_app,
auth_manager_with_appbuilder):
class TestSecurityManager(FabAirflowSecurityManagerOverride):
pass
@@ -702,7 +698,6 @@ class TestFabAuthManager:
flask_app.config["SECURITY_MANAGER_CLASS"] = TestSecurityManager
assert isinstance(auth_manager_with_appbuilder.security_manager,
TestSecurityManager)
- @pytest.mark.db_test
def test_security_manager_wrong_inheritance_raise_exception(
self, flask_app, auth_manager_with_appbuilder
):
