Every repository, service, Blazor page/dialog, and test now uses
async/await. Single atomic diff; the codebase does not compile in
intermediate states.
- BaseRepository: LoadListAsync/LoadAsync/SaveAsync return Task<T>;
per-file locks use SemaphoreSlim so waiters can await; Save
serialises to a MemoryStream sync (XmlSerializer has no async
form), then File.WriteAllBytesAsync + sync File.Move.
- RepositoryCache.GetOrLoadAsync takes a Func<Task<List<T>>>.
- All 7 repository interfaces + implementations async.
- All service interfaces + implementations async (except vendor
IFxService and stateless IFxConverter / SettingsService).
- Every Blazor OnInitializedAsync switches to await base.
- Test suite fully async, 42 tests pass.
AccountRepository.EnsureAccountsFile keeps two .GetAwaiter().GetResult()
bridges because it runs from the constructor.
Null-render guard follow-up (folded in):
Blazor now renders the component once with fields at their initial
values while OnInitializedAsync awaits — so fields declared `= null!`
are actually null on that first render and things like
`accounts.GroupBy(...)` throw ArgumentNullException. Fixed across
Transactions, BalanceReport, BalanceSheetReport, ProfitLossReport,
DetailReport, Assets, Spendings, SpendingsOverTime, TransactionDialog,
TransactionSplitDialog, and BookingRuleDialog:
- Collection fields initialise to [] so first-render loops are empty.
- Single-object data fields become nullable; the razor wraps
consumption in `@if (field is null) { <p>Lädt…</p> return; }`.
- <PlotlyChart> guarded behind a null check on config/layout/data so
Plotly.Blazor's @bind doesn't see nulls.
- Header/footer strings initialise to "" instead of null!.
Architectural hygiene on a single-user local Blazor Server app: the
observed win is one File.ReadAllBytesAsync and one
File.WriteAllBytesAsync per Load/Save, and after PR D each file is
loaded at most once per SignalR circuit.
Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
113 lines
4.1 KiB
C#
113 lines
4.1 KiB
C#
using Microsoft.Extensions.Logging.Abstractions;
|
|
using NSubstitute;
|
|
using Schaad.Accounting.Interfaces;
|
|
using Schaad.Accounting.Models;
|
|
using Schaad.Accounting.Repositories;
|
|
using Shouldly;
|
|
|
|
namespace Schaad.Accounting.Tests;
|
|
|
|
public class AccountRepositoryTestShould : IDisposable
|
|
{
|
|
private readonly string dbDir;
|
|
private readonly ISettingsService settingsService;
|
|
private readonly AccountRepository sut;
|
|
|
|
public AccountRepositoryTestShould()
|
|
{
|
|
dbDir = Path.Combine(Path.GetTempPath(), "AccountingNext-tests-" + Guid.NewGuid().ToString("N"));
|
|
Directory.CreateDirectory(dbDir);
|
|
|
|
settingsService = Substitute.For<ISettingsService>();
|
|
settingsService.GetDbPath().Returns(dbDir);
|
|
settingsService.GetLastYearDbPath().Returns(Path.Combine(dbDir, "no-such-dir"));
|
|
|
|
sut = new AccountRepository(settingsService, new RepositoryCache(), NullLogger<AccountRepository>.Instance);
|
|
}
|
|
|
|
public void Dispose()
|
|
{
|
|
if (Directory.Exists(dbDir))
|
|
{
|
|
Directory.Delete(dbDir, recursive: true);
|
|
}
|
|
}
|
|
|
|
[Fact]
|
|
public async Task AssignNewIdWhenSavingAccountWithoutId()
|
|
{
|
|
var account = new Account { Number = 1000, Name = "Cash", Currency = "CHF" };
|
|
|
|
await sut.SaveAccountAsync(account);
|
|
|
|
var stored = (await sut.GetAccountListAsync()).Single();
|
|
stored.Id.ShouldNotBeNullOrEmpty();
|
|
stored.Name.ShouldBe("Cash");
|
|
}
|
|
|
|
[Fact]
|
|
public async Task UpdateInPlaceWhenSavingAccountWithExistingId()
|
|
{
|
|
await sut.SaveAccountAsync(new Account { Number = 1000, Name = "Cash", Currency = "CHF" });
|
|
var id = (await sut.GetAccountListAsync()).Single().Id;
|
|
|
|
await sut.SaveAccountAsync(new Account { Id = id, Number = 1000, Name = "Cash renamed", Currency = "CHF" });
|
|
|
|
var accounts = await sut.GetAccountListAsync();
|
|
accounts.Count.ShouldBe(1);
|
|
accounts[0].Name.ShouldBe("Cash renamed");
|
|
}
|
|
|
|
[Fact]
|
|
public async Task DefaultCurrencyToChfWhenSavingAccountWithoutCurrency()
|
|
{
|
|
await sut.SaveAccountAsync(new Account { Number = 1000, Name = "Cash", Currency = null! });
|
|
|
|
(await sut.GetAccountListAsync()).Single().Currency.ShouldBe("CHF");
|
|
}
|
|
|
|
[Fact]
|
|
public async Task RemoveAccountWhenDeletingById()
|
|
{
|
|
await sut.SaveAccountAsync(new Account { Number = 1000, Name = "Cash", Currency = "CHF" });
|
|
var id = (await sut.GetAccountListAsync()).Single().Id;
|
|
|
|
await sut.DeleteAccountAsync(id);
|
|
|
|
(await sut.GetAccountListAsync()).ShouldBeEmpty();
|
|
}
|
|
|
|
[Fact]
|
|
public async Task ReturnAccountWhenLookingUpByBankAccountNumber()
|
|
{
|
|
await sut.SaveAccountAsync(new Account { Number = 1010, Name = "Checking", Currency = "CHF", BankAccountNumber = "CH01" });
|
|
await sut.SaveAccountAsync(new Account { Number = 1020, Name = "Savings", Currency = "CHF", BankAccountNumber = "CH02" });
|
|
|
|
(await sut.GetAccountForBankAccountNumberAsync("CH01")).Name.ShouldBe("Checking");
|
|
(await sut.GetAccountForBankAccountNumberAsync("CH02")).Name.ShouldBe("Savings");
|
|
(await sut.GetAccountForBankAccountNumberAsync("CH99")).ShouldBeNull();
|
|
}
|
|
|
|
[Fact]
|
|
public async Task UpdateLastBankBalanceWhenSavingBankBalance()
|
|
{
|
|
await sut.SaveAccountAsync(new Account { Number = 1010, Name = "Checking", Currency = "CHF", BankAccountNumber = "CH01" });
|
|
|
|
await sut.SaveBankAccountBalanceAsync("CH01", 1234.56m);
|
|
|
|
(await sut.GetAccountForBankAccountNumberAsync("CH01")).LastBankBalance.ShouldBe(1234.56m);
|
|
}
|
|
|
|
[Fact]
|
|
public async Task SurviveConstructorReRunWhenAccountsFileAlreadyExists()
|
|
{
|
|
await sut.SaveAccountAsync(new Account { Number = 1000, Name = "Cash", Currency = "CHF" });
|
|
|
|
// Re-construct with the same directory: existing file, no year rollover, no data loss.
|
|
var fresh = new AccountRepository(settingsService, new RepositoryCache(), NullLogger<AccountRepository>.Instance);
|
|
|
|
var accounts = await fresh.GetAccountListAsync();
|
|
accounts.Count.ShouldBe(1);
|
|
accounts[0].Name.ShouldBe("Cash");
|
|
}
|
|
}
|