좋은 테스트 코드, 나쁜 테스트 코드 — util편 (1)

좋은 테스트 코드, 나쁜 테스트 코드 — util편

테스트 이름은 문서입니다

describe('formatPrice', () => {
  it('should work correctly', () => {
    expect(formatPrice(1000)).toBe('1,000원');
    expect(formatPrice(0)).toBe('0원');
    expect(formatPrice(-500)).toBe('-500원');
  });
});

describe('formatPrice', () => {
  it('양수 금액에 천 단위 콤마와 "원"을 붙인다', () => {
    expect(formatPrice(1000)).toBe('1,000원');
  });

  it('0원은 콤마 없이 "0원"을 반환한다', () => {
    expect(formatPrice(0)).toBe('0원');
  });

  it('음수 금액은 마이너스 부호를 유지한다', () => {
    expect(formatPrice(-500)).toBe('-500원');
  });
});

구현이 아니라 동작을 테스트합니다

it('Array.prototype.sort를 호출한다', () => {
  const spy = vi.spyOn(Array.prototype, 'sort');
  sortUsers([{ name: '김철수', age: 30 }, { name: '이영희', age: 25 }]);
  expect(spy).toHaveBeenCalled();
  spy.mockRestore();
});

it('사용자를 나이 오름차순으로 정렬한다', () => {
  const users = [
    { name: '김철수', age: 30 },
    { name: '이영희', age: 25 },
    { name: '박민수', age: 35 },
  ];

  expect(sortUsers(users)).toEqual([
    { name: '이영희', age: 25 },
    { name: '김철수', age: 30 },
    { name: '박민수', age: 35 },
  ]);
});

it('나이가 같으면 이름순으로 정렬한다', () => {
  const users = [
    { name: '김철수', age: 30 },
    { name: '강민지', age: 30 },
  ];

  expect(sortUsers(users)).toEqual([
    { name: '강민지', age: 30 },
    { name: '김철수', age: 30 },
  ]);
});

it('VIP 할인 로직을 실행한다', () => {
  const spy = vi.spyOn(discountModule, 'applyVipDiscount');
  calculateTotal({ items: [{ price: 10000 }], memberGrade: 'vip' });
  expect(spy).toHaveBeenCalled();
});

it('VIP 등급은 10% 할인된 금액을 반환한다', () => {
  const result = calculateTotal({
    items: [{ price: 10000 }],
    memberGrade: 'vip',
  });

  expect(result).toBe(9000);
});

it('일반 등급은 할인 없이 원래 금액을 반환한다', () => {
  const result = calculateTotal({
    items: [{ price: 10000 }],
    memberGrade: 'normal',
  });

  expect(result).toBe(10000);
});

각 테스트는 독립적으로 실행 가능해야 합니다

let cart: Cart;

beforeAll(() => {
  cart = new Cart();
});

it('상품을 추가할 수 있다', () => {
  cart.add({ id: '1', name: '키보드', price: 50000 });
  expect(cart.items).toHaveLength(1);
});

it('총 금액을 계산할 수 있다', () => {
  // 위 테스트에서 추가한 '키보드'가 남아 있다고 가정
  expect(cart.totalPrice()).toBe(50000);
});

const createCart = () => new Cart();

it('상품을 추가하면 items에 포함된다', () => {
  const cart = createCart();
  cart.add({ id: '1', name: '키보드', price: 50000 });
  expect(cart.items).toHaveLength(1);
});

it('추가된 상품의 금액을 합산하여 총 금액을 반환한다', () => {
  const cart = createCart();
  cart.add({ id: '1', name: '키보드', price: 50000 });
  expect(cart.totalPrice()).toBe(50000);
});

it('오늘 날짜 기준으로 D-day를 계산한다', () => {
  // 이 테스트는 실행하는 날짜에 따라 결과가 달라진다
  const dday = calculateDday('2025-12-31');
  expect(dday).toBe(275);
});

beforeEach(() => {
  vi.useFakeTimers();
  vi.setSystemTime(new Date('2025-03-31'));
});

afterEach(() => {
  vi.useRealTimers();
});

it('기준일로부터 대상일까지 남은 일수를 반환한다', () => {
  expect(calculateDday('2025-12-31')).toBe(275);
});

it('대상일이 지났으면 음수를 반환한다', () => {
  expect(calculateDday('2025-03-01')).toBe(-30);
});

테스트는 실패할 수 있어야 의미가 있습니다

it('입력값을 파싱한다', () => {
  try {
    const result = parseConfig('{"port": 3000}');
    expect(result).toBeDefined();
  } catch {
    // 에러가 나도 통과
  }
});

it('JSON 문자열을 파싱하여 설정 객체를 반환한다', () => {
  const config = parseConfig('{"port": 3000, "host": "localhost"}');

  expect(config).toEqual({
    port: 3000,
    host: 'localhost',
  });
});

it('유효하지 않은 JSON이면 ConfigParseError를 던진다', () => {
  expect(() => parseConfig('not-json')).toThrow(ConfigParseError);
});

it('필수 필드가 누락되면 MissingFieldError를 던진다', () => {
  expect(() => parseConfig('{"host": "localhost"}')).toThrow(
    MissingFieldError
  );
});

마무리