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 MonitoringService #435

Closed
5 tasks
vjohnslhm opened this issue Sep 24, 2024 · 3 comments · Fixed by #578
Closed
5 tasks

Tests-Naming Convention für MonitoringService #435

vjohnslhm opened this issue Sep 24, 2024 · 3 comments · Fixed by #578
Assignees
Labels
enhancement New feature or request service-monitoring

Comments

@vjohnslhm
Copy link
Contributor

vjohnslhm commented Sep 24, 2024

UPDATE:

Reopen, da nach der Umsetzung noch Tests vorhanden sind, die nicht der Convention entsprechen.

Die sich wiederholenden Tests im .configuration-Package wurden bereits im Broadcast-Service umbenannt. Bei der neuen Benennung kann sich hier daran orientiert werden.


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 vjohnslhm added enhancement New feature or request service-monitoring labels Sep 24, 2024
@vjohnslhm
Copy link
Contributor Author

vjohnslhm commented Sep 24, 2024

Die Implementierung hat noch nciht stattgefunden. Da die Convention ab sofort gültig ist wird hier kein extra Ticket benötigt, sondern direkt in der Umsetzung berücksichtigt. (#169, #170)

@vjohnslhm
Copy link
Contributor Author

Bei der Umsetzung sind die Tests im .configuration-Package nicht berücksichtigt worden, daher wurde das Ticket wieder eröffnet.

@vjohnslhm vjohnslhm reopened this Nov 15, 2024
vjohnslhm added a commit that referenced this issue Nov 20, 2024
vjohnslhm added a commit that referenced this issue Nov 20, 2024
@vjohnslhm vjohnslhm mentioned this issue Nov 20, 2024
1 task
@vjohnslhm vjohnslhm linked a pull request Nov 20, 2024 that will close this issue
1 task
@vjohnslhm vjohnslhm self-assigned this Nov 20, 2024
vjohnslhm added a commit that referenced this issue Nov 21, 2024
vjohnslhm added a commit that referenced this issue Nov 21, 2024
@vjohnslhm
Copy link
Contributor Author

commit 106c80f gehört zu issue #437

vjohnslhm added a commit that referenced this issue Nov 22, 2024
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-monitoring
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant