Testmethoden im Backend auf snake_case umbenennen #153

New issue

Open

opened 2026-10-25 10:21:12 +00:00 by claude · 0 comments

claude commented

2026-10-25 10:21:12 +00:00

Collaborator

Worum es geht

Testmethoden sollen in snake_case heissen, weil das bei langen deutschen Testnamen besser lesbar ist (Olli, 25.10.2026). Neue Tests folgen dem ab der Einführung von PMD; dieses Issue zieht den Bestand nach.

Stand (gemessen 25.10.2026, `backend/src/test/java`)

Schreibweise	Anzahl	Beispiel
camelCase	2103	`mitAnnotationLaeuftDieSaison`
gemischt: Methode, Unterstrich, Verhalten	198	`seed_setztDieStartwerteNichtZweimal`
reines snake_case	13	`nimmt_den_benutzer_aus_der_umgebung_wenn_sie_einen_nennt`

Zusammen 2314 Methoden mit @Test oder @ParameterizedTest; 2301 davon sind umzubenennen.

Vorgehen

Mechanisch per Skript, ein Commit, keine inhaltliche Änderung an den Tests.
Zeitpunkt: wenn keine Feature-Branches mit Backend-Tests offen sind, sonst gibt es Konflikte in jedem davon.
Danach in der PMD-Regel MethodNamingConventions das Testmuster von [a-z][a-zA-Z0-9_]* (beides erlaubt) auf [a-z][a-z0-9_]* (nur snake_case) verschärfen.

Offen

Wie heisst bei der gemischten Form der Teil vor dem Unterstrich, wenn er eine Methode benennt? Zum Beispiel findScored_nimmtBestaetigteCupbegegnungenMit: wird daraus find_scored_nimmt_… oder bleibt findScored_ als Methodenname stehen?

Nicht betroffen

Die Frontend-Tests (vitest, Playwright) benennen Fälle mit Strings in it('…'); dort gibt es nichts umzubenennen.

## Worum es geht Testmethoden sollen in snake_case heissen, weil das bei langen deutschen Testnamen besser lesbar ist (Olli, 25.10.2026). Neue Tests folgen dem ab der Einführung von PMD; dieses Issue zieht den Bestand nach. ## Stand (gemessen 25.10.2026, `backend/src/test/java`) | Schreibweise | Anzahl | Beispiel | |---|---|---| | camelCase | 2103 | `mitAnnotationLaeuftDieSaison` | | gemischt: Methode, Unterstrich, Verhalten | 198 | `seed_setztDieStartwerteNichtZweimal` | | reines snake_case | 13 | `nimmt_den_benutzer_aus_der_umgebung_wenn_sie_einen_nennt` | Zusammen 2314 Methoden mit `@Test` oder `@ParameterizedTest`; 2301 davon sind umzubenennen. ## Vorgehen - Mechanisch per Skript, ein Commit, keine inhaltliche Änderung an den Tests. - Zeitpunkt: wenn keine Feature-Branches mit Backend-Tests offen sind, sonst gibt es Konflikte in jedem davon. - Danach in der PMD-Regel `MethodNamingConventions` das Testmuster von `[a-z][a-zA-Z0-9_]*` (beides erlaubt) auf `[a-z][a-z0-9_]*` (nur snake_case) verschärfen. ## Offen - Wie heisst bei der gemischten Form der Teil vor dem Unterstrich, wenn er eine Methode benennt? Zum Beispiel `findScored_nimmtBestaetigteCupbegegnungenMit`: wird daraus `find_scored_nimmt_…` oder bleibt `findScored_` als Methodenname stehen? ## Nicht betroffen Die Frontend-Tests (vitest, Playwright) benennen Fälle mit Strings in `it('…')`; dort gibt es nichts umzubenennen.