SVI Исправление ошибок

ya_news/conf_test.py

@@ -17,6 +17,8 @@ User = get_user_model()
 
 @pytest.fixture
 def auth_client():
+    """Тест ваторизации клиента."""
+
     client = Client()
     user = User.objects.create(username='Мимо Крокодил')
     client.force_login(user)

ya_news/news/migrations/0001_initial.py → ya_news/news/migrations/mig001.py

@@ -1,4 +1,5 @@
 # Generated by Django 3.2.15 on 2022-10-25 13:03
+"""Миграция для новостей."""
 
 import datetime
 from django.conf import settings
@@ -7,8 +8,9 @@ import django.db.models.deletion
 
 
 class Migration(migrations.Migration):
+    """Класс миграции для приложения."""
 
-    initial = True
+    is_initial: bool = True
 
     dependencies = [
         migrations.swappable_dependency(settings.AUTH_USER_MODEL),
@@ -18,7 +20,12 @@ class Migration(migrations.Migration):
         migrations.CreateModel(
             name='News',
             fields=[
-                ('id', models.BigAutoField(auto_created=True, primary_key=True, serialize=False, verbose_name='ID')),
+                ('id',
+                 models.BigAutoField(
+                     auto_created=True,
+                     primary_key=True,
+                     serialize=False,
+                     verbose_name='ID')),
                 ('title', models.CharField(max_length=50)),
                 ('text', models.TextField()),
                 ('date', models.DateField(default=datetime.datetime.today)),
@@ -32,11 +39,23 @@ class Migration(migrations.Migration):
         migrations.CreateModel(
             name='Comment',
             fields=[
-                ('id', models.BigAutoField(auto_created=True, primary_key=True, serialize=False, verbose_name='ID')),
+                ('id',
+                 models.BigAutoField(
+                     auto_created=True,
+                     primary_key=True,
+                     serialize=False,
+                     verbose_name='ID')),
                 ('text', models.TextField()),
-                ('created', models.DateTimeField(auto_now_add=True)),
-                ('author', models.ForeignKey(on_delete=django.db.models.deletion.CASCADE, to=settings.AUTH_USER_MODEL)),
-                ('news', models.ForeignKey(on_delete=django.db.models.deletion.CASCADE, to='news.news')),
+                ('created',
+                 models.DateTimeField(auto_now_add=True)
+                 ),
+                ('author',
+                 models.ForeignKey(
+                     on_delete=django.db.models.deletion.CASCADE,
+                     to=settings.AUTH_USER_MODEL)),
+                ('news', models.ForeignKey(
+                    on_delete=django.db.models.deletion.CASCADE,
+                    to='news.news')),
             ],
             options={
                 'ordering': ('created',),

ya_news/news/pytest_tests/test_routes.py

@@ -1,7 +1,9 @@
-from django.urls import reverse
-from http import HTTPStatus
+"""Тестирование роутов."""
 
+from typing import Dict
+from http import HTTPStatus
 import pytest
+from django.urls import reverse
 from django.contrib.auth import get_user_model
 from news.models import Comment, News
 
@@ -16,7 +18,7 @@ FULL_URLS = {
 }
 
 # Зависимые URL'ы
-DEPENDENT_URLS = {
+DEPENDENT_URLS: Dict[str, str] = {
     "detail": "news:detail",
     "delete": "news:delete",
     "edit": "news:edit",
@@ -24,6 +26,8 @@ DEPENDENT_URLS = {
 
 
 def format_urls(pk):
+    """Форматирует урлы."""
+
     formatted_urls = {}
     for name, url_info in FULL_URLS.items():
         url_name, args = url_info
@@ -37,6 +41,8 @@ def format_urls(pk):
 
 @pytest.fixture
 def initial_data(db):
+    """Настройка данных."""
+
     news = News.objects.create(title='Заголовок', text='Текст')
     author = User.objects.create(username='Лев Толстой')
     urls = format_urls(news.id)
@@ -44,6 +50,8 @@ def initial_data(db):
 
 
 def test_pages_availability(client, initial_data):
+    """Тест доступности страниц."""
+
     _, _, urls = initial_data
     for url in [urls["home"], urls["login"], urls["logout"], urls["signup"]]:
         response = client.get(url)
@@ -51,6 +59,8 @@ def test_pages_availability(client, initial_data):
 
 
 def test_availability_for_comment_edit_and_delete(client, initial_data):
+    """Тест доступности редактирования и удаления комментариев."""
+
     news, author, urls = initial_data
     reader = User.objects.create(username='Читатель простой')
     Comment.objects.create(
@@ -70,6 +80,8 @@ def test_availability_for_comment_edit_and_delete(client, initial_data):
 
 
 def test_redirect_for_anonymous_client(client, initial_data):
+    """Проверка редиректа для анонимных клиентов."""
+
     news, author, urls = initial_data
     Comment.objects.create(
         news=news,

ya_note/notes/forms/form_note.py

@@ -4,7 +4,6 @@ from typing import List
 from pytils.translit import slugify
 from django import forms
 from django.core.exceptions import ValidationError
-
 from ya_note.notes.models import ModelNote
 
 
@@ -25,15 +24,18 @@ class Meta:
 
 class FormNote(forms.ModelForm):
     """Форма для создания или обновления заметки."""
+
     meta: Meta = Meta()
 
     @property
     def fields(self) -> List[str]:
         """Возвращает список полей."""
+
         return self.meta.fields
 
     def clean_slug(self):
         """Обрабатывает случай, если slug не уникален."""
+
         cleaned_data = super().clean()
         slug = cleaned_data.get('slug')
         if not slug:

ya_note/notes/tests/test_routes.py

@@ -1,7 +1,7 @@
 """Тест роутов приложения."""
 
+from typing import Any
 from http import HTTPStatus
-
 from django.contrib.auth import get_user_model
 from django.test import TestCase, Client
 from django.urls import reverse
@@ -12,15 +12,19 @@ from notes.models import ModelNote
 class TestRoutes(TestCase):
     """Тестирует роуты приложения."""
 
+    test_user1: Any = None
+    test_user2: Any = None
+
     @classmethod
     def setUpTestData(cls):  # pylint: disable=C0103
         """Устанавливает предварительные условия."""
 
-        cls.test_user1 = get_user_model().objects.create_user(
+        user_model = get_user_model().objects
+        cls.test_user1 = user_model.create_user(  # bandit disable
             username='testuser1',
             password='12345'
         )
-        cls.test_user2 = get_user_model().objects.create_user(
+        cls.test_user2 = user_model.create_user(
             username='testuser2',
             password='12345'
         )

ya_note/notes/views/view_note_base.py

@@ -1,5 +1,6 @@
 """Базовое представление примечания."""
 from django.contrib.auth.mixins import LoginRequiredMixin
+from django.urls import reverse_lazy
 from ya_note.notes.models import ModelNote