Browse Source

Merge branch 'master' of github.com:Squidex/squidex

pull/333/head
Sebastian Stehle 7 years ago
parent
commit
8091a4f8fb
  1. 5
      src/Squidex.Domain.Apps.Core.Operations/HandleRules/RuleService.cs
  2. 15
      tests/Squidex.Domain.Apps.Core.Tests/Operations/HandleRules/RuleServiceTests.cs

5
src/Squidex.Domain.Apps.Core.Operations/HandleRules/RuleService.cs

@ -56,6 +56,11 @@ namespace Squidex.Domain.Apps.Core.HandleRules
Guard.NotNull(rule, nameof(rule));
Guard.NotNull(@event, nameof(@event));
if (!rule.IsEnabled)
{
return null;
}
if (!(@event.Payload is AppEvent appEvent))
{
return null;

15
tests/Squidex.Domain.Apps.Core.Tests/Operations/HandleRules/RuleServiceTests.cs

@ -253,5 +253,18 @@ namespace Squidex.Domain.Apps.Core.Operations.HandleRules
Assert.Equal((ruleError.ToString(), RuleResult.Failed, TimeSpan.Zero), result);
}
[Fact]
public async Task Should_not_create_if_rule_disabled()
{
var ruleConfig = new Rule(new ContentChangedTrigger(), new ValidAction());
var ruleEnvelope = Envelope.Create(new ContentCreated());
ruleConfig = ruleConfig.Disable();
var job = await sut.CreateJobAsync(ruleConfig, ruleEnvelope);
Assert.Null(job);
}
}
}
}

Loading…
Cancel
Save