Merge pull request #23120 from abpframework/FeatureAppService

fix: ensure provider policy check before deleting feature

Author: hikalkan

modules/feature-management/src/Volo.Abp.FeatureManagement.Application/Volo/Abp/FeatureManagement/FeatureAppService.cs

@@ -137,6 +137,7 @@ protected virtual async Task CheckProviderPolicy(string providerName, string pro
 
     public virtual async Task DeleteAsync([NotNull] string providerName, string providerKey)
     {
+        await CheckProviderPolicy(providerName, providerKey);
         await FeatureManager.DeleteAsync(providerName, providerKey);
     }
 }

modules/feature-management/test/Volo.Abp.FeatureManagement.Application.Tests/Volo/Abp/FeatureManagement/FeatureManagementApplicationTestModule.cs

@@ -8,5 +8,11 @@ namespace Volo.Abp.FeatureManagement;
     )]
 public class FeatureManagementApplicationTestModule : AbpModule
 {
-
+    public override void ConfigureServices(ServiceConfigurationContext context)
+    {
+        Configure<FeatureManagementOptions>(options =>
+        {
+            options.ProviderPolicies["test"] = FeatureManagementPermissions.ManageHostFeatures;
+        });
+    }
 }