Do not double-filter Reports in ReportPaths

Previously, the `ReportPaths.files` accessor would manually filter through the `unfiltered_report`.
Even though the `self.report` was already filtered.

Now the custom filtering logic is a bit updated, but taking care of existing bugs in `FilteredReport`, as that is still yielding files even though they are not matching any `flags`.

Author: Swatinem

services/path.py

@@ -9,6 +9,7 @@
 from asgiref.sync import async_to_sync
 from django.conf import settings
 from shared.reports.resources import Report
+from shared.reports.filtered import FilteredReport, FilteredReportFile
 from shared.reports.types import ReportTotals
 from shared.torngit.exceptions import TorngitClientError
 from shared.utils.match import Matcher
@@ -150,13 +151,12 @@ def __init__(
         report: Report,
         path: str | None = None,
         search_term: str | None = None,
-        filter_flags: list[str] = [],
-        filter_paths: list[str] = [],
+        filter_flags: list[str] = None,
+        filter_paths: list[str] = None,
     ):
-        self.report = report
-        self.unfiltered_report = report
-        self.filter_flags = filter_flags
-        self.filter_paths = filter_paths
+        self.report: Report | FilteredReport = report
+        self.filter_flags = filter_flags or []
+        self.filter_paths = filter_paths or []
         self.prefix = path or ""
 
         # Filter report if flags or paths exist
@@ -172,37 +172,37 @@ def __init__(
         ]
 
         if search_term:
+            search_term = search_term.lower()
             self._paths = [
                 path
-                for path in self.paths
-                if search_term.lower() in path.relative_path.lower()
+                for path in self._paths
+                if search_term in path.relative_path.lower()
             ]
 
     @cached_property
     def files(self) -> list[str]:
-        # No filtering, just return files in Report
-        if not self.filter_flags and not self.filter_paths:
+        # No flags filtering, just return (path-filtered) files in Report
+        if not self.filter_flags:
             return self.report.files
 
+        # When there is a flag filter, `FilteredReport` currently yields
+        # `FilteredReportFile`s without actually checking whether they match the sessions.
+        # Before that bug is fixed, lets do the filtering manually here. Once that bug is fixed,
+        # this should just forward to `self.report.files` like above.
         files = []
-        # Do flag filtering if needed
-        if self.filter_flags:
-            files = report_service.files_belonging_to_flags(
-                commit_report=self.unfiltered_report, flags=self.filter_flags
-            )
-        else:
-            files = [file.name for file in self.unfiltered_report]
-
-        # Do path filtering if needed
-        if self.filter_paths:
-            matcher = Matcher(self.filter_paths)
-            files = [file for file in files if matcher.match(file)]
-
+        for file in self.report:
+            if isinstance(file, FilteredReportFile):
+                found = False
+                for _ln, line in self.report_file.lines:
+                    if line and any(s.id in file.session_ids for s in line.sessions):
+                        found = True
+                        break
+                if not found:
+                    continue
+
+            files.append(file.name)
         return files
 
-    def _filter_commit_report(self) -> None:
-        self.report = self.report.filter(flags=self.filter_flags)
-
     @property
     def paths(self) -> list[PrefixedPath]:
         return self._paths