diff options
| author | crupest <crupest@outlook.com> | 2020-11-15 20:31:22 +0800 |
|---|---|---|
| committer | crupest <crupest@outlook.com> | 2020-11-15 20:31:22 +0800 |
| commit | 5b7b0120406ef4c609c07ab57074a76a753cd1fd (patch) | |
| tree | dd34d028426627375a3183c30d1494fcec55969b /BackEnd/Timeline.Tests | |
| parent | fa7b123be84afe020fc582535cc270e8cf24e85b (diff) | |
| download | timeline-5b7b0120406ef4c609c07ab57074a76a753cd1fd.tar.gz timeline-5b7b0120406ef4c609c07ab57074a76a753cd1fd.tar.bz2 timeline-5b7b0120406ef4c609c07ab57074a76a753cd1fd.zip | |
feat: Now changing user permission returns 400.
Diffstat (limited to 'BackEnd/Timeline.Tests')
| -rw-r--r-- | BackEnd/Timeline.Tests/IntegratedTests/UserPermissionTest.cs | 18 | ||||
| -rw-r--r-- | BackEnd/Timeline.Tests/Services/UserPermissionServiceTest.cs | 14 |
2 files changed, 8 insertions, 24 deletions
diff --git a/BackEnd/Timeline.Tests/IntegratedTests/UserPermissionTest.cs b/BackEnd/Timeline.Tests/IntegratedTests/UserPermissionTest.cs index 77cae590..3fb581f0 100644 --- a/BackEnd/Timeline.Tests/IntegratedTests/UserPermissionTest.cs +++ b/BackEnd/Timeline.Tests/IntegratedTests/UserPermissionTest.cs @@ -36,23 +36,15 @@ namespace Timeline.Tests.IntegratedTests [Theory]
[MemberData(nameof(EveryPermissionTestData))]
- public async Task ModifyRootUserPermissionShouldHaveNoEffect(UserPermission permission)
+ public async Task ModifyRootUserPermission_Should_Error(UserPermission permission)
{
using var client = await CreateClientAsAdministrator();
- await client.TestDeleteAsync($"users/admin/permissions/{permission}");
+ await client.TestPutAssertErrorAsync($"users/admin/permissions/{permission}",
+ errorCode: ErrorCodes.UserController.ChangePermission_RootUser);
- {
- var body = await client.GetUserAsync("admin");
- body.Permissions.Should().BeEquivalentTo(Enum.GetNames<UserPermission>());
- }
-
- await client.TestPutAsync($"users/admin/permissions/{permission}");
-
- {
- var body = await client.GetUserAsync("admin");
- body.Permissions.Should().BeEquivalentTo(Enum.GetNames<UserPermission>());
- }
+ await client.TestDeleteAssertErrorAsync($"users/admin/permissions/{permission}",
+ errorCode: ErrorCodes.UserController.ChangePermission_RootUser);
}
[Theory]
diff --git a/BackEnd/Timeline.Tests/Services/UserPermissionServiceTest.cs b/BackEnd/Timeline.Tests/Services/UserPermissionServiceTest.cs index 5a4e4954..ea20bd18 100644 --- a/BackEnd/Timeline.Tests/Services/UserPermissionServiceTest.cs +++ b/BackEnd/Timeline.Tests/Services/UserPermissionServiceTest.cs @@ -48,18 +48,10 @@ namespace Timeline.Tests.Services }
[Fact]
- public async Task ModifyPermissionOnRootUserShouldHaveNoEffect()
+ public async Task ModifyPermissionOnRootUser_Should_Throw()
{
- await _service.AddPermissionToUserAsync(1, UserPermission.AllTimelineManagement);
- {
- var permission = await _service.GetPermissionsOfUserAsync(1);
- permission.Should().BeEquivalentTo(Enum.GetValues<UserPermission>());
- }
- await _service.RemovePermissionFromUserAsync(1, UserPermission.AllTimelineManagement);
- {
- var permission = await _service.GetPermissionsOfUserAsync(1);
- permission.Should().BeEquivalentTo(Enum.GetValues<UserPermission>());
- }
+ await _service.Awaiting(s => s.AddPermissionToUserAsync(1, UserPermission.AllTimelineManagement)).Should().ThrowAsync<InvalidOperationOnRootUserException>();
+ await _service.Awaiting(s => s.RemovePermissionFromUserAsync(1, UserPermission.AllTimelineManagement)).Should().ThrowAsync<InvalidOperationOnRootUserException>();
}
[Fact]
|