fix jobs view, add regression test

Author: tykling

src/files/tests/test_views.py

@@ -267,6 +267,17 @@ def test_file_tag_list_view(self) -> None:
         rows = soup.select("div.table-container > table > tbody > tr")
         assert len(rows) == 2, "did not get 2 rows in file tag list view"
 
+    ######### FILE JOB LIST ####################################
+
+    def test_file_job_list_view(self) -> None:
+        """Test the file job list view."""
+        self.client.login(username="creator2", password="secret")
+        response = self.client.get(reverse("files:file_jobs", kwargs={"file_uuid": self.files[0]}))
+        content = response.content.decode()
+        soup = BeautifulSoup(content, "html.parser")
+        rows = soup.select("div.table-container > table > tbody > tr")
+        assert len(rows) == 25, "did not get 25 rows in file job list view"
+
     ######### FILE TAG CREATE ####################################
 
     def test_file_tag_create_view(self) -> None:

src/files/views.py

@@ -333,7 +333,7 @@ def form_invalid(self, form: Form) -> HttpResponse:
 ########## File job and album views ######################################################
 
 
-class FileJobsView(SingleTableMixin, FilterView):
+class FileJobsView(FileViewMixin, SingleTableMixin, FilterView):
     """File jobs view. Shows all jobs for a file."""
 
     template_name = "file_jobs.html"
@@ -342,18 +342,15 @@ class FileJobsView(SingleTableMixin, FilterView):
     table_class = JobTable
     filterset_class = JobFilter
 
-    def get_queryset(self, queryset: models.QuerySet[BaseJob] | None = None) -> models.QuerySet[BaseJob]:
+    def get_queryset(self, queryset: models.QuerySet[BaseJob] | None = None) -> models.QuerySet[BaseJob]:  # type: ignore[override]
         """Get jobs."""
-        self.file = get_object_or_404(
-            BaseFile.objects.get_permitted(user=self.request.user), uuid=self.kwargs["file_uuid"]
-        )
         return BaseJob.objects.filter(basefile=self.file)
 
     def get_context_data(self, **kwargs: dict[str, str]) -> dict[str, str]:
         """Add total_jobs to context."""
         context = super().get_context_data(**kwargs)
         context["total_jobs"] = self.file.jobs.count()
-        return context  # type: ignore[no-any-return]
+        return context
 
 
 class FileAlbumsView(FileViewMixin, SingleTableMixin, FilterView):