Cache not working

[banitt]

In this example

var cache = new Cache();
cache.Add("data", "test", TimeSpan.FromMilliseconds(100));

while(true)
{
    var item = cache.Get<string,string>("data");
    Console.WriteLine(item.ToString());
}

item in cache should be available for 100 milliseconds, but instead it is always available.

[xXLAOKOONXx]

I am not that confident in this caching.
I found a code snipped that refreshes the expiry duration every time you access it:

Call of .Viewed() inside the Get function

RiotSharp/RiotSharp/Caching/Cache.cs

Line 51 in 7ae756c

_slidingTimes[key].Viewed();

Code inside .Viewed()

RiotSharp/RiotSharp/Caching/Cache.cs

Line 282 in 7ae756c

_expireAt = DateTime.Now.Add(_relativeExpiry);

This behavior makes kinda sense, so I guess it is intended.
@banitt may you check if caching works if you delay for a second before accessing the data in an endless loop?

[banitt]

@xXLAOKOONXx
Oh, I thought item should be available up to 100 milliseconds.

[xXLAOKOONXx]

FileCache works as you expect it to work.
As Cache and FileCache claim to do the same with different backend implementation it might be a good point to align both approaches regarding expiry.
In the same change Cache could implement CacheData.
@JanOuborny or @BenFradet your thoughts?

[BenFradet]

As Cache and FileCache claim to do the same with different backend implementation it might be a good point to align both approaches regarding expiry.

agreed 👍