From 5a62952229d784bab73260c6ff76b8d25500bbb4 Mon Sep 17 00:00:00 2001 From: transtrike Date: Thu, 14 Jan 2021 09:40:55 +0200 Subject: Renamed async methods to include Async in the name in UserRepository --- src/DevHive.Data/Interfaces/IUserRepository.cs | 6 +++--- src/DevHive.Data/Repositories/UserRepository.cs | 6 +++--- src/DevHive.Services/Services/PostService.cs | 6 +++--- src/DevHive.Services/Services/UserService.cs | 4 ++-- src/DevHive.Tests/DevHive.Data.Tests/UserRepositoryTests.cs | 7 +++++++ 5 files changed, 18 insertions(+), 11 deletions(-) create mode 100644 src/DevHive.Tests/DevHive.Data.Tests/UserRepositoryTests.cs (limited to 'src') diff --git a/src/DevHive.Data/Interfaces/IUserRepository.cs b/src/DevHive.Data/Interfaces/IUserRepository.cs index bca8f71..b1cd0cb 100644 --- a/src/DevHive.Data/Interfaces/IUserRepository.cs +++ b/src/DevHive.Data/Interfaces/IUserRepository.cs @@ -12,15 +12,15 @@ namespace DevHive.Data.Interfaces Task AddLanguageToUserAsync(User user, Language language); Task AddTechnologyToUserAsync(User user, Technology technology); - Task GetByUsername(string username); + Task GetByUsernameAsync(string username); Language GetUserLanguage(User user, Language language); IList GetUserLanguages(User user); IList GetUserTechnologies(User user); Technology GetUserTechnology(User user, Technology technology); IEnumerable QueryAll(); - Task EditUserLanguage(User user, Language oldLang, Language newLang); - Task EditUserTechnologies(User user, Technology oldTech, Technology newTech); + Task EditUserLanguageAsync(User user, Language oldLang, Language newLang); + Task EditUserTechnologiesAsync(User user, Technology oldTech, Technology newTech); Task RemoveFriendAsync(User user, User friend); Task RemoveLanguageFromUserAsync(User user, Language language); diff --git a/src/DevHive.Data/Repositories/UserRepository.cs b/src/DevHive.Data/Repositories/UserRepository.cs index b4deacd..075ccd3 100644 --- a/src/DevHive.Data/Repositories/UserRepository.cs +++ b/src/DevHive.Data/Repositories/UserRepository.cs @@ -75,7 +75,7 @@ namespace DevHive.Data.Repositories .FirstOrDefaultAsync(x => x.Id == id); } - public async Task GetByUsername(string username) + public async Task GetByUsernameAsync(string username) { return await this._context.Users .Include(u => u.Roles) @@ -119,7 +119,7 @@ namespace DevHive.Data.Repositories return await RepositoryMethods.SaveChangesAsync(this._context); } - public async Task EditUserLanguage(User user, Language oldLang, Language newLang) + public async Task EditUserLanguageAsync(User user, Language oldLang, Language newLang) { this._context.Update(user); @@ -129,7 +129,7 @@ namespace DevHive.Data.Repositories return await RepositoryMethods.SaveChangesAsync(this._context); } - public async Task EditUserTechnologies(User user, Technology oldTech, Technology newTech) + public async Task EditUserTechnologiesAsync(User user, Technology oldTech, Technology newTech) { this._context.Update(user); diff --git a/src/DevHive.Services/Services/PostService.cs b/src/DevHive.Services/Services/PostService.cs index da9e76b..24ca8f3 100644 --- a/src/DevHive.Services/Services/PostService.cs +++ b/src/DevHive.Services/Services/PostService.cs @@ -101,7 +101,7 @@ namespace DevHive.Services.Services return result; } - //Validate + //Validate public async Task ValidateJwtForComment(Guid commentId, string rawTokenData) { Comment comment = await this._postRepository.GetCommentByIdAsync(commentId); @@ -120,7 +120,7 @@ namespace DevHive.Services.Services string jwtUserName = this.GetClaimTypeValues("unique_name", jwt.Claims)[0]; //List jwtRoleNames = this.GetClaimTypeValues("role", jwt.Claims); - User user = await this._userRepository.GetByUsername(jwtUserName) + User user = await this._userRepository.GetByUsernameAsync(jwtUserName) ?? throw new ArgumentException("User does not exist!"); return user; @@ -137,4 +137,4 @@ namespace DevHive.Services.Services return toReturn; } } -} \ No newline at end of file +} diff --git a/src/DevHive.Services/Services/UserService.cs b/src/DevHive.Services/Services/UserService.cs index 012ec1b..44cb0e2 100644 --- a/src/DevHive.Services/Services/UserService.cs +++ b/src/DevHive.Services/Services/UserService.cs @@ -50,7 +50,7 @@ namespace DevHive.Services.Services if (!await this._userRepository.DoesUsernameExistAsync(loginModel.UserName)) throw new ArgumentException("Invalid username!"); - User user = await this._userRepository.GetByUsername(loginModel.UserName); + User user = await this._userRepository.GetByUsernameAsync(loginModel.UserName); if (user.PasswordHash != GeneratePasswordHash(loginModel.Password)) throw new ArgumentException("Incorrect password!"); @@ -276,7 +276,7 @@ namespace DevHive.Services.Services string jwtUserName = this.GetClaimTypeValues("unique_name", jwt.Claims)[0]; List jwtRoleNames = this.GetClaimTypeValues("role", jwt.Claims); - User user = await this._userRepository.GetByUsername(jwtUserName) + User user = await this._userRepository.GetByUsernameAsync(jwtUserName) ?? throw new ArgumentException("User does not exist!"); /* Username check, only when user isn't admin */ diff --git a/src/DevHive.Tests/DevHive.Data.Tests/UserRepositoryTests.cs b/src/DevHive.Tests/DevHive.Data.Tests/UserRepositoryTests.cs new file mode 100644 index 0000000..39868a6 --- /dev/null +++ b/src/DevHive.Tests/DevHive.Data.Tests/UserRepositoryTests.cs @@ -0,0 +1,7 @@ +namespace DevHive.Data.Tests +{ + public class UserRepositoryTests + { + + } +} -- cgit v1.2.3