diff --git a/tests/conftest.py b/tests/conftest.py index f830986..4296c2a 100644 --- a/tests/conftest.py +++ b/tests/conftest.py @@ -2,6 +2,9 @@ from typing import TYPE_CHECKING, Iterator +from django.contrib.sessions.backends.db import SessionStore +from django.test import RequestFactory + import pytest if TYPE_CHECKING: @@ -57,3 +60,12 @@ def keycloak_config(db) -> Iterator[OpenIDConnectConfig]: yield config OpenIDConnectConfig.clear_cache() + + +@pytest.fixture +def auth_request(rf: RequestFactory): + request = rf.get("/some-auth", {"next": "/ignored"}) + session = SessionStore() + session.save() + request.session = session + return request diff --git a/tests/test_init_flow_custom_config.py b/tests/test_init_flow_custom_config.py index e96a229..7772357 100644 --- a/tests/test_init_flow_custom_config.py +++ b/tests/test_init_flow_custom_config.py @@ -4,9 +4,7 @@ from urllib.parse import parse_qs, urlsplit -from django.contrib.sessions.backends.db import SessionStore from django.core.exceptions import DisallowedRedirect -from django.test import RequestFactory import pytest @@ -17,18 +15,6 @@ pytestmark = [pytest.mark.django_db] -@pytest.fixture -def auth_request(rf: RequestFactory): - request = rf.get("/some-auth", {"next": "/ignored"}) - session = SessionStore() - session.save() - request.session = session - return request - - -# Use a proxy model to modify behaviour without needing migrations/models machinery. - - class static_setting: def __init__(self, val): self.val = val @@ -41,6 +27,7 @@ def __set__(self, obj, val): class CustomConfig(OpenIDConnectConfig): + # Use a proxy model to modify behaviour without needing migrations/models machinery. class Meta: proxy = True app_label = "mozilla_django_oidc_db"