filipw / Strathweb.CacheOutput

ASP.NET Web API CacheOutput - library to allow you to cache the output of ApiControllers
Other
883 stars 254 forks source link

Problem with registrations #268

Open dmalanij opened 2 years ago

dmalanij commented 2 years ago

I'm uncertain whether this is a real bug or a wrong unit tests.

Basically: registrations in the CacheOutputConfiguration will only take into account whateve is used in the first invocation due to the inner implmentation with GetOrAdd.

That's fine if is the intended behaviour, but then the CacheKeyGeneratorRegistrationTests.last_registered_default_is_used states other thing.

Due to a strange combination this was not breaking test itself, you can check comment about that.