Skip to content

Conversation

@drazisil-codecov
Copy link
Contributor

@drazisil-codecov drazisil-codecov commented Dec 12, 2025

  • Updated ReportEmptyError to include archive_path and reportid for better error handling.
  • Modified exception handling in ReportService to log detailed information about empty reports.
  • Adjusted raw_upload_processor to raise ReportEmptyError with the session archive path.

Note

Adds context (archive_path, reportid) to ReportEmptyError, propagates it to Sentry/logs, and raises it with the session archive when no files are found.

  • Worker/Report processing
    • ReportEmptyError: Now carries archive_path and reportid; sets a default message.
    • Logging/Sentry: In services/report/__init__.py, on ReportEmptyError capture to Sentry and log reportid and archive_path (fallbacks to upload.storage_path).
    • Raw upload: In raw_upload_processor.process_raw_upload, raise ReportEmptyError with archive_path=session.archive when the merged report is empty.

Written by Cursor Bugbot for commit bf34e7a. This will update automatically on new commits. Configure here.

- Updated ReportEmptyError to include archive_path and reportid for better error handling.
- Modified exception handling in ReportService to log detailed information about empty reports.
- Adjusted raw_upload_processor to raise ReportEmptyError with the session archive path.
@drazisil-codecov drazisil-codecov linked an issue Dec 12, 2025 that may be closed by this pull request
@drazisil-codecov drazisil-codecov marked this pull request as ready for review December 12, 2025 18:21
@drazisil-codecov drazisil-codecov requested a review from a team December 12, 2025 18:21
@sentry
Copy link

sentry bot commented Dec 12, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 93.87%. Comparing base (d1ef920) to head (bf34e7a).
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #613   +/-   ##
=======================================
  Coverage   93.87%   93.87%           
=======================================
  Files        1285     1285           
  Lines       46674    46680    +6     
  Branches     1522     1522           
=======================================
+ Hits        43817    43823    +6     
  Misses       2547     2547           
  Partials      310      310           
Flag Coverage Δ
workerintegration 58.58% <11.11%> (-0.03%) ⬇️
workerunit 91.22% <100.00%> (+<0.01%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@codecov-notifications
Copy link

codecov-notifications bot commented Dec 12, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ All tests successful. No failed tests found.

📢 Thoughts on this report? Let us know!

@drazisil-codecov drazisil-codecov added this pull request to the merge queue Dec 12, 2025
Merged via the queue into main with commit f3bc54c Dec 12, 2025
40 checks passed
@drazisil-codecov drazisil-codecov deleted the 612-bug-report-is-empty-error-doesnt-log-filenamearchive-path branch December 12, 2025 19:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Bug: "Report is empty" error doesn't log filename/archive path

3 participants