Unit-Tests für Legacy-Code schreiben, bevor du ihn änderst

Füge die komplette Funktion oder Klasse ein und nenne bekannte externe Abhängigkeiten. Führe die Tests gegen den unveränderten Code aus: Alle müssen grün sein, bevor das Refactoring beginnt.

Sprache: Python 3.11 mit pytest · Abhängigkeiten: datetime.now() und MySQL-Abfragen · Ziel: eine 300-Zeilen-Funktion zur Provisionsberechnung aufteilen.

Angenommen, die vollständige Funktion liest die Bestellungen des Kunden über PDO und nutzt date() für die Kundendauer…

2. Seams

• Datenbank: optionalen Parameter ?PDO $pdo = null ergänzen, der bei null auf die bisherige Verbindung zurückfällt.
• Datum: ?DateTimeImmutable $jetzt = null ergänzen und date() durch $jetzt->format(...) ersetzen.

Beide Änderungen lassen alle bestehenden Aufrufe unverändert.

3. Charakterisierungstests

<?php
use PHPUnit\Framework\TestCase;

final class BerechneRabattTest extends TestCase
{
    private PDO $pdo;

    protected function setUp(): void
    {
        $this->pdo = new PDO('sqlite::memory:');
        // Minimales Schema, das die echte Tabelle nachbildet: [AUSFÜLLEN]
        $this->pdo->exec('CREATE TABLE bestellungen (kunden_id INT, summe REAL, erstellt_am TEXT)');
    }

    public function test_gegeben_kunde_ohne_bestellungen_wenn_summe_100_dann_aktuelles_ergebnis(): void
    {
        $jetzt = new DateTimeImmutable('2025-03-15 10:00:00');

        $ergebnis = berechneRabatt(['id' => 42], 100.0, $this->pdo, $jetzt);

        $this->assertSame(0.0, $ergebnis); // [AUSFÜLLEN: ausführen und echte Ausgabe einfügen]
    }

    public function test_gegeben_negative_summe_dann_keine_exception(): void
    {
        // MÖGLICHER BUG: negative Summe wird nicht abgelehnt; aktuelles Verhalten festschreiben
        $ergebnis = berechneRabatt(['id' => 42], -50.0, $this->pdo, new DateTimeImmutable());
        $this->assertIsFloat($ergebnis);
    }
}

Erst jeden Test ausführen, den echten Wert in die Assertion übernehmen und erst danach mit dem Refactoring beginnen…