fix: Filter components by id in addition to name

Author: suejung-sentry

api/internal/coverage/views.py

@@ -43,7 +43,7 @@ def get_object(self) -> ReportPaths:
         component_paths = []
         if components:
             all_components = components_service.commit_components(commit, self.owner)
-            filtered_components = components_service.filter_components_by_name(
+            filtered_components = components_service.filter_components_by_name_or_id(
                 all_components, components
             )
 

compare/commands/compare/interactors/fetch_impacted_files.py

@@ -45,7 +45,7 @@ def _apply_filters(
             all_components = components.commit_components(
                 comparison.head_commit, comparison.user
             )
-            filtered_components = components.filter_components_by_name(
+            filtered_components = components.filter_components_by_name_or_id(
                 all_components, components_filter
             )
             for component in filtered_components:

graphql_api/types/commit/commit.py

@@ -178,7 +178,7 @@ def resolve_path_contents(commit: Commit, info, path: str = None, filters=None):
 
     if component_filter:
         all_components = components_service.commit_components(commit, current_owner)
-        filtered_components = components_service.filter_components_by_name(
+        filtered_components = components_service.filter_components_by_name_or_id(
             all_components, component_filter
         )
 
@@ -268,7 +268,7 @@ def resolve_deprecated_path_contents(
 
     if component_filter:
         all_components = components_service.commit_components(commit, current_owner)
-        filtered_components = components_service.filter_components_by_name(
+        filtered_components = components_service.filter_components_by_name_or_id(
             all_components, component_filter
         )
 
@@ -394,7 +394,7 @@ def resolve_coverage_file(commit, info, path, flags=None, components=None):
         all_components = components_service.commit_components(
             commit, info.context["request"].current_owner
         )
-        filtered_components = components_service.filter_components_by_name(
+        filtered_components = components_service.filter_components_by_name_or_id(
             all_components, components
         )
         for fc in filtered_components:
@@ -423,7 +423,7 @@ def resolve_coverage_components(commit: Commit, info, filters=None) -> List[Comp
     all_components = components_service.commit_components(commit, current_owner)
 
     if filters and filters.get("components"):
-        return components_service.filter_components_by_name(
+        return components_service.filter_components_by_name_or_id(
             all_components, filters["components"]
         )
 

graphql_api/types/comparison/comparison.py

@@ -203,7 +203,7 @@ def resolve_component_comparisons(
     list_components = comparison_report.commit_comparison.component_comparisons.all()
 
     if filters and filters.get("components"):
-        components = components_service.filter_components_by_name(
+        components = components_service.filter_components_by_name_or_id(
             components, filters["components"]
         )
 

services/components.py

@@ -39,15 +39,20 @@ def component_filtered_report(
     return filtered_report
 
 
-def filter_components_by_name(
+def filter_components_by_name_or_id(
     components: List[Component], terms: List[str]
 ) -> List[Component]:
     """
     Given a list of Components and a list of strings (terms),
     return a new list of Components only including Components with names in terms (case insensitive)
+    OR component_id in terms (case insensitive)
     """
     terms = [v.lower() for v in terms]
-    return list(filter(lambda c: c.name.lower() in terms, components))
+    return [
+        component
+        for component in components
+        if component.name.lower() in terms or component.component_id.lower() in terms
+    ]
 
 
 class ComponentComparison: