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. #428

Closed ManuelRauber closed 7 months ago

ManuelRauber commented 7 months ago

Sowas wie Methode_ErwartetesVerhalten_Bedingung finde ich persönlich ansprechender, in der Weise sind auch die meisten bisher benannt. Aber du hast die meisten Tests geschrieben, wirst wohl auch noch die meisten schreiben, und bist Project Owner :) Also ich passe mich da einfach an dann 👍

Passt für mich aber auch, wenn wir die Underscore-Variant wählen, das dürfte besser lesbar sein. Ich mach ein Ticket, um die Name anzupassen. :)

Originally posted by @ManuelRauber in https://github.com/BoundfoxStudios/fairy-tale-defender/issues/427#issuecomment-1886726602

M4lph commented 7 months ago

.meine-aufgabe

github-actions[bot] commented 7 months ago

Danke, dass Du diese Aufgabe übernimmst! Falls Du Fragen hast, schreib sie einfach ins Issue!

M4lph commented 7 months ago

In der asmdef ist als Rootnamespace nicht der vollständige Pfad angegeben, bei allen anderen glaube ich schon. grafik

Soll ich den auch gerade noch mit anpassen? BoundfoxStudios.FairyTaleDefender.Tests.Runtime?

ManuelRauber commented 7 months ago

In der asmdef ist als Rootnamespace nicht der vollständige Pfad angegeben, bei allen anderen glaube ich schon. grafik

Soll ich den auch gerade noch mit anpassen? BoundfoxStudios.FairyTaleDefender.Tests.Runtime?

Ich würde fast behaupten, dass das bei der asmdef für die normale Runtime (also nicht tests) genau so ist. :)

Ist gewollt so, weil ich kein "Runtime" im Namespace haben will für das "normale". Nur bei Editor steht's mit drin, damit's eben im Editor-Namespace ist.

M4lph commented 7 months ago

Ich würde fast behaupten, dass das bei der asmdef für die normale Runtime (also nicht tests) genau so ist. :)

Jepp, hab ich wohl übersehen.

Ist gewollt so, weil ich kein "Runtime" im Namespace haben will für das "normale".

Okidoki 👍