From b40e6937485d92cad4eb83ba7b15892542e7817c Mon Sep 17 00:00:00 2001 From: juancwu Date: Sat, 14 Mar 2026 17:56:25 -0400 Subject: [PATCH] chore: update test data --- internal/repository/expense_test.go | 28 +++++++++++------------ internal/repository/money_account_test.go | 18 +++++++-------- internal/service/expense_test.go | 28 +++++++++++------------ internal/service/money_account_test.go | 28 +++++++++++------------ 4 files changed, 51 insertions(+), 51 deletions(-) diff --git a/internal/repository/expense_test.go b/internal/repository/expense_test.go index 10a4497..cc72577 100644 --- a/internal/repository/expense_test.go +++ b/internal/repository/expense_test.go @@ -25,7 +25,7 @@ func TestExpenseRepository_Create(t *testing.T) { SpaceID: space.ID, CreatedBy: user.ID, Description: "Lunch", - Amount: decimal.RequireFromString("15.00"), + Amount: decimal.RequireFromString("15.49"), Type: model.ExpenseTypeExpense, Date: now, CreatedAt: now, @@ -39,7 +39,7 @@ func TestExpenseRepository_Create(t *testing.T) { require.NoError(t, err) assert.Equal(t, expense.ID, fetched.ID) assert.Equal(t, "Lunch", fetched.Description) - assert.True(t, decimal.RequireFromString("15.00").Equal(fetched.Amount)) + assert.True(t, decimal.RequireFromString("15.49").Equal(fetched.Amount)) assert.Equal(t, model.ExpenseTypeExpense, fetched.Type) }) } @@ -50,9 +50,9 @@ func TestExpenseRepository_GetBySpaceIDPaginated(t *testing.T) { user := testutil.CreateTestUser(t, dbi.DB, "test@example.com", nil) space := testutil.CreateTestSpace(t, dbi.DB, user.ID, "Test Space") - testutil.CreateTestExpense(t, dbi.DB, space.ID, user.ID, "Expense 1", decimal.RequireFromString("10.00"), model.ExpenseTypeExpense) - testutil.CreateTestExpense(t, dbi.DB, space.ID, user.ID, "Expense 2", decimal.RequireFromString("20.00"), model.ExpenseTypeExpense) - testutil.CreateTestExpense(t, dbi.DB, space.ID, user.ID, "Expense 3", decimal.RequireFromString("30.00"), model.ExpenseTypeExpense) + testutil.CreateTestExpense(t, dbi.DB, space.ID, user.ID, "Expense 1", decimal.RequireFromString("10.75"), model.ExpenseTypeExpense) + testutil.CreateTestExpense(t, dbi.DB, space.ID, user.ID, "Expense 2", decimal.RequireFromString("20.50"), model.ExpenseTypeExpense) + testutil.CreateTestExpense(t, dbi.DB, space.ID, user.ID, "Expense 3", decimal.RequireFromString("30.25"), model.ExpenseTypeExpense) expenses, err := repo.GetBySpaceIDPaginated(space.ID, 2, 0) require.NoError(t, err) @@ -66,8 +66,8 @@ func TestExpenseRepository_CountBySpaceID(t *testing.T) { user := testutil.CreateTestUser(t, dbi.DB, "test@example.com", nil) space := testutil.CreateTestSpace(t, dbi.DB, user.ID, "Test Space") - testutil.CreateTestExpense(t, dbi.DB, space.ID, user.ID, "Expense 1", decimal.RequireFromString("10.00"), model.ExpenseTypeExpense) - testutil.CreateTestExpense(t, dbi.DB, space.ID, user.ID, "Expense 2", decimal.RequireFromString("20.00"), model.ExpenseTypeExpense) + testutil.CreateTestExpense(t, dbi.DB, space.ID, user.ID, "Expense 1", decimal.RequireFromString("10.75"), model.ExpenseTypeExpense) + testutil.CreateTestExpense(t, dbi.DB, space.ID, user.ID, "Expense 2", decimal.RequireFromString("20.50"), model.ExpenseTypeExpense) count, err := repo.CountBySpaceID(space.ID) require.NoError(t, err) @@ -88,7 +88,7 @@ func TestExpenseRepository_GetTagsByExpenseIDs(t *testing.T) { SpaceID: space.ID, CreatedBy: user.ID, Description: "Weekly groceries", - Amount: decimal.RequireFromString("50.00"), + Amount: decimal.RequireFromString("49.99"), Type: model.ExpenseTypeExpense, Date: now, CreatedAt: now, @@ -120,7 +120,7 @@ func TestExpenseRepository_GetPaymentMethodsByExpenseIDs(t *testing.T) { SpaceID: space.ID, CreatedBy: user.ID, Description: "Online purchase", - Amount: decimal.RequireFromString("30.00"), + Amount: decimal.RequireFromString("29.95"), Type: model.ExpenseTypeExpense, Date: now, PaymentMethodID: &method.ID, @@ -157,7 +157,7 @@ func TestExpenseRepository_GetExpensesByTag(t *testing.T) { SpaceID: space.ID, CreatedBy: user.ID, Description: "Lunch", - Amount: decimal.RequireFromString("15.00"), + Amount: decimal.RequireFromString("15.49"), Type: model.ExpenseTypeExpense, Date: now, CreatedAt: now, @@ -171,7 +171,7 @@ func TestExpenseRepository_GetExpensesByTag(t *testing.T) { SpaceID: space.ID, CreatedBy: user.ID, Description: "Dinner", - Amount: decimal.RequireFromString("25.00"), + Amount: decimal.RequireFromString("24.52"), Type: model.ExpenseTypeExpense, Date: now, CreatedAt: now, @@ -185,7 +185,7 @@ func TestExpenseRepository_GetExpensesByTag(t *testing.T) { require.Len(t, summaries, 1) assert.Equal(t, tag.ID, summaries[0].TagID) assert.Equal(t, "Food", summaries[0].TagName) - assert.True(t, decimal.RequireFromString("40.00").Equal(summaries[0].TotalAmount)) + assert.True(t, decimal.RequireFromString("40.01").Equal(summaries[0].TotalAmount)) }) } @@ -203,7 +203,7 @@ func TestExpenseRepository_Update(t *testing.T) { SpaceID: space.ID, CreatedBy: user.ID, Description: "Original", - Amount: decimal.RequireFromString("10.00"), + Amount: decimal.RequireFromString("10.75"), Type: model.ExpenseTypeExpense, Date: now, CreatedAt: now, @@ -235,7 +235,7 @@ func TestExpenseRepository_Delete(t *testing.T) { user := testutil.CreateTestUser(t, dbi.DB, "test@example.com", nil) space := testutil.CreateTestSpace(t, dbi.DB, user.ID, "Test Space") - expense := testutil.CreateTestExpense(t, dbi.DB, space.ID, user.ID, "To Delete", decimal.RequireFromString("5.00"), model.ExpenseTypeExpense) + expense := testutil.CreateTestExpense(t, dbi.DB, space.ID, user.ID, "To Delete", decimal.RequireFromString("4.99"), model.ExpenseTypeExpense) err := repo.Delete(expense.ID) require.NoError(t, err) diff --git a/internal/repository/money_account_test.go b/internal/repository/money_account_test.go index 541bb65..24055f3 100644 --- a/internal/repository/money_account_test.go +++ b/internal/repository/money_account_test.go @@ -99,7 +99,7 @@ func TestMoneyAccountRepository_CreateTransfer(t *testing.T) { transfer := &model.AccountTransfer{ ID: uuid.NewString(), AccountID: account.ID, - Amount: decimal.RequireFromString("50.00"), + Amount: decimal.RequireFromString("49.95"), Direction: model.TransferDirectionDeposit, Note: "Initial deposit", CreatedBy: user.ID, @@ -113,7 +113,7 @@ func TestMoneyAccountRepository_CreateTransfer(t *testing.T) { require.NoError(t, err) require.Len(t, transfers, 1) assert.Equal(t, transfer.ID, transfers[0].ID) - assert.True(t, decimal.RequireFromString("50.00").Equal(transfers[0].Amount)) + assert.True(t, decimal.RequireFromString("49.95").Equal(transfers[0].Amount)) assert.Equal(t, model.TransferDirectionDeposit, transfers[0].Direction) }) } @@ -124,7 +124,7 @@ func TestMoneyAccountRepository_DeleteTransfer(t *testing.T) { user := testutil.CreateTestUser(t, dbi.DB, "test@example.com", nil) space := testutil.CreateTestSpace(t, dbi.DB, user.ID, "Test Space") account := testutil.CreateTestMoneyAccount(t, dbi.DB, space.ID, "Checking", user.ID) - transfer := testutil.CreateTestTransfer(t, dbi.DB, account.ID, decimal.RequireFromString("10.00"), model.TransferDirectionDeposit, user.ID) + transfer := testutil.CreateTestTransfer(t, dbi.DB, account.ID, decimal.RequireFromString("10.25"), model.TransferDirectionDeposit, user.ID) err := repo.DeleteTransfer(transfer.ID) require.NoError(t, err) @@ -142,12 +142,12 @@ func TestMoneyAccountRepository_GetAccountBalance(t *testing.T) { space := testutil.CreateTestSpace(t, dbi.DB, user.ID, "Test Space") account := testutil.CreateTestMoneyAccount(t, dbi.DB, space.ID, "Checking", user.ID) - testutil.CreateTestTransfer(t, dbi.DB, account.ID, decimal.RequireFromString("10.00"), model.TransferDirectionDeposit, user.ID) - testutil.CreateTestTransfer(t, dbi.DB, account.ID, decimal.RequireFromString("3.00"), model.TransferDirectionWithdrawal, user.ID) + testutil.CreateTestTransfer(t, dbi.DB, account.ID, decimal.RequireFromString("10.50"), model.TransferDirectionDeposit, user.ID) + testutil.CreateTestTransfer(t, dbi.DB, account.ID, decimal.RequireFromString("3.25"), model.TransferDirectionWithdrawal, user.ID) balance, err := repo.GetAccountBalance(account.ID) require.NoError(t, err) - assert.True(t, decimal.RequireFromString("7.00").Equal(balance)) + assert.True(t, decimal.RequireFromString("7.25").Equal(balance)) }) } @@ -160,11 +160,11 @@ func TestMoneyAccountRepository_GetTotalAllocatedForSpace(t *testing.T) { account1 := testutil.CreateTestMoneyAccount(t, dbi.DB, space.ID, "Account A", user.ID) account2 := testutil.CreateTestMoneyAccount(t, dbi.DB, space.ID, "Account B", user.ID) - testutil.CreateTestTransfer(t, dbi.DB, account1.ID, decimal.RequireFromString("20.00"), model.TransferDirectionDeposit, user.ID) - testutil.CreateTestTransfer(t, dbi.DB, account2.ID, decimal.RequireFromString("30.00"), model.TransferDirectionDeposit, user.ID) + testutil.CreateTestTransfer(t, dbi.DB, account1.ID, decimal.RequireFromString("20.75"), model.TransferDirectionDeposit, user.ID) + testutil.CreateTestTransfer(t, dbi.DB, account2.ID, decimal.RequireFromString("29.50"), model.TransferDirectionDeposit, user.ID) total, err := repo.GetTotalAllocatedForSpace(space.ID) require.NoError(t, err) - assert.True(t, decimal.RequireFromString("50.00").Equal(total)) + assert.True(t, decimal.RequireFromString("50.25").Equal(total)) }) } diff --git a/internal/service/expense_test.go b/internal/service/expense_test.go index 1c7b32f..99f9aaf 100644 --- a/internal/service/expense_test.go +++ b/internal/service/expense_test.go @@ -25,7 +25,7 @@ func TestExpenseService_CreateExpense(t *testing.T) { SpaceID: space.ID, UserID: user.ID, Description: "Lunch", - Amount: decimal.RequireFromString("15.00"), + Amount: decimal.RequireFromString("15.49"), Type: model.ExpenseTypeExpense, Date: time.Now(), TagIDs: []string{tag.ID}, @@ -33,7 +33,7 @@ func TestExpenseService_CreateExpense(t *testing.T) { require.NoError(t, err) assert.NotEmpty(t, expense.ID) assert.Equal(t, "Lunch", expense.Description) - assert.True(t, decimal.RequireFromString("15.00").Equal(expense.Amount)) + assert.True(t, decimal.RequireFromString("15.49").Equal(expense.Amount)) assert.Equal(t, model.ExpenseTypeExpense, expense.Type) }) } @@ -47,7 +47,7 @@ func TestExpenseService_CreateExpense_EmptyDescription(t *testing.T) { SpaceID: "some-space", UserID: "some-user", Description: "", - Amount: decimal.RequireFromString("10.00"), + Amount: decimal.RequireFromString("10.75"), Type: model.ExpenseTypeExpense, Date: time.Now(), }) @@ -88,7 +88,7 @@ func TestExpenseService_GetExpensesWithTagsForSpacePaginated(t *testing.T) { SpaceID: space.ID, UserID: user.ID, Description: "Bus fare", - Amount: decimal.RequireFromString("2.50"), + Amount: decimal.RequireFromString("2.49"), Type: model.ExpenseTypeExpense, Date: time.Now(), TagIDs: []string{tag.ID}, @@ -100,7 +100,7 @@ func TestExpenseService_GetExpensesWithTagsForSpacePaginated(t *testing.T) { SpaceID: space.ID, UserID: user.ID, Description: "Coffee", - Amount: decimal.RequireFromString("5.00"), + Amount: decimal.RequireFromString("5.01"), Type: model.ExpenseTypeExpense, Date: time.Now(), }) @@ -133,12 +133,12 @@ func TestExpenseService_GetBalanceForSpace(t *testing.T) { user := testutil.CreateTestUser(t, dbi.DB, "exp-svc-balance@example.com", nil) space := testutil.CreateTestSpace(t, dbi.DB, user.ID, "Expense Svc Balance Space") - testutil.CreateTestExpense(t, dbi.DB, space.ID, user.ID, "Topup", decimal.RequireFromString("100.00"), model.ExpenseTypeTopup) - testutil.CreateTestExpense(t, dbi.DB, space.ID, user.ID, "Groceries", decimal.RequireFromString("30.00"), model.ExpenseTypeExpense) + testutil.CreateTestExpense(t, dbi.DB, space.ID, user.ID, "Topup", decimal.RequireFromString("100.50"), model.ExpenseTypeTopup) + testutil.CreateTestExpense(t, dbi.DB, space.ID, user.ID, "Groceries", decimal.RequireFromString("30.75"), model.ExpenseTypeExpense) balance, err := svc.GetBalanceForSpace(space.ID) require.NoError(t, err) - assert.True(t, decimal.RequireFromString("70.00").Equal(balance)) + assert.True(t, decimal.RequireFromString("69.75").Equal(balance)) }) } @@ -157,7 +157,7 @@ func TestExpenseService_GetExpensesByTag(t *testing.T) { SpaceID: space.ID, UserID: user.ID, Description: "Dinner", - Amount: decimal.RequireFromString("25.00"), + Amount: decimal.RequireFromString("24.99"), Type: model.ExpenseTypeExpense, Date: now, TagIDs: []string{tag.ID}, @@ -170,7 +170,7 @@ func TestExpenseService_GetExpensesByTag(t *testing.T) { require.NoError(t, err) require.Len(t, summaries, 1) assert.Equal(t, tag.ID, summaries[0].TagID) - assert.True(t, decimal.RequireFromString("25.00").Equal(summaries[0].TotalAmount)) + assert.True(t, decimal.RequireFromString("24.99").Equal(summaries[0].TotalAmount)) }) } @@ -186,7 +186,7 @@ func TestExpenseService_UpdateExpense(t *testing.T) { SpaceID: space.ID, UserID: user.ID, Description: "Old Description", - Amount: decimal.RequireFromString("10.00"), + Amount: decimal.RequireFromString("10.75"), Type: model.ExpenseTypeExpense, Date: time.Now(), }) @@ -196,13 +196,13 @@ func TestExpenseService_UpdateExpense(t *testing.T) { ID: created.ID, SpaceID: space.ID, Description: "New Description", - Amount: decimal.RequireFromString("20.00"), + Amount: decimal.RequireFromString("19.49"), Type: model.ExpenseTypeExpense, Date: time.Now(), }) require.NoError(t, err) assert.Equal(t, "New Description", updated.Description) - assert.True(t, decimal.RequireFromString("20.00").Equal(updated.Amount)) + assert.True(t, decimal.RequireFromString("19.49").Equal(updated.Amount)) }) } @@ -218,7 +218,7 @@ func TestExpenseService_DeleteExpense(t *testing.T) { SpaceID: space.ID, UserID: user.ID, Description: "Doomed Expense", - Amount: decimal.RequireFromString("5.00"), + Amount: decimal.RequireFromString("4.99"), Type: model.ExpenseTypeExpense, Date: time.Now(), }) diff --git a/internal/service/money_account_test.go b/internal/service/money_account_test.go index 9fb65dc..fda2819 100644 --- a/internal/service/money_account_test.go +++ b/internal/service/money_account_test.go @@ -54,13 +54,13 @@ func TestMoneyAccountService_GetAccountsForSpace(t *testing.T) { user := testutil.CreateTestUser(t, dbi.DB, "acct-svc-list@example.com", nil) space := testutil.CreateTestSpace(t, dbi.DB, user.ID, "Account Svc List Space") account := testutil.CreateTestMoneyAccount(t, dbi.DB, space.ID, "Checking", user.ID) - testutil.CreateTestTransfer(t, dbi.DB, account.ID, decimal.RequireFromString("50.00"), model.TransferDirectionDeposit, user.ID) + testutil.CreateTestTransfer(t, dbi.DB, account.ID, decimal.RequireFromString("49.95"), model.TransferDirectionDeposit, user.ID) accounts, err := svc.GetAccountsForSpace(space.ID) require.NoError(t, err) require.Len(t, accounts, 1) assert.Equal(t, "Checking", accounts[0].Name) - assert.True(t, decimal.RequireFromString("50.00").Equal(accounts[0].Balance)) + assert.True(t, decimal.RequireFromString("49.95").Equal(accounts[0].Balance)) }) } @@ -75,14 +75,14 @@ func TestMoneyAccountService_CreateTransfer_Deposit(t *testing.T) { transfer, err := svc.CreateTransfer(CreateTransferDTO{ AccountID: account.ID, - Amount: decimal.RequireFromString("30.00"), + Amount: decimal.RequireFromString("29.75"), Direction: model.TransferDirectionDeposit, Note: "Initial deposit", CreatedBy: user.ID, - }, decimal.RequireFromString("100.00")) + }, decimal.RequireFromString("100.50")) require.NoError(t, err) assert.NotEmpty(t, transfer.ID) - assert.True(t, decimal.RequireFromString("30.00").Equal(transfer.Amount)) + assert.True(t, decimal.RequireFromString("29.75").Equal(transfer.Amount)) assert.Equal(t, model.TransferDirectionDeposit, transfer.Direction) }) } @@ -98,11 +98,11 @@ func TestMoneyAccountService_CreateTransfer_InsufficientBalance(t *testing.T) { transfer, err := svc.CreateTransfer(CreateTransferDTO{ AccountID: account.ID, - Amount: decimal.RequireFromString("50.00"), + Amount: decimal.RequireFromString("50.25"), Direction: model.TransferDirectionDeposit, Note: "Too much", CreatedBy: user.ID, - }, decimal.RequireFromString("10.00")) + }, decimal.RequireFromString("10.50")) assert.Error(t, err) assert.Nil(t, transfer) }) @@ -116,18 +116,18 @@ func TestMoneyAccountService_CreateTransfer_Withdrawal(t *testing.T) { user := testutil.CreateTestUser(t, dbi.DB, "acct-svc-withdraw@example.com", nil) space := testutil.CreateTestSpace(t, dbi.DB, user.ID, "Account Svc Withdraw Space") account := testutil.CreateTestMoneyAccount(t, dbi.DB, space.ID, "Withdraw Account", user.ID) - testutil.CreateTestTransfer(t, dbi.DB, account.ID, decimal.RequireFromString("50.00"), model.TransferDirectionDeposit, user.ID) + testutil.CreateTestTransfer(t, dbi.DB, account.ID, decimal.RequireFromString("49.75"), model.TransferDirectionDeposit, user.ID) transfer, err := svc.CreateTransfer(CreateTransferDTO{ AccountID: account.ID, - Amount: decimal.RequireFromString("20.00"), + Amount: decimal.RequireFromString("19.50"), Direction: model.TransferDirectionWithdrawal, Note: "Withdrawal", CreatedBy: user.ID, }, decimal.Zero) require.NoError(t, err) assert.NotEmpty(t, transfer.ID) - assert.True(t, decimal.RequireFromString("20.00").Equal(transfer.Amount)) + assert.True(t, decimal.RequireFromString("19.50").Equal(transfer.Amount)) assert.Equal(t, model.TransferDirectionWithdrawal, transfer.Direction) }) } @@ -141,14 +141,14 @@ func TestMoneyAccountService_GetTotalAllocatedForSpace(t *testing.T) { space := testutil.CreateTestSpace(t, dbi.DB, user.ID, "Account Svc Total Space") account1 := testutil.CreateTestMoneyAccount(t, dbi.DB, space.ID, "Account 1", user.ID) - testutil.CreateTestTransfer(t, dbi.DB, account1.ID, decimal.RequireFromString("30.00"), model.TransferDirectionDeposit, user.ID) + testutil.CreateTestTransfer(t, dbi.DB, account1.ID, decimal.RequireFromString("30.25"), model.TransferDirectionDeposit, user.ID) account2 := testutil.CreateTestMoneyAccount(t, dbi.DB, space.ID, "Account 2", user.ID) - testutil.CreateTestTransfer(t, dbi.DB, account2.ID, decimal.RequireFromString("20.00"), model.TransferDirectionDeposit, user.ID) + testutil.CreateTestTransfer(t, dbi.DB, account2.ID, decimal.RequireFromString("19.50"), model.TransferDirectionDeposit, user.ID) total, err := svc.GetTotalAllocatedForSpace(space.ID) require.NoError(t, err) - assert.True(t, decimal.RequireFromString("50.00").Equal(total)) + assert.True(t, decimal.RequireFromString("49.75").Equal(total)) }) } @@ -178,7 +178,7 @@ func TestMoneyAccountService_DeleteTransfer(t *testing.T) { user := testutil.CreateTestUser(t, dbi.DB, "acct-svc-deltx@example.com", nil) space := testutil.CreateTestSpace(t, dbi.DB, user.ID, "Account Svc DelTx Space") account := testutil.CreateTestMoneyAccount(t, dbi.DB, space.ID, "DelTx Account", user.ID) - transfer := testutil.CreateTestTransfer(t, dbi.DB, account.ID, decimal.RequireFromString("10.00"), model.TransferDirectionDeposit, user.ID) + transfer := testutil.CreateTestTransfer(t, dbi.DB, account.ID, decimal.RequireFromString("10.25"), model.TransferDirectionDeposit, user.ID) err := svc.DeleteTransfer(transfer.ID) require.NoError(t, err)