Best Practices zum Nachrüsten von Legacy-Code mit automatisierten Tests

Ich werde die Aufgabe übernehmen, eine bereits definierte Schnittstelle (eine Reihe von C ++ - Header-Dateien) in einer relativ großen und alten Codebasis neu zu implementieren. Zuvor möchte ich eine möglichst vollständige Testabdeckung haben, damit ich Implementierungsfehler so früh und einfach wie möglich erkennen kann. Das Problem ist, dass die bereits vorhandene Codebasis nicht einfach zu testen ist, da sie (sehr) große Klassen und Funktionen, einen hohen Kopplungsgrad, Funktionen mit (vielen) Nebenwirkungen usw. enthält.

Es wäre schön, von früheren Erfahrungen mit ähnlichen Aufgaben und einigen guten und konkreten Tipps zu hören, wie Sie automatisierte Tests (Einheiten, Integrationen, Regressionen usw.) in Ihren Legacy-Code nachrüsten.

Lassen Sie sich zunächst von Michael Feathers Effektiv mit Legacy-Code arbeiten und lesen Sie ihn - er ist eine unverzichtbare Hilfe für solche Aufgaben.

Dann ein paar Notizen:

• Haben Sie eine genaue Spezifikation / einen genauen Vertrag für die Schnittstelle oder haben Sie praktisch nur die vorhandene Implementierung als "Spezifikation"? Im ersteren Fall ist es einfacher, ein vollständiges Neuschreiben von Grund auf durchzuführen, im letzteren Fall ist es schwierig bis unmöglich.
• Wenn Sie die Schnittstelle erneut implementieren möchten, ist es am sinnvollsten, Tests nur für die Schnittstelle zu schreiben. Natürlich ist dies kein Unit-Test im eigentlichen Sinne, sondern ein Funktions- / Abnahmetest, aber ich bin kein Purist :-) Diese Tests sind jedoch wiederverwendbar und ermöglichen es Ihnen, die Ergebnisse der beiden Implementierungen direkt nebeneinander zu vergleichen .
• Insgesamt würde ich es vorziehen, den vorhandenen Code umzugestalten, anstatt ihn von Grund auf neu zu schreiben, es sei denn, er ist absolut nicht zu warten. (Aber wie schreibt man in diesem Fall überhaupt Unit-Tests dagegen?) Lesen Sie diesen Beitrag von Joel, um eine ausführlichere Diskussion zu diesem Thema zu erhalten. Wenn Sie eine Reihe von Abnahmetests für die Benutzeroberfläche erstellt haben, erhalten Sie ein dünnes, aber nützliches Sicherheitsnetz, anhand dessen Sie den vorhandenen Code vorsichtig überarbeiten können, um die Einheit testbar zu machen (unter Verwendung der Ideen aus Feathers 'Buch).

Die beste bekannte Methode ist die Mikado-Methode. http://mikadomethod.wordpress.com/2010/08/04/the-mikado-method-book/ Dies ist nur die Verallgemeinerung einer einfachen Technik, aber es ist die einzige Möglichkeit, die Codequalität in einer großen Codebasis zu verbessern ohne unnötige Risiken einzugehen.

WEWLC ist auch ein sehr gutes Buch darüber, aber in C ++ geschrieben zu sein ist mit Java- oder Ruby-Code nicht immer nützlich.

Nachträgliche Anpassungstests an eine alte Codebasis können recht schwierig sein, wenn sie monolithisch aufgebaut ist.

Wenn möglich (haben Sie die Zeit / das Geld), wäre eine Möglichkeit, voranzukommen, den Code in testbarere Einheiten umzugestalten.

Ich möchte einen Link hinzufügen . Es gibt nur wenige Beispiele für nicht so einfach zu testende Implementierungen, die in mehr xUnit-freundlichem Code umgesetzt wurden. Als allgemeine Vorgehensweise versuchen Sie es mit den bereits erwähnten Links (Joel Post, Working With Legacy Code)