Skip to content

refactor: Contract-Validierung delegieren - #31

Draft
mohammadtaiba wants to merge 1 commit into
mainfrom
refactor/contract-validation-service
Draft

refactor: Contract-Validierung delegieren#31
mohammadtaiba wants to merge 1 commit into
mainfrom
refactor/contract-validation-service

Conversation

@mohammadtaiba

Copy link
Copy Markdown
Owner

Änderungen

  • ContractValidationService ergänzt, der ContractValidator aufruft und Validierungsfehler in eine IllegalArgumentException übersetzt.
  • ContractService delegiert die Vertragsvalidierung an den neuen Service.
  • Manuelles Contract-Test-Setup um die neue Dependency ergänzt.
  • Unit-Tests für gültige Sale-/Rental-Verträge und zusammengefügte Fehlermeldungen hinzugefügt.

Gründe

  • ContractService enthält damit keine eigene Validierungs- und Fehlerlistenlogik mehr.
  • Die Übersetzung von Validator-Ergebnissen in Service-Exceptions ist zentral testbar.
  • Der Contract-Service bleibt stärker auf den Ablauf von Create/Update/Delete fokussiert.

Validierung

  • ./mvnw.cmd test
  • Ergebnis: 111 Tests, 0 Fehler, 0 Errors, 0 übersprungen

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant