Browse Source

fixed duplicate permission

pull/338/head
seamys 7 years ago
parent
commit
564440e1c7
  1. 2
      src/Squidex.Domain.Apps.Entities/Apps/RoleExtensions.cs
  2. 8
      tests/Squidex.Domain.Apps.Entities.Tests/Apps/RoleExtensionsRests.cs

2
src/Squidex.Domain.Apps.Entities/Apps/RoleExtensions.cs

@ -30,7 +30,7 @@ namespace Squidex.Domain.Apps.Entities.Apps
} }
} }
permissions = result; permissions = result.Distinct().ToArray();
return permissions; return permissions;
} }

8
tests/Squidex.Domain.Apps.Entities.Tests/Apps/RoleExtensionsRests.cs

@ -23,6 +23,14 @@ namespace Squidex.Domain.Apps.Entities.Apps
Assert.Equal(new[] { "squidex.apps.my-app.common" }, result); Assert.Equal(new[] { "squidex.apps.my-app.common" }, result);
} }
[Fact]
public void Should_not_have_duplicate_permission()
{
var source = new[] { "common", "common", "common" };
var result = source.Prefix("my-app");
Assert.Single(result);
}
[Fact] [Fact]
public void Should_prefix_permission() public void Should_prefix_permission()
{ {

Loading…
Cancel
Save