Skip to content

Handle duplicate setting, feature and permission records - #26209

Merged
EngincanV merged 3 commits into
devfrom
maliming/null-provider-key-duplicates
Oct 1, 2026
Merged

EngincanV merged 3 commits into
devfrom
maliming/null-provider-key-duplicates

Delete all duplicate grants in ResourcePermissionManager.DeleteAsync

769ccc4
Select commit
Loading
Failed to load commit list.
Codecov / codecov/project succeeded Sep 24, 2026 in 1s

49.95% (+0.04%) compared to c80c6f8

View this Pull Request on Codecov

49.95% (+0.04%) compared to c80c6f8

Details

Codecov Report

❌ Patch coverage is 69.70954% with 73 lines in your changes missing coverage. Please review.
✅ Project coverage is 49.95%. Comparing base (c80c6f8) to head (769ccc4).
⚠️ Report is 33 commits behind head on dev.

Files with missing lines Patch % Lines
...ment/ResourcePermissionManagementProvider_Tests.cs 0.00% 30 Missing ⚠️
...ssionManagement/ResourcePermissionManager_Tests.cs 0.00% 30 Missing ⚠️
...onManagement/PermissionManagementProvider_Tests.cs 0.00% 11 Missing ⚠️
...lo/Abp/FeatureManagement/FeatureManagementStore.cs 97.22% 0 Missing and 1 partial ⚠️
...lo/Abp/SettingManagement/SettingManagementStore.cs 97.95% 0 Missing and 1 partial ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##              dev   #26209      +/-   ##
==========================================
+ Coverage   49.90%   49.95%   +0.04%     
==========================================
  Files        3842     3843       +1     
  Lines      135458   135679     +221     
  Branches    10271    10282      +11     
==========================================
+ Hits        67597    67774     +177     
- Misses      65823    65867      +44     
  Partials     2038     2038              

☔ View full report in Codecov by Harness.
📢 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.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.