Skip to content

Commit

Permalink
Change: Log task id on failure to get last report in time period.
Browse files Browse the repository at this point in the history
  • Loading branch information
a-h-abdelsalam committed Jul 26, 2024
1 parent c7b8f73 commit f1ba00a
Showing 1 changed file with 5 additions and 1 deletion.
6 changes: 5 additions & 1 deletion scripts/create-consolidated-report.gmp.py
Original file line number Diff line number Diff line change
Expand Up @@ -235,7 +235,11 @@ def get_last_report_in_time_period(
)
)
# should always be max 1 report
reports.append(reports_xml.xpath("report/@id")[0])
reports_id = reports_xml.xpath("report/@id")
if reports_id:
reports.append(reports_id[0])

Check warning on line 240 in scripts/create-consolidated-report.gmp.py

View check run for this annotation

Codecov / codecov/patch

scripts/create-consolidated-report.gmp.py#L238-L240

Added lines #L238 - L240 were not covered by tests
else:
print(f"Failed to get report for task {task_id}")

Check warning on line 242 in scripts/create-consolidated-report.gmp.py

View check run for this annotation

Codecov / codecov/patch

scripts/create-consolidated-report.gmp.py#L242

Added line #L242 was not covered by tests
return reports


Expand Down

0 comments on commit f1ba00a

Please sign in to comment.