Tests that don't actually check anything

TSПрактика

Main has `calculateDiscount.ts` with no tests. Leadership wants coverage pushed to 95%. A colleague sent a PR — a new file with 5 tests. All green, coverage jumped. But you know: coverage ≠ quality. Walk the diff and find the tests that don't actually check anything.

Клик по строке diff — напиши свой комментарий. Когда закончишь, жми «Отправить ревью».Туториал этого урока

1-// calculateDiscount.test.ts — new file

1+import { describe, it, expect, vi } from 'vitest'

2+import { calculateDiscount } from './calculateDiscount'

3+import { db } from './db'

4+

5+vi.mock('./db')

6+

7+describe('calculateDiscount', () => {

8+ it('should work', () => {

9+ const result = calculateDiscount(100, 'SILVER')

10+ expect(result).toBeDefined()

11+ })

12+

13+ it('applies discount', () => {

14+ const result = calculateDiscount(100, 'GOLD')

15+ expect(result).not.toBe(100)

16+ })

17+

18+ it('handles edge case', () => {

19+ expect(() => calculateDiscount(0, 'SILVER')).not.toThrow()

20+ })

21+

22+ it('saves to db', async () => {

23+ vi.mocked(db.discounts.save).mockResolvedValue({ id: '1' })

24+ await calculateDiscount(100, 'GOLD')

25+ expect(db.discounts.save).toHaveBeenCalled()

26+ })

27+

28+ it('handles invalid tier', () => {

29+ const result = calculateDiscount(100, 'INVALID')

30+ expect(result).toBe(result)

31+ })

32+})

0 комментариев

ArchCase

ArchCase

Tests that don't actually check anything