huahaofeng
/
aspnetcore
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

Don't set Redis timeout (#3154)

This commit is contained in:
BrennanConroy 2018-10-19 09:14:33 -07:00 committed by GitHub
parent d3aaf1f542
commit e68c8025d5
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
4 changed files with 2 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
test/Microsoft.AspNetCore.SignalR.Redis.Tests/RedisEndToEnd.cs
View File

@ -92,7 +92,7 @@ namespace Microsoft.AspNetCore.SignalR.Redis.Tests
} }
} }
[ConditionalTheory(Skip= "https://github.com/aspnet/SignalR/issues/3058")] [ConditionalTheory]
[SkipIfDockerNotPresent] [SkipIfDockerNotPresent]
[MemberData(nameof(TransportTypesAndProtocolTypes))] [MemberData(nameof(TransportTypesAndProtocolTypes))]
public async Task CanSendAndReceiveUserMessagesFromMultipleConnectionsWithSameUser(HttpTransportType transportType, string protocolName) public async Task CanSendAndReceiveUserMessagesFromMultipleConnectionsWithSameUser(HttpTransportType transportType, string protocolName)

2
test/Microsoft.AspNetCore.SignalR.Redis.Tests/Startup.cs
View File

@ -20,8 +20,6 @@ namespace Microsoft.AspNetCore.SignalR.Redis.Tests
.AddMessagePackProtocol() .AddMessagePackProtocol()
.AddRedis(options => .AddRedis(options =>
{ {
// We start the servers before starting redis so we want to time them out ASAP
options.Configuration.ConnectTimeout = 1;
options.Configuration.EndPoints.Add(Environment.GetEnvironmentVariable("REDIS_CONNECTION-PREV")); options.Configuration.EndPoints.Add(Environment.GetEnvironmentVariable("REDIS_CONNECTION-PREV"));
}); });

2
test/Microsoft.AspNetCore.SignalR.StackExchangeRedis.Tests/RedisEndToEnd.cs
View File

@ -92,7 +92,7 @@ namespace Microsoft.AspNetCore.SignalR.StackExchangeRedis.Tests
} }
} }
[ConditionalTheory(Skip= "https://github.com/aspnet/SignalR/issues/3058")] [ConditionalTheory]
[SkipIfDockerNotPresent] [SkipIfDockerNotPresent]
[MemberData(nameof(TransportTypesAndProtocolTypes))] [MemberData(nameof(TransportTypesAndProtocolTypes))]
public async Task CanSendAndReceiveUserMessagesFromMultipleConnectionsWithSameUser(HttpTransportType transportType, string protocolName) public async Task CanSendAndReceiveUserMessagesFromMultipleConnectionsWithSameUser(HttpTransportType transportType, string protocolName)

2
test/Microsoft.AspNetCore.SignalR.StackExchangeRedis.Tests/Startup.cs
View File

@ -20,8 +20,6 @@ namespace Microsoft.AspNetCore.SignalR.StackExchangeRedis.Tests
.AddMessagePackProtocol() .AddMessagePackProtocol()
.AddStackExchangeRedis(options => .AddStackExchangeRedis(options =>
{ {
// We start the servers before starting redis so we want to time them out ASAP
options.Configuration.ConnectTimeout = 1;
options.Configuration.EndPoints.Add(Environment.GetEnvironmentVariable("REDIS_CONNECTION")); options.Configuration.EndPoints.Add(Environment.GetEnvironmentVariable("REDIS_CONNECTION"));
}); });