From 376cfab88fb1bf5b3069cfdbe5007afa445db3ac Mon Sep 17 00:00:00 2001 From: Jan Dittberner Date: Tue, 2 May 2023 17:57:13 +0200 Subject: [PATCH] Fix output of response_for_exception in tests Django has a django.core.handlers.exception.response_for_exception that handles SuspiciousOperationException, that is expected behaviour and should not be logged in test output. --- gnuviechadmin/gnuviechadmin/settings.py | 5 +++++ gnuviechadmin/userdbs/tests/test_views.py | 7 ++----- 2 files changed, 7 insertions(+), 5 deletions(-) diff --git a/gnuviechadmin/gnuviechadmin/settings.py b/gnuviechadmin/gnuviechadmin/settings.py index d32f9c0..baf6d1c 100644 --- a/gnuviechadmin/gnuviechadmin/settings.py +++ b/gnuviechadmin/gnuviechadmin/settings.py @@ -451,6 +451,11 @@ elif GVA_ENVIRONMENT == "test": ] ) ) + LOGGING["loggers"]["django"] = { + "handlers": ["console"], + "level": "CRITICAL", + "propagate": True, + } BROKER_URL = BROKER_URL + "_test" CELERY_RESULT_PERSISTENT = False else: diff --git a/gnuviechadmin/userdbs/tests/test_views.py b/gnuviechadmin/userdbs/tests/test_views.py index b52cbdf..c51662d 100644 --- a/gnuviechadmin/userdbs/tests/test_views.py +++ b/gnuviechadmin/userdbs/tests/test_views.py @@ -2,10 +2,10 @@ This module provides tests for :py:mod:`userdbs.views`. """ -from unittest.mock import patch, MagicMock +from unittest.mock import MagicMock, patch -from django.test import TestCase from django.contrib.auth import get_user_model +from django.test import TestCase from django.urls import reverse from hostingpackages.models import ( @@ -14,11 +14,9 @@ from hostingpackages.models import ( HostingPackageTemplate, UserDatabaseOption, ) - from userdbs.models import DB_TYPES, UserDatabase from userdbs.views import AddUserDatabase, ChangeDatabaseUserPassword - User = get_user_model() TEST_USER = "test" @@ -27,7 +25,6 @@ TEST_EMAIL = "test@example.org" class HostingPackageAwareTestMixin(object): - # noinspection PyMethodMayBeStatic def _setup_hosting_package(self, customer): template = HostingPackageTemplate.objects.create(