diff options
author | crupest <crupest@outlook.com> | 2020-11-15 20:54:33 +0800 |
---|---|---|
committer | GitHub <noreply@github.com> | 2020-11-15 20:54:33 +0800 |
commit | 515430ecc976008c50cf27a38dc869c2b1aa73c0 (patch) | |
tree | 945016b5c2a911a2a19b8f4f1472ecfdbe32a602 /BackEnd/Timeline.Tests/Services | |
parent | fa7b123be84afe020fc582535cc270e8cf24e85b (diff) | |
parent | 45873d9115840c9db596c2dffebc7bb29df13686 (diff) | |
download | timeline-515430ecc976008c50cf27a38dc869c2b1aa73c0.tar.gz timeline-515430ecc976008c50cf27a38dc869c2b1aa73c0.tar.bz2 timeline-515430ecc976008c50cf27a38dc869c2b1aa73c0.zip |
Merge pull request #188 from crupest/root-user
Forbid some operation on root user.
Diffstat (limited to 'BackEnd/Timeline.Tests/Services')
-rw-r--r-- | BackEnd/Timeline.Tests/Services/UserDeleteServiceTest.cs | 27 | ||||
-rw-r--r-- | BackEnd/Timeline.Tests/Services/UserPermissionServiceTest.cs | 14 |
2 files changed, 30 insertions, 11 deletions
diff --git a/BackEnd/Timeline.Tests/Services/UserDeleteServiceTest.cs b/BackEnd/Timeline.Tests/Services/UserDeleteServiceTest.cs new file mode 100644 index 00000000..be11564e --- /dev/null +++ b/BackEnd/Timeline.Tests/Services/UserDeleteServiceTest.cs @@ -0,0 +1,27 @@ +using FluentAssertions;
+using Microsoft.Extensions.Logging.Abstractions;
+using Moq;
+using System.Threading.Tasks;
+using Timeline.Services;
+using Timeline.Services.Exceptions;
+using Xunit;
+
+namespace Timeline.Tests.Services
+{
+ public class UserDeleteServiceTest : DatabaseBasedTest
+ {
+ private readonly Mock<ITimelineService> _mockTimelineService = new Mock<ITimelineService>();
+ private UserDeleteService _service = default!;
+
+ protected override void OnDatabaseCreated()
+ {
+ _service = new UserDeleteService(NullLogger<UserDeleteService>.Instance, Database, _mockTimelineService.Object);
+ }
+
+ [Fact]
+ public async Task DeleteRootUser_Should_Throw()
+ {
+ await _service.Awaiting(s => s.DeleteUser("admin")).Should().ThrowAsync<InvalidOperationOnRootUserException>();
+ }
+ }
+}
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]
|