From 2a545d81f62d2168470454eec8152218940b9e44 Mon Sep 17 00:00:00 2001
From: RobinTTY <muellerobin95@gmail.com>
Date: Thu, 31 Oct 2024 23:27:59 +0100
Subject: [PATCH] Use projections on the BankAccountQueryResolvers

---
 .../Queries/BankAccountQueryResolvers.cs        | 17 ++++++++++-------
 .../Repositories/BankAccountRepository.cs       |  8 ++++----
 2 files changed, 14 insertions(+), 11 deletions(-)

diff --git a/src/server/RobinTTY.PersonalFinanceDashboard.API/Resolvers/Queries/BankAccountQueryResolvers.cs b/src/server/RobinTTY.PersonalFinanceDashboard.API/Resolvers/Queries/BankAccountQueryResolvers.cs
index b64e22d..717cd66 100644
--- a/src/server/RobinTTY.PersonalFinanceDashboard.API/Resolvers/Queries/BankAccountQueryResolvers.cs
+++ b/src/server/RobinTTY.PersonalFinanceDashboard.API/Resolvers/Queries/BankAccountQueryResolvers.cs
@@ -1,4 +1,5 @@
-using HotChocolate.Types;
+using HotChocolate.Data;
+using HotChocolate.Types;
 using RobinTTY.PersonalFinanceDashboard.Core.Models;
 using RobinTTY.PersonalFinanceDashboard.Infrastructure.Repositories;
 
@@ -15,19 +16,21 @@ public sealed class BankAccountQueryResolvers
     /// </summary>
     /// <param name="repository">The repository to use for data retrieval.</param>
     /// <param name="accountId">The id of the account to lookup.</param>
-    public async Task<BankAccount?> GetBankAccount(BankAccountRepository repository, Guid accountId)
+    [UseSingleOrDefault]
+    [UseProjection]
+    public IQueryable<BankAccount?> GetBankAccount(BankAccountRepository repository, Guid accountId)
     {
-        return await repository.GetBankAccount(accountId);
+        return repository.GetBankAccount(accountId);
     }
 
-    // TODO: there is no input here like the comment indicates
     /// <summary>
-    /// Look up accounts by a list of ids.
+    /// Look up accounts.
     /// </summary>
     /// <param name="repository">The injected repository to use for data retrieval.</param>
     [UsePaging]
-    public async Task<IEnumerable<BankAccount>> GetBankAccounts(BankAccountRepository repository)
+    [UseProjection]
+    public IQueryable<BankAccount> GetBankAccounts(BankAccountRepository repository)
     {
-        return await repository.GetBankAccounts();
+        return repository.GetBankAccounts();
     }
 }
diff --git a/src/server/RobinTTY.PersonalFinanceDashboard.Infrastructure/Repositories/BankAccountRepository.cs b/src/server/RobinTTY.PersonalFinanceDashboard.Infrastructure/Repositories/BankAccountRepository.cs
index 10d2643..089b3f8 100644
--- a/src/server/RobinTTY.PersonalFinanceDashboard.Infrastructure/Repositories/BankAccountRepository.cs
+++ b/src/server/RobinTTY.PersonalFinanceDashboard.Infrastructure/Repositories/BankAccountRepository.cs
@@ -24,18 +24,18 @@ public BankAccountRepository(ApplicationDbContext dbContext)
     /// </summary>
     /// <param name="accountId">The id of the <see cref="BankAccount"/> to retrieve.</param>
     /// <returns>The <see cref="BankAccount"/> if one ist matched otherwise <see langword="null"/>.</returns>
-    public async Task<BankAccount?> GetBankAccount(Guid accountId)
+    public IQueryable<BankAccount?> GetBankAccount(Guid accountId)
     {
-        return await _dbContext.BankAccounts.SingleOrDefaultAsync(account => account.Id == accountId);
+        return _dbContext.BankAccounts.Where(account => account.Id == accountId);
     }
 
     /// <summary>
     /// Gets a list of <see cref="BankAccount"/>s.
     /// </summary>
     /// <returns>A list of <see cref="BankAccount"/>s.</returns>
-    public async Task<List<BankAccount>> GetBankAccounts()
+    public IQueryable<BankAccount> GetBankAccounts()
     {
-        return await _dbContext.BankAccounts.ToListAsync();
+        return _dbContext.BankAccounts;
     }
 
     /// <summary>