Cahce.Remove is now void

Added tests for Cache.Remove

NzbDrone.Common.Test/CacheTests/CachedFixture.cs

@@ -48,6 +48,23 @@ namespace NzbDrone.Common.Test.CacheTests
             _cachedString.Find("Key").Should().Be("New");
         }
 
+
+        [Test]
+        public void should_be_able_to_remove_key()
+        {
+            _cachedString.Set("Key", "Value");
+
+            _cachedString.Remove("Key");
+
+            _cachedString.Find("Key").Should().BeNull();
+        }
+
+        [Test]
+        public void should_be_able_to_remove_non_existing_key()
+        {
+            _cachedString.Remove("Key");
+        }
+
         [Test]
         public void should_store_null()
         {

NzbDrone.Common/Cache/Cached.cs

@@ -61,17 +61,10 @@ namespace NzbDrone.Common.Cache
             return value.Object;
         }
 
-        public T Remove(string key)
+        public void Remove(string key)
         {
             CacheItem value;
             _store.TryRemove(key, out value);
-
-            if (value == null)
-            {
-                return default(T);
-            }
-
-            return value.Object;
         }
 
         public T Get(string key, Func<T> function, TimeSpan? lifeTime = null)

NzbDrone.Common/Cache/ICached.cs

@@ -13,7 +13,7 @@ namespace NzbDrone.Common.Cache
         void Set(string key, T value, TimeSpan? lifetime = null);
         T Get(string key, Func<T> function, TimeSpan? lifeTime = null);
         T Find(string key);
-        T Remove(string key);
+        void Remove(string key);
 
         ICollection<T> Values { get; }
     }