Browse Source

Role improvement.

pull/432/head
Sebastian Stehle 6 years ago
parent
commit
f965507a78
  1. 5
      src/Squidex.Domain.Apps.Core.Model/Apps/Roles.cs
  2. 8
      tests/Squidex.Domain.Apps.Core.Tests/Model/Apps/RolesTests.cs

5
src/Squidex.Domain.Apps.Core.Model/Apps/Roles.cs

@ -95,6 +95,11 @@ namespace Squidex.Domain.Apps.Core.Apps
throw new ArgumentException("Name already exists.", nameof(name)); throw new ArgumentException("Name already exists.", nameof(name));
} }
if (IsDefault(name))
{
return this;
}
return new Roles(inner.With(name, newRole)); return new Roles(inner.With(name, newRole));
} }

8
tests/Squidex.Domain.Apps.Core.Tests/Model/Apps/RolesTests.cs

@ -51,6 +51,14 @@ namespace Squidex.Domain.Apps.Core.Model.Apps
Assert.Throws<ArgumentException>(() => roles_1.Add(role)); Assert.Throws<ArgumentException>(() => roles_1.Add(role));
} }
[Fact]
public void Should_do_nothing_if_role_to_add_is_default()
{
var roles_1 = roles_0.Add(Role.Developer);
Assert.True(roles_1.CustomCount > 0);
}
[Fact] [Fact]
public void Should_update_role() public void Should_update_role()
{ {

Loading…
Cancel
Save