Test-Analyst

describe('calculateDiscount', () => {
  it('wendet 10% für Bestellungen über 100€ an', () => {
    expect(calculateDiscount(150)).toBe(15)
  })

  it('wendet 5% für Bestellungen über 50€ an', () => {
    expect(calculateDiscount(75)).toBe(3.75)
  })

  // Was ist mit 0€? Negativ? Genau 50€? 100€?
})

Fehlende Grenzwert-Tests: 0€, 50€, 100€, negativ

describe('calculateDiscount', () => {
  it('wendet 10% für Bestellungen über 100€ an', () => {
    expect(calculateDiscount(150)).toBe(15)
  })

  it('wendet 5% für Bestellungen über 50€ an', () => {
    expect(calculateDiscount(75)).toBe(3.75)
  })

  it('gibt 0 für Bestellungen an der Grenze zurück', () => {
    expect(calculateDiscount(50)).toBe(0)
    expect(calculateDiscount(100)).toBe(5) // 5% Stufe
  })

  it('behandelt Null und Negativ ordnungsgemäß', () => {
    expect(calculateDiscount(0)).toBe(0)
    expect(calculateDiscount(-10)).toBe(0)
  })
})

Edge Cases für Grenzwerte und ungültige Eingaben hinzufügen

it('zeigt Benachrichtigung nach Speichern', async () => {
  await user.click(saveButton)

  // Instabil! Abhängig vom Timing
  await waitFor(() => {
    expect(screen.getByText('Gespeichert!')).toBeVisible()
  })

  // Noch schlimmer: willkürliches Timeout
  await new Promise(r => setTimeout(r, 100))
  expect(notificationCount).toBe(1)
})

Willkürliche Timeouts und Timing-Annahmen

it('zeigt Benachrichtigung nach Speichern', async () => {
  await user.click(saveButton)

  // Warte auf spezifische Zustandsänderung
  await waitFor(() => {
    expect(screen.getByRole('alert')).toHaveTextContent('Gespeichert!')
  })

  // Behauptung auf beobachtbares Verhalten, nicht Timing
  expect(
    await screen.findByRole('alert', { name: /gespeichert/i })
  ).toBeVisible()
})

Auf beobachtbare Zustandsänderungen warten, nicht auf Zeit

let testUser: User

beforeAll(async () => {
  // Gemeinsam für ALLE Tests — Mutationen sickern durch!
  testUser = await createUser({ name: 'Test' })
})

it('aktualisiert Benutzernamen', async () => {
  await updateUser(testUser.id, { name: 'Aktualisiert' })
  // Jetzt ist testUser.name 'Aktualisiert' für alle folgenden Tests
})

it('prüft ursprünglichen Namen', () => {
  // FEHLER! Vorheriger Test hat gemeinsamen State mutiert
  expect(testUser.name).toBe('Test')
})

Gemeinsamer State in beforeAll sickert zwischen Tests durch

describe('Benutzeraktualisierungen', () => {
  let testUser: User

  beforeEach(async () => {
    // Frischer Benutzer für JEDEN Test
    testUser = await createUser({ name: 'Test' })
  })

  afterEach(async () => {
    await deleteUser(testUser.id)
  })

  it('aktualisiert Benutzernamen', async () => {
    await updateUser(testUser.id, { name: 'Aktualisiert' })
    expect(testUser.name).toBe('Aktualisiert')
  })

  it('prüft ursprünglichen Namen', () => {
    // Funktioniert! Frischer testUser mit ursprünglichem Namen
    expect(testUser.name).toBe('Test')
  })
})

beforeEach für Test-Isolation verwenden