Merge pull request #6509 from abpframework/maliming/AbpAuthorizationException

Localize exception messages for AbpAuthorizationException.

framework/src/Volo.Abp.Authorization/Microsoft/AspNetCore/Authorization/AbpAuthorizationServiceExtensions.cs

@@ -95,7 +95,8 @@ namespace Microsoft.AspNetCore.Authorization
         {
             if (!await authorizationService.IsGrantedAsync(policyName))
             {
-                throw new AbpAuthorizationException("Authorization failed! Given policy has not granted: " + policyName);
+                throw new AbpAuthorizationException(code: AbpAuthorizationErrorCodes.GivenPolicyHasNotGrantedWithPolicyName)
+                    .WithData("PolicyName", policyName);
             }
         }
 
@@ -103,7 +104,8 @@ namespace Microsoft.AspNetCore.Authorization
         {
             if (!await authorizationService.IsGrantedAsync(resource, requirement))
             {
-                throw new AbpAuthorizationException("Authorization failed! Given requirement has not granted for given resource: " + resource);
+                throw new AbpAuthorizationException(code: AbpAuthorizationErrorCodes.GivenRequirementHasNotGrantedForGivenResource)
+                    .WithData("PolicyName", resource);
             }
         }
 
@@ -111,7 +113,8 @@ namespace Microsoft.AspNetCore.Authorization
         {
             if (!await authorizationService.IsGrantedAsync(resource, policy))
             {
-                throw new AbpAuthorizationException("Authorization failed! Given policy has not granted for given resource: " + resource);
+                throw new AbpAuthorizationException(code: AbpAuthorizationErrorCodes.GivenPolicyHasNotGrantedForGivenResource)
+                    .WithData("ResourceName", resource);
             }
         }
 
@@ -119,7 +122,7 @@ namespace Microsoft.AspNetCore.Authorization
         {
             if (!await authorizationService.IsGrantedAsync(policy))
             {
-                throw new AbpAuthorizationException("Authorization failed! Given policy has not granted.");
+                throw new AbpAuthorizationException(code: AbpAuthorizationErrorCodes.GivenPolicyHasNotGranted);
             }
         }
 
@@ -127,7 +130,8 @@ namespace Microsoft.AspNetCore.Authorization
         {
             if (!await authorizationService.IsGrantedAsync(resource, requirements))
             {
-                throw new AbpAuthorizationException("Authorization failed! Given requirements have not granted for given resource: " + resource);
+                throw new AbpAuthorizationException(code: AbpAuthorizationErrorCodes.GivenRequirementsHasNotGrantedForGivenResource)
+                    .WithData("ResourceName", resource);
             }
         }
 
@@ -135,7 +139,8 @@ namespace Microsoft.AspNetCore.Authorization
         {
             if (!await authorizationService.IsGrantedAsync(resource, policyName))
             {
-                throw new AbpAuthorizationException("Authorization failed! Given polist has not granted for given resource: " + resource);
+                throw new AbpAuthorizationException(code: AbpAuthorizationErrorCodes.GivenPolicyHasNotGrantedForGivenResource)
+                    .WithData("ResourceName", resource);
             }
         }
 
@@ -149,4 +154,4 @@ namespace Microsoft.AspNetCore.Authorization
             return abpAuthorizationService;
         }
     }
-}
+}

framework/src/Volo.Abp.Authorization/Volo.Abp.Authorization.csproj

@@ -19,7 +19,12 @@
   </ItemGroup>
 
   <ItemGroup>
-    <ProjectReference Include="..\Volo.Abp.Localization.Abstractions\Volo.Abp.Localization.Abstractions.csproj" />
+    <None Remove="Volo\Abp\Authorization\Localization\*.json" />
+    <EmbeddedResource Include="Volo\Abp\Authorization\Localization\*.json" />
+  </ItemGroup>
+
+  <ItemGroup>
+    <ProjectReference Include="..\Volo.Abp.Localization\Volo.Abp.Localization.csproj" />
     <ProjectReference Include="..\Volo.Abp.MultiTenancy\Volo.Abp.MultiTenancy.csproj" />
     <ProjectReference Include="..\Volo.Abp.Security\Volo.Abp.Security.csproj" />
   </ItemGroup>

framework/src/Volo.Abp.Authorization/Volo/Abp/Authorization/AbpAuthorizationErrorCodes.cs

@@ -0,0 +1,15 @@
+namespace Volo.Abp.Authorization
+{
+    public static class AbpAuthorizationErrorCodes
+    {
+        public const string GivenPolicyHasNotGranted = "Volo.Authorization:010001";
+
+        public const string GivenPolicyHasNotGrantedWithPolicyName = "Volo.Authorization:010002";
+
+        public const string GivenPolicyHasNotGrantedForGivenResource = "Volo.Authorization:010003";
+
+        public const string GivenRequirementHasNotGrantedForGivenResource = "Volo.Authorization:010004";
+
+        public const string GivenRequirementsHasNotGrantedForGivenResource = "Volo.Authorization:010005";
+    }
+}

framework/src/Volo.Abp.Authorization/Volo/Abp/Authorization/AbpAuthorizationModule.cs

@@ -3,17 +3,20 @@ using System.Collections.Generic;
 using Microsoft.AspNetCore.Authorization;
 using Microsoft.Extensions.DependencyInjection;
 using Microsoft.Extensions.DependencyInjection.Extensions;
+using Volo.Abp.Authorization.Localization;
 using Volo.Abp.Authorization.Permissions;
 using Volo.Abp.Localization;
+using Volo.Abp.Localization.ExceptionHandling;
 using Volo.Abp.Modularity;
 using Volo.Abp.MultiTenancy;
 using Volo.Abp.Security;
+using Volo.Abp.VirtualFileSystem;
 
 namespace Volo.Abp.Authorization
 {
     [DependsOn(
         typeof(AbpSecurityModule),
-        typeof(AbpLocalizationAbstractionsModule),
+        typeof(AbpLocalizationModule),
         typeof(AbpMultiTenancyModule)
         )]
     public class AbpAuthorizationModule : AbpModule
@@ -38,6 +41,23 @@ namespace Volo.Abp.Authorization
                 options.ValueProviders.Add<RolePermissionValueProvider>();
                 options.ValueProviders.Add<ClientPermissionValueProvider>();
             });
+
+            Configure<AbpVirtualFileSystemOptions>(options =>
+            {
+                options.FileSets.AddEmbedded<AbpAuthorizationResource>();
+            });
+
+            Configure<AbpLocalizationOptions>(options =>
+            {
+                options.Resources
+                    .Add<AbpAuthorizationResource>("en")
+                    .AddVirtualJson("/Volo/Abp/Authorization/Localization");
+            });
+
+            Configure<AbpExceptionLocalizationOptions>(options =>
+            {
+                options.MapCodeNamespace("Volo.Authorization", typeof(AbpAuthorizationResource));
+            });
         }
 
         private static void AutoAddDefinitionProviders(IServiceCollection services)

framework/src/Volo.Abp.Authorization/Volo/Abp/Authorization/Localization/AbpAuthorizationResource.cs

@@ -0,0 +1,10 @@
+using Volo.Abp.Localization;
+
+namespace Volo.Abp.Authorization.Localization
+{
+    [LocalizationResourceName("AbpAuthorization")]
+    public class AbpAuthorizationResource
+    {
+
+    }
+}

framework/src/Volo.Abp.Authorization/Volo/Abp/Authorization/Localization/en.json

@@ -0,0 +1,10 @@
+{
+  "culture": "en",
+  "texts": {
+    "Volo.Authorization:010001": "Authorization failed! Given policy has not granted.",
+    "Volo.Authorization:010002": "Authorization failed! Given policy has not granted: {PolicyName}",
+    "Volo.Authorization:010003": "Authorization failed! Given policy has not granted for given resource: {ResourceName}",
+    "Volo.Authorization:010004": "Authorization failed! Given requirement has not granted for given resource: {ResourceName}",
+    "Volo.Authorization:010005": "Authorization failed! Given requirements has not granted for given resource: {ResourceName}"
+  }
+}

framework/src/Volo.Abp.Authorization/Volo/Abp/Authorization/Localization/tr.json

@@ -0,0 +1,10 @@
+{
+  "culture": "tr",
+  "texts": {
+    "Volo.Authorization:010001": "Yetkilendirme başarısız oldu! Verilen politika verilmemiştir.",
+    "Volo.Authorization:010002": "Yetkilendirme başarısız oldu! Verilen politika vermedi: {PolicyName}",
+    "Volo.Authorization:010003": "Yetkilendirme başarısız oldu! Verilen politika, verilen kaynak için verilmemiştir: {ResourceName}",
+    "Volo.Authorization:010004": "Yetkilendirme başarısız oldu! Belirtilen kaynak için verilen şart verilmedi {ResourceName}",
+    "Volo.Authorization:010005": "Yetkilendirme başarısız oldu! Verilen gereksinimler, verilen kaynak için verilmemiştir: {ResourceName}"
+  }
+}

framework/src/Volo.Abp.Authorization/Volo/Abp/Authorization/Localization/zh-Hans.json

@@ -0,0 +1,10 @@
+{
+  "culture": "zh-Hans",
+  "texts": {
+    "Volo.Authorization:010001": "授权失败! 提供的策略尚未授予.",
+    "Volo.Authorization:010002": "授权失败! 提供的策略尚未授予: {PolicyName}",
+    "Volo.Authorization:010003": "授权失败! 提供的策略未授予提供的资源: {ResourceName}",
+    "Volo.Authorization:010004": "授权失败! 提供的要求未授予提供的资源: {ResourceName}",
+    "Volo.Authorization:010005": "授权失败! 提供的要求未授予提供的资源: {ResourceName}"
+  }
+}

framework/src/Volo.Abp.ExceptionHandling/Volo/Abp/AspNetCore/ExceptionHandling/DefaultExceptionToErrorInfoConverter.cs

@@ -62,12 +62,6 @@ namespace Volo.Abp.AspNetCore.ExceptionHandling
                 return CreateEntityNotFoundError(exception as EntityNotFoundException);
             }
 
-            if (exception is AbpAuthorizationException)
-            {
-                var authorizationException = exception as AbpAuthorizationException;
-                return new RemoteServiceErrorInfo(authorizationException.Message);
-            }
-
             var errorInfo = new RemoteServiceErrorInfo();
 
             if (exception is IUserFriendlyException)

framework/src/Volo.Abp.Features/Volo.Abp.Features.csproj

@@ -15,7 +15,12 @@
   </PropertyGroup>
 
   <ItemGroup>
-    <ProjectReference Include="..\Volo.Abp.Localization.Abstractions\Volo.Abp.Localization.Abstractions.csproj" />
+    <None Remove="Volo\Abp\Features\Localization\*.json" />
+    <EmbeddedResource Include="Volo\Abp\Features\Localization\*.json" />
+  </ItemGroup>
+
+  <ItemGroup>
+    <ProjectReference Include="..\Volo.Abp.Localization\Volo.Abp.Localization.csproj" />
     <ProjectReference Include="..\Volo.Abp.MultiTenancy\Volo.Abp.MultiTenancy.csproj" />
     <ProjectReference Include="..\Volo.Abp.Validation\Volo.Abp.Validation.csproj" />
   </ItemGroup>

framework/src/Volo.Abp.Features/Volo/Abp/Features/AbpFeatureErrorCodes.cs

@@ -0,0 +1,11 @@
+namespace Volo.Abp.Features
+{
+    public static class AbpFeatureErrorCodes
+    {
+        public const string FeatureIsNotEnabled = "Volo.Feature:010001";
+
+        public const string AllOfTheseFeaturesMustBeEnabled = "Volo.Feature:010002";
+
+        public const string AtLeastOneOfTheseFeaturesMustBeEnabled = "Volo.Feature:010003";
+    }
+}

framework/src/Volo.Abp.Features/Volo/Abp/Features/AbpFeaturesModule.cs

@@ -1,15 +1,18 @@
 using Microsoft.Extensions.DependencyInjection;
 using System;
 using System.Collections.Generic;
+using Volo.Abp.Features.Localization;
 using Volo.Abp.Localization;
+using Volo.Abp.Localization.ExceptionHandling;
 using Volo.Abp.Modularity;
 using Volo.Abp.MultiTenancy;
 using Volo.Abp.Validation;
+using Volo.Abp.VirtualFileSystem;
 
 namespace Volo.Abp.Features
 {
     [DependsOn(
-        typeof(AbpLocalizationAbstractionsModule),
+        typeof(AbpLocalizationModule),
         typeof(AbpMultiTenancyModule),
         typeof(AbpValidationModule)
         )]
@@ -29,6 +32,23 @@ namespace Volo.Abp.Features
                 options.ValueProviders.Add<EditionFeatureValueProvider>();
                 options.ValueProviders.Add<TenantFeatureValueProvider>();
             });
+
+            Configure<AbpVirtualFileSystemOptions>(options =>
+            {
+                options.FileSets.AddEmbedded<AbpFeatureResource>();
+            });
+
+            Configure<AbpLocalizationOptions>(options =>
+            {
+                options.Resources
+                    .Add<AbpFeatureResource>("en")
+                    .AddVirtualJson("/Volo/Abp/Features/Localization");
+            });
+
+            Configure<AbpExceptionLocalizationOptions>(options =>
+            {
+                options.MapCodeNamespace("Volo.Feature", typeof(AbpFeatureResource));
+            });
         }
 
         private static void AutoAddDefinitionProviders(IServiceCollection services)

framework/src/Volo.Abp.Features/Volo/Abp/Features/FeatureCheckerExtensions.cs

@@ -9,8 +9,8 @@ namespace Volo.Abp.Features
     public static class FeatureCheckerExtensions
     {
         public static async Task<T> GetAsync<T>(
-            [NotNull] this IFeatureChecker featureChecker, 
-            [NotNull] string name, 
+            [NotNull] this IFeatureChecker featureChecker,
+            [NotNull] string name,
             T defaultValue = default)
             where T : struct
         {
@@ -56,10 +56,11 @@ namespace Volo.Abp.Features
         {
             if (!(await featureChecker.IsEnabledAsync(featureName)))
             {
-                throw new AbpAuthorizationException("Feature is not enabled: " + featureName);
+                throw new AbpAuthorizationException(code: AbpFeatureErrorCodes.FeatureIsNotEnabled).WithData(
+                    "FeatureName", featureName);
             }
         }
-        
+
         public static async Task CheckEnabledAsync(this IFeatureChecker featureChecker, bool requiresAll, params string[] featureNames)
         {
             if (featureNames.IsNullOrEmpty())
@@ -73,10 +74,8 @@ namespace Volo.Abp.Features
                 {
                     if (!(await featureChecker.IsEnabledAsync(featureName)))
                     {
-                        throw new AbpAuthorizationException(
-                            "Required features are not enabled. All of these features must be enabled: " +
-                            string.Join(", ", featureNames)
-                        );
+                        throw new AbpAuthorizationException(code: AbpFeatureErrorCodes.AllOfTheseFeaturesMustBeEnabled)
+                            .WithData("FeatureNames", string.Join(", ", featureNames));
                     }
                 }
             }
@@ -90,11 +89,9 @@ namespace Volo.Abp.Features
                     }
                 }
 
-                throw new AbpAuthorizationException(
-                    "Required features are not enabled. At least one of these features must be enabled: " +
-                    string.Join(", ", featureNames)
-                );
+                throw new AbpAuthorizationException(code: AbpFeatureErrorCodes.AtLeastOneOfTheseFeaturesMustBeEnabled)
+                    .WithData("FeatureNames", string.Join(", ", featureNames));
             }
         }
     }
-}
+}

framework/src/Volo.Abp.Features/Volo/Abp/Features/Localization/AbpFeatureResource.cs

@@ -0,0 +1,10 @@
+using Volo.Abp.Localization;
+
+namespace Volo.Abp.Features.Localization
+{
+    [LocalizationResourceName("AbpFeature")]
+    public class AbpFeatureResource
+    {
+
+    }
+}

framework/src/Volo.Abp.Features/Volo/Abp/Features/Localization/en.json

@@ -0,0 +1,8 @@
+{
+  "culture": "en",
+  "texts": {
+    "Volo.Feature:010001": "Feature is not enabled: {FeatureName}",
+    "Volo.Feature:010002": "Required features are not enabled. All of these features must be enabled: {FeatureNames}",
+    "Volo.Feature:010003": "Required features are not enabled. At least one of these features must be enabled: {FeatureNames}"
+  }
+}

framework/src/Volo.Abp.Features/Volo/Abp/Features/Localization/tr.json

@@ -0,0 +1,8 @@
+{
+  "culture": "tr",
+  "texts": {
+    "Volo.Feature:010001": "Özellik etkinleştirilmedi: {FeatureName}",
+    "Volo.Feature:010002": "Gerekli özellikler etkinleştirilmedi. Bu özelliklerin tümü etkinleştirilmelidir: {FeatureNames}",
+    "Volo.Feature:010003": "Gerekli özellikler etkinleştirilmedi. Bu özelliklerden en az birinin etkinleştirilmesi gerekir: {FeatureNames}"
+  }
+}

framework/src/Volo.Abp.Features/Volo/Abp/Features/Localization/zh-Hans.json

@@ -0,0 +1,8 @@
+{
+  "culture": "zh-Hans",
+  "texts": {
+    "Volo.Feature:010001": "功能未启用: {FeatureName}",
+    "Volo.Feature:010002": "必要的功能未启用. 这些功能需要启用: {FeatureNames}",
+    "Volo.Feature:010003": "必要的功能未启用. 需要启用这些功能中的一项：{FeatureNames}"
+  }
+}

framework/src/Volo.Abp.Security/Volo/Abp/Authorization/AbpAuthorizationException.cs

@@ -1,6 +1,7 @@
 using System;
 using System.Runtime.Serialization;
 using Microsoft.Extensions.Logging;
+using Volo.Abp.ExceptionHandling;
 using Volo.Abp.Logging;
 
 namespace Volo.Abp.Authorization
@@ -9,7 +10,7 @@ namespace Volo.Abp.Authorization
     /// This exception is thrown on an unauthorized request.
     /// </summary>
     [Serializable]
-    public class AbpAuthorizationException : AbpException, IHasLogLevel
+    public class AbpAuthorizationException : AbpException, IHasLogLevel, IHasErrorCode
     {
         /// <summary>
         /// Severity of the exception.
@@ -17,6 +18,11 @@ namespace Volo.Abp.Authorization
         /// </summary>
         public LogLevel LogLevel { get; set; }
 
+        /// <summary>
+        /// Error code.
+        /// </summary>
+        public string Code { get; }
+
         /// <summary>
         /// Creates a new <see cref="AbpAuthorizationException"/> object.
         /// </summary>
@@ -54,5 +60,24 @@ namespace Volo.Abp.Authorization
         {
             LogLevel = LogLevel.Warning;
         }
+
+        /// <summary>
+        /// Creates a new <see cref="AbpAuthorizationException"/> object.
+        /// </summary>
+        /// <param name="message">Exception message</param>
+        /// <param name="code">Exception code</param>
+        /// <param name="innerException">Inner exception</param>
+        public AbpAuthorizationException(string message = null, string code = null, Exception innerException = null)
+            : base(message, innerException)
+        {
+            Code = code;
+            LogLevel = LogLevel.Warning;
+        }
+
+        public AbpAuthorizationException WithData(string name, object value)
+        {
+            Data[name] = value;
+            return this;
+        }
     }
-}
+}

framework/test/Volo.Abp.Authorization.Tests/Microsoft/AspNetCore/Authorization/AbpAuthorizationServiceExtensions_Tests.cs

@@ -0,0 +1,49 @@
+using Shouldly;
+using Volo.Abp.AspNetCore.ExceptionHandling;
+using Volo.Abp.Authorization;
+using Volo.Abp.Localization;
+using Xunit;
+
+namespace Microsoft.AspNetCore.Authorization
+{
+    public class AbpAuthorizationServiceExtensions_Tests : AuthorizationTestBase
+    {
+        private readonly IExceptionToErrorInfoConverter _exceptionToErrorInfoConverter;
+
+        public AbpAuthorizationServiceExtensions_Tests()
+        {
+            _exceptionToErrorInfoConverter = GetRequiredService<IExceptionToErrorInfoConverter>();
+        }
+
+        [Fact]
+        public void Test_AbpAuthorizationException_Localization()
+        {
+            using (CultureHelper.Use("zh-Hans"))
+            {
+                var exception = new AbpAuthorizationException(code: AbpAuthorizationErrorCodes.GivenPolicyHasNotGranted);
+                var errorInfo = _exceptionToErrorInfoConverter.Convert(exception, false);
+                errorInfo.Message.ShouldBe("授权失败! 提供的策略尚未授予.");
+
+                exception = new AbpAuthorizationException(code: AbpAuthorizationErrorCodes.GivenPolicyHasNotGrantedWithPolicyName)
+                    .WithData("PolicyName", "my_policy_name");
+                errorInfo = _exceptionToErrorInfoConverter.Convert(exception, false);
+                errorInfo.Message.ShouldBe("授权失败! 提供的策略尚未授予: my_policy_name");
+
+                exception =  new AbpAuthorizationException(code: AbpAuthorizationErrorCodes.GivenPolicyHasNotGrantedForGivenResource)
+                    .WithData("ResourceName", "my_resource_name");
+                errorInfo = _exceptionToErrorInfoConverter.Convert(exception, false);
+                errorInfo.Message.ShouldBe("授权失败! 提供的策略未授予提供的资源: my_resource_name");
+
+                exception = new AbpAuthorizationException(code: AbpAuthorizationErrorCodes.GivenRequirementHasNotGrantedForGivenResource)
+                    .WithData("ResourceName", "my_resource_name");
+                errorInfo = _exceptionToErrorInfoConverter.Convert(exception, false);
+                errorInfo.Message.ShouldBe("授权失败! 提供的要求未授予提供的资源: my_resource_name");
+
+                exception =  new AbpAuthorizationException(code: AbpAuthorizationErrorCodes.GivenRequirementsHasNotGrantedForGivenResource)
+                    .WithData("ResourceName", "my_resource_name");
+                errorInfo = _exceptionToErrorInfoConverter.Convert(exception, false);
+                errorInfo.Message.ShouldBe("授权失败! 提供的要求未授予提供的资源: my_resource_name");
+            }
+        }
+    }
+}

framework/test/Volo.Abp.Authorization.Tests/Volo.Abp.Authorization.Tests.csproj

@@ -12,6 +12,7 @@
 
   <ItemGroup>
     <ProjectReference Include="..\..\src\Volo.Abp.Autofac\Volo.Abp.Autofac.csproj" />
+    <ProjectReference Include="..\..\src\Volo.Abp.ExceptionHandling\Volo.Abp.ExceptionHandling.csproj" />
     <ProjectReference Include="..\AbpTestBase\AbpTestBase.csproj" />
     <ProjectReference Include="..\..\src\Volo.Abp.Authorization\Volo.Abp.Authorization.csproj" />
     <PackageReference Include="Microsoft.NET.Test.Sdk" Version="$(MicrosoftNETTestSdkPackageVersion)" />

framework/test/Volo.Abp.Authorization.Tests/Volo/Abp/Authorization/AbpAuthorizationTestModule.cs

@@ -2,12 +2,14 @@
 using Volo.Abp.Authorization.TestServices;
 using Volo.Abp.Autofac;
 using Volo.Abp.DynamicProxy;
+using Volo.Abp.ExceptionHandling;
 using Volo.Abp.Modularity;
 
 namespace Volo.Abp.Authorization
 {
     [DependsOn(typeof(AbpAutofacModule))]
     [DependsOn(typeof(AbpAuthorizationModule))]
+    [DependsOn(typeof(AbpExceptionHandlingModule))]
     public class AbpAuthorizationTestModule : AbpModule
     {
         public override void PreConfigureServices(ServiceConfigurationContext context)
@@ -22,4 +24,4 @@ namespace Volo.Abp.Authorization
             });
         }
     }
-}
+}

framework/test/Volo.Abp.Features.Tests/Volo.Abp.Features.Tests.csproj

@@ -9,6 +9,7 @@
 
   <ItemGroup>
     <ProjectReference Include="..\..\src\Volo.Abp.Autofac\Volo.Abp.Autofac.csproj" />
+    <ProjectReference Include="..\..\src\Volo.Abp.ExceptionHandling\Volo.Abp.ExceptionHandling.csproj" />
     <ProjectReference Include="..\AbpTestBase\AbpTestBase.csproj" />
     <ProjectReference Include="..\..\src\Volo.Abp.Features\Volo.Abp.Features.csproj" />
     <PackageReference Include="Microsoft.NET.Test.Sdk" Version="$(MicrosoftNETTestSdkPackageVersion)" />

framework/test/Volo.Abp.Features.Tests/Volo/Abp/Features/AbpFeaturesTestModule.cs

@@ -1,10 +1,12 @@
 using Volo.Abp.Autofac;
+using Volo.Abp.ExceptionHandling;
 using Volo.Abp.Modularity;
 
 namespace Volo.Abp.Features
 {
     [DependsOn(
         typeof(AbpFeaturesModule),
+        typeof(AbpExceptionHandlingModule),
         typeof(AbpTestBaseModule),
         typeof(AbpAutofacModule)
         )]
@@ -12,7 +14,7 @@ namespace Volo.Abp.Features
     {
         public override void ConfigureServices(ServiceConfigurationContext context)
         {
-            
+
         }
     }
 }

framework/test/Volo.Abp.Features.Tests/Volo/Abp/Features/FeatureCheckerExtensions_Tests.cs

@@ -0,0 +1,40 @@
+using Shouldly;
+using Volo.Abp.AspNetCore.ExceptionHandling;
+using Volo.Abp.Authorization;
+using Volo.Abp.Localization;
+using Xunit;
+
+namespace Volo.Abp.Features
+{
+    public class FeatureCheckerExtensions_Tests : FeatureTestBase
+    {
+        private readonly IExceptionToErrorInfoConverter _exceptionToErrorInfoConverter;
+
+        public FeatureCheckerExtensions_Tests()
+        {
+            _exceptionToErrorInfoConverter = GetRequiredService<IExceptionToErrorInfoConverter>();
+        }
+
+        [Fact]
+        public void Test_AbpAuthorizationException_Localization()
+        {
+            using (CultureHelper.Use("zh-Hans"))
+            {
+                var exception = new AbpAuthorizationException(code: AbpFeatureErrorCodes.FeatureIsNotEnabled)
+                    .WithData("FeatureName", "my_feature_name");
+                var errorInfo = _exceptionToErrorInfoConverter.Convert(exception, false);
+                errorInfo.Message.ShouldBe("功能未启用: my_feature_name");
+
+                exception = new AbpAuthorizationException(code: AbpFeatureErrorCodes.AllOfTheseFeaturesMustBeEnabled)
+                    .WithData("FeatureNames", "my_feature_name, my_feature_name2");
+                errorInfo = _exceptionToErrorInfoConverter.Convert(exception, false);
+                errorInfo.Message.ShouldBe("必要的功能未启用. 这些功能需要启用: my_feature_name, my_feature_name2");
+
+                exception = new AbpAuthorizationException(code: AbpFeatureErrorCodes.AtLeastOneOfTheseFeaturesMustBeEnabled)
+                    .WithData("FeatureNames", "my_feature_name, my_feature_name2");
+                errorInfo = _exceptionToErrorInfoConverter.Convert(exception, false);
+                errorInfo.Message.ShouldBe("必要的功能未启用. 需要启用这些功能中的一项：my_feature_name, my_feature_name2");
+            }
+        }
+    }
+}