Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. Weโ€™ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[AC-2847] Simplify OrganizationUser and Group PUT methods and tests #4479

Merged
merged 22 commits into from
Jul 16, 2024

Add comment

b5194e4
Select commit
Loading
Failed to load commit list.
Merged

[AC-2847] Simplify OrganizationUser and Group PUT methods and tests #4479

Add comment
b5194e4
Select commit
Loading
Failed to load commit list.
Codecov / codecov/project succeeded Jul 12, 2024 in 0s

41.60% (+0.07%) compared to 25dc0c9

View this Pull Request on Codecov

41.60% (+0.07%) compared to 25dc0c9

Details

Codecov Report

Attention: Patch coverage is 85.38462% with 19 lines in your changes missing coverage. Please review.

Project coverage is 41.60%. Comparing base (25dc0c9) to head (b5194e4).

Files Patch % Lines
.../OrganizationFeatures/Groups/UpdateGroupCommand.cs 82.69% 7 Missing and 2 partials โš ๏ธ
...OrganizationUsers/UpdateOrganizationUserCommand.cs 88.67% 3 Missing and 3 partials โš ๏ธ
...c/Api/AdminConsole/Controllers/GroupsController.cs 71.42% 2 Missing and 2 partials โš ๏ธ
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #4479      +/-   ##
==========================================
+ Coverage   41.52%   41.60%   +0.07%     
==========================================
  Files        1268     1268              
  Lines       60023    60131     +108     
  Branches     5492     5516      +24     
==========================================
+ Hits        24925    25015      +90     
- Misses      33948    33962      +14     
- Partials     1150     1154       +4     

โ˜” View full report in Codecov by Sentry.
๐Ÿ“ข Have feedback on the report? Share it here.