search

AliBazzi / IdentityServer4.Contrib.RedisStore

A persistence layer using Redis DB for operational data and for caching capability for Identity Server 4
https://www.nuget.org/packages/IdentityServer4.Contrib.RedisStore
MIT License
137 stars 48 forks source link

Adding test to ensure Redis multiplexer is only created once per Redi… #7

Closed RyanWChild closed 6 years ago

RyanWChild commented 6 years ago

@AliBazzi , thank you for passing along the issue with the Lazy method and for fixing it! I added a test to ensure that calling RedisOptions.Multiplexer isn't creating a new Multiplexer each time. I'm using System.Runtime.CompilerServices to expose the internal methods to the test project and XUnit's Assert.Same to check reference equality.

AliBazzi commented 6 years ago

Hi @RyanWChild It's Ok, I will not add a test case for a code that will not change virtually So I will skip this PR Thank you for taking the time though to open it