Skip to content
This repository was archived by the owner on May 5, 2025. It is now read-only.

Commit 3845bec

Browse files
authored
"Fix" errors calling get_flag_names on None (#1207)
1 parent 7189c13 commit 3845bec

File tree

2 files changed

+3
-2
lines changed

2 files changed

+3
-2
lines changed

graphql_api/types/commit/commit.py

+1-1
Original file line numberDiff line numberDiff line change
@@ -326,7 +326,7 @@ def resolve_coverage_totals(
326326
@commit_coverage_analytics_bindable.field("flagNames")
327327
@sync_to_async
328328
def resolve_coverage_flags(commit: Commit, info: GraphQLResolveInfo) -> list[str]:
329-
return commit.full_report.get_flag_names()
329+
return commit.full_report.get_flag_names() if commit.full_report else []
330330

331331

332332
@sentry_sdk.trace

services/components.py

+2-1
Original file line numberDiff line numberDiff line change
@@ -32,8 +32,9 @@ def component_filtered_report(
3232
Filter a report such that the totals, etc. are only pertaining to the given component.
3333
"""
3434
flags, paths = [], []
35+
report_flags = report.get_flag_names() if report else []
3536
for component in components:
36-
flags.extend(component.get_matching_flags(report.get_flag_names()))
37+
flags.extend(component.get_matching_flags(report_flags))
3738
paths.extend(component.paths)
3839
filtered_report = report.filter(flags=flags, paths=paths)
3940
return filtered_report

0 commit comments

Comments
 (0)