Browse Source

Merge pull request #338 from seamys/duplicatePermissions

fixed duplicate permission
pull/339/head
Sebastian Stehle 7 years ago
committed by GitHub
parent
commit
c8dcae1d59
No known key found for this signature in database GPG Key ID: 4AEE18F83AFDEB23
  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;
}

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);
}
[Fact]
public void Should_not_have_duplicate_permission()
{
var source = new[] { "common", "common", "common" };
var result = source.Prefix("my-app");
Assert.Single(result);
}
[Fact]
public void Should_prefix_permission()
{

Loading…
Cancel
Save