Skip to content

Commit

Permalink
When core installers are disabled nothing should run
Browse files Browse the repository at this point in the history
Rename the test

Remove redundant line
  • Loading branch information
danielmarbach committed Nov 20, 2020
1 parent d60e246 commit 128dfbc
Show file tree
Hide file tree
Showing 3 changed files with 14 additions and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -13,6 +13,13 @@ public SubscriptionStorageInstallerFeature()
protected override void Setup(FeatureConfigurationContext context)
{
var settings = context.Settings.Get<SubscriptionStorageInstallerSettings>();

// if it hasn't been explicitly disabled installer settings need to be considered
if (!settings.Disabled)
{
settings.Disabled = !context.Settings.GetOrDefault<bool>("Installers.Enable");
}

if (settings.Disabled)
{
return;
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -13,6 +13,12 @@ protected override void Setup(FeatureConfigurationContext context)
{
var settings = context.Settings.Get<SynchronizedStorageInstallerSettings>();

// if it hasn't been explicitly disabled installer settings need to be considered
if (!settings.Disabled)
{
settings.Disabled = !context.Settings.GetOrDefault<bool>("Installers.Enable");
}

if (settings.Disabled)
{
return;
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,7 @@ namespace NServiceBus.AcceptanceTests
using EndpointTemplates;
using NUnit.Framework;

public partial class When_saga_table_doesnt_exist_and_schema_creation_off : NServiceBusAcceptanceTest
public partial class When_saga_table_doesnt_exist_and_table_creation_disabled : NServiceBusAcceptanceTest
{
[Test]
public void Should_throw_not_supported()
Expand Down

0 comments on commit 128dfbc

Please sign in to comment.