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.

Sign up for GitHub

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

Jump to bottom

189 lesen und schreiben des status inklusive update an monitoring service #621

Merged
merged 59 commits into from
Dec 10, 2024
Merged

189 lesen und schreiben des status inklusive update an monitoring service #621

merged 59 commits into from
Dec 10, 2024

Conversation

MrSebastian
Copy link
Member

@MrSebastian MrSebastian commented Dec 6, 2024 •
edited by coderabbitai bot
Loading

Beschreibung:

  • da mehrere Clients erstellt werden wurden die gemeinsamen Teile der Konfiguration aus den Executions ausgelagert in die Pluginkonfiguration
  • im Alten Code war im Service die Logik zum Versenden von Nachrichten an den MonitoringService enthalten. Diese wurde in kleinere Senderklasse ausgelagert. Ich denke das macht den Code wartbarer
  • Tests zu meiner Erweiterung vom DummyClient habe ich nicht erstellt da nur geloggt wird

Definition of Done (DoD):

Backend

  • Doku aktualisiert
  • Swagger-API vollständig
  • Unit-Tests gepflegt
  • Integrationstests gepflegt
  • Beispiel-Requests gepflegt

Dokumentation

  • Links geprüft

Referenzen1:

Closes #189

Summary by CodeRabbit

Release Notes

  • New Features

    • Enhanced documentation for the Ergebnismeldung-Service, detailing the status update process.
    • Introduced new roles for managing status actions in Keycloak.
    • Added new REST API endpoints for retrieving and updating election status.
    • Implemented new classes for managing and validating status updates, including logging capabilities for timestamps.

  • Bug Fixes

    • Improved error handling for status retrieval and updates.

  • Tests

    • Comprehensive unit and integration tests added for new functionalities, ensuring robust error handling and validation.

  • Configuration

    • Updated application configuration to include new monitoring client settings.

Footnotes

  1. Nicht zutreffende Referenzen vor dem Speichern entfernen ↩

@MrSebastian
added monitoring service api and defined client
42d2cb5
@MrSebastian
add mapstruct
9edfe53
@MrSebastian
add dependency wls-common:security
f867739
@MrSebastian
use offsetdatetime as localdatetime
50b9c97
@MrSebastian
✨ impl of get and post of status - first runnable draft
86e87f8
@MrSebastian
add example http requests for status
2ca3ca3
@MrSebastian
Merge remote-tracking branch 'origin/185-erfassung-awerte-inklusive-e…
8317aa9 
…ai-anbindung' into 189-lesen-und-schreiben-des-status-inklusive-update-an-monitoring-service

# Conflicts:
#	wls-ergebnismeldung-service/pom.xml
#	wls-ergebnismeldung-service/src/main/java/de/muenchen/oss/wahllokalsystem/ergebnismeldungservice/MicroServiceApplication.java
#	wls-ergebnismeldung-service/src/main/java/de/muenchen/oss/wahllokalsystem/ergebnismeldungservice/configuration/BasePathConfiguration.java
#	wls-ergebnismeldung-service/src/main/java/de/muenchen/oss/wahllokalsystem/ergebnismeldungservice/configuration/ClientConfiguration.java
#	wls-ergebnismeldung-service/src/main/java/de/muenchen/oss/wahllokalsystem/ergebnismeldungservice/configuration/Profiles.java
#	wls-ergebnismeldung-service/src/main/java/de/muenchen/oss/wahllokalsystem/ergebnismeldungservice/exception/ExceptionConstants.java
@MrSebastian
move status client to new package
f2ff99e
@MrSebastian
redesign status client interaction checks into small sender classes
c996d14
@MrSebastian
🚧 some tests
33bf974
@MrSebastian
created tests for implementation
049c1b7
@MrSebastian
remove unused class
e3f2e5c
@MrSebastian
bugfixing
fc834cc
@MrSebastian
spotless:apply
415289d
@MrSebastian
Merge remote-tracking branch 'origin/189-lesen-und-schreiben-des-stat…
af37894 
…us-inklusive-update-an-monitoring-service' into 189-lesen-und-schreiben-des-status-inklusive-update-an-monitoring-service

# Conflicts:
#	wls-ergebnismeldung-service/src/test/java/de/muenchen/oss/wahllokalsystem/ergebnismeldungservice/service/status/StatusModelMapperTest.java
#	wls-ergebnismeldung-service/src/test/java/de/muenchen/oss/wahllokalsystem/ergebnismeldungservice/service/status/StatusServiceTest.java
#	wls-ergebnismeldung-service/src/test/java/de/muenchen/oss/wahllokalsystem/ergebnismeldungservice/service/status/StatusValidatorTest.java
#	wls-ergebnismeldung-service/src/test/java/de/muenchen/oss/wahllokalsystem/ergebnismeldungservice/service/status/sender/NiederschriftDruckuhrzeitSenderTest.java
#	wls-ergebnismeldung-service/src/test/java/de/muenchen/oss/wahllokalsystem/ergebnismeldungservice/service/status/sender/NiederschriftSendungsuhrzeitSenderTest.java
@MrSebastian MrSebastian self-assigned this Dec 6, 2024
@MrSebastian MrSebastian linked an issue Dec 6, 2024 that may be closed by this pull request
Lesen und Schreiben des Status inklusive Update an Monitoring-Service #189
Closed
2 tasks
vjohnslhm
vjohnslhm requested changes Dec 6, 2024
View reviewed changes
Copy link
Contributor

@vjohnslhm vjohnslhm left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

review teil 1, tests folgen

docs/src/services/ergebnismeldung-service/index.md Outdated Show resolved Hide resolved
...ava/de/muenchen/oss/wahllokalsystem/ergebnismeldungservice/rest/status/StatusController.java Outdated Show resolved Hide resolved
wls-ergebnismeldung-service/src/test/resources/status.http Outdated Show resolved Hide resolved
wls-ergebnismeldung-service/pom.xml Show resolved Hide resolved
wls-ergebnismeldung-service/pom.xml Show resolved Hide resolved
...a/de/muenchen/oss/wahllokalsystem/ergebnismeldungservice/service/status/StatusValidator.java Outdated Show resolved Hide resolved
...nchen/oss/wahllokalsystem/ergebnismeldungservice/service/status/ValidierungsstatusModel.java Outdated Show resolved Hide resolved
...system/ergebnismeldungservice/service/status/sender/SchnellmeldungSendungsuhrzeitSender.java Outdated Show resolved Hide resolved
...system/ergebnismeldungservice/service/status/sender/SchnellmeldungSendungsuhrzeitSender.java Outdated Show resolved Hide resolved
...system/ergebnismeldungservice/service/status/sender/SchnellmeldungSendungsuhrzeitSender.java Outdated Show resolved Hide resolved
vjohnslhm
vjohnslhm requested changes Dec 6, 2024
View reviewed changes
Copy link
Contributor

@vjohnslhm vjohnslhm left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

review der tests teil 1, der rest folgt morgen :)

...va/de/muenchen/oss/wahllokalsystem/ergebnismeldungservice/client/StatusClientMapperTest.java Outdated Show resolved Hide resolved
...va/de/muenchen/oss/wahllokalsystem/ergebnismeldungservice/client/StatusClientMapperTest.java Outdated Show resolved Hide resolved
.../de/muenchen/oss/wahllokalsystem/ergebnismeldungservice/rest/status/StatusDTOMapperTest.java Outdated Show resolved Hide resolved
.../de/muenchen/oss/wahllokalsystem/ergebnismeldungservice/rest/status/StatusDTOMapperTest.java Outdated Show resolved Hide resolved
...uenchen/oss/wahllokalsystem/ergebnismeldungservice/service/status/StatusModelMapperTest.java Show resolved Hide resolved
...uenchen/oss/wahllokalsystem/ergebnismeldungservice/service/status/StatusModelMapperTest.java Outdated Show resolved Hide resolved
@MrSebastian
Copy link
Member Author

review der tests teil 1, der rest folgt morgen :)

@vjohnslhm
Morgen ist Wochenende 😄 - bitte erholen und wir schauen nächste Woche weiter

vjohnslhm
vjohnslhm requested changes Dec 9, 2024
View reviewed changes
Copy link
Contributor

@vjohnslhm vjohnslhm left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

über die namen ALL_AUTHORITIES_SET_STATUS_MISSING_WILL_RESULT_IN_WLS_EXCEPTION und ALL_AUTHORITIES_SET_STATUS_MISSING_WILL_RESULT_IN_ACCESS_DENIED bin ich gestolpert und habe etwas gebraucht, bis ich verstanden habe um was es geht. mir ist allerdings auch kein besserer name eingefallen, ohne dass er noch länger wird.

...okalsystem/ergebnismeldungservice/service/status/sender/NiederschriftDruckuhrzeitSender.java Outdated Show resolved Hide resolved
...lsystem/ergebnismeldungservice/service/status/sender/NiederschriftSendungsuhrzeitSender.java Outdated Show resolved Hide resolved
...va/de/muenchen/oss/wahllokalsystem/ergebnismeldungservice/client/StatusClientMapperTest.java Outdated Show resolved Hide resolved
...va/de/muenchen/oss/wahllokalsystem/ergebnismeldungservice/client/StatusClientMapperTest.java Outdated Show resolved Hide resolved
...de/muenchen/oss/wahllokalsystem/ergebnismeldungservice/service/status/StatusServiceTest.java Outdated Show resolved Hide resolved
...ava/de/muenchen/oss/wahllokalsystem/ergebnismeldungservice/rest/status/StatusController.java Outdated Show resolved Hide resolved
...system/ergebnismeldungservice/service/status/sender/SchnellmeldungSendungsuhrzeitSender.java Outdated Show resolved Hide resolved
...chen/oss/wahllokalsystem/ergebnismeldungservice/configuration/SecurityConfigurationTest.java Outdated Show resolved Hide resolved
...de/muenchen/oss/wahllokalsystem/ergebnismeldungservice/service/status/StatusServiceTest.java Outdated Show resolved Hide resolved
.../src/test/java/de/muenchen/oss/wahllokalsystem/ergebnismeldungservice/utils/Authorities.java Outdated Show resolved Hide resolved
@MrSebastian MrSebastian requested a review from vjohnslhm December 9, 2024 16:15
@MrSebastian MrSebastian merged commit dded6a4 into dev Dec 10, 2024
6 checks passed
@MrSebastian MrSebastian deleted the 189-lesen-und-schreiben-des-status-inklusive-update-an-monitoring-service branch December 10, 2024 07:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@coderabbitai coderabbitai[bot] coderabbitai[bot] left review comments

@vjohnslhm vjohnslhm vjohnslhm approved these changes

Assignees

@MrSebastian MrSebastian

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Lesen und Schreiben des Status inklusive Update an Monitoring-Service
2 participants
@MrSebastian @vjohnslhm