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 | 667143d870679deb4be55122237e66d2d480946f (patch) | |
| tree | da5c5c4f4b0edcc827182a448bed0e7cc5ec1729 /BackEnd/Timeline.Tests | |
| parent | 63ec1050dd24e4123f73e9ed757376dc8128803d (diff) | |
| download | timeline-667143d870679deb4be55122237e66d2d480946f.tar.gz timeline-667143d870679deb4be55122237e66d2d480946f.tar.bz2 timeline-667143d870679deb4be55122237e66d2d480946f.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]
|