Amend exception message on incorrect expiration, throw on double.Nan …

…passed to Trim() and test for that case

src/FastCache.Cached/CacheManager.cs

@@ -46,7 +46,7 @@ public static void QueueFullClear<K, V>() where K : notnull
     /// <returns>True: trim is performed inline. False: the count to trim is above threshold and removal is queued to thread pool.</returns>
     public static bool Trim<K, V>(double percentage) where K : notnull
     {
-        if (percentage is > 100.0 or <= double.Epsilon)
+        if (percentage is > 100.0 or <= double.Epsilon or double.NaN)
         {
             ThrowHelpers.ArgumentOutOfRange(percentage, nameof(percentage));
         }

src/FastCache.Cached/Helpers/ThrowHelpers.cs

@@ -21,7 +21,7 @@ public static void ArgumentOutOfRange<T>(T argument, string argumentName)
 #endif
     public static void InvalidExpiration(TimeSpan expiration)
     {
-        throw new ArgumentOutOfRangeException(nameof(expiration), expiration, "Expiration must not be negative, zero or exceed multiple years.");
+        throw new ArgumentOutOfRangeException(nameof(expiration), expiration, "Expiration must not be negative or zero.");
     }
 
 #if NETSTANDARD2_0

tests/FastCache.CachedTests/Internals/CacheManager.cs

@@ -0,0 +1,22 @@
+using FastCache.Services;
+
+namespace FastCache.CachedTests.Internals;
+
+public sealed class CacheManagerTests
+{
+    [Theory]
+    [InlineData(double.NaN)]
+    [InlineData(double.MinValue)]
+    [InlineData(double.NegativeInfinity)]
+    [InlineData(-double.Epsilon)]
+    [InlineData(100.00001)]
+    [InlineData(100.1D)]
+    [InlineData(double.MaxValue)]
+    [InlineData(double.PositiveInfinity)]
+    public void Trim_Throws_OnInvalidPercentage(double percentage)
+    {
+        void Trim() => CacheManager.Trim<int, object>(percentage);
+
+        Assert.Throws<ArgumentOutOfRangeException>(Trim);
+    }
+}