Усилена фильтрация вакансий в Feed

Author: Toksi86

docs/modules/feed.md

@@ -21,6 +21,7 @@ handlers для служебных записей.
 - отображение проектных новостей;
 - отображение служебных записей для проектов и вакансий;
 - исключение записей непубличных или черновых проектов;
+- исключение записей закрытых вакансий и вакансий из недоступных проектов;
 - передача признака лайка текущим пользователем.
 
 ## Архитектура
@@ -67,6 +68,11 @@ View выбирает подходящие `news.News`, сериализует 
 Если проект черновой или непубличный, связанные с ним записи не возвращаются в
 `/feed/`.
 
+### 5. Вакансия становится недоступной для публичной ленты
+
+Если вакансия закрыта, находится в черновом проекте или относится к непубличному
+проекту, связанная с ней служебная feed-запись не возвращается в `/feed/`.
+
 ## API
 
 - `GET /feed/?type=news` - новости пользователей.
@@ -96,6 +102,9 @@ View выбирает подходящие `news.News`, сериализует 
   `type_model = "vacancy"`;
 - новости непубличных проектов не попадают в feed;
 - новости черновых проектов не попадают в feed;
+- служебные записи закрытых вакансий не попадают в feed;
+- служебные записи вакансий из черновых и непубличных проектов не попадают в
+  feed;
 - liked flag выставляется для новостей, лайкнутых текущим пользователем;
 - `get_liked_news()` возвращает лайкнутые текущим пользователем записи;
 - `create_news_for_model()` создает одну служебную feed-запись без дублей;

feed/tests/test_feed_api.py

@@ -58,6 +58,36 @@ def test_feed_returns_vacancy_feed_record_as_vacancy_content(self):
         self.assertEqual(item["content"]["id"], vacancy.id)
         self.assertEqual(item["content"]["role"], "Backend developer")
 
+    def test_feed_excludes_feed_record_for_inactive_vacancy(self):
+        vacancy = create_vacancy(role="Inactive vacancy", is_active=False)
+        create_news_for_model(vacancy)
+
+        response = self.client.get("/feed/?type=vacancy")
+
+        self.assertEqual(response.status_code, 200)
+        self.assertEqual(response.data["results"], [])
+
+    def test_feed_excludes_vacancy_feed_record_for_draft_project(self):
+        draft_project = create_project(name="Draft vacancy project", draft=True)
+        create_vacancy(project=draft_project, role="Draft project vacancy")
+
+        response = self.client.get("/feed/?type=vacancy")
+
+        self.assertEqual(response.status_code, 200)
+        self.assertEqual(response.data["results"], [])
+
+    def test_feed_excludes_vacancy_feed_record_for_private_project(self):
+        private_project = create_project(
+            name="Private vacancy project",
+            is_public=False,
+        )
+        create_vacancy(project=private_project, role="Private project vacancy")
+
+        response = self.client.get("/feed/?type=vacancy")
+
+        self.assertEqual(response.status_code, 200)
+        self.assertEqual(response.data["results"], [])
+
     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)

feed/views.py

@@ -37,7 +37,11 @@ def get_queryset(self) -> QuerySet[News]:
             "project": Project.objects.filter(draft=False, is_public=True).values_list(
                 "id", flat=True
             ),
-            "vacancy": Vacancy.objects.values_list("id", flat=True),
+            "vacancy": Vacancy.objects.filter(
+                is_active=True,
+                project__draft=False,
+                project__is_public=True,
+            ).values_list("id", flat=True),
         }
         for model_name, ids_queryset in existing_object_filters.items():
             queryset = queryset.exclude(