search

BoundfoxStudios / fairy-tale-defender

Boundfox Studios Community-Projekt - Entwicklung eines Tower Defense Spiels
https://docs.fairytaledefender.de/
MIT License
20 stars 23 forks source link

Namen von Tests anpassen. #430

Closed M4lph closed 7 months ago

M4lph commented 7 months ago

Dokumentation(Coding conventions): Bezeichnung/ Aufbau von Tests hinzugefügt Code: Bezeichnung von Testmethoden entsprechend Doku angepasst.

closes #428

Wie soll mit Fällen wie der Extensionmethode hier umgegangen werden? https://github.com/BoundfoxStudios/fairy-tale-defender/blob/01e753a064f9c85dd2fb61833c6d2115bf17179d/FairyTaleDefender/Assets/_Game/Scripts/Tests/Editor/Extensions/BezierKnotExtensions.cs#L9 Eigentlich ist es ja auch ein Test, sollte man daher hier auch die "neue" Bezeichnung anwenden oder in solchen Fällen eine Ausnahme machen?

ManuelRauber commented 7 months ago

Eigentlich ist es ja auch ein Test, sollte man daher hier auch die "neue" Bezeichnung anwenden oder in solchen Fällen eine Ausnahme machen?

Da es nur eine Hilfsmethode ist, die kein Test an sich darstellt, ist's keine Ausnahme und wir können es ganz normal belassen. Oder?

M4lph commented 7 months ago

Ja, kann gerne dabei bleiben 👍 Von der Funktion her würde ich die Methode ja schon als Test ansehen, weswegen ich sie wohl auch so bezeichnet hatte. Aber ohne ein passendes Attribut, wird sie ja nicht als Test ausgeführt. Auch wieder solche Haarspaltereien :)

ManuelRauber commented 7 months ago

Ja, kann gerne dabei bleiben 👍 Von der Funktion her würde ich die Methode ja schon als Test ansehen, weswegen ich sie wohl auch so bezeichnet hatte. Aber ohne ein passendes Attribut, wird sie ja nicht als Test ausgeführt. Auch wieder solche Haarspaltereien :)

Sie hilft ja zum Testen und ist kein eigener Test :)