From c5f3c69b3a008ab87542e523e2a59f37801bd65a Mon Sep 17 00:00:00 2001 From: crupest Date: Sat, 18 Jan 2020 00:50:31 +0800 Subject: ... --- Timeline.Tests/Controllers/TokenControllerTest.cs | 4 ++-- Timeline.Tests/Controllers/UserControllerTest.cs | 13 +++++++------ 2 files changed, 9 insertions(+), 8 deletions(-) (limited to 'Timeline.Tests') diff --git a/Timeline.Tests/Controllers/TokenControllerTest.cs b/Timeline.Tests/Controllers/TokenControllerTest.cs index 238fc237..371884bb 100644 --- a/Timeline.Tests/Controllers/TokenControllerTest.cs +++ b/Timeline.Tests/Controllers/TokenControllerTest.cs @@ -10,7 +10,7 @@ using Timeline.Models.Http; using Timeline.Services; using Timeline.Tests.Helpers; using Xunit; -using static Timeline.ErrorCodes.Http.Token; +using static Timeline.ErrorCodes.Token; namespace Timeline.Tests.Controllers { @@ -98,7 +98,7 @@ namespace Timeline.Tests.Controllers public static IEnumerable Verify_BadRequest_Data() { - yield return new object[] { new JwtVerifyException(JwtVerifyException.ErrorCodes.Expired), Verify.Expired }; + yield return new object[] { new JwtVerifyException(JwtVerifyException.ErrorCodes.Expired), Verify.TimeExpired }; yield return new object[] { new JwtVerifyException(JwtVerifyException.ErrorCodes.IdClaimBadFormat), Verify.BadFormat }; yield return new object[] { new JwtVerifyException(JwtVerifyException.ErrorCodes.OldVersion), Verify.OldVersion }; yield return new object[] { new UserNotExistException(), Verify.UserNotExist }; diff --git a/Timeline.Tests/Controllers/UserControllerTest.cs b/Timeline.Tests/Controllers/UserControllerTest.cs index a5ca7a2b..7a6541fb 100644 --- a/Timeline.Tests/Controllers/UserControllerTest.cs +++ b/Timeline.Tests/Controllers/UserControllerTest.cs @@ -13,7 +13,8 @@ using Timeline.Models.Http; using Timeline.Services; using Timeline.Tests.Helpers; using Xunit; -using static Timeline.ErrorCodes.Http.User; +using static Timeline.ErrorCodes.User; +using static Timeline.ErrorCodes.UserCommon; namespace Timeline.Tests.Controllers { @@ -61,7 +62,7 @@ namespace Timeline.Tests.Controllers var action = await _controller.Get(username); action.Result.Should().BeAssignableTo() .Which.Value.Should().BeAssignableTo() - .Which.Code.Should().Be(Get.NotExist); + .Which.Code.Should().Be(UserNotExist); } [Theory] @@ -114,7 +115,7 @@ namespace Timeline.Tests.Controllers }, username); action.Should().BeAssignableTo() .Which.Value.Should().BeAssignableTo() - .Which.Code.Should().Be(Patch.NotExist); + .Which.Code.Should().Be(UserNotExist); } [Fact] @@ -154,8 +155,8 @@ namespace Timeline.Tests.Controllers } [Theory] - [InlineData(typeof(UserNotExistException), Op.ChangeUsername.NotExist)] - [InlineData(typeof(UsernameConfictException), Op.ChangeUsername.AlreadyExist)] + [InlineData(typeof(UserNotExistException), UserNotExist)] + [InlineData(typeof(UsernameConfictException), ChangeUsername_Conflict)] public async Task Op_ChangeUsername_Failure(Type exceptionType, int code) { const string oldUsername = "aaa"; @@ -212,7 +213,7 @@ namespace Timeline.Tests.Controllers var action = await _controller.ChangePassword(new ChangePasswordRequest { OldPassword = oldPassword, NewPassword = newPassword }); action.Should().BeAssignableTo() .Which.Value.Should().BeAssignableTo() - .Which.Code.Should().Be(Op.ChangePassword.BadOldPassword); + .Which.Code.Should().Be(ChangePassword_BadOldPassword); } } } -- cgit v1.2.3