Hausaufgabe 2

[JordanHubb]

No description provided.

[ProfWider]

ok, sieht schon gar nicht so schlecht aus. Der 1. rote Test und der passende Fix sind gut. Aber der 2. rote Test ist zwar etwas einfach aber vor allem glaube ich nicht, dass er durch den letzten Commit gefixt wird (zumindest ist das nicht bei mir lokal der Fall). Es kann sein, dass Sie diesen Test bereits durch Ihren 1. Bugfix grün gemacht haben, aber dann zählt es nicht als 2 Bugfixes. Test+Bugfix bedeutet immer, dass der Test rot sein muss vor dem Bugfix und grün danach. Aber außerdem fehlt Ihnen auch noch folgendes aus der Aufgabenstellung zum Bestehen: "Schreiben Sie zwei neue zusätzliche Tests, die zwei unterschiedliche Fehler in zwei unterschiedlichen Methoden aufdecken (d.h. Kombinationen von unterschiedliche Methoden testen) und somit fehlschlagen. Da die Kombination pressOperationKey, pressDigitKey, pressEquals schon getestet ist, sollte also noch mindestens eine der Methoden pressClear, pressDotKey, oder pressNegative getestet werden."
D.h. Sie müssen noch einen Test + Bugfix hinzufügen, der eine dieser Methoden benutzt. Ich denke aber Sie können das noch schaffen!

[ProfWider]

Ihre Nachbesserungen reichen leider noch nicht ganz: Die Methode pressClearKey, so wie Sie sie verwenden, hat bereits vorher funktioniert und Ihr Test war nur rot, weil Sie die nicht existierende Methode pressClear aufgerufen haben. Insofern haben Sie da nichts gefixt. Probieren Sie doch z.B. mal, was passiert (in einem Test), wenn die Taste pressNegative gedrückt wird. Das ist die Taste, auf dem online-calculator unten in der Mitte, auf der +/- steht.

[ProfWider]

Das ist ein echter Bugfix! 👍 Das reicht zum Bestehen.
Die Commit-Struktur ist natürlich jetzt arg durcheinander geraten. Wenn Sie Lust haben, können Sie jetzt noch neu forken, clonen, dann Stück für Stück all Ihre Änderungen rüber ziehen, dabei immer sauber Commits erzeugen, und dann einen neuen "schönen" PR stellen.