Задокументирован и покрыт тестами модуль Feed

Toksi86 · Toksi86 · commit 56e25cbe78be · 2026-05-25T11:11:20.000+05:00
diff --git a/docs/modules/feed.md b/docs/modules/feed.md
@@ -11,7 +11,8 @@ Feed отвечает за общую ленту `/feed/`.
 ## Статус модуля
 
 Модуль рабочий, но небольшой и связан с `news`, `projects` и `vacancy`.
-Основная логика сосредоточена во view, serializer и service helpers.
+Основная логика сосредоточена во view, serializer, service helpers и signal
+handlers для служебных записей.
 
 ## Основные возможности
 
@@ -31,8 +32,9 @@ Feed отвечает за общую ленту `/feed/`.
 - `feed/services.py` - helpers для лайков и служебных feed-записей.
 - `feed/mapping.py` - соответствие content object типам и serializers.
 - `feed/constants.py` - типы моделей, для которых signals создают feed-записи.
-- `feed/signals.py` - подключение signal handlers.
-- `feed/tests/` - regression-тесты API и service helpers.
+- `feed/signals.py` - создание и удаление служебных feed-записей для проектов
+  и вакансий.
+- `feed/tests/` - regression-тесты API, service helpers и signal handlers.
 
 ## Основные сценарии
 
@@ -56,6 +58,10 @@ View выбирает подходящие `news.News`, сериализует 
 Они используют `news.News` с пустым `text` и связью на объект, например проект
 или вакансию.
 
+Для служебной записи проекта `type_model = "project"`, а `content` содержит
+проект. Для служебной записи вакансии `type_model = "vacancy"`, а `content`
+содержит вакансию.
+
 ### 4. Проект становится недоступным для публичной ленты
 
 Если проект черновой или непубличный, связанные с ним записи не возвращаются в
@@ -65,26 +71,35 @@ View выбирает подходящие `news.News`, сериализует 
 
 - `GET /feed/?type=news` - новости пользователей.
 - `GET /feed/?type=project` - проектные новости и проектные feed-записи.
-- `GET /feed/?type=project|news` - комбинированная выдача по нескольким типам.
+- `GET /feed/?type=vacancy` - служебные feed-записи вакансий.
+- `GET /feed/?type=project|news|vacancy` - комбинированная выдача по нескольким
+  типам.
 
 ## Ограничения и правила
 
 - Feed читает данные из `news.News`, но не отвечает за создание обычных
   project/user/program news.
 - Служебная feed-запись определяется через пустой `text`.
-- Signals проектов могут создавать или удалять feed-записи, но тесты этих
-  side effects остаются в модуле `projects`.
-- `DevScript` в `feed/views.py` остается служебным legacy-инструментом и не
-  является основным пользовательским API.
+- Signals `feed` создают или удаляют служебные feed-записи для проектов и
+  вакансий. Более широкие сценарии публикации проекта остаются в модуле
+  `projects`.
 
 ## Тесты
 
 Текущие regression-тесты проверяют:
 
 - `/feed/?type=news` возвращает пользовательские новости;
 - `/feed/?type=project` возвращает проектные новости в frontend-формате;
+- `/feed/?type=project` возвращает служебную feed-запись проекта как
+  `type_model = "project"`;
+- `/feed/?type=vacancy` возвращает служебную feed-запись вакансии как
+  `type_model = "vacancy"`;
 - новости непубличных проектов не попадают в feed;
+- новости черновых проектов не попадают в feed;
+- liked flag выставляется для новостей, лайкнутых текущим пользователем;
 - `get_liked_news()` возвращает лайкнутые текущим пользователем записи;
 - `create_news_for_model()` создает одну служебную feed-запись без дублей;
 - `delete_news_for_model()` удаляет только служебную feed-запись и не трогает
-  обычную новость с текстом.
+  обычную новость с текстом;
+- signal handlers создают и удаляют служебные feed-записи при изменении проекта
+  или вакансии.
diff --git a/feed/serializers.py b/feed/serializers.py
@@ -28,8 +28,6 @@ def get_type_model(self, obj) -> str:
 
     def get_content_object(self, obj) -> dict:
         type_model = obj.content_type.model
-        if is_content_news(obj) and self.get_type_model(obj) == "project":
-            type_model = "news"
         serializer = CONTENT_OBJECT_SERIALIZER_MAPPING[type_model](obj.content_object)
         return serializer.data
 
diff --git a/feed/tests/helpers.py b/feed/tests/helpers.py
@@ -0,0 +1,17 @@
+from news.tests.helpers import create_project
+from projects.models import Project
+from vacancy.models import Vacancy
+
+
+def create_vacancy(
+    *,
+    project: Project | None = None,
+    role: str = "Feed vacancy",
+    is_active: bool = True,
+) -> Vacancy:
+    return Vacancy.objects.create(
+        project=project or create_project(name="Feed vacancy project"),
+        role=role,
+        description="Vacancy description",
+        is_active=is_active,
+    )
diff --git a/feed/tests/test_feed_api.py b/feed/tests/test_feed_api.py
@@ -1,6 +1,9 @@
 from django.test import TestCase
 from rest_framework.test import APIClient
 
+from core.services import set_like
+from feed.services import create_news_for_model
+from feed.tests.helpers import create_vacancy
 from news.tests.helpers import create_news_for, create_project, create_user
 
 
@@ -33,6 +36,39 @@ def test_feed_returns_project_news_as_news_content(self):
         self.assertEqual(item["content"]["id"], news.id)
         self.assertEqual(item["content"]["text"], "Project feed news")
 
+    def test_feed_returns_project_feed_record_as_project_content(self):
+        project = create_project(name="Feed record project")
+        create_news_for_model(project)
+
+        response = self.client.get("/feed/?type=project")
+
+        self.assertEqual(response.status_code, 200)
+        item = response.data["results"][0]
+        self.assertEqual(item["type_model"], "project")
+        self.assertEqual(item["content"]["id"], project.id)
+
+    def test_feed_returns_vacancy_feed_record_as_vacancy_content(self):
+        vacancy = create_vacancy(role="Backend developer")
+
+        response = self.client.get("/feed/?type=vacancy")
+
+        self.assertEqual(response.status_code, 200)
+        item = response.data["results"][0]
+        self.assertEqual(item["type_model"], "vacancy")
+        self.assertEqual(item["content"]["id"], vacancy.id)
+        self.assertEqual(item["content"]["role"], "Backend developer")
+
+    def test_feed_marks_news_liked_by_current_user(self):
+        news = create_news_for(self.user, text="Liked user feed news")
+        set_like(news, self.user, True)
+
+        response = self.client.get("/feed/?type=news")
+
+        self.assertEqual(response.status_code, 200)
+        item = response.data["results"][0]
+        self.assertEqual(item["type_model"], "news")
+        self.assertTrue(item["content"]["is_user_liked"])
+
     def test_feed_excludes_news_for_private_project(self):
         private_project = create_project(name="Private project", is_public=False)
         create_news_for(private_project, text="Private project news")
@@ -41,3 +77,12 @@ def test_feed_excludes_news_for_private_project(self):
 
         self.assertEqual(response.status_code, 200)
         self.assertEqual(response.data["results"], [])
+
+    def test_feed_excludes_news_for_draft_project(self):
+        draft_project = create_project(name="Draft project", draft=True)
+        create_news_for(draft_project, text="Draft project news")
+
+        response = self.client.get("/feed/?type=project")
+
+        self.assertEqual(response.status_code, 200)
+        self.assertEqual(response.data["results"], [])
diff --git a/feed/tests/test_feed_signals.py b/feed/tests/test_feed_signals.py
@@ -0,0 +1,62 @@
+from django.test import TestCase
+
+from feed.tests.helpers import create_vacancy
+from news.models import News
+from news.services import FEED_RECORD_TEXT
+from news.tests.helpers import create_project
+
+
+class FeedSignalTests(TestCase):
+    def test_project_signal_removes_feed_record_when_project_becomes_draft(self):
+        project = create_project(name="Draft signal project")
+        self.assertTrue(
+            News.objects.get_news(project).filter(text=FEED_RECORD_TEXT).exists()
+        )
+
+        project.draft = True
+        project.save(update_fields=["draft"])
+
+        self.assertFalse(
+            News.objects.get_news(project).filter(text=FEED_RECORD_TEXT).exists()
+        )
+
+    def test_project_delete_signal_removes_feed_record(self):
+        project = create_project(name="Deleted signal project")
+        project_id = project.id
+
+        project.delete()
+
+        self.assertFalse(
+            News.objects.filter(
+                object_id=project_id,
+                content_type__model="project",
+                text=FEED_RECORD_TEXT,
+            ).exists()
+        )
+
+    def test_vacancy_signal_creates_and_removes_feed_record_by_active_state(self):
+        vacancy = create_vacancy(role="Signal vacancy")
+        self.assertTrue(
+            News.objects.get_news(vacancy).filter(text=FEED_RECORD_TEXT).exists()
+        )
+
+        vacancy.is_active = False
+        vacancy.save(update_fields=["is_active"])
+
+        self.assertFalse(
+            News.objects.get_news(vacancy).filter(text=FEED_RECORD_TEXT).exists()
+        )
+
+    def test_vacancy_delete_signal_removes_feed_record(self):
+        vacancy = create_vacancy(role="Deleted signal vacancy")
+        vacancy_id = vacancy.id
+
+        vacancy.delete()
+
+        self.assertFalse(
+            News.objects.filter(
+                object_id=vacancy_id,
+                content_type__model="vacancy",
+                text=FEED_RECORD_TEXT,
+            ).exists()
+        )
diff --git a/feed/urls.py b/feed/urls.py
@@ -1,10 +1,9 @@
 from django.urls import path
 
-from feed.views import NewSimpleFeed, DevScript
+from feed.views import NewSimpleFeed
 
 app_name = "feed"
 
 urlpatterns = [
     path("", NewSimpleFeed.as_view()),
-    path("dev-needs-script", DevScript.as_view()),
 ]
diff --git a/feed/views.py b/feed/views.py
@@ -1,10 +1,6 @@
-from django.contrib.contenttypes.models import ContentType
 from django.db.models import Q, QuerySet
-from rest_framework.generics import CreateAPIView
-from rest_framework.response import Response
 from rest_framework.views import APIView
 
-from core.serializers import EmptySerializer
 from feed.pagination import FeedPagination
 from feed.services import get_liked_news
 from news.models import News
@@ -67,52 +63,13 @@ def get(self, *args, **kwargs):
         # временная подстройка данных под фронт
         for data in serializer.data:
             if data["type_model"] in ["project", "vacancy", None]:
-                fomated_data = {
+                formatted_data = {
                     "type_model": data["type_model"],
                     "content": data["content_object"],
                 }
             elif data["type_model"] == "news":
                 del data["type_model"]
-                fomated_data = {"type_model": "news", "content": data}
-            new_data.append(fomated_data)
+                formatted_data = {"type_model": "news", "content": data}
+            new_data.append(formatted_data)
 
         return paginator.get_paginated_response(new_data)
-
-
-class DevScript(CreateAPIView):
-    serializer_class = EmptySerializer
-
-    def create(self, request):
-        content_type_project = ContentType.objects.filter(model="project").first()
-        for project in Project.objects.filter(draft=False):
-            if not News.objects.filter(
-                content_type=content_type_project, object_id=project.id
-            ).exists():
-                News.objects.create(
-                    content_type=content_type_project,
-                    object_id=project.id,
-                    datetime_created=project.datetime_created,
-                )
-
-        content_type_vacancy = ContentType.objects.filter(model="vacancy").first()
-        for vacancy in Vacancy.objects.filter(is_active=True):
-            if not News.objects.filter(
-                content_type=content_type_vacancy, object_id=vacancy.id
-            ).exists():
-                News.objects.create(
-                    content_type=content_type_vacancy,
-                    object_id=vacancy.id,
-                    datetime_created=vacancy.datetime_created,
-                )
-
-        news_to_delete = list(
-            News.objects.filter(
-                content_type__in=[content_type_vacancy, content_type_project]
-            )
-        )
-
-        for news in news_to_delete:
-            if not news.content_object:
-                news.delete()
-
-        return Response({"status": "success"}, status=201)

Original file line number	Diff line number	Diff line change
`@@ -1,10 +1,9 @@`
`1`	`1`	`from django.urls import path`
`2`	`2`
`3`		`-from feed.views import NewSimpleFeed, DevScript`
	`3`	`+from feed.views import NewSimpleFeed`
`4`	`4`
`5`	`5`	`app_name = "feed"`
`6`	`6`
`7`	`7`	`urlpatterns = [`
`8`	`8`	`path("", NewSimpleFeed.as_view()),`
`9`		`- path("dev-needs-script", DevScript.as_view()),`
`10`	`9`	`]`