Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Tests-Naming Convention für BroadcastService #431

Open
5 tasks
vjohnslhm opened this issue Sep 24, 2024 · 1 comment
Open
5 tasks

Tests-Naming Convention für BroadcastService #431

vjohnslhm opened this issue Sep 24, 2024 · 1 comment
Labels
enhancement New feature or request service-broadcast

Comments

@vjohnslhm
Copy link
Contributor

Wie in der Doku festgehalten, wollen wir die Namen der Tests des Services nach dem Schema should_<result>_when_<input> umformulieren.

Akzeptanzkriterien:

  • alle Testnamen sind auf englsich
  • alle Testnamen folgen dem Muster should_<result>_when_<input>
  • result und input sind in CamelCase gehalten
  • die @Nested class bekommt den Namen der zu testenden Methode
  • method overloads werden durch zusätliche @Nested-Blöcke innerhalb gruppiert
@vjohnslhm
Copy link
Contributor Author

Bei der Bearbeitung eines anderen Issues ist aufgefallen, dass es im BroadcastService keinen ControllerTest gibt. In dem Zuge wäre es sinnvoll, dieses Issue zu erweitern und zusätzlich zu der Umbenennung der Tests den gesamten Service nochmal mit den neueren zu vergleichen und bei Bedarf zu überarbeiten und anzupassen.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request service-broadcast
Projects
None yet
Development

No branches or pull requests

1 participant