Rename parametrs to snack_case. Pylint 10.

README.rst

@@ -172,6 +172,9 @@ Quickstart
 
 pylint --django-settings-module=access_controller_new.access_controller.settings ../access_controller_new
 
+Вместо "access_controller_new" необходимо указать папку проекта.
+
+
 Для приведения файлов к стандарту PEP8 используем:
 --------------------------------------------------
 

main/templates/registration/password_reset_form.html

@@ -12,7 +12,7 @@
   <p>Введте свой e-mail адрес для восстановления пароля.</p>
   <form action="." method="post">
     {{ form.as_p }}
-    <p><input class="btn btn-success" type="submit" value="Отпрваить e-mail"></p>
+    <p><input class="btn btn-success" type="submit" value="Отправить e-mail"></p>
     {% csrf_token %}
   </form>
 {% endblock %}

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()]
+        tickets_mock.return_value = [Mock()]
-        ZenpyViewsMock.get_user.return_value = Mock(role='agent', custom_role_id=sets.ZENDESK_ROLES['engineer'])
+        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
+        tickets = zenpy_views_mock.update_tickets.call_args
-        self.assertEqual(tickets[0][0][0].assignee, ZenpyViewsMock.get_user.return_value)
+        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
+        tickets_mock.return_value = [Mock()] * 3
-        ZenpyMock.get_user.return_value = Mock(role='agent', custom_role_id=sets.ZENDESK_ROLES['engineer'])
+        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
+        tickets_mock.return_value = [Mock()] * 3
-        ZenpyMock.get_user.return_value = Mock(role='agent', custom_role_id=sets.ZENDESK_ROLES['engineer'])
+        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
+        tickets_mock.return_value = [Mock()] * 3
-        ZenpyMock.get_user.return_value = Mock(role='agent', custom_role_id=sets.ZENDESK_ROLES['engineer'])
+        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)