Feat/mongodb transactions

[Kmario19]

This pull request introduces transaction session management to ensure database consistency during account credit, debit, and transaction creation operations. The changes primarily involve adding session handling to the controllers and updating the corresponding tests to mock session operations.

Transaction session management:

• src/routes/account/credit/controller.ts: Added session management to start, commit, and abort transactions for credit operations.
• src/routes/account/debit/controller.ts: Added session management to start, commit, and abort transactions for debit operations.
• src/routes/transaction/create/controller.ts: Added session management to handle transactions and update account balances accordingly.

Test updates:

• src/routes/account/credit/controller.test.ts: Updated tests to mock session operations and validate transaction handling for credit operations. [1] [2]
• src/routes/account/debit/controller.test.ts: Updated tests to mock session operations and validate transaction handling for debit operations. [1] [2] [3]
• src/routes/transaction/create/controller.test.ts: Added tests to validate session handling and account balance updates for both credit and debit transactions.
• src/routes/transaction/delete/controller.test.ts: Updated tests to mock session operations for transaction deletion scenarios. [1] [2] [3] [4] [5] [6]

[Copilot]

Copilot reviewed 10 out of 10 changed files in this pull request and generated no comments.

Comments suppressed due to low confidence (1)

src/routes/transaction/delete/controller.ts:14

• The session variable is used without being initialized in this controller. Please add session initialization (e.g., const session = await Transaction.startSession();) and begin the transaction before invoking any session methods.

const transaction = await Transaction.findById(transactionId).session(session);

[Copilot]

Copilot reviewed 19 out of 20 changed files in this pull request and generated no comments.

Files not reviewed (1)

• swagger.json: Language not supported

Comments suppressed due to low confidence (3)

src/routes/transaction/edit/controller.ts:23

• Add a return statement immediately after sending the 'account not found' error response to prevent further execution of the controller logic.

if (!account) { await session.abortTransaction(); res.status(StatusCodes.NOT_FOUND).json({ error: 'Associated account not found' }); }

src/routes/transaction/edit/controller.ts:42

• Add a return statement after the insufficient funds error response to ensure no further processing is executed after aborting the transaction.

if (account.balance + balanceAdjustment < 0) { await session.abortTransaction(); res.status(StatusCodes.BAD_REQUEST).json({ error: 'Insufficient funds' }); }

src/routes/transaction/create/controller.ts:38

• When using an array with Transaction.create, Mongoose returns an array of documents; consider extracting the created document or passing a single object to ensure the response aligns with the expected format.

const transaction = await Transaction.create([ { account: accountId, amount, cost, date, type } ], { session });