#349 default to no load balancer so that in load balaner house we dont always rebuild the loadbalancer infrastructure because the load balancer options type is null (#358)

This commit is contained in:
Tom Pallister 2018-05-15 18:29:35 +01:00 committed by GitHub
parent 1e2e953b2c
commit 061a90f1dd
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
2 changed files with 20 additions and 1 deletions

View File

@ -1,10 +1,12 @@
using Ocelot.LoadBalancer.LoadBalancers;
namespace Ocelot.Configuration namespace Ocelot.Configuration
{ {
public class LoadBalancerOptions public class LoadBalancerOptions
{ {
public LoadBalancerOptions(string type, string key, int expiryInMs) public LoadBalancerOptions(string type, string key, int expiryInMs)
{ {
Type = type; Type = type ?? nameof(NoLoadBalancer);
Key = key; Key = key;
ExpiryInMs = expiryInMs; ExpiryInMs = expiryInMs;
} }

View File

@ -0,0 +1,17 @@
using Ocelot.Configuration;
using Ocelot.LoadBalancer.LoadBalancers;
using Shouldly;
using Xunit;
namespace Ocelot.UnitTests.LoadBalancer
{
public class LoadBalancerOptionsTests
{
[Fact]
public void should_default_to_no_load_balancer()
{
var options = new LoadBalancerOptionsBuilder().Build();
options.Type.ShouldBe(nameof(NoLoadBalancer));
}
}
}