From aba5d9d563c9e9745cd8af4fca692298c1d8779c 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: Sat, 15 May 2021 15:44:21 +0300 Subject: [PATCH] Rename parametrs to snack_case. Pylint 10. --- README.rst | 3 + .../registration/password_reset_form.html | 2 +- main/tests.py | 88 ++++++++++++++----- 3 files changed, 68 insertions(+), 25 deletions(-) diff --git a/README.rst b/README.rst index bfd4aff..e3c1a03 100644 --- a/README.rst +++ b/README.rst @@ -172,6 +172,9 @@ Quickstart pylint --django-settings-module=access_controller_new.access_controller.settings ../access_controller_new +Вместо "access_controller_new" необходимо указать папку проекта. + + Для приведения файлов к стандарту PEP8 используем: -------------------------------------------------- diff --git a/main/templates/registration/password_reset_form.html b/main/templates/registration/password_reset_form.html index 39cf045..f2980e4 100644 --- a/main/templates/registration/password_reset_form.html +++ b/main/templates/registration/password_reset_form.html @@ -12,7 +12,7 @@

Введте свой e-mail адрес для восстановления пароля.

{{ form.as_p }} -

+

{% csrf_token %}
{% endblock %} diff --git a/main/tests.py b/main/tests.py index 61ff9e3..9d87037 100644 --- a/main/tests.py +++ b/main/tests.py @@ -196,7 +196,7 @@ class PasswordResetTestCase(TestCase): def setUp(self): """ - Предустановленные значения для проведения тестов + Предустановленные значения для проведения тестов. """ self.user = '123@test.ru' self.email_backend = 'django.core.mail.backends.locmem.EmailBackend' @@ -204,12 +204,18 @@ class PasswordResetTestCase(TestCase): self.client.force_login(get_user_model().objects.get(email=self.user)) def test_redirect(self): + """ + Функция проверки переадресации на страницу уведомления о сбросе пароля на email. + """ with self.settings(EMAIL_BACKEND=self.email_backend): resp = self.client.post(reverse_lazy('password_reset'), data={'email': self.user}) self.assertRedirects(resp, reverse('password_reset_done')) self.assertEqual(resp.status_code, 302) def test_send_email(self): + """ + Функция проверки содержания и отправки письма для установки пароля. + """ with self.settings(EMAIL_BACKEND=self.email_backend): response: HttpResponseRedirect = \ self.client.post(reverse_lazy('password_reset'), data={'email': self.user}) @@ -225,11 +231,17 @@ class PasswordResetTestCase(TestCase): self.assertEqual(mail.outbox[0].body, correct_body) def test_email_invalid(self): + """ + Функция проверки уведомления клиента о некорректности введенного email. + """ with self.settings(EMAIL_BACKEND=self.email_backend) and translation.override('ru'): resp = self.client.post(reverse_lazy('password_reset'), data={'email': 1}) self.assertContains(resp, 'Введите правильный адрес электронной почты.', count=1, status_code=200) def test_user_does_not_exist(self): + """ + Функция корректности отработки неверно введенного email. + """ with self.settings(EMAIL_BACKEND=self.email_backend): resp = self.client.post(reverse_lazy('password_reset'), data={'email': self.user + str(random.random())}) self.assertRedirects(resp, reverse('password_reset_done')) @@ -238,21 +250,33 @@ class PasswordResetTestCase(TestCase): class PasswordChangeTestCase(TestCase): + """ + Класс тестирования смены пароля. + """ fixtures = ['fixtures/test_make_engineer.json'] def setUp(self): + """ + Предустановленные значения для проведения тестов. + """ self.user = '123@test.ru' self.client = Client() self.client.force_login(get_user_model().objects.get(email=self.user)) self.set_password() def set_password(self): + """ + Пароль, сформированный для тестирования. + """ user: get_user_model() = get_user_model().objects.get(email=self.user) user.set_password('ImpossiblyHardPassword') user.save() self.client.force_login(get_user_model().objects.get(email=self.user)) def test_change_successful(self): + """ + Функция тестирования успешного изменения пароля. + """ self.client.post( reverse_lazy('password_change'), data={ @@ -265,6 +289,9 @@ class PasswordChangeTestCase(TestCase): self.assertTrue(user.check_password('EasyPassword')) def test_invalid_old_password(self): + """ + Функция тестирования отработки неверно введенного старого пароля при смене. + """ with translation.override('ru'): resp = self.client.post( reverse_lazy('password_change'), @@ -277,6 +304,9 @@ class PasswordChangeTestCase(TestCase): self.assertContains(resp, 'Ваш старый пароль введен неправильно', count=1, status_code=200) def test_different_new_passwords(self): + """ + Функция тестирования случая с вводом двух разных новых паролей. + """ with translation.override('ru'): resp = self.client.post( reverse_lazy('password_change'), @@ -289,6 +319,9 @@ class PasswordChangeTestCase(TestCase): self.assertContains(resp, 'Введенные пароли не совпадают', count=1, status_code=200) def test_invalid_new_password1(self): + """ + Функция тестирования случая с неправильно подобранным новым паролем (слишком короткий). + """ with translation.override('ru'): resp = self.client.post( reverse_lazy('password_change'), @@ -301,6 +334,9 @@ class PasswordChangeTestCase(TestCase): self.assertContains(resp, 'Введённый пароль слишком короткий', count=1, status_code=200) def test_invalid_new_password2(self): + """ + Функция тестирования случая с неправильно подобранным новым паролем (употребляются только цифры). + """ with translation.override('ru'): resp = self.client.post( reverse_lazy('password_change'), @@ -313,6 +349,9 @@ class PasswordChangeTestCase(TestCase): self.assertContains(resp, 'Введённый пароль состоит только из цифр', count=1, status_code=200) def test_invalid_new_password3(self): + """ + Функция тестирования случая с неправильно подобранным новым паролем (совпадает с именем пользователя). + """ with translation.override('ru'): resp = self.client.post( reverse_lazy('password_change'), @@ -344,74 +383,75 @@ class GetTicketsTestCase(TestCase): @patch('main.views.zenpy.get_user') @patch('main.extra_func.zenpy') - def test_redirect(self, ZenpyMock, GetUserMock): + def test_redirect(self, zenpy_mock, get_user_mock): """ Функция проверки переадресации пользователя на рабочую страницу. """ - GetUserMock.return_value = Mock() + get_user_mock.return_value = Mock() user = get_user_model().objects.get(email=self.engineer) resp = self.client.post(reverse('work_get_tickets')) self.assertRedirects(resp, reverse('work', args=[user.id])) self.assertEqual(resp.status_code, 302) + self.assertFalse(zenpy_mock.called) @patch('main.views.zenpy') @patch('main.views.get_tickets_list_for_group') - def test_take_one_ticket(self, TicketsMock, ZenpyViewsMock): + def test_take_one_ticket(self, tickets_mock, zenpy_views_mock): """ Функция проверки назначения одного тикета на engineer. """ - TicketsMock.return_value = [Mock()] - ZenpyViewsMock.get_user.return_value = Mock(role='agent', custom_role_id=sets.ZENDESK_ROLES['engineer']) + tickets_mock.return_value = [Mock()] + zenpy_views_mock.get_user.return_value = Mock(role='agent', custom_role_id=sets.ZENDESK_ROLES['engineer']) self.client.post(reverse('work_get_tickets'), data={'count_tickets': 1}) - tickets = ZenpyViewsMock.update_tickets.call_args - self.assertEqual(tickets[0][0][0].assignee, ZenpyViewsMock.get_user.return_value) + tickets = zenpy_views_mock.update_tickets.call_args + self.assertEqual(tickets[0][0][0].assignee, zenpy_views_mock.get_user.return_value) @patch('main.views.get_tickets_list_for_group') @patch('main.views.zenpy') - def test_take_many_tickets(self, ZenpyMock, TicketsMock): + def test_take_many_tickets(self, zenpy_mock, tickets_mock): """ Функция проверки назначения нескольких тикетов на engineer. """ - TicketsMock.return_value = [Mock()] * 3 - ZenpyMock.get_user.return_value = Mock(role='agent', custom_role_id=sets.ZENDESK_ROLES['engineer']) + tickets_mock.return_value = [Mock()] * 3 + zenpy_mock.get_user.return_value = Mock(role='agent', custom_role_id=sets.ZENDESK_ROLES['engineer']) self.client.post(reverse('work_get_tickets'), data={'count_tickets': 3}) - tickets = ZenpyMock.update_tickets.call_args + tickets = zenpy_mock.update_tickets.call_args for ticket in tickets[0][0]: - self.assertEqual(ticket.assignee, ZenpyMock.get_user.return_value) + self.assertEqual(ticket.assignee, zenpy_mock.get_user.return_value) @patch('main.views.zenpy.get_user') @patch('main.views.zenpy') - def test_light_agent_take_ticket(self, ZenpyMock, GetUserMock): + def test_light_agent_take_ticket(self, zenpy_mock, get_user_mock): """ Функция проверки попытки назначения тикета на light_agent. """ - GetUserMock.return_value = Mock(role='agent', custom_role_id=sets.ZENDESK_ROLES['light_agent']) + get_user_mock.return_value = Mock(role='agent', custom_role_id=sets.ZENDESK_ROLES['light_agent']) self.light_agent_client.post(reverse('work_get_tickets'), data={'count_tickets': 3}) - tickets = ZenpyMock.update_tickets.call_args + tickets = zenpy_mock.update_tickets.call_args self.assertIsNone(tickets) @patch('main.views.zenpy') @patch('main.views.get_tickets_list_for_group') - def test_take_zero_tickets(self, TicketsMock, ZenpyMock): + def test_take_zero_tickets(self, tickets_mock, zenpy_mock): """ Функция проверки попытки назначения нуля тикета на engineer. """ - TicketsMock.return_value = [Mock()] * 3 - ZenpyMock.get_user.return_value = Mock(role='agent', custom_role_id=sets.ZENDESK_ROLES['engineer']) + tickets_mock.return_value = [Mock()] * 3 + zenpy_mock.get_user.return_value = Mock(role='agent', custom_role_id=sets.ZENDESK_ROLES['engineer']) self.client.post(reverse('work_get_tickets'), data={'count_tickets': 0}) - tickets = ZenpyMock.update_tickets.call_args[0][0] + tickets = zenpy_mock.update_tickets.call_args[0][0] self.assertListEqual(tickets, []) @patch('main.views.get_tickets_list_for_group') @patch('main.views.zenpy') - def test_take_invalid_count_tickets(self, ZenpyMock, TicketsMock, ): + def test_take_invalid_count_tickets(self, zenpy_mock, tickets_mock): """ Функция проверки попытки назначения нуля тикетов на engineer. """ - TicketsMock.return_value = [Mock()] * 3 - ZenpyMock.get_user.return_value = Mock(role='agent', custom_role_id=sets.ZENDESK_ROLES['engineer']) + tickets_mock.return_value = [Mock()] * 3 + zenpy_mock.get_user.return_value = Mock(role='agent', custom_role_id=sets.ZENDESK_ROLES['engineer']) self.client.post(reverse('work_get_tickets'), data={'count_tickets': 'asd'}) - tickets = ZenpyMock.update_tickets.call_args + tickets = zenpy_mock.update_tickets.call_args self.assertIsNone(tickets)