From 975b6085770f055eff16f6bd053b28cd45575a12 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?=D0=A1=D1=82=D0=B5=D0=BF=D0=B0=D0=BD=D0=B5=D0=BD=D0=BA?= =?UTF-8?q?=D0=BE=20=D0=9E=D0=BB=D1=8C=D0=B3=D0=B0?= Date: Sun, 23 May 2021 17:09:52 +0300 Subject: [PATCH] Add tests docs --- access_controller/settings.py | 1 + main/extra_func.py | 1 - main/tests.py | 345 ++++++++++++++++++++++++++++------ 3 files changed, 288 insertions(+), 59 deletions(-) diff --git a/access_controller/settings.py b/access_controller/settings.py index 6293efc..30875f6 100644 --- a/access_controller/settings.py +++ b/access_controller/settings.py @@ -10,6 +10,7 @@ For the full list of settings and their values, see https://docs.djangoproject.com/en/3.1/ref/settings/ """ import os + from pathlib import Path diff --git a/main/extra_func.py b/main/extra_func.py index 53bf919..e652a7e 100644 --- a/main/extra_func.py +++ b/main/extra_func.py @@ -29,7 +29,6 @@ def update_role(user_profile: UserProfile, role: int, who_changes: get_user_mode :param user_profile: Профиль пользователя :param role: Новая роль :param who_changes: Пользователь, меняющий роль - :return: Пользователь с обновленной ролью """ zendesk = zenpy user = zendesk.get_user(user_profile.user.email) diff --git a/main/tests.py b/main/tests.py index 60f31ed..4353ddf 100644 --- a/main/tests.py +++ b/main/tests.py @@ -1,7 +1,8 @@ import random from unittest.mock import patch, Mock -from django.contrib.auth.models import User +from django.contrib.auth import get_user_model +# from django.contrib.auth.models import User from django.core import mail from django.http import HttpResponseRedirect from django.template.loader import render_to_string @@ -16,47 +17,112 @@ from main.extra_func import log class UsersBaseTestCase(TestCase): - """Базовый класс загружения данных для тестов с пользователями""" + """ + Базовый класс загрузки данных для тестов с пользователями. + + Для тестов используются фикстуры тестовых пользователей (test_users.json). + """ fixtures = ['fixtures/test_users.json'] - def setUp(self): - """Добавление в переменные почт и клиентов для пользователей""" + def setUp(self) -> None: + """ + Функция предустановки значений переменных. + + Добавляем email тестовых пользователей и создаем клиентов для тестов. + + :param light_agent: email тестового пользователя с правами light_agent + :type light_agent: :class:`str` + :param engineer: email тестового пользователя с правами engineer + :type engineer: :class:`str` + :param admin: email тестового пользователя с правами admin + :type admin: :class:`str` + :param agent_client: клиент, залогиненный как пользователь с email light_agent + :type agent_client: :class:`django.test.client.Client` + :param engineer_client: клиент, залогиненный как пользователь с email engineer + :type engineer_client: :class:`django.test.client.Client` + :param admin_client: клиент, залогиненный как пользователь с email admin + :type admin_client: :class:`django.test.client.Client` + """ self.light_agent = '123@test.ru' self.admin = 'admin@gmail.com' - self.engineer = 'customer@example.com' + self.engineer = 'customer@example.com' self.agent_client = Client() - self.agent_client.force_login(User.objects.get(email=self.light_agent)) + self.agent_client.force_login(get_user_model().objects.get(email=self.light_agent)) self.admin_client = Client() - self.admin_client.force_login(User.objects.get(email=self.admin)) + self.admin_client.force_login(get_user_model().objects.get(email=self.admin)) self.engineer_client = Client() - self.engineer_client.force_login(User.objects.get(email=self.engineer)) + self.engineer_client.force_login(get_user_model().objects.get(email=self.engineer)) class RegistrationTestCase(TestCase): + """ + Класс тестирования регистрации. + + Для тестов используются фикстуры с данными пользователей engeneer и light_agent (data.json). + """ fixtures = ['fixtures/data.json'] - def setUp(self): + def setUp(self) -> None: + """ + Функция предустановки значений переменных. + + Добавляем email тестовых пользователей и создаем клиентов для тестов. + + :param email_backend: locmem бэкенд со списком отправленных писем + :type email_backend: :class:`str` + :param any_zendesk_user_email: email пользователя, зарегистрированного на Zendesk + :type any_zendesk_user_email: :class:`str` + :param zendesk_admin_email: email администратора + :type zendesk_admin_email: :class:`str` + :param client: новый клиент + :type client: :class:`django.test.client.Client` + + """ self.email_backend = 'django.core.mail.backends.locmem.EmailBackend' self.any_zendesk_user_email = 'idar.sokurov.05@mail.ru' self.zendesk_admin_email = 'idar.sokurov.05@mail.ru' self.client = Client() - def test_registration_complete_redirect(self): + def test_registration_complete_redirect(self) -> None: + """ + Функция тестирования успешной регистрации пользователя. + + Проверяет, что в случае если email пользователя зарегистрирован на Zendesk, была заполнена форма регистрации + и направлено письмо со ссылкой для завершения регистрации, происходит редирект на страницу завершения + регистрации. + """ with self.settings(EMAIL_BACKEND=self.email_backend): resp = self.client.post(reverse('registration'), data={'email': self.any_zendesk_user_email}) self.assertRedirects(resp, reverse('password_reset_done')) - def test_registration_fail_redirect(self): + def test_registration_fail_redirect(self) -> None: + """ + Функция тестирования неуспешной регистрации пользователя (введен email, не зарегистированный на Zendesk). + + Проверяет, что происходит редирект на страницу "registration disallowed" + """ with self.settings(EMAIL_BACKEND=self.email_backend): resp = self.client.post(reverse('registration'), data={'email': self.any_zendesk_user_email + 'asd'}) self.assertRedirects(resp, reverse('django_registration_disallowed')) - def test_registration_user_already_exist(self): + def test_registration_user_already_exist(self) -> None: + """ + Функция тестирования попытки зарегистрироваться, используя email уже зарегистрированного в приложении + пользователя ("123@test.ru"). + + Проверяет, что пользователь получает сообщение "Этот адрес электронной почты уже используется" + """ with self.settings(EMAIL_BACKEND=self.email_backend) and translation.override('ru'): resp = self.client.post(reverse('registration'), data={'email': '123@test.ru'}) self.assertContains(resp, 'Этот адрес электронной почты уже используется', count=1, status_code=200) - def test_registration_send_email(self): + def test_registration_send_email(self) -> None: + """ + Функция тестирования отправки email пользователю при регистрации. + + Проверяет отправку уведомления на указанный пользователем адрес, а также содержание письма (заголовка и тела) + через email locmem backend. + """ with self.settings(EMAIL_BACKEND=self.email_backend): response: HttpResponseRedirect = \ self.client.post(reverse('registration'), data={'email': self.any_zendesk_user_email}) @@ -71,50 +137,88 @@ class RegistrationTestCase(TestCase): correct_body = render_to_string('registration/password_reset_email.html', email_context, response.request) self.assertEqual(mail.outbox[0].body, correct_body) - def test_registration_user_creating(self): + def test_registration_user_creating(self) -> None: + """ + Функция тестирования создания пользователя приложения при регистрации. + + Проверяет соответствие имени созданного пользователя с именем пользователя в Zendesk + """ with self.settings(EMAIL_BACKEND=self.email_backend): self.client.post(reverse('registration'), data={'email': self.any_zendesk_user_email}) - user = User.objects.get(email=self.any_zendesk_user_email) + user = get_user_model().objects.get(email=self.any_zendesk_user_email) zendesk_user = zenpy.get_user(self.any_zendesk_user_email) self.assertEqual(user.userprofile.name, zendesk_user.name) - def test_permissions_applying(self): + def test_permissions_applying(self) -> None: + """ + Функция тестирования создания администратора и присвоения ему соответствующих прав. + + Проверяет, что у созданного пользователя роль "admin" и права "has_control_access". + """ with self.settings(EMAIL_BACKEND=self.email_backend): self.client.post(reverse('registration'), data={'email': self.zendesk_admin_email}) - user = User.objects.get(email=self.zendesk_admin_email) + user = get_user_model().objects.get(email=self.zendesk_admin_email) self.assertEqual(user.userprofile.role, 'admin') self.assertTrue(user.has_perm('main.has_control_access')) class MakeEngineerTestCase(UsersBaseTestCase): + """ + Класс тестирования присвоения пользователю роли engineer. + + В тестах используется @patch('main.extra_func.zenpy') замещающий API Zendesk. + """ @patch('main.extra_func.zenpy') - def test_become_engineer_redirect(self, _zenpy_mock): - user = User.objects.get(email=self.light_agent) + def test_become_engineer_redirect(self, _zenpy_mock: zenpy) -> None: + """ + Функция тестирования редиректа на рабочую страницу тестового пользователя при назначении его инженером. + """ + user = get_user_model().objects.get(email=self.light_agent) resp = self.agent_client.post(reverse_lazy('work_become_engineer')) self.assertRedirects(resp, reverse('work', args=[user.id])) self.assertEqual(resp.status_code, 302) @patch('main.extra_func.zenpy') - def test_light_agent_make_engineer(self, zenpy_mock): + def test_light_agent_make_engineer(self, zenpy_mock: zenpy) -> None: + """ + Функция тестирования назначения легкого агента на роль инженера. + + Проверяет установку роли "engineer" в Zendesk. + """ self.agent_client.post(reverse_lazy('work_become_engineer')) self.assertEqual(zenpy_mock.update_user.call_args[0][0].custom_role_id, sets.ZENDESK_ROLES['engineer']) @patch('main.extra_func.zenpy') - def test_admin_make_engineer(self, zenpy_mock): + def test_admin_make_engineer(self, zenpy_mock: zenpy) -> None: + """ + Функция тестирования назначения администратора на роль инженера. + + Проверяет установку роли "engineer" в Zendesk. + """ self.admin_client.post(reverse_lazy('work_become_engineer')) self.assertEqual(zenpy_mock.update_user.call_args[0][0].custom_role_id, sets.ZENDESK_ROLES['engineer']) @patch('main.extra_func.zenpy') - def test_engineer_make_engineer(self, zenpy_mock): + def test_engineer_make_engineer(self, zenpy_mock: zenpy) -> None: + """ + Функция тестирования назначения инженера на роль инженера. + + Проверяет установку роли "engineer" в Zendesk. + """ self.engineer_client.post(reverse_lazy('work_become_engineer')) self.assertEqual(zenpy_mock.update_user.call_args[0][0].custom_role_id, sets.ZENDESK_ROLES['engineer']) @patch('main.extra_func.zenpy') - def test_control_page_make_engineer_one(self, zenpy_mock): + def test_control_page_make_engineer_one(self, zenpy_mock: zenpy) -> None: + """ + Функция тестирования назначения администратором одного инженера на странице "Управление". + + Проверяет обновление администратором роли пользователя с light_agent на engineer. + """ self.admin_client.post( reverse_lazy('control'), - data={'users': [User.objects.get(email=self.light_agent).userprofile.id], 'engineer': 'engineer'} + data={'users': [get_user_model().objects.get(email=self.light_agent).userprofile.id], 'engineer': 'engineer'} ) call_list = zenpy_mock.update_user.call_args_list mock_object = call_list[0][0][0] @@ -122,13 +226,18 @@ class MakeEngineerTestCase(UsersBaseTestCase): self.assertEqual(mock_object.custom_role_id, sets.ZENDESK_ROLES['engineer']) @patch('main.extra_func.zenpy') - def test_control_page_make_engineer_many(self, zenpy_mock): + def test_control_page_make_engineer_many(self, zenpy_mock: zenpy) -> None: + """ + Функция тестирования назначения администратором нескольких инженеров на странице "Управление". + + Проверяет обновление администратором ролей двух пользователей с light_agent на engineer. + """ self.admin_client.post( reverse_lazy('control'), data={ 'users': [ - User.objects.get(email=self.light_agent).userprofile.id, - User.objects.get(email=self.engineer).userprofile.id, + get_user_model().objects.get(email=self.light_agent).userprofile.id, + get_user_model().objects.get(email=self.engineer).userprofile.id, ], 'engineer': 'engineer' } @@ -141,18 +250,33 @@ class MakeEngineerTestCase(UsersBaseTestCase): class MakeLightAgentTestCase(UsersBaseTestCase): + """ + Класс тестирования присвоения пользователю роли light_agent. + + В тестах используется @patch('main.extra_func.zenpy') замещающий API Zendesk, а также + @patch('main.requester.TicketListRequester.get_tickets_list_for_user', side_effect=[[]]), предоставляющий список + тикетов. + """ @patch('main.requester.TicketListRequester.get_tickets_list_for_user', side_effect=[[]]) @patch('main.extra_func.zenpy') - def test_hand_over_redirect(self, _zenpy_mock, _user_tickets_mock): - user = User.objects.get(email=self.engineer) + def test_hand_over_redirect(self, _zenpy_mock: zenpy, _user_tickets_mock: list) -> None: + """ + Функция тестирования переадресации инженера на рабочую страницу, после сдачи прав. + """ + user = get_user_model().objects.get(email=self.engineer) resp = self.engineer_client.post(reverse_lazy('work_hand_over')) self.assertRedirects(resp, reverse('work', args=[user.id])) self.assertEqual(resp.status_code, 302) @patch('main.requester.TicketListRequester.get_tickets_list_for_user', side_effect=[[]]) @patch('main.extra_func.zenpy') - def test_engineer_make_light_agent_no_tickets(self, zenpy_mock, _user_tickets_mock): + def test_engineer_make_light_agent_no_tickets(self, zenpy_mock: zenpy, _user_tickets_mock: list) -> None: + """ + Функция тестирования назначения инженера легким агентом, в случае, когда у него в работе нет тикетов. + + Проверяет назначение роли light_agent в Zendesk. + """ self.engineer_client.post(reverse_lazy('work_hand_over')) self.assertEqual(zenpy_mock.update_user.call_args[0][0].custom_role_id, sets.ZENDESK_ROLES['light_agent']) @@ -160,7 +284,15 @@ class MakeLightAgentTestCase(UsersBaseTestCase): [Mock(id=1, status='solved'), Mock(id=2, status='open'), Mock(id=3, status='open')] ]) @patch('main.extra_func.zenpy') - def test_engineer_make_light_agent_with_tickets(self, zenpy_mock, _user_tickets_mock): + def test_engineer_make_light_agent_with_tickets(self, zenpy_mock: zenpy, _user_tickets_mock: list): + """ + Функция тестирования назначения инженера легким агентом, в случае, когда у него в работе есть тикеты. + + Для тестирования принимается, что в работе у инженера находится 3 тикета, один в состоянии: решен, + два в состоянии: открыт. + Проверяет распределение тикетов (поместить в решенные или назначить нового ответственного), + а также назначение роли light_agent в Zendesk. + """ zenpy_mock.solved_tickets_user_id = Mock() self.engineer_client.post(reverse_lazy('work_hand_over')) @@ -172,7 +304,12 @@ class MakeLightAgentTestCase(UsersBaseTestCase): @patch('main.requester.TicketListRequester.get_tickets_list_for_user', side_effect=[[]]) @patch('main.extra_func.zenpy') - def test_admin_make_light_agent_no_tickets(self, zenpy_mock, _user_tickets_mock): + def test_admin_make_light_agent_no_tickets(self, zenpy_mock: zenpy, _user_tickets_mock: list) -> None: + """ + Функция тестирования назначения администратора на роль легкого агента. + + Проверяет назначение роли light_agent в Zendesk. + """ self.admin_client.post(reverse_lazy('work_hand_over')) self.assertEqual(zenpy_mock.update_user.call_args[0][0].custom_role_id, sets.ZENDESK_ROLES['light_agent']) @@ -180,7 +317,15 @@ class MakeLightAgentTestCase(UsersBaseTestCase): [Mock(id=1, status='solved'), Mock(id=2, status='open'), Mock(id=3, status='open')] ]) @patch('main.extra_func.zenpy') - def test_admin_make_light_agent_with_tickets(self, zenpy_mock, _user_tickets_mock): + def test_admin_make_light_agent_with_tickets(self, zenpy_mock: zenpy, _user_tickets_mock: list) -> None: + """ + Функция тестирования назначения администратора легким агентом, в случае, когда у него в работе есть тикеты. + + Для тестирования принимается, что в работе находится 3 тикета, один в состоянии: решен, + два в состоянии: открыт. + Проверяет распределение тикетов (поместить в решенные или назначить нового ответственного), + а также назначение роли light_agent в Zendesk. + """ zenpy_mock.solved_tickets_user_id = Mock() self.admin_client.post(reverse_lazy('work_hand_over')) @@ -192,16 +337,26 @@ class MakeLightAgentTestCase(UsersBaseTestCase): @patch('main.requester.TicketListRequester.get_tickets_list_for_user', side_effect=[[]]) @patch('main.extra_func.zenpy') - def test_light_agent_make_light_agent(self, zenpy_mock, _user_tickets_mock): + def test_light_agent_make_light_agent(self, zenpy_mock: zenpy, _user_tickets_mock: list): + """ + Функция тестирования назначения легкого агента на роль легкого агента. + + Проверяет назначение роли light_agent в Zendesk. + """ self.agent_client.post(reverse_lazy('work_hand_over')) self.assertEqual(zenpy_mock.update_user.call_args[0][0].custom_role_id, sets.ZENDESK_ROLES['light_agent']) @patch('main.requester.TicketListRequester.get_tickets_list_for_user', side_effect=[[]]) @patch('main.extra_func.zenpy') - def test_control_page_make_light_agent_one(self, zenpy_mock, _user_tickets_mock): + def test_control_page_make_light_agent_one(self, zenpy_mock: zenpy, _user_tickets_mock: list) -> None: + """ + Функция тестирования назначения администратором одного легкого агента на странице "Управление". + + Проверяет обновление администратором роли пользователя с engineer на light_agent. + """ self.admin_client.post( reverse_lazy('control'), - data={'users': [User.objects.get(email=self.engineer).userprofile.id], 'light_agent': 'light_agent'} + data={'users': [get_user_model().objects.get(email=self.engineer).userprofile.id], 'light_agent': 'light_agent'} ) call_list = zenpy_mock.update_user.call_args_list mock_object = call_list[0][0][0] @@ -210,13 +365,19 @@ class MakeLightAgentTestCase(UsersBaseTestCase): @patch('main.requester.TicketListRequester.get_tickets_list_for_user', side_effect=[[], []]) @patch('main.extra_func.zenpy') - def test_control_page_make_light_agent_many(self, zenpy_mock, _user_tickets_mock): + def test_control_page_make_light_agent_many(self, zenpy_mock: zenpy, _user_tickets_mock: list) -> None: + """ + Функция тестирования назначения администратором нескольких легких агентов на странице "Управление". + + Проверяет обновление администратором ролей двух пользователей с engineer на light_agent. + """ + self.admin_client.post( reverse_lazy('control'), data={ 'users': [ - User.objects.get(email=self.light_agent).userprofile.id, - User.objects.get(email=self.engineer).userprofile.id, + get_user_model().objects.get(email=self.light_agent).userprofile.id, + get_user_model().objects.get(email=self.engineer).userprofile.id, ], 'light_agent': 'light_agent' } @@ -229,18 +390,32 @@ class MakeLightAgentTestCase(UsersBaseTestCase): class PasswordResetTestCase(UsersBaseTestCase): + """ + Класс тестирования сброса пароля. + """ def setUp(self): super().setUp() self.email_backend = 'django.core.mail.backends.locmem.EmailBackend' - def test_redirect(self): + def test_redirect(self) -> None: + """ + Функция тестирования успешной смены пароля. + + Проверяется переадресация на страницу завершения смены пароля, в случае, когда пользователь существует и на его + email было направлено письмо для сброса пароля. + """ with self.settings(EMAIL_BACKEND=self.email_backend): resp = self.agent_client.post(reverse_lazy('password_reset'), data={'email': self.light_agent}) self.assertRedirects(resp, reverse('password_reset_done')) self.assertEqual(resp.status_code, 302) - def test_send_email(self): + def test_send_email(self) -> None: + """ + Функция тестирования отправки email для сброса пароля. + + Проверяет наличие отправленного письма, и его содержание, сверяет email адресата с email пользователя. + """ with self.settings(EMAIL_BACKEND=self.email_backend): response: HttpResponseRedirect = \ self.agent_client.post(reverse_lazy('password_reset'), data={'email': self.light_agent}) @@ -255,12 +430,22 @@ class PasswordResetTestCase(UsersBaseTestCase): correct_body = render_to_string('registration/password_reset_email.html', email_context, response.request) self.assertEqual(mail.outbox[0].body, correct_body) - def test_email_invalid(self): + def test_email_invalid(self) -> None: + """ + Функция тестирования попытки смены пароля с некорректным email. + + Проверяет уведомление пользователя о неверном адресе электронной почты. + """ with self.settings(EMAIL_BACKEND=self.email_backend) and translation.override('ru'): resp = self.agent_client.post(reverse_lazy('password_reset'), data={'email': 1}) self.assertContains(resp, 'Введите правильный адрес электронной почты.', count=1, status_code=200) - def test_user_does_not_exist(self): + def test_user_does_not_exist(self) -> None: + """ + Функция тестирования попытки смены пароля с email, который не зарегистрирован. + + Проверяет отсутствие отправки письма о смене пароля. + """ with self.settings(EMAIL_BACKEND=self.email_backend): resp = self.agent_client.post(reverse_lazy('password_reset'), data={'email': self.light_agent + str(random.random())}) self.assertRedirects(resp, reverse('password_reset_done')) @@ -269,18 +454,31 @@ class PasswordResetTestCase(UsersBaseTestCase): class PasswordChangeTestCase(UsersBaseTestCase): + """ + Класс тестирования смены пароля. + """ - def setUp(self): + def setUp(self) -> None: super().setUp() self.set_password() - def set_password(self): - user: User = User.objects.get(email=self.light_agent) + def set_password(self) -> None: + """ + Функция предустанавливает тестовому пользователю с ролью light_agent пароль 'ImpossiblyHardPassword' и создает + клиента с соответствующими данным для тестирования. + """ + user = get_user_model().objects.get(email=self.light_agent) user.set_password('ImpossiblyHardPassword') user.save() - self.agent_client.force_login(User.objects.get(email=self.light_agent)) + self.agent_client.force_login(get_user_model().objects.get(email=self.light_agent)) - def test_change_successful(self): + def test_change_successful(self) -> None: + """ + Функция тестирования успешной смены пароля. + + Проверяет установку нового пароля пользователю при вводе корректных данных: старый пароль, новый пароль + (2 раза). + """ self.agent_client.post( reverse_lazy('password_change'), data={ @@ -289,10 +487,15 @@ class PasswordChangeTestCase(UsersBaseTestCase): 'new_password2': 'EasyPassword', } ) - user = User.objects.get(email=self.light_agent) + user = get_user_model().objects.get(email=self.light_agent) self.assertTrue(user.check_password('EasyPassword')) - def test_invalid_old_password(self): + def test_invalid_old_password(self) -> None: + """ + Функция тестирования смены пароля, при неверном вводе старого пароля. + + Проверяет текст уведомления пользователя 'Ваш старый пароль введен неправильно'. + """ with translation.override('ru'): resp = self.agent_client.post( reverse_lazy('password_change'), @@ -304,7 +507,12 @@ class PasswordChangeTestCase(UsersBaseTestCase): ) self.assertContains(resp, 'Ваш старый пароль введен неправильно', count=1, status_code=200) - def test_different_new_passwords(self): + def test_different_new_passwords(self) -> None: + """ + Функция тестирования смены пароля, при вводе несовпадающих новых паролей. + + Проверяет текст уведомления пользователя 'Введенные пароли не совпадают'. + """ with translation.override('ru'): resp = self.agent_client.post( reverse_lazy('password_change'), @@ -317,6 +525,11 @@ class PasswordChangeTestCase(UsersBaseTestCase): self.assertContains(resp, 'Введенные пароли не совпадают', count=1, status_code=200) def test_invalid_new_password1(self): + """ + Функция тестирования попытки смены пароля, когда новый пароль не соответствует требованиям: слишком короткий. + + Проверяет текст уведомления пользователя 'Введённый пароль слишком короткий'. + """ with translation.override('ru'): resp = self.agent_client.post( reverse_lazy('password_change'), @@ -328,7 +541,13 @@ class PasswordChangeTestCase(UsersBaseTestCase): ) self.assertContains(resp, 'Введённый пароль слишком короткий', count=1, status_code=200) - def test_invalid_new_password2(self): + def test_invalid_new_password2(self) -> None: + """ + Функция тестирования попытки смены пароля, когда новый пароль не соответствует требованиям: состоит + только из цифр. + + Проверяет текст уведомления пользователя 'Введённый пароль состоит только из цифр'. + """ with translation.override('ru'): resp = self.agent_client.post( reverse_lazy('password_change'), @@ -341,6 +560,12 @@ class PasswordChangeTestCase(UsersBaseTestCase): self.assertContains(resp, 'Введённый пароль состоит только из цифр', count=1, status_code=200) def test_invalid_new_password3(self): + """ + Функция тестирования попытки смены пароля, когда новый пароль не соответствует требованиям: аналогчен имени + пользователя. + + Проверяет текст уведомления пользователя 'Введённый пароль слишком похож на имя пользователя'. + """ with translation.override('ru'): resp = self.agent_client.post( reverse_lazy('password_change'), @@ -356,6 +581,8 @@ class PasswordChangeTestCase(UsersBaseTestCase): class GetTicketsTestCase(UsersBaseTestCase): """ Класс тестов для проверки функции получения тикетов. + + """ @patch('main.views.zenpy.get_user') @@ -365,7 +592,7 @@ class GetTicketsTestCase(UsersBaseTestCase): Функция проверки переадресации пользователя на рабочую страницу. """ get_user_mock.return_value = Mock() - user = User.objects.get(email=self.engineer) + user = get_user_model().objects.get(email=self.engineer) resp = self.engineer_client.post(reverse('work_get_tickets')) self.assertRedirects(resp, reverse('work', args=[user.id])) self.assertEqual(resp.status_code, 302) @@ -434,6 +661,8 @@ class GetTicketsTestCase(UsersBaseTestCase): class ProfileTestCase(TestCase): """ Класс тестов для проверки синхронизации профиля пользователя. + + """ fixtures = ['fixtures/profile.json'] @@ -444,9 +673,9 @@ class ProfileTestCase(TestCase): self.zendesk_agent_email = 'krav-88@mail.ru' self.zendesk_admin_email = 'idar.sokurov.05@mail.ru' self.client = Client() - self.client.force_login(User.objects.get(email=self.zendesk_agent_email)) + self.client.force_login(get_user_model().objects.get(email=self.zendesk_agent_email)) self.admin_client = Client() - self.admin_client.force_login(User.objects.get(email=self.zendesk_admin_email)) + self.admin_client.force_login(get_user_model().objects.get(email=self.zendesk_admin_email)) def test_correct_username(self): """ @@ -495,9 +724,9 @@ class LoggingTestCase(UsersBaseTestCase): def setUp(self): super().setUp() - self.admin_profile = User.objects.get(email=self.admin).userprofile - self.agent_profile = User.objects.get(email=self.light_agent).userprofile - self.engineer_profile = User.objects.get(email=self.engineer).userprofile + self.admin_profile = get_user_model().objects.get(email=self.admin).userprofile + self.agent_profile = get_user_model().objects.get(email=self.light_agent).userprofile + self.engineer_profile = get_user_model().objects.get(email=self.engineer).userprofile @staticmethod def get_file_output():