Skip to content

Commit

Permalink
finalize
Browse files Browse the repository at this point in the history
  • Loading branch information
Pl8tinium committed Dec 12, 2024
1 parent 63b4f85 commit b181062
Show file tree
Hide file tree
Showing 3 changed files with 6 additions and 2 deletions.
1 change: 0 additions & 1 deletion .github/workflows/sq.yml
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,6 @@ on:
push:
branches:
- main
- feature/abgabe-alex-p

jobs:
build:
Expand Down
1 change: 0 additions & 1 deletion .github/workflows/tests.yml
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,6 @@ on:
push:
branches:
- main
- feature/abgabe-alex-p
pull_request:
branches:
- main
Expand Down
6 changes: 6 additions & 0 deletions bericht.md → README_Aufgabe2.md
Original file line number Diff line number Diff line change
Expand Up @@ -71,3 +71,9 @@ Ein weiterer Hotspot war, dass Sonarqube davon ausgeht, dass Express Versionsinf
Eine Sache störte mich noch. Ich habe bisher 0% Code coverage. Ich machte mich also ans Werk und began weitere Tests zu schreiben um mehr Funktionalität des Programs abzudecken. Eine Vermutung macht sich jedoch breit, dass die Code Coverade, da es ein API Test ist, sich nicht erhöhen wird. Ich müsste unit tests erstellen und Funktionen in dem Hauptteil des Programs abkapseln. Da das Hauptprogram aber sehr wenig Logik enthält entschied ich mich dazu es bei erweiterten API Tests zu belassen.

Nun schlug die Quality gate durch die fehlende Anzahl an Code Coverage bei neuem Code fehl und ich knickte ein und implementierte Unit tests, zumindest für die Datenbank Klasse.

Dies hat die Code coverage leider auch nicht verbessert. Ich entschied, dass die Anwendung, Ihrem Umfang entsprechend, gut genug getestet ist.

## Finalization

Ich entferne zum Schluss noch die Ausführung der GitHub actions auf meinem Branch. Dies war für Testzwecke konfiguriert worden um den Feedback Loop möglichst gering zu halten.

0 comments on commit b181062

Please sign in to comment.