Extracted PermissionGrantCacheItemInvalidator.

src/Volo.Abp.Permissions.Domain/Volo/Abp/Permissions/PermissionGrantCacheItem.cs

@@ -19,5 +19,10 @@ namespace Volo.Abp.Permissions
             Name = name;
             IsGranted = isGranted;
         }
+
+        public static string CalculateCacheKey(string name, string providerName, string providerKey)
+        {
+            return "pn:" + providerName + ",pk:" + providerKey + ",n:" + name;
+        }
     }
 }

src/Volo.Abp.Permissions.Domain/Volo/Abp/Permissions/PermissionGrantCacheItemInvalidator.cs

@@ -0,0 +1,41 @@
+using System.Threading.Tasks;
+using Volo.Abp.Caching;
+using Volo.Abp.DependencyInjection;
+using Volo.Abp.Domain.Entities.Events;
+using Volo.Abp.EventBus;
+using Volo.Abp.MultiTenancy;
+
+namespace Volo.Abp.Permissions
+{
+    public class PermissionGrantCacheItemInvalidator : IAsyncEventHandler<EntityChangedEventData<PermissionGrant>>, ITransientDependency
+    {
+        protected ICurrentTenant CurrentTenant { get; }
+
+        protected IDistributedCache<PermissionGrantCacheItem> Cache { get; }
+
+        public PermissionGrantCacheItemInvalidator(IDistributedCache<PermissionGrantCacheItem> cache, ICurrentTenant currentTenant)
+        {
+            Cache = cache;
+            CurrentTenant = currentTenant;
+        }
+
+        public virtual async Task HandleEventAsync(EntityChangedEventData<PermissionGrant> eventData)
+        {
+            var cacheKey = CalculateCacheKey(
+                eventData.Entity.Name,
+                eventData.Entity.ProviderName,
+                eventData.Entity.ProviderKey
+            );
+
+            using (CurrentTenant.Change(eventData.Entity.TenantId))
+            {
+                await Cache.RemoveAsync(cacheKey);
+            }
+        }
+
+        protected virtual string CalculateCacheKey(string name, string providerName, string providerKey)
+        {
+            return PermissionGrantCacheItem.CalculateCacheKey(name, providerName, providerKey);
+        }
+    }
+}

src/Volo.Abp.Permissions.Domain/Volo/Abp/Permissions/PermissionStore.cs

@@ -2,47 +2,27 @@
 using Volo.Abp.Authorization.Permissions;
 using Volo.Abp.Caching;
 using Volo.Abp.DependencyInjection;
-using Volo.Abp.Domain.Entities.Events;
-using Volo.Abp.EventBus;
-using Volo.Abp.MultiTenancy;
 
 namespace Volo.Abp.Permissions
 {
-    //TODO: Extract cache invalidate logic to another class
+    public class PermissionStore : IPermissionStore, ITransientDependency
-    public class PermissionStore : IPermissionStore, IAsyncEventHandler<EntityChangedEventData<PermissionGrant>>, ITransientDependency
     {
         protected IPermissionGrantRepository PermissionGrantRepository { get; }
+
         protected IDistributedCache<PermissionGrantCacheItem> Cache { get; }
-        protected ICurrentTenant CurrentTenant { get; }
 
         public PermissionStore(
             IPermissionGrantRepository permissionGrantRepository,
-            IDistributedCache<PermissionGrantCacheItem> cache, 
+            IDistributedCache<PermissionGrantCacheItem> cache)
-            ICurrentTenant currentTenant)
         {
             PermissionGrantRepository = permissionGrantRepository;
             Cache = cache;
-            CurrentTenant = currentTenant;
         }
 
         public async Task<bool> IsGrantedAsync(string name, string providerName, string providerKey)
         {
             return (await GetCacheItemAsync(name, providerName, providerKey)).IsGranted;
         }
-        
-        public virtual async Task HandleEventAsync(EntityChangedEventData<PermissionGrant> eventData)
-        {
-            var cacheKey = CalculateCacheKey(
-                eventData.Entity.Name,
-                eventData.Entity.ProviderName,
-                eventData.Entity.ProviderKey
-            );
-
-            using (CurrentTenant.Change(eventData.Entity.TenantId))
-            {
-                await Cache.RemoveAsync(cacheKey);
-            }
-        }
 
         protected virtual async Task<PermissionGrantCacheItem> GetCacheItemAsync(string name, string providerName, string providerKey)
         {
@@ -66,10 +46,10 @@ namespace Volo.Abp.Permissions
 
             return cacheItem;
         }
-        
+
         protected virtual string CalculateCacheKey(string name, string providerName, string providerKey)
         {
-            return "pn:" + providerName + ",pk:" + providerKey + ",n:" + name;
+            return PermissionGrantCacheItem.CalculateCacheKey(name, providerName, providerKey);
         }
     }
 }