refactor: single bank operation projection and domain rules (WealthLedger/backend#168) (#189)

src/modules/banking/bank-cards/application/bank-operations.service.ts

@@ -7,6 +7,10 @@ import {
 import { PrismaService } from '../../../../prisma/prisma.service';
 import { FIRST_PAGE } from '../domain/bank-card-limits';
 import { BankOperationKind } from '../domain/bank-operation-kind';
+import {
+  operationFieldConflict,
+  operationKindChanged,
+} from '../domain/bank-operation.rules';
 import { rubMoney, ZERO_KOPECKS } from '../domain/rub-money';
 import { BankCardsRepository } from '../infrastructure/bank-cards.repository';
 import {
@@ -42,53 +46,7 @@ export class BankOperationsService {
     tx: Tx,
     row: BankCardOperation,
   ): Promise<BankOperationDto> {
-    const card = await this.repository.card(tx, row.userId, row.cardId);
-    const target = row.targetCardId
-      ? await this.repository.card(tx, row.userId, row.targetCardId)
-      : null;
-    const source = row.originalExpenseId
-      ? await this.repository.operation(tx, row.userId, row.originalExpenseId)
-      : null;
-    const categoryId = source?.categoryId ?? row.categoryId;
-    const category = categoryId
-      ? await tx.expenseCategory.findFirst({
-          where: { id: categoryId, userId: row.userId },
-        })
-      : null;
-    const refunds =
-      row.kind === BankOperationKind.Expense || source
-        ? await tx.bankCardOperation.findMany({
-            where: {
-              userId: row.userId,
-              originalExpenseId: source?.id ?? row.id,
-              deletedAt: null,
-              ...(source ? { id: { not: row.id } } : {}),
-            },
-          })
-        : [];
-    return {
-      id: row.id,
-      kind: row.kind as BankOperationDto['kind'],
-      cardId: row.cardId,
-      cardName: card.name,
-      targetCardId: row.targetCardId,
-      targetCardName: target?.name ?? null,
-      categoryId,
-      categoryName: category?.name ?? null,
-      originalExpenseId: row.originalExpenseId,
-      amount: rubMoney(row.amountKopecks),
-      availableRefund:
-        row.kind === BankOperationKind.Expense || source
-          ? rubMoney(
-              (source?.amountKopecks ?? row.amountKopecks) -
-              refunds.reduce((sum, r) => sum + r.amountKopecks, ZERO_KOPECKS),
-            )
-          : null,
-      date: row.date,
-      description: row.description,
-      version: row.version,
-      createdAt: row.createdAt.toISOString(),
-    };
+    return (await this.pageViews(tx, [row]))[0];
   }
 
   private async pageViews(tx: Tx, rows: BankCardOperation[]) {
@@ -207,25 +165,12 @@ export class BankOperationsService {
         fail(BankErrorCode.InvalidDate);
       }
     }
-    if (
-      old &&
-      (old.kind !== input.kind ||
-        old.originalExpenseId !== input.originalExpenseId)
-    ) {
+    if (old && operationKindChanged(old.kind, input.kind,
+      old.originalExpenseId, input.originalExpenseId)) {
       fail(BankErrorCode.ImmutableOperation);
     }
-    if (input.kind === BankOperationKind.Transfer) {
-      if (
-        !input.targetCardId ||
-        input.targetCardId === input.cardId ||
-        input.categoryId ||
-        input.originalExpenseId
-      ) {
-        fail();
-      }
-    } else if (input.targetCardId) fail();
+    if (operationFieldConflict(input)) fail();
     if (input.kind === BankOperationKind.Expense) {
-      if (input.originalExpenseId) fail();
       if (!input.categoryId) {
         const category = await tx.expenseCategory.findUnique({
           where: { userId_seedKey: { userId, seedKey: 'other' } },
@@ -240,16 +185,12 @@ export class BankOperationsService {
       ) {
         missing();
       }
-    } else if (input.categoryId) fail();
-    if (input.kind !== BankOperationKind.Refund && input.originalExpenseId) {
-      fail();
     }
     if (input.kind === BankOperationKind.Refund) {
-      if (!input.originalExpenseId) fail();
       const source = await this.repository.operation(
         tx,
         userId,
-        input.originalExpenseId,
+        input.originalExpenseId as string,
       );
       if (
         source.deletedAt ||