Skip to content

feat/DEVSU-2616 create project when submitting a report with report_async#59

Merged
sshugsc merged 8 commits intodevelopfrom
feat/DEVSU-2616-create-project-when-submitting-a-report-with-report_async
Aug 19, 2025
Merged

feat/DEVSU-2616 create project when submitting a report with report_async#59
sshugsc merged 8 commits intodevelopfrom
feat/DEVSU-2616-create-project-when-submitting-a-report-with-report_async

Conversation

@sshugsc
Copy link
Contributor

@sshugsc sshugsc commented Apr 9, 2025

DEVSU-2616

  • check projects in report async upload. if not exist, create one

@sshugsc sshugsc self-assigned this Apr 9, 2025
@codecov-commenter
Copy link

codecov-commenter commented Apr 10, 2025

Codecov Report

❌ Patch coverage is 0% with 8 lines in your changes missing coverage. Please review.
✅ Project coverage is 83.02%. Comparing base (2d35f42) to head (9ddc55d).

Files with missing lines Patch % Lines
pori_python/ipr/connection.py 0.00% 8 Missing ⚠️

❌ Your patch status has failed because the patch coverage (0.00%) is below the target coverage (80.00%). You can increase the patch coverage or adjust the target coverage.

Additional details and impacted files
@@             Coverage Diff             @@
##           develop      #59      +/-   ##
===========================================
- Coverage    83.30%   83.02%   -0.28%     
===========================================
  Files           18       18              
  Lines         2396     2404       +8     
===========================================
  Hits          1996     1996              
- Misses         400      408       +8     
Flag Coverage Δ
unittests 83.02% <0.00%> (-0.28%) ⬇️

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.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

dustinbleile
dustinbleile previously approved these changes Apr 10, 2025
sshugsc added 2 commits June 3, 2025 11:45
…-with-report_async' of github.com:bcgsc/pori_python into feat/DEVSU-2616-create-project-when-submitting-a-report-with-report_async
@mathieulemieux
Copy link
Member

@elewis2 ,
in async mode, creating a new project will only work for users with admin credential because you need read access on that newly created project in order to check the upload status (see the issue Dustin ran into when investigating SDEV-4922).

@sshugsc sshugsc requested a review from elewis2 July 16, 2025 23:24
elewis2
elewis2 previously approved these changes Aug 5, 2025
@sshugsc sshugsc merged commit 091eddb into develop Aug 19, 2025
5 checks passed
@mathieulemieux mathieulemieux mentioned this pull request Oct 7, 2025
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.

5 participants