fix: #dev async fixes 2

rag-2-backend/Infrastructure/Dao/CourseDao.cs

@@ -16,9 +16,4 @@ public virtual async Task<Course> GetCourseByIdOrThrow(int id)
         return await dbContext.Courses.SingleOrDefaultAsync(u => u.Id == id) ??
                throw new NotFoundException("Course not found");
     }
-
-    public virtual async Task<List<Course>> GetAllCourses()
-    {
-        return await dbContext.Courses.ToListAsync();
-    }
 }

rag-2-backend/Infrastructure/Dao/RefreshTokenDao.cs

@@ -12,6 +12,7 @@ public class RefreshTokenDao(DatabaseContext context)
 {
     public virtual async Task RemoveTokensForUser(User user)
     {
+        context.Attach(user);
         var unusedTokens = await context.RefreshTokens.Where(r => r.User.Id == user.Id).ToListAsync();
         context.RefreshTokens.RemoveRange(unusedTokens);
         await context.SaveChangesAsync();

rag-2-backend/Infrastructure/Module/Auth/AuthService.cs

@@ -67,6 +67,7 @@ public async Task LogoutUser(string token)
     private async Task<RefreshToken> GenerateRefreshToken(double refreshTokenExpirationTimeDays,
         Database.Entity.User user)
     {
+        databaseContext.Attach(user);
         var refreshToken = new RefreshToken
         {
             User = user,

rag-2-backend/Infrastructure/Module/Course/CourseService.cs

@@ -13,11 +13,11 @@ namespace rag_2_backend.Infrastructure.Module.Course;
 
 public class CourseService(DatabaseContext context, CourseDao courseDao)
 {
-    public async Task<IEnumerable<CourseResponse>> GetCourses()
+    public async Task<List<CourseResponse>> GetCourses()
     {
         var courses = await context.Courses.ToListAsync();
 
-        return courses.Select(CourseMapper.Map);
+        return courses.Select(CourseMapper.Map).ToList();
     }
 
     public async Task AddCourse(CourseRequest request)

rag-2-backend/Infrastructure/Module/User/UserController.cs

@@ -77,9 +77,6 @@ public async Task ChangePassword([Required] string oldPassword, [Required] strin
     [Authorize]
     public async Task DeleteAccount()
     {
-        var header = HttpContext.Request.Headers.Authorization.FirstOrDefault() ??
-                     throw new UnauthorizedAccessException("Unauthorized");
-
-        await userService.DeleteAccount(AuthDao.GetPrincipalEmail(User), header);
+        await userService.DeleteAccount(AuthDao.GetPrincipalEmail(User));
     }
 }

rag-2-backend/Infrastructure/Module/User/UserService.cs

@@ -128,7 +128,7 @@ public async Task ChangePassword(string email, string oldPassword, string newPas
         await context.SaveChangesAsync();
     }
 
-    public async Task DeleteAccount(string email, string header)
+    public async Task DeleteAccount(string email)
     {
         var user = await userDao.GetUserByEmailOrThrow(email);
 
@@ -147,7 +147,6 @@ public async Task DeleteAccount(string email, string header)
 
         context.Users.Remove(user);
         await context.SaveChangesAsync();
-
         await refreshTokenDao.RemoveTokensForUser(user);
     }
 
@@ -157,6 +156,8 @@ private async Task UpdateUserProperties(
         int? studyCycleYearA, int? studyCycleYearB, int? courseId, string? group, Database.Entity.User user
     )
     {
+        context.Attach(user);
+
         if (user.Role == Role.Student && (
                 !studyCycleYearA.HasValue || !studyCycleYearB.HasValue || !courseId.HasValue ||
                 string.IsNullOrWhiteSpace(group))

rag-2-backend/Test/Service/UserServiceTest.cs

@@ -170,7 +170,7 @@ await Assert.ThrowsAsync<BadRequestException>(() =>
     [Fact]
     public async Task ShouldDeleteAccount()
     {
-        await _userService.DeleteAccount("[email protected]", "Bearer header");
+        await _userService.DeleteAccount("[email protected]");
 
         _contextMock.Verify(c => c.Users.Remove(It.IsAny<User>()), Times.Once);
     }