Remove CachePolicy TryTrimExpired (#499)

Author: bitfaster

BitFaster.Caching.UnitTests/CachePolicyTests.cs

@@ -19,40 +19,5 @@ public void WhenCtorFieldsAreAssigned()
             cp.ExpireAfterAccess.HasValue.Should().BeFalse();
             cp.ExpireAfter.HasValue.Should().BeFalse();
         }
-
-        [Fact]
-        public void TryTrimWhenTrimNotSupportedReturnsFalse()
-        {
-            var cp = new CachePolicy(Optional<IBoundedPolicy>.None(), Optional<ITimePolicy>.None());
-
-            cp.TryTrimExpired().Should().BeFalse();
-        }
-
-        [Fact]
-        public void TryTrimWhenExpireAfterWriteReturnsTrue()
-        {
-            var expire = new Mock<ITimePolicy>();
-            var cp = new CachePolicy(Optional<IBoundedPolicy>.None(), new Optional<ITimePolicy>(expire.Object));
-
-            cp.TryTrimExpired().Should().BeTrue();
-        }
-
-        [Fact]
-        public void TryTrimWhenExpireAfterAccessReturnsTrue()
-        {
-            var expire = new Mock<ITimePolicy>();
-            var cp = new CachePolicy(Optional<IBoundedPolicy>.None(), Optional<ITimePolicy>.None(), new Optional<ITimePolicy>(expire.Object), Optional<IDiscreteTimePolicy>.None());
-
-            cp.TryTrimExpired().Should().BeTrue();
-        }
-
-        [Fact]
-        public void TryTrimWhenExpireAfterReturnsTrue()
-        {
-            var expire = new Mock<IDiscreteTimePolicy>();
-            var cp = new CachePolicy(Optional<IBoundedPolicy>.None(), Optional<ITimePolicy>.None(), Optional<ITimePolicy>.None(), new Optional<IDiscreteTimePolicy>(expire.Object));
-
-            cp.TryTrimExpired().Should().BeTrue();
-        }
     }
 }

BitFaster.Caching/CachePolicy.cs

@@ -55,32 +55,5 @@ public CachePolicy(Optional<IBoundedPolicy> eviction, Optional<ITimePolicy> expi
         /// a time to expire computed from the key and value.
         /// </summary>
         public Optional<IDiscreteTimePolicy> ExpireAfter { get; }
-
-		/// <summary>
-        /// If supported, trim expired items from the cache.
-        /// </summary>
-        /// <returns>True if expiry is supported and expired items were trimmed, otherwise false.</returns>
-        public bool TryTrimExpired()
-        {
-            if (ExpireAfterWrite.HasValue)
-            {
-                ExpireAfterWrite.Value.TrimExpired();
-                return true;
-            }
-
-            if (ExpireAfterAccess.HasValue)
-            {
-                ExpireAfterAccess.Value.TrimExpired();
-                return true;
-            }
-
-            if (ExpireAfter.HasValue)
-            {
-                ExpireAfter.Value.TrimExpired();
-                return true;
-            }
-
-            return false;
-        }
     }
 }