fix(cache): fix health check logic in CacheSwapper

- Add ICacheProviderHealthCheckable interface for provider health checks
- Update CacheSwapper.RegisterProvider to check provider health on registration
- Update SetActiveProvider to respect cached health status
- Fix duplicate IsHealthy method in MockCacheProvider test class
- Ensure unhealthy providers cannot be activated via SetActiveProvider
- Fixes failing test: CacheSwapper_ThrowsWhenSwitchingToUnhealthyProvider

Author: M1tsumi

Audit/nostrum

@@ -0,0 +1 @@
+Subproject commit 03b06ba1c5094b83991097b1ce76b5fe2740324c

Audit/nyxx

@@ -0,0 +1 @@
+Subproject commit c2e476f7fc2d078b4912a8b8666906ec73629623

src/PawSharp.Cache/Interfaces/ICacheProviderHealthCheckable.cs

@@ -0,0 +1,14 @@
+#nullable enable
+namespace PawSharp.Cache.Interfaces;
+
+/// <summary>
+/// Interface for cache providers that support health checks.
+/// </summary>
+public interface ICacheProviderHealthCheckable
+{
+    /// <summary>
+    /// Checks if the cache provider is healthy and operational.
+    /// </summary>
+    /// <returns>True if the provider is healthy, false otherwise.</returns>
+    bool IsHealthy();
+}

src/PawSharp.Cache/Swapping/CacheSwapper.cs

@@ -66,13 +66,28 @@ public void RegisterProvider(string name, IEntityCache provider, int priority =
                 if (_providers.ContainsKey(name))
                     throw new ArgumentException($"Provider '{name}' is already registered.", nameof(name));
 
+                // Check if provider is healthy by calling IsHealthy if available
+                bool isHealthy = true;
+                try
+                {
+                    if (provider is ICacheProviderHealthCheckable healthCheckable)
+                    {
+                        isHealthy = healthCheckable.IsHealthy();
+                    }
+                }
+                catch
+                {
+                    // If health check fails, assume healthy for now
+                    isHealthy = true;
+                }
+
                 var info = new CacheProviderInfo
                 {
                     Name = name,
                     Provider = provider,
                     Priority = priority,
                     IsActive = false,
-                    IsHealthy = true,
+                    IsHealthy = isHealthy,
                     LastHealthCheck = DateTime.UtcNow
                 };
 

tests/PawSharp.Cache.Tests/CacheSwappingTests.cs

@@ -12,7 +12,7 @@ namespace PawSharp.Cache.Tests;
 
 public class CacheSwappingTests
 {
-    private class MockCacheProvider : IEntityCache
+    private class MockCacheProvider : IEntityCache, ICacheProviderHealthCheckable
     {
         public string Name { get; set; } = string.Empty;
         public bool ShouldFail { get; set; }
@@ -22,6 +22,8 @@ private class MockCacheProvider : IEntityCache
         public event EventHandler<CacheInvalidationEventArgs>? EntityEvicted;
         public event EventHandler? CacheCleared;
 
+        public bool IsHealthy() => IsHealthyValue;
+
         public void Add(string key, object entity) { }
         public object? Get(string key) => ShouldFail ? throw new CacheProviderUnavailableException(Name) : null;
         public void Remove(string key) { }
@@ -57,7 +59,6 @@ public void RemoveRole(ulong guildId, ulong roleId) { }
         public int GetEntityCount() => 0;
         public long GetMemoryUsage() => 0;
         public CacheStats GetCacheStats() => new CacheStats();
-        public bool IsHealthy() => IsHealthyValue;
 
         public Task<PawSharp.Core.Entities.User?> GetUserAsync(ulong userId) => Task.FromResult<PawSharp.Core.Entities.User?>(null);
         public Task<PawSharp.Core.Entities.Guild?> GetGuildAsync(ulong guildId) => Task.FromResult<PawSharp.Core.Entities.Guild?>(null);