- Firestore Timestamp가 res.json으로 그대로 나가 {_seconds,_nanoseconds}로 직렬화되던 문제 수정
- 앱의 포인트 내역(/reward/ledger)과 예측 기록(/stats/history) 크래시 원인 제거
- 클라 소비 7개 핸들러(reward/attendance/prediction/user/stats/kbo/chat)의 모든 응답에 명시적 DTO 타입 도입
- 날짜 와이어 형식을 순수 UTC ISO 8601(...Z)로 통일 — 채팅의 기존 KST(+09:00) 출력도 UTC로 전환
- DTO는 필드를 명시적으로 나열해 조립 (스프레드 덤프 제거) — 문서에 새 Timestamp 필드가 생겨도 다시 새지 않는다
- 재사용되는 주문 형태만 toOrderDto로 분리, 나머지는 응답 경계에서 직접 조립
- Firestore 저장 형식은 변경하지 않음. 출석 idempotent 재요청 경로는 저장된 Timestamp/문자열을 모두 처리
- DTO 직렬화 회귀 테스트 추가 (_seconds 부재, UTC ISO 형식, optional 키 생략, 미지 필드 차단)
153 lines
5.2 KiB
TypeScript
153 lines
5.2 KiB
TypeScript
import { describe, expect, it } from "vitest";
|
|
import { Timestamp } from "firebase-admin/firestore";
|
|
import {
|
|
EMPTY_WALLET_DTO,
|
|
toLedgerEntryDto,
|
|
toOrderDto,
|
|
toWalletDto,
|
|
} from "../../src/types/dto/rewardDto";
|
|
import { PointLedgerType, type PointLedgerEntry, type WalletDoc } from "../../src/types/points";
|
|
import type { OrderDoc } from "../../src/types/reward";
|
|
|
|
const UTC_ISO = /^\d{4}-\d{2}-\d{2}T\d{2}:\d{2}:\d{2}\.\d{3}Z$/;
|
|
|
|
const ts = (iso: string) => Timestamp.fromDate(new Date(iso));
|
|
|
|
/** 와이어에 실제로 나가는 모양. Timestamp 유출은 여기서만 드러난다. */
|
|
function wire(dto: unknown): Record<string, unknown> {
|
|
return JSON.parse(JSON.stringify(dto));
|
|
}
|
|
|
|
/** 중첩 구조 어디에도 admin SDK Timestamp 의 내부 필드가 남아 있지 않은지 확인. */
|
|
function expectNoTimestampLeak(dto: unknown): void {
|
|
const json = JSON.stringify(dto);
|
|
expect(json).not.toContain("_seconds");
|
|
expect(json).not.toContain("_nanoseconds");
|
|
}
|
|
|
|
const wallet: WalletDoc = {
|
|
availableBalance: 1200,
|
|
reservedBalance: 300,
|
|
totalEarned: 5000,
|
|
totalSpent: 3500,
|
|
version: 7,
|
|
createdAt: ts("2026-01-02T03:04:05.678Z"),
|
|
updatedAt: ts("2026-07-20T05:30:00.000Z"),
|
|
};
|
|
|
|
const ledgerEntry: PointLedgerEntry = {
|
|
txId: "uid:2026-07-20:attendance_daily",
|
|
uid: "uid",
|
|
type: PointLedgerType.AttendanceDaily,
|
|
op: "credit",
|
|
amount: 20,
|
|
availableBefore: 1180,
|
|
availableAfter: 1200,
|
|
reservedBefore: 300,
|
|
reservedAfter: 300,
|
|
relatedDate: "2026-07-20",
|
|
createdAt: ts("2026-07-20T05:30:00.000Z"),
|
|
};
|
|
|
|
const baseOrder: OrderDoc = {
|
|
uid: "uid",
|
|
items: [{ productId: "p1", qty: 2, pointPrice: 500, name: "굿즈" }],
|
|
totalPoints: 1000,
|
|
recipient: { name: "홍길동", phone: "01000000000", address1: "서울", postalCode: "00000" },
|
|
status: "reserved",
|
|
clientIdempotencyKey: "key-1",
|
|
reserveLedgerTxIds: ["uid:order:o1:reserve"],
|
|
orderedAt: ts("2026-07-20T05:30:00.000Z"),
|
|
statusHistory: [{ status: "reserved", at: ts("2026-07-20T05:30:00.000Z"), actor: "uid" }],
|
|
createdAt: ts("2026-07-20T05:30:00.000Z"),
|
|
updatedAt: ts("2026-07-20T05:30:00.000Z"),
|
|
};
|
|
|
|
describe("toWalletDto", () => {
|
|
it("날짜를 UTC ISO 문자열로 내보내고 Timestamp 를 유출하지 않는다", () => {
|
|
const dto = toWalletDto(wallet);
|
|
expectNoTimestampLeak(dto);
|
|
expect(wire(dto)).toEqual({
|
|
availableBalance: 1200,
|
|
reservedBalance: 300,
|
|
totalEarned: 5000,
|
|
totalSpent: 3500,
|
|
version: 7,
|
|
createdAt: "2026-01-02T03:04:05.678Z",
|
|
updatedAt: "2026-07-20T05:30:00.000Z",
|
|
});
|
|
});
|
|
|
|
it("지갑 문서가 없는 유저는 잔액 0 응답을 쓴다", () => {
|
|
expect(wire(EMPTY_WALLET_DTO)).toEqual({
|
|
availableBalance: 0,
|
|
reservedBalance: 0,
|
|
totalEarned: 0,
|
|
totalSpent: 0,
|
|
version: 0,
|
|
});
|
|
});
|
|
});
|
|
|
|
describe("toLedgerEntryDto", () => {
|
|
it("createdAt 이 UTC ISO 문자열이고 문서 id 가 붙는다", () => {
|
|
const dto = toLedgerEntryDto("entry-1", ledgerEntry);
|
|
expectNoTimestampLeak(dto);
|
|
expect(dto.id).toBe("entry-1");
|
|
expect(dto.createdAt).toMatch(UTC_ISO);
|
|
expect(dto.createdAt).toBe("2026-07-20T05:30:00.000Z");
|
|
});
|
|
|
|
it("relatedDate 의 YYYY-MM-DD 형식은 그대로 둔다", () => {
|
|
expect(toLedgerEntryDto("entry-1", ledgerEntry).relatedDate).toBe("2026-07-20");
|
|
});
|
|
|
|
it("선택 필드가 없으면 응답에서 키가 사라진다", () => {
|
|
const minimal: PointLedgerEntry = { ...ledgerEntry, relatedDate: undefined };
|
|
const json = wire(toLedgerEntryDto("entry-1", minimal));
|
|
expect(json).not.toHaveProperty("relatedDate");
|
|
expect(json).not.toHaveProperty("orderId");
|
|
});
|
|
});
|
|
|
|
describe("toOrderDto", () => {
|
|
it("중첩된 statusHistory 까지 UTC ISO 문자열로 바꾼다", () => {
|
|
const dto = toOrderDto("o1", baseOrder);
|
|
expectNoTimestampLeak(dto);
|
|
expect(dto.orderedAt).toMatch(UTC_ISO);
|
|
expect(dto.createdAt).toMatch(UTC_ISO);
|
|
expect(dto.updatedAt).toMatch(UTC_ISO);
|
|
expect(dto.statusHistory[0].at).toMatch(UTC_ISO);
|
|
expect(dto.statusHistory[0].at).toBe("2026-07-20T05:30:00.000Z");
|
|
});
|
|
|
|
it("선택 날짜 필드가 채워지면 모두 문자열로 나간다", () => {
|
|
const settled: OrderDoc = {
|
|
...baseOrder,
|
|
status: "refunded",
|
|
cancelledBy: "admin",
|
|
cancelledAt: ts("2026-07-21T00:00:00.000Z"),
|
|
confirmedAt: ts("2026-07-22T00:00:00.000Z"),
|
|
refundedAt: ts("2026-07-23T00:00:00.000Z"),
|
|
};
|
|
const dto = toOrderDto("o1", settled);
|
|
expectNoTimestampLeak(dto);
|
|
for (const value of [dto.cancelledAt, dto.confirmedAt, dto.refundedAt]) {
|
|
expect(value).toMatch(UTC_ISO);
|
|
}
|
|
});
|
|
|
|
it("선택 날짜 필드가 비면 응답에서 키가 사라진다", () => {
|
|
const json = wire(toOrderDto("o1", baseOrder));
|
|
expect(json).not.toHaveProperty("cancelledAt");
|
|
expect(json).not.toHaveProperty("confirmedAt");
|
|
expect(json).not.toHaveProperty("refundedAt");
|
|
expect(json).not.toHaveProperty("cancelledBy");
|
|
});
|
|
|
|
it("문서에 없는 필드를 임의로 흘려보내지 않는다", () => {
|
|
const polluted = { ...baseOrder, internalOnly: "secret" } as OrderDoc;
|
|
expect(wire(toOrderDto("o1", polluted))).not.toHaveProperty("internalOnly");
|
|
});
|
|
});
|