From e38ccc9d83706d19c5585fad5aaee648fd429537 Mon Sep 17 00:00:00 2001 From: Michael Hoennig Date: Sat, 9 Nov 2024 14:17:13 +0100 Subject: [PATCH 1/6] use .uuid in API instead of a directly attached Uuid --- .../hs-booking/hs-booking-item-schemas.yaml | 6 ++--- .../hs-booking-project-schemas.yaml | 4 +-- .../hs-hosting/hs-hosting-asset-schemas.yaml | 18 ++++++------- .../hs-office-coopassets-schemas.yaml | 6 ++--- .../hs-office-coopshares-schemas.yaml | 6 ++--- .../hs-office/hs-office-debitor-schemas.yaml | 8 +++--- .../hs-office-membership-schemas.yaml | 4 +-- .../hs-office/hs-office-partner-schemas.yaml | 14 +++++----- .../hs-office/hs-office-relation-schemas.yaml | 26 +++++++++---------- .../hs-office-sepamandate-schemas.yaml | 8 +++--- .../rbac/rbac-grant-schemas.yaml | 10 +++---- .../rbac/rbac-role-schemas.yaml | 2 +- .../rbac/rbac-subject-schemas.yaml | 6 ++--- 13 files changed, 59 insertions(+), 59 deletions(-) diff --git a/src/main/resources/api-definition/hs-booking/hs-booking-item-schemas.yaml b/src/main/resources/api-definition/hs-booking/hs-booking-item-schemas.yaml index ef0ac307..97ed69f6 100644 --- a/src/main/resources/api-definition/hs-booking/hs-booking-item-schemas.yaml +++ b/src/main/resources/api-definition/hs-booking/hs-booking-item-schemas.yaml @@ -52,11 +52,11 @@ components: HsBookingItemInsert: type: object properties: - projectUuid: + project.uuid: type: string format: uuid nullable: false - parentItemUuid: + parentItem.uuid: type: string format: uuid nullable: false @@ -77,7 +77,7 @@ components: $ref: '../hs-hosting/hs-hosting-asset-schemas.yaml#/components/schemas/HsHostingAssetAutoInsert' required: - caption - - projectUuid + - project.uuid - validFrom - resources additionalProperties: false diff --git a/src/main/resources/api-definition/hs-booking/hs-booking-project-schemas.yaml b/src/main/resources/api-definition/hs-booking/hs-booking-project-schemas.yaml index de95203d..56626290 100644 --- a/src/main/resources/api-definition/hs-booking/hs-booking-project-schemas.yaml +++ b/src/main/resources/api-definition/hs-booking/hs-booking-project-schemas.yaml @@ -25,7 +25,7 @@ components: HsBookingProjectInsert: type: object properties: - debitorUuid: + debitor.uuid: type: string format: uuid nullable: false @@ -35,6 +35,6 @@ components: maxLength: 80 nullable: false required: - - debitorUuid + - debitor.uuid - caption additionalProperties: false diff --git a/src/main/resources/api-definition/hs-hosting/hs-hosting-asset-schemas.yaml b/src/main/resources/api-definition/hs-hosting/hs-hosting-asset-schemas.yaml index 44813162..39fafaeb 100644 --- a/src/main/resources/api-definition/hs-hosting/hs-hosting-asset-schemas.yaml +++ b/src/main/resources/api-definition/hs-hosting/hs-hosting-asset-schemas.yaml @@ -54,7 +54,7 @@ components: caption: type: string nullable: true - alarmContactUuid: + alarmContact.uuid: type: string format: uuid nullable: true @@ -64,11 +64,11 @@ components: HsHostingAssetInsert: type: object properties: - bookingItemUuid: + bookingItem.uuid: type: string format: uuid nullable: true - parentAssetUuid: + parentAsset.uuid: type: string format: uuid nullable: true @@ -84,7 +84,7 @@ components: minLength: 3 maxLength: 80 nullable: false - alarmContactUuid: + alarmContact.uuid: type: string format: uuid nullable: true @@ -99,11 +99,11 @@ components: HsHostingAssetAutoInsert: type: object properties: - parentAssetUuid: + parentAsset.uuid: type: string format: uuid nullable: true - assignedToAssetUuid: + assignedToAsset.uuid: type: string format: uuid type: @@ -118,7 +118,7 @@ components: minLength: 3 maxLength: 80 nullable: false - alarmContactUuid: + alarmContact.uuid: type: string format: uuid nullable: true @@ -147,10 +147,10 @@ components: minLength: 3 maxLength: 80 nullable: false - assignedToAssetUuid: + assignedToAsset.uuid: type: string format: uuid - alarmContactUuid: + alarmContact.uuid: type: string format: uuid nullable: true diff --git a/src/main/resources/api-definition/hs-office/hs-office-coopassets-schemas.yaml b/src/main/resources/api-definition/hs-office/hs-office-coopassets-schemas.yaml index 0c937767..16f4806b 100644 --- a/src/main/resources/api-definition/hs-office/hs-office-coopassets-schemas.yaml +++ b/src/main/resources/api-definition/hs-office/hs-office-coopassets-schemas.yaml @@ -62,7 +62,7 @@ components: HsOfficeCoopAssetsTransactionInsert: type: object properties: - membershipUuid: + membership.uuid: type: string format: uuid nullable: false @@ -80,11 +80,11 @@ components: maxLength: 48 comment: type: string - reverseEntryUuid: + reverseEntry.uuid: type: string format: uuid required: - - membershipUuid + - membership.uuid - transactionType - assetValue - valueDate diff --git a/src/main/resources/api-definition/hs-office/hs-office-coopshares-schemas.yaml b/src/main/resources/api-definition/hs-office/hs-office-coopshares-schemas.yaml index 680321be..50ea65a5 100644 --- a/src/main/resources/api-definition/hs-office/hs-office-coopshares-schemas.yaml +++ b/src/main/resources/api-definition/hs-office/hs-office-coopshares-schemas.yaml @@ -56,7 +56,7 @@ components: HsOfficeCoopSharesTransactionInsert: type: object properties: - membershipUuid: + membership.uuid: type: string format: uuid nullable: false @@ -73,11 +73,11 @@ components: maxLength: 48 comment: type: string - adjustedShareTxUuid: + adjustedShareTx.uuid: type: string format: uuid required: - - membershipUuid + - membership.uuid - transactionType - shareCount - valueDate diff --git a/src/main/resources/api-definition/hs-office/hs-office-debitor-schemas.yaml b/src/main/resources/api-definition/hs-office/hs-office-debitor-schemas.yaml index ed4d8c26..bc3d402e 100644 --- a/src/main/resources/api-definition/hs-office/hs-office-debitor-schemas.yaml +++ b/src/main/resources/api-definition/hs-office/hs-office-debitor-schemas.yaml @@ -41,7 +41,7 @@ components: HsOfficeDebitorPatch: type: object properties: - debitorRelUuid: + debitorRel.uuid: type: string format: uuid nullable: true @@ -61,7 +61,7 @@ components: vatReverseCharge: type: boolean nullable: false - refundBankAccountUuid: + refundBankAccount.uuid: type: string format: uuid nullable: true @@ -75,7 +75,7 @@ components: properties: debitorRel: $ref: 'hs-office-relation-schemas.yaml#/components/schemas/HsOfficeRelationSubInsert' - debitorRelUuid: + debitorRel.uuid: type: string format: uuid debitorNumberSuffix: @@ -92,7 +92,7 @@ components: type: boolean vatReverseCharge: type: boolean - refundBankAccountUuid: + refundBankAccount.uuid: type: string format: uuid defaultPrefix: diff --git a/src/main/resources/api-definition/hs-office/hs-office-membership-schemas.yaml b/src/main/resources/api-definition/hs-office/hs-office-membership-schemas.yaml index 7132cff4..20459f4e 100644 --- a/src/main/resources/api-definition/hs-office/hs-office-membership-schemas.yaml +++ b/src/main/resources/api-definition/hs-office/hs-office-membership-schemas.yaml @@ -62,7 +62,7 @@ components: HsOfficeMembershipInsert: type: object properties: - partnerUuid: + partner.uuid: type: string format: uuid nullable: false @@ -86,7 +86,7 @@ components: nullable: false type: boolean required: - - partnerUuid + - partner.uuid - memberNumberSuffix - validFrom - membershipFeeBillable diff --git a/src/main/resources/api-definition/hs-office/hs-office-partner-schemas.yaml b/src/main/resources/api-definition/hs-office/hs-office-partner-schemas.yaml index 0e5952e1..bd327e10 100644 --- a/src/main/resources/api-definition/hs-office/hs-office-partner-schemas.yaml +++ b/src/main/resources/api-definition/hs-office/hs-office-partner-schemas.yaml @@ -50,7 +50,7 @@ components: HsOfficePartnerPatch: type: object properties: - partnerRelUuid: + partnerRel.uuid: type: string format: uuid nullable: true @@ -103,19 +103,19 @@ components: type: object nullable: false properties: - anchorUuid: + anchor.uuid: type: string format: uuid - holderUuid: + holder.uuid: type: string format: uuid - contactUuid: + contact.uuid: type: string format: uuid required: - - anchorUuid - - holderUuid - - relContactUuid + - anchor.uuid + - holder.uuid + - relContact.uuid HsOfficePartnerDetailsInsert: type: object diff --git a/src/main/resources/api-definition/hs-office/hs-office-relation-schemas.yaml b/src/main/resources/api-definition/hs-office/hs-office-relation-schemas.yaml index cbad776c..16ba6070 100644 --- a/src/main/resources/api-definition/hs-office/hs-office-relation-schemas.yaml +++ b/src/main/resources/api-definition/hs-office/hs-office-relation-schemas.yaml @@ -36,7 +36,7 @@ components: HsOfficeRelationPatch: type: object properties: - contactUuid: + contact.uuid: type: string format: uuid nullable: true @@ -45,10 +45,10 @@ components: HsOfficeRelationInsert: type: object properties: - anchorUuid: + anchor.uuid: type: string format: uuid - holderUuid: + holder.uuid: type: string format: uuid type: @@ -57,32 +57,32 @@ components: mark: type: string nullable: true - contactUuid: + contact.uuid: type: string format: uuid required: - - anchorUuid - - holderUuid + - anchor.uuid + - holder.uuid - type - - contactUuid + - contact.uuid # relation created as a sub-element with implicitly known type HsOfficeRelationSubInsert: type: object properties: - anchorUuid: + anchor.uuid: type: string format: uuid - holderUuid: + holder.uuid: type: string format: uuid mark: type: string nullable: true - contactUuid: + contact.uuid: type: string format: uuid required: - - anchorUuid - - holderUuid - - contactUuid + - anchor.uuid + - holder.uuid + - contact.uuid diff --git a/src/main/resources/api-definition/hs-office/hs-office-sepamandate-schemas.yaml b/src/main/resources/api-definition/hs-office/hs-office-sepamandate-schemas.yaml index 80668ba8..2afd4127 100644 --- a/src/main/resources/api-definition/hs-office/hs-office-sepamandate-schemas.yaml +++ b/src/main/resources/api-definition/hs-office/hs-office-sepamandate-schemas.yaml @@ -48,11 +48,11 @@ components: HsOfficeSepaMandateInsert: type: object properties: - debitorUuid: + debitor.uuid: type: string format: uuid nullable: false - bankAccountUuid: + bankAccount.uuid: type: string format: uuid nullable: false @@ -72,8 +72,8 @@ components: format: date nullable: true required: - - debitorUuid - - bankAccountUuid + - debitor.uuid + - bankAccount.uuid - reference - agreement - validFrom diff --git a/src/main/resources/api-definition/rbac/rbac-grant-schemas.yaml b/src/main/resources/api-definition/rbac/rbac-grant-schemas.yaml index 90700354..c713ebc6 100644 --- a/src/main/resources/api-definition/rbac/rbac-grant-schemas.yaml +++ b/src/main/resources/api-definition/rbac/rbac-grant-schemas.yaml @@ -8,21 +8,21 @@ components: properties: grantedByRoleIdName: type: string - grantedByRoleUuid: + grantedByRole.uuid: type: string format: uuid assumed: type: boolean grantedRoleIdName: type: string - grantedRoleUuid: + grantedRole.uuid: type: string format: uuid granteeSubjectName: type: string - granteeSubjectUuid: + granteeSubject.uuid: type: string format: uuid required: - - grantedRoleUuid - - granteeSubjectUuid + - grantedRole.uuid + - granteeSubject.uuid diff --git a/src/main/resources/api-definition/rbac/rbac-role-schemas.yaml b/src/main/resources/api-definition/rbac/rbac-role-schemas.yaml index 4e5b5f4d..bf72a8e5 100644 --- a/src/main/resources/api-definition/rbac/rbac-role-schemas.yaml +++ b/src/main/resources/api-definition/rbac/rbac-role-schemas.yaml @@ -9,7 +9,7 @@ components: uuid: type: string format: uuid - objectUuid: + object.uuid: type: string format: uuid objectTable: diff --git a/src/main/resources/api-definition/rbac/rbac-subject-schemas.yaml b/src/main/resources/api-definition/rbac/rbac-subject-schemas.yaml index 9cb8ec0f..b907093a 100644 --- a/src/main/resources/api-definition/rbac/rbac-subject-schemas.yaml +++ b/src/main/resources/api-definition/rbac/rbac-subject-schemas.yaml @@ -14,7 +14,7 @@ components: RbacSubjectPermission: type: object properties: - objectUuid: + object.uuid: type: string format: uuid objectTable: @@ -23,10 +23,10 @@ components: type: string roleName: type: string - roleUuid: + role.uuid: type: string format: uuid - permissionUuid: + permission.uuid: type: string format: uuid op: -- 2.39.5 From 1f19bbbcac501d7004bc3304bb65b80f2b760046 Mon Sep 17 00:00:00 2001 From: Michael Hoennig Date: Sat, 9 Nov 2024 14:28:20 +0100 Subject: [PATCH 2/6] amend ScenarioTests according to .uuid --- .../hs/office/scenarios/contact/ReplaceContactData.java | 2 +- .../debitor/CreateExternalDebitorForPartner.java | 8 ++++---- .../scenarios/debitor/CreateSelfDebitorForPartner.java | 8 ++++---- .../scenarios/debitor/CreateSepaMandateForDebitor.java | 4 ++-- .../hs/office/scenarios/membership/CreateMembership.java | 2 +- .../scenarios/partner/AddOperationsContactToPartner.java | 6 +++--- .../scenarios/partner/AddRepresentativeToPartner.java | 6 +++--- .../hs/office/scenarios/partner/CreatePartner.java | 6 +++--- .../scenarios/subscription/SubscribeToMailinglist.java | 6 +++--- 9 files changed, 24 insertions(+), 24 deletions(-) diff --git a/src/test/java/net/hostsharing/hsadminng/hs/office/scenarios/contact/ReplaceContactData.java b/src/test/java/net/hostsharing/hsadminng/hs/office/scenarios/contact/ReplaceContactData.java index e60bbc8e..2638a4e5 100644 --- a/src/test/java/net/hostsharing/hsadminng/hs/office/scenarios/contact/ReplaceContactData.java +++ b/src/test/java/net/hostsharing/hsadminng/hs/office/scenarios/contact/ReplaceContactData.java @@ -47,7 +47,7 @@ public class ReplaceContactData extends UseCase { withTitle("Replace the Contact-Reference in the Partner-Relation", () -> httpPatch("/api/hs/office/relations/%{partnerRelationUuid}", usingJsonBody(""" { - "contactUuid": ${Contact: %{newContactCaption}} + "contact.uuid": ${Contact: %{newContactCaption}} } """)) .expecting(OK) diff --git a/src/test/java/net/hostsharing/hsadminng/hs/office/scenarios/debitor/CreateExternalDebitorForPartner.java b/src/test/java/net/hostsharing/hsadminng/hs/office/scenarios/debitor/CreateExternalDebitorForPartner.java index 194d4513..cf525c30 100644 --- a/src/test/java/net/hostsharing/hsadminng/hs/office/scenarios/debitor/CreateExternalDebitorForPartner.java +++ b/src/test/java/net/hostsharing/hsadminng/hs/office/scenarios/debitor/CreateExternalDebitorForPartner.java @@ -55,9 +55,9 @@ public class CreateExternalDebitorForPartner extends UseCase { protected HttpResponse run() { return httpPost("/api/hs/office/memberships", usingJsonBody(""" { - "partnerUuid": ${Partner: Test AG}, + "partner.uuid": ${Partner: Test AG}, "memberNumberSuffix": ${memberNumberSuffix}, "status": "ACTIVE", "validFrom": ${validFrom}, diff --git a/src/test/java/net/hostsharing/hsadminng/hs/office/scenarios/partner/AddOperationsContactToPartner.java b/src/test/java/net/hostsharing/hsadminng/hs/office/scenarios/partner/AddOperationsContactToPartner.java index 8cef6e10..f862c782 100644 --- a/src/test/java/net/hostsharing/hsadminng/hs/office/scenarios/partner/AddOperationsContactToPartner.java +++ b/src/test/java/net/hostsharing/hsadminng/hs/office/scenarios/partner/AddOperationsContactToPartner.java @@ -57,9 +57,9 @@ public class AddOperationsContactToPartner extends UseCase { { "partnerNumber": ${partnerNumber}, "partnerRel": { - "anchorUuid": ${Person: Hostsharing eG}, - "holderUuid": ${Person: %{%{tradeName???}???%{givenName???} %{familyName???}}}, - "contactUuid": ${Contact: %{contactCaption}} + "anchor.uuid": ${Person: Hostsharing eG}, + "holder.uuid": ${Person: %{%{tradeName???}???%{givenName???} %{familyName???}}}, + "contact.uuid": ${Contact: %{contactCaption}} }, "details": { "registrationOffice": "Registergericht Hamburg", diff --git a/src/test/java/net/hostsharing/hsadminng/hs/office/scenarios/subscription/SubscribeToMailinglist.java b/src/test/java/net/hostsharing/hsadminng/hs/office/scenarios/subscription/SubscribeToMailinglist.java index 3e4ae74b..7cca9ba4 100644 --- a/src/test/java/net/hostsharing/hsadminng/hs/office/scenarios/subscription/SubscribeToMailinglist.java +++ b/src/test/java/net/hostsharing/hsadminng/hs/office/scenarios/subscription/SubscribeToMailinglist.java @@ -52,9 +52,9 @@ public class SubscribeToMailinglist extends UseCase { { "type": "SUBSCRIBER", "mark": ${mailingList}, - "anchorUuid": ${Person: %{partnerPersonTradeName}}, - "holderUuid": ${Person: %{subscriberGivenName} %{subscriberFamilyName}}, - "contactUuid": ${Contact: %{subscriberGivenName} %{subscriberFamilyName}} + "anchor.uuid": ${Person: %{partnerPersonTradeName}}, + "holder.uuid": ${Person: %{subscriberGivenName} %{subscriberFamilyName}}, + "contact.uuid": ${Contact: %{subscriberGivenName} %{subscriberFamilyName}} } """)) .expecting(CREATED).expecting(JSON); -- 2.39.5 From 3a910468b403863cc0ceeae8eceff22de3bf13ee Mon Sep 17 00:00:00 2001 From: Michael Hoennig Date: Sat, 9 Nov 2024 14:55:31 +0100 Subject: [PATCH 3/6] amend AcceptanceTests according to .uuid --- ...HsBookingItemControllerAcceptanceTest.java | 14 ++++---- .../item/HsBookingItemControllerRestTest.java | 4 +-- ...ookingProjectControllerAcceptanceTest.java | 2 +- ...sHostingAssetControllerAcceptanceTest.java | 14 ++++---- ...omainSetupHostingAssetFactoryUnitTest.java | 8 ++--- ...tsTransactionControllerAcceptanceTest.java | 12 +++---- ...opAssetsTransactionControllerRestTest.java | 2 +- ...esTransactionControllerAcceptanceTest.java | 8 ++--- ...opSharesTransactionControllerRestTest.java | 4 +-- ...OfficeDebitorControllerAcceptanceTest.java | 18 +++++------ ...iceMembershipControllerAcceptanceTest.java | 2 +- .../HsOfficeMembershipControllerRestTest.java | 8 ++--- ...OfficePartnerControllerAcceptanceTest.java | 32 +++++++++---------- .../HsOfficePartnerControllerRestTest.java | 20 ++++++------ ...fficeRelationControllerAcceptanceTest.java | 28 ++++++++-------- ...ceSepaMandateControllerAcceptanceTest.java | 14 ++++---- 16 files changed, 95 insertions(+), 95 deletions(-) diff --git a/src/test/java/net/hostsharing/hsadminng/hs/booking/item/HsBookingItemControllerAcceptanceTest.java b/src/test/java/net/hostsharing/hsadminng/hs/booking/item/HsBookingItemControllerAcceptanceTest.java index 6b4a4b29..3598c0ce 100644 --- a/src/test/java/net/hostsharing/hsadminng/hs/booking/item/HsBookingItemControllerAcceptanceTest.java +++ b/src/test/java/net/hostsharing/hsadminng/hs/booking/item/HsBookingItemControllerAcceptanceTest.java @@ -150,7 +150,7 @@ class HsBookingItemControllerAcceptanceTest extends ContextBasedTestWithCleanup .contentType(ContentType.JSON) .body(""" { - "projectUuid": "{projectUuid}", + "project.uuid": "{projectUuid}", "type": "MANAGED_SERVER", "caption": "some new booking", "validTo": "{validTo}", @@ -200,8 +200,8 @@ class HsBookingItemControllerAcceptanceTest extends ContextBasedTestWithCleanup .contentType(ContentType.JSON) .body(""" { - "projectUuid": "{projectUuid}", - "parentItemUuid": "{managedServerUuid}", + "project.uuid": "{projectUuid}", + "parentItem.uuid": "{managedServerUuid}", "type": "MANAGED_WEBSPACE", "caption": "some managed webspace", "resources": { @@ -270,7 +270,7 @@ class HsBookingItemControllerAcceptanceTest extends ContextBasedTestWithCleanup .contentType(ContentType.JSON) .body(""" { - "projectUuid": "{projectUuid}", + "project.uuid": "{projectUuid}", "type": "DOMAIN_SETUP", "caption": "Domain-Setup for example.org", "resources": { @@ -285,7 +285,7 @@ class HsBookingItemControllerAcceptanceTest extends ContextBasedTestWithCleanup }, { "type": "DOMAIN_HTTP_SETUP", - "assignedToAssetUuid": "{unixUserUuid}" + "assignedToAsset.uuid": "{unixUserUuid}" }, { "type": "DOMAIN_MBOX_SETUP" @@ -360,7 +360,7 @@ class HsBookingItemControllerAcceptanceTest extends ContextBasedTestWithCleanup .contentType(ContentType.JSON) .body(""" { - "projectUuid": "{projectUuid}", + "project.uuid": "{projectUuid}", "type": "DOMAIN_SETUP", "caption": "some new domain-setup booking", "resources": { @@ -375,7 +375,7 @@ class HsBookingItemControllerAcceptanceTest extends ContextBasedTestWithCleanup }, { "type": "DOMAIN_HTTP_SETUP", - "assignedToAssetUuid": "{unixUserUuid}" + "assignedToAsset.uuid": "{unixUserUuid}" }, { "type": "DOMAIN_MBOX_SETUP" diff --git a/src/test/java/net/hostsharing/hsadminng/hs/booking/item/HsBookingItemControllerRestTest.java b/src/test/java/net/hostsharing/hsadminng/hs/booking/item/HsBookingItemControllerRestTest.java index 11ecf5bc..6b2bf5df 100644 --- a/src/test/java/net/hostsharing/hsadminng/hs/booking/item/HsBookingItemControllerRestTest.java +++ b/src/test/java/net/hostsharing/hsadminng/hs/booking/item/HsBookingItemControllerRestTest.java @@ -105,7 +105,7 @@ class HsBookingItemControllerRestTest { .contentType(MediaType.APPLICATION_JSON) .content(""" { - "projectUuid": "{projectUuid}", + "project.uuid": "{projectUuid}", "type": "MANAGED_SERVER", "caption": "some new booking", "validTo": "{validTo}", @@ -155,7 +155,7 @@ class HsBookingItemControllerRestTest { .contentType(MediaType.APPLICATION_JSON) .content(""" { - "projectUuid": "{projectUuid}", + "project.uuid": "{projectUuid}", "type": "MANAGED_SERVER", "caption": "some new booking", "validFrom": "{validFrom}", diff --git a/src/test/java/net/hostsharing/hsadminng/hs/booking/project/HsBookingProjectControllerAcceptanceTest.java b/src/test/java/net/hostsharing/hsadminng/hs/booking/project/HsBookingProjectControllerAcceptanceTest.java index 69c11d9f..2e281882 100644 --- a/src/test/java/net/hostsharing/hsadminng/hs/booking/project/HsBookingProjectControllerAcceptanceTest.java +++ b/src/test/java/net/hostsharing/hsadminng/hs/booking/project/HsBookingProjectControllerAcceptanceTest.java @@ -92,7 +92,7 @@ class HsBookingProjectControllerAcceptanceTest extends ContextBasedTestWithClean .contentType(ContentType.JSON) .body(""" { - "debitorUuid": "%s", + "debitor.uuid": "%s", "caption": "some new project" } """.formatted(givenDebitor.getUuid())) diff --git a/src/test/java/net/hostsharing/hsadminng/hs/hosting/asset/HsHostingAssetControllerAcceptanceTest.java b/src/test/java/net/hostsharing/hsadminng/hs/hosting/asset/HsHostingAssetControllerAcceptanceTest.java index 30b62a5a..2eff25de 100644 --- a/src/test/java/net/hostsharing/hsadminng/hs/hosting/asset/HsHostingAssetControllerAcceptanceTest.java +++ b/src/test/java/net/hostsharing/hsadminng/hs/hosting/asset/HsHostingAssetControllerAcceptanceTest.java @@ -165,10 +165,10 @@ class HsHostingAssetControllerAcceptanceTest extends ContextBasedTestWithCleanup .contentType(ContentType.JSON) .body(""" { - "bookingItemUuid": "%s", + "bookingItem.uuid": "%s", "type": "MANAGED_WEBSPACE", "identifier": "fir10", - "parentAssetUuid": "%s", + "parentAsset.uuid": "%s", "caption": "some separate ManagedWebspace HA", "config": {} } @@ -227,7 +227,7 @@ class HsHostingAssetControllerAcceptanceTest extends ContextBasedTestWithCleanup .contentType(ContentType.JSON) .body(""" { - "parentAssetUuid": "%s", + "parentAsset.uuid": "%s", "type": "UNIX_USER", "identifier": "fir01-temp", "caption": "some new UnixUser in client's ManagedWebspace", @@ -280,7 +280,7 @@ class HsHostingAssetControllerAcceptanceTest extends ContextBasedTestWithCleanup .contentType(ContentType.JSON) .body(""" { - "bookingItemUuid": "%s", + "bookingItem.uuid": "%s", "type": "DOMAIN_SETUP", "identifier": "example.com", "caption": "some unrelated domain-setup", @@ -326,7 +326,7 @@ class HsHostingAssetControllerAcceptanceTest extends ContextBasedTestWithCleanup .contentType(ContentType.JSON) .body(""" { - "bookingItemUuid": "%s", + "bookingItem.uuid": "%s", "type": "MANAGED_SERVER", "identifier": "vm1400", "caption": "some new ManagedServer", @@ -381,7 +381,7 @@ class HsHostingAssetControllerAcceptanceTest extends ContextBasedTestWithCleanup .contentType(ContentType.JSON) .body(""" { - "parentAssetUuid": "%s", + "parentAsset.uuid": "%s", "type": "UNIX_USER", "identifier": "fir01-extra", "caption": "some extra UnixUser", @@ -508,7 +508,7 @@ class HsHostingAssetControllerAcceptanceTest extends ContextBasedTestWithCleanup .contentType(ContentType.JSON) .body(""" { - "alarmContactUuid": "%s", + "alarmContact.uuid": "%s", "config": { "monit_max_ssd_usage": 85, "monit_max_hdd_usage": null, diff --git a/src/test/java/net/hostsharing/hsadminng/hs/hosting/asset/factories/DomainSetupHostingAssetFactoryUnitTest.java b/src/test/java/net/hostsharing/hsadminng/hs/hosting/asset/factories/DomainSetupHostingAssetFactoryUnitTest.java index 14762bd0..cf2fc350 100644 --- a/src/test/java/net/hostsharing/hsadminng/hs/hosting/asset/factories/DomainSetupHostingAssetFactoryUnitTest.java +++ b/src/test/java/net/hostsharing/hsadminng/hs/hosting/asset/factories/DomainSetupHostingAssetFactoryUnitTest.java @@ -83,7 +83,7 @@ class DomainSetupHostingAssetFactoryUnitTest { "subHostingAssets": [ { "type": "DOMAIN_HTTP_SETUP", - "assignedToAssetUuid": "{unixUserHostingAssetUuid}" + "assignedToAsset.uuid": "{unixUserHostingAssetUuid}" }, { "type": "DOMAIN_DNS_SETUP" @@ -124,7 +124,7 @@ class DomainSetupHostingAssetFactoryUnitTest { "subHostingAssets": [ { "type": "DOMAIN_HTTP_SETUP", - "assignedToAssetUuid": "{unixUserHostingAssetUuid}" + "assignedToAsset.uuid": "{unixUserHostingAssetUuid}" }, { "type": "DOMAIN_DNS_SETUP" @@ -164,7 +164,7 @@ class DomainSetupHostingAssetFactoryUnitTest { "subHostingAssets": [ { "type": "DOMAIN_HTTP_SETUP", - "assignedToAssetUuid": "{unixUserHostingAssetUuid}" + "assignedToAsset.uuid": "{unixUserHostingAssetUuid}" }, { "type": "DOMAIN_DNS_SETUP" @@ -206,7 +206,7 @@ class DomainSetupHostingAssetFactoryUnitTest { "subHostingAssets": [ { "type": "DOMAIN_HTTP_SETUP", - "assignedToAssetUuid": "{unixUserHostingAssetUuid}" + "assignedToAsset.uuid": "{unixUserHostingAssetUuid}" }, { "type": "DOMAIN_DNS_SETUP" diff --git a/src/test/java/net/hostsharing/hsadminng/hs/office/coopassets/HsOfficeCoopAssetsTransactionControllerAcceptanceTest.java b/src/test/java/net/hostsharing/hsadminng/hs/office/coopassets/HsOfficeCoopAssetsTransactionControllerAcceptanceTest.java index e8766490..6ac97292 100644 --- a/src/test/java/net/hostsharing/hsadminng/hs/office/coopassets/HsOfficeCoopAssetsTransactionControllerAcceptanceTest.java +++ b/src/test/java/net/hostsharing/hsadminng/hs/office/coopassets/HsOfficeCoopAssetsTransactionControllerAcceptanceTest.java @@ -180,7 +180,7 @@ class HsOfficeCoopAssetsTransactionControllerAcceptanceTest extends ContextBased .contentType(ContentType.JSON) .body(""" { - "membershipUuid": "%s", + "membership.uuid": "%s", "transactionType": "DEPOSIT", "assetValue": 1024.00, "valueDate": "2022-10-13", @@ -237,13 +237,13 @@ class HsOfficeCoopAssetsTransactionControllerAcceptanceTest extends ContextBased .contentType(ContentType.JSON) .body(""" { - "membershipUuid": "%s", + "membership.uuid": "%s", "transactionType": "ADJUSTMENT", "assetValue": %s, "valueDate": "2022-10-30", "reference": "test ref adjustment", "comment": "some coop assets adjustment transaction", - "reverseEntryUuid": "%s" + "reverseEntry.uuid": "%s" } """.formatted( givenMembership.getUuid(), @@ -270,7 +270,7 @@ class HsOfficeCoopAssetsTransactionControllerAcceptanceTest extends ContextBased "reference": "test ref" } } - """.formatted(givenTransaction.getUuid()))) + """)) .header("Location", startsWith("http://localhost")) .extract().header("Location"); // @formatter:on @@ -293,14 +293,14 @@ class HsOfficeCoopAssetsTransactionControllerAcceptanceTest extends ContextBased .contentType(ContentType.JSON) .body(""" { - "membershipUuid": "%s", + "membership.uuid": "%s", "transactionType": "DISBURSAL", "assetValue": -10240.00, "valueDate": "2022-10-13", "reference": "temp ref X", "comment": "just some test coop assets transaction" } - """.formatted(givenMembership.getUuid())) + """.formatted(givenMembership.getUuid())) .port(port) .when() .post("http://localhost/api/hs/office/coopassetstransactions") diff --git a/src/test/java/net/hostsharing/hsadminng/hs/office/coopassets/HsOfficeCoopAssetsTransactionControllerRestTest.java b/src/test/java/net/hostsharing/hsadminng/hs/office/coopassets/HsOfficeCoopAssetsTransactionControllerRestTest.java index 03a1c71b..f00edff8 100644 --- a/src/test/java/net/hostsharing/hsadminng/hs/office/coopassets/HsOfficeCoopAssetsTransactionControllerRestTest.java +++ b/src/test/java/net/hostsharing/hsadminng/hs/office/coopassets/HsOfficeCoopAssetsTransactionControllerRestTest.java @@ -37,7 +37,7 @@ class HsOfficeCoopAssetsTransactionControllerRestTest { static final String VALID_INSERT_REQUEST_BODY = """ { - "membershipUuid": "%s", + "membership.uuid": "%s", "transactionType": "DEPOSIT", "assetValue": 128.00, "valueDate": "2022-10-13", diff --git a/src/test/java/net/hostsharing/hsadminng/hs/office/coopshares/HsOfficeCoopSharesTransactionControllerAcceptanceTest.java b/src/test/java/net/hostsharing/hsadminng/hs/office/coopshares/HsOfficeCoopSharesTransactionControllerAcceptanceTest.java index 28a7723e..a1e8a675 100644 --- a/src/test/java/net/hostsharing/hsadminng/hs/office/coopshares/HsOfficeCoopSharesTransactionControllerAcceptanceTest.java +++ b/src/test/java/net/hostsharing/hsadminng/hs/office/coopshares/HsOfficeCoopSharesTransactionControllerAcceptanceTest.java @@ -168,7 +168,7 @@ class HsOfficeCoopSharesTransactionControllerAcceptanceTest extends ContextBased final var location = RestAssured // @formatter:off .given().header("current-subject", "superuser-alex@hostsharing.net").contentType(ContentType.JSON).body(""" { - "membershipUuid": "%s", + "membership.uuid": "%s", "transactionType": "SUBSCRIPTION", "shareCount": 8, "valueDate": "2022-10-13", @@ -214,13 +214,13 @@ class HsOfficeCoopSharesTransactionControllerAcceptanceTest extends ContextBased .contentType(ContentType.JSON) .body(""" { - "membershipUuid": "%s", + "membership.uuid": "%s", "transactionType": "ADJUSTMENT", "shareCount": %s, "valueDate": "2022-10-30", "reference": "test ref adjustment", "comment": "some coop shares adjustment transaction", - "adjustedShareTxUuid": "%s" + "adjustedShareTx.uuid": "%s" } """.formatted( givenMembership.getUuid(), @@ -267,7 +267,7 @@ class HsOfficeCoopSharesTransactionControllerAcceptanceTest extends ContextBased RestAssured // @formatter:off .given().header("current-subject", "superuser-alex@hostsharing.net").contentType(ContentType.JSON).body(""" { - "membershipUuid": "%s", + "membership.uuid": "%s", "transactionType": "CANCELLATION", "shareCount": -80, "valueDate": "2022-10-13", diff --git a/src/test/java/net/hostsharing/hsadminng/hs/office/coopshares/HsOfficeCoopSharesTransactionControllerRestTest.java b/src/test/java/net/hostsharing/hsadminng/hs/office/coopshares/HsOfficeCoopSharesTransactionControllerRestTest.java index 1e33fde9..7024c668 100644 --- a/src/test/java/net/hostsharing/hsadminng/hs/office/coopshares/HsOfficeCoopSharesTransactionControllerRestTest.java +++ b/src/test/java/net/hostsharing/hsadminng/hs/office/coopshares/HsOfficeCoopSharesTransactionControllerRestTest.java @@ -37,7 +37,7 @@ class HsOfficeCoopSharesTransactionControllerRestTest { static final String VALID_INSERT_REQUEST_BODY = """ { - "membershipUuid": "%s", + "membership.uuid": "%s", "transactionType": "SUBSCRIPTION", "shareCount": 8, "valueDate": "2022-10-13", @@ -48,7 +48,7 @@ class HsOfficeCoopSharesTransactionControllerRestTest { enum BadRequestTestCases { MEMBERSHIP_UUID_MISSING( - requestBody -> requestBody.without("membershipUuid"), + requestBody -> requestBody.without("membership.uuid"), "[membershipUuid must not be null but is \"null\"]"), TRANSACTION_TYPE_MISSING( diff --git a/src/test/java/net/hostsharing/hsadminng/hs/office/debitor/HsOfficeDebitorControllerAcceptanceTest.java b/src/test/java/net/hostsharing/hsadminng/hs/office/debitor/HsOfficeDebitorControllerAcceptanceTest.java index a15ffbce..f6623fe0 100644 --- a/src/test/java/net/hostsharing/hsadminng/hs/office/debitor/HsOfficeDebitorControllerAcceptanceTest.java +++ b/src/test/java/net/hostsharing/hsadminng/hs/office/debitor/HsOfficeDebitorControllerAcceptanceTest.java @@ -287,14 +287,14 @@ class HsOfficeDebitorControllerAcceptanceTest extends ContextBasedTestWithCleanu .contentType(ContentType.JSON) .body(""" { - "debitorRelUuid": "%s", + "debitorRel.uuid": "%s", "debitorNumberSuffix": "%s", "billable": "true", "vatId": "VAT123456", "vatCountryCode": "DE", "vatBusiness": true, "vatReverseCharge": "false", - "refundBankAccountUuid": "%s", + "refundBankAccount.uuid": "%s", "defaultPrefix": "for" } """.formatted( givenDebitorRelUUid, ++nextDebitorSuffix, givenBankAccount.getUuid())) @@ -333,9 +333,9 @@ class HsOfficeDebitorControllerAcceptanceTest extends ContextBasedTestWithCleanu .body(""" { "debitorRel": { - "anchorUuid": "%s", - "holderUuid": "%s", - "contactUuid": "%s" + "anchor.uuid": "%s", + "holder.uuid": "%s", + "contact.uuid": "%s" }, "debitorNumberSuffix": "%s", "defaultPrefix": "for", @@ -384,9 +384,9 @@ class HsOfficeDebitorControllerAcceptanceTest extends ContextBasedTestWithCleanu .body(""" { "debitorRel": { - "anchorUuid": "%s", - "holderUuid": "%s", - "contactUuid": "%s" + "anchor.uuid": "%s", + "holder.uuid": "%s", + "contact.uuid": "%s" }, "debitorNumberSuffix": "%s", "defaultPrefix": "for", @@ -418,7 +418,7 @@ class HsOfficeDebitorControllerAcceptanceTest extends ContextBasedTestWithCleanu .contentType(ContentType.JSON) .body(""" { - "debitorRelUuid": "%s", + "debitorRel.uuid": "%s", "debitorNumberSuffix": "%s", "defaultPrefix": "for", "billable": "true", diff --git a/src/test/java/net/hostsharing/hsadminng/hs/office/membership/HsOfficeMembershipControllerAcceptanceTest.java b/src/test/java/net/hostsharing/hsadminng/hs/office/membership/HsOfficeMembershipControllerAcceptanceTest.java index 55f2fbd7..f3793da1 100644 --- a/src/test/java/net/hostsharing/hsadminng/hs/office/membership/HsOfficeMembershipControllerAcceptanceTest.java +++ b/src/test/java/net/hostsharing/hsadminng/hs/office/membership/HsOfficeMembershipControllerAcceptanceTest.java @@ -182,7 +182,7 @@ class HsOfficeMembershipControllerAcceptanceTest extends ContextBasedTestWithCle .contentType(ContentType.JSON) .body(""" { - "partnerUuid": "%s", + "partner.uuid": "%s", "memberNumberSuffix": "%s", "validFrom": "2022-10-13", "membershipFeeBillable": "true" diff --git a/src/test/java/net/hostsharing/hsadminng/hs/office/membership/HsOfficeMembershipControllerRestTest.java b/src/test/java/net/hostsharing/hsadminng/hs/office/membership/HsOfficeMembershipControllerRestTest.java index 1fcc9f11..455008a2 100644 --- a/src/test/java/net/hostsharing/hsadminng/hs/office/membership/HsOfficeMembershipControllerRestTest.java +++ b/src/test/java/net/hostsharing/hsadminng/hs/office/membership/HsOfficeMembershipControllerRestTest.java @@ -56,12 +56,12 @@ public class HsOfficeMembershipControllerRestTest { .contentType(MediaType.APPLICATION_JSON) .content(""" { - "partnerUuid": null, + "partner.uuid": null, "memberNumberSuffix": "01", "validFrom": "2022-10-13", "membershipFeeBillable": "true" } - """.formatted(UUID.randomUUID())) + """) .accept(MediaType.APPLICATION_JSON)) // then @@ -86,7 +86,7 @@ public class HsOfficeMembershipControllerRestTest { .contentType(MediaType.APPLICATION_JSON) .content(""" { - "partnerUuid": "%s", + "partner.uuid": "%s", "memberNumberSuffix": "01", "validFrom": "2022-10-13", "membershipFeeBillable": "true" @@ -112,7 +112,7 @@ public class HsOfficeMembershipControllerRestTest { .contentType(MediaType.APPLICATION_JSON) .content(""" { - "partnerUuid": "%s", + "partner.uuid": "%s", %s "validFrom": "2022-10-13", "membershipFeeBillable": "true" diff --git a/src/test/java/net/hostsharing/hsadminng/hs/office/partner/HsOfficePartnerControllerAcceptanceTest.java b/src/test/java/net/hostsharing/hsadminng/hs/office/partner/HsOfficePartnerControllerAcceptanceTest.java index 8c75ed16..c2ec1271 100644 --- a/src/test/java/net/hostsharing/hsadminng/hs/office/partner/HsOfficePartnerControllerAcceptanceTest.java +++ b/src/test/java/net/hostsharing/hsadminng/hs/office/partner/HsOfficePartnerControllerAcceptanceTest.java @@ -102,9 +102,9 @@ class HsOfficePartnerControllerAcceptanceTest extends ContextBasedTestWithCleanu { "partnerNumber": "20002", "partnerRel": { - "anchorUuid": "%s", - "holderUuid": "%s", - "contactUuid": "%s" + "anchor.uuid": "%s", + "holder.uuid": "%s", + "contact.uuid": "%s" }, "details": { "registrationOffice": "Temp Registergericht Aurich", @@ -161,12 +161,12 @@ class HsOfficePartnerControllerAcceptanceTest extends ContextBasedTestWithCleanu { "partnerNumber": "20003", "partnerRel": { - "anchorUuid": "%s", - "holderUuid": "%s", - "contactUuid": "%s" + "anchor.uuid": "%s", + "holder.uuid": "%s", + "contact.uuid": "%s" }, - "personUuid": "%s", - "contactUuid": "%s", + "person.uuid": "%s", + "contact.uuid": "%s", "details": {} } """.formatted( @@ -199,12 +199,12 @@ class HsOfficePartnerControllerAcceptanceTest extends ContextBasedTestWithCleanu { "partnerNumber": "20004", "partnerRel": { - "anchorUuid": "%s", - "holderUuid": "%s", - "contactUuid": "%s" + "anchor.uuid": "%s", + "holder.uuid": "%s", + "contact.uuid": "%s" }, - "personUuid": "%s", - "contactUuid": "%s", + "person.uuid": "%s", + "contact.uuid": "%s", "details": {} } """.formatted( @@ -321,7 +321,7 @@ class HsOfficePartnerControllerAcceptanceTest extends ContextBasedTestWithCleanu .body(""" { "partnerNumber": "20011", - "partnerRelUuid": "%s", + "partnerRel.uuid": "%s", "details": { "registrationOffice": "Temp Registergericht Aurich", "registrationNumber": "222222", @@ -330,7 +330,7 @@ class HsOfficePartnerControllerAcceptanceTest extends ContextBasedTestWithCleanu "dateOfDeath": "2022-01-12" } } - """.formatted(givenPartnerRel.getUuid())) + """.formatted(givenPartnerRel.getUuid())) .port(port) .when() .patch("http://localhost/api/hs/office/partners/" + givenPartner.getUuid()) @@ -387,7 +387,7 @@ class HsOfficePartnerControllerAcceptanceTest extends ContextBasedTestWithCleanu .contentType(ContentType.JSON) .body(""" { - "partnerRelUuid": "%s" + "partnerRel.uuid": "%s" } """.formatted(givenPartnerRel.getUuid())) .port(port) diff --git a/src/test/java/net/hostsharing/hsadminng/hs/office/partner/HsOfficePartnerControllerRestTest.java b/src/test/java/net/hostsharing/hsadminng/hs/office/partner/HsOfficePartnerControllerRestTest.java index 42d0566c..889e776f 100644 --- a/src/test/java/net/hostsharing/hsadminng/hs/office/partner/HsOfficePartnerControllerRestTest.java +++ b/src/test/java/net/hostsharing/hsadminng/hs/office/partner/HsOfficePartnerControllerRestTest.java @@ -101,12 +101,12 @@ class HsOfficePartnerControllerRestTest { { "partnerNumber": "20002", "partnerRel": { - "anchorUuid": "%s", - "holderUuid": "%s", - "contactUuid": "%s" + "anchor.uuid": "%s", + "holder.uuid": "%s", + "contact.uuid": "%s" }, - "personUuid": "%s", - "contactUuid": "%s", + "person.uuid": "%s", + "contact.uuid": "%s", "details": { "registrationOffice": "Temp Registergericht Aurich", "registrationNumber": "111111" @@ -138,12 +138,12 @@ class HsOfficePartnerControllerRestTest { { "partnerNumber": "20002", "partnerRel": { - "anchorUuid": "%s", - "holderUuid": "%s", - "contactUuid": "%s" + "anchor.uuid": "%s", + "holder.uuid": "%s", + "contact.uuid": "%s" }, - "personUuid": "%s", - "contactUuid": "%s", + "person.uuid": "%s", + "contact.uuid": "%s", "details": { "registrationOffice": "Temp Registergericht Aurich", "registrationNumber": "111111" diff --git a/src/test/java/net/hostsharing/hsadminng/hs/office/relation/HsOfficeRelationControllerAcceptanceTest.java b/src/test/java/net/hostsharing/hsadminng/hs/office/relation/HsOfficeRelationControllerAcceptanceTest.java index 2dac741b..9bc4eafa 100644 --- a/src/test/java/net/hostsharing/hsadminng/hs/office/relation/HsOfficeRelationControllerAcceptanceTest.java +++ b/src/test/java/net/hostsharing/hsadminng/hs/office/relation/HsOfficeRelationControllerAcceptanceTest.java @@ -235,9 +235,9 @@ class HsOfficeRelationControllerAcceptanceTest extends ContextBasedTestWithClean { "type": "%s", "mark": "%s", - "anchorUuid": "%s", - "holderUuid": "%s", - "contactUuid": "%s" + "anchor.uuid": "%s", + "holder.uuid": "%s", + "contact.uuid": "%s" } """.formatted( HsOfficeRelationTypeResource.SUBSCRIBER, @@ -281,9 +281,9 @@ class HsOfficeRelationControllerAcceptanceTest extends ContextBasedTestWithClean .body(""" { "type": "%s", - "anchorUuid": "%s", - "holderUuid": "%s", - "contactUuid": "%s" + "anchor.uuid": "%s", + "holder.uuid": "%s", + "contact.uuid": "%s" } """.formatted( HsOfficeRelationTypeResource.DEBITOR, @@ -313,9 +313,9 @@ class HsOfficeRelationControllerAcceptanceTest extends ContextBasedTestWithClean .body(""" { "type": "%s", - "anchorUuid": "%s", - "holderUuid": "%s", - "contactUuid": "%s" + "anchor.uuid": "%s", + "holder.uuid": "%s", + "contact.uuid": "%s" } """.formatted( HsOfficeRelationTypeResource.DEBITOR, @@ -346,9 +346,9 @@ class HsOfficeRelationControllerAcceptanceTest extends ContextBasedTestWithClean .body(""" { "type": "%s", - "anchorUuid": "%s", - "holderUuid": "%s", - "contactUuid": "%s" + "anchor.uuid": "%s", + "holder.uuid": "%s", + "contact.uuid": "%s" } """.formatted( HsOfficeRelationTypeResource.DEBITOR, @@ -461,9 +461,9 @@ class HsOfficeRelationControllerAcceptanceTest extends ContextBasedTestWithClean .contentType(ContentType.JSON) .body(""" { - "contactUuid": "%s" + "contact.uuid": "%s" } - """.formatted(givenContact.getUuid())) + """.formatted(givenContact.getUuid())) .port(port) .when() .patch("http://localhost/api/hs/office/relations/" + givenRelation.getUuid()) diff --git a/src/test/java/net/hostsharing/hsadminng/hs/office/sepamandate/HsOfficeSepaMandateControllerAcceptanceTest.java b/src/test/java/net/hostsharing/hsadminng/hs/office/sepamandate/HsOfficeSepaMandateControllerAcceptanceTest.java index 48ed0d9c..82a011dc 100644 --- a/src/test/java/net/hostsharing/hsadminng/hs/office/sepamandate/HsOfficeSepaMandateControllerAcceptanceTest.java +++ b/src/test/java/net/hostsharing/hsadminng/hs/office/sepamandate/HsOfficeSepaMandateControllerAcceptanceTest.java @@ -144,8 +144,8 @@ class HsOfficeSepaMandateControllerAcceptanceTest extends ContextBasedTestWithCl .contentType(ContentType.JSON) .body(""" { - "debitorUuid": "%s", - "bankAccountUuid": "%s", + "debitor.uuid": "%s", + "bankAccount.uuid": "%s", "reference": "temp ref CAT A", "agreement": "2020-01-02", "validFrom": "2022-10-13" @@ -186,7 +186,7 @@ class HsOfficeSepaMandateControllerAcceptanceTest extends ContextBasedTestWithCl .contentType(ContentType.JSON) .body(""" { - "bankAccountUuid": "%s", + "bankAccount.uuid": "%s", "reference": "temp ref CAT B", "validFrom": "2022-10-13" } @@ -211,8 +211,8 @@ class HsOfficeSepaMandateControllerAcceptanceTest extends ContextBasedTestWithCl .contentType(ContentType.JSON) .body(""" { - "debitorUuid": "%s", - "bankAccountUuid": "%s", + "debitor.uuid": "%s", + "bankAccount.uuid": "%s", "reference": "temp ref CAT C", "agreement": "2022-10-12", "validFrom": "2022-10-13", @@ -241,8 +241,8 @@ class HsOfficeSepaMandateControllerAcceptanceTest extends ContextBasedTestWithCl .contentType(ContentType.JSON) .body(""" { - "debitorUuid": "%s", - "bankAccountUuid": "%s", + "debitor.uuid": "%s", + "bankAccount.uuid": "%s", "reference": "temp refCAT D", "agreement": "2022-10-12", "validFrom": "2022-10-13", -- 2.39.5 From ca4397736108aa5b2cdae5d1fae3f026a6837a83 Mon Sep 17 00:00:00 2001 From: Michael Hoennig Date: Sat, 9 Nov 2024 17:03:50 +0100 Subject: [PATCH 4/6] amend RestTests according to .uuid --- .../HsOfficeCoopAssetsTransactionControllerRestTest.java | 2 +- .../rbac/grant/RbacGrantControllerAcceptanceTest.java | 8 ++++---- .../hsadminng/rbac/role/RbacRoleControllerRestTest.java | 6 +++--- 3 files changed, 8 insertions(+), 8 deletions(-) diff --git a/src/test/java/net/hostsharing/hsadminng/hs/office/coopassets/HsOfficeCoopAssetsTransactionControllerRestTest.java b/src/test/java/net/hostsharing/hsadminng/hs/office/coopassets/HsOfficeCoopAssetsTransactionControllerRestTest.java index f00edff8..7fa7e3f5 100644 --- a/src/test/java/net/hostsharing/hsadminng/hs/office/coopassets/HsOfficeCoopAssetsTransactionControllerRestTest.java +++ b/src/test/java/net/hostsharing/hsadminng/hs/office/coopassets/HsOfficeCoopAssetsTransactionControllerRestTest.java @@ -49,7 +49,7 @@ class HsOfficeCoopAssetsTransactionControllerRestTest { enum BadRequestTestCases { MEMBERSHIP_UUID_MISSING( requestBody -> requestBody.without("membershipUuid"), - "[membershipUuid must not be null but is \"null\"]"), + "[membership.uuid must not be null but is \"null\"]"), TRANSACTION_TYPE_MISSING( requestBody -> requestBody.without("transactionType"), diff --git a/src/test/java/net/hostsharing/hsadminng/rbac/grant/RbacGrantControllerAcceptanceTest.java b/src/test/java/net/hostsharing/hsadminng/rbac/grant/RbacGrantControllerAcceptanceTest.java index 5906f3e3..0dff3e75 100644 --- a/src/test/java/net/hostsharing/hsadminng/rbac/grant/RbacGrantControllerAcceptanceTest.java +++ b/src/test/java/net/hostsharing/hsadminng/rbac/grant/RbacGrantControllerAcceptanceTest.java @@ -388,8 +388,8 @@ class RbacGrantControllerAcceptanceTest extends ContextBasedTest { .body(""" { "assumed": true, - "grantedRoleUuid": "%s", - "granteeSubjectUuid": "%s" + "grantedRole.uuid": "%s", + "granteeSubject.uuid": "%s" } """.formatted( grantedRole.getUuid(), @@ -424,8 +424,8 @@ class RbacGrantControllerAcceptanceTest extends ContextBasedTest { .body(""" { "assumed": true, - "grantedRoleUuid": "%s", - "granteeSubjectUuid": "%s" + "grantedRole.uuid": "%s", + "granteeSubject.uuid": "%s" } """.formatted( grantedRole.getUuid(), diff --git a/src/test/java/net/hostsharing/hsadminng/rbac/role/RbacRoleControllerRestTest.java b/src/test/java/net/hostsharing/hsadminng/rbac/role/RbacRoleControllerRestTest.java index 0f1abce6..a9f09345 100644 --- a/src/test/java/net/hostsharing/hsadminng/rbac/role/RbacRoleControllerRestTest.java +++ b/src/test/java/net/hostsharing/hsadminng/rbac/role/RbacRoleControllerRestTest.java @@ -76,8 +76,8 @@ class RbacRoleControllerRestTest { .andExpect(jsonPath("$[1].roleName", is("rbactest.customer#xxx:OWNER"))) .andExpect(jsonPath("$[2].roleName", is("rbactest.customer#xxx:ADMIN"))) .andExpect(jsonPath("$[2].uuid", is(customerXxxAdmin.getUuid().toString()))) - .andExpect(jsonPath("$[2].objectUuid", is(customerXxxAdmin.getObjectUuid().toString()))) - .andExpect(jsonPath("$[2].objectTable", is(customerXxxAdmin.getObjectTable().toString()))) - .andExpect(jsonPath("$[2].objectIdName", is(customerXxxAdmin.getObjectIdName().toString()))); + .andExpect(jsonPath("$[2].['object.uuid']", is(customerXxxAdmin.getObjectUuid().toString()))) + .andExpect(jsonPath("$[2].objectTable", is(customerXxxAdmin.getObjectTable()))) + .andExpect(jsonPath("$[2].objectIdName", is(customerXxxAdmin.getObjectIdName()))); } } -- 2.39.5 From 1d4d985a3e9ae4b928ed8815022f004294f1b47c Mon Sep 17 00:00:00 2001 From: Michael Hoennig Date: Sat, 9 Nov 2024 18:34:27 +0100 Subject: [PATCH 5/6] amend RestTests according to .uuid --- .../HsOfficeCoopAssetsTransactionControllerRestTest.java | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/test/java/net/hostsharing/hsadminng/hs/office/coopassets/HsOfficeCoopAssetsTransactionControllerRestTest.java b/src/test/java/net/hostsharing/hsadminng/hs/office/coopassets/HsOfficeCoopAssetsTransactionControllerRestTest.java index 7fa7e3f5..b64e0cae 100644 --- a/src/test/java/net/hostsharing/hsadminng/hs/office/coopassets/HsOfficeCoopAssetsTransactionControllerRestTest.java +++ b/src/test/java/net/hostsharing/hsadminng/hs/office/coopassets/HsOfficeCoopAssetsTransactionControllerRestTest.java @@ -121,10 +121,10 @@ class HsOfficeCoopAssetsTransactionControllerRestTest { .accept(MediaType.APPLICATION_JSON)) // then - .andExpect(status().is4xxClientError()) + .andExpect(jsonPath("message", is("ERROR: [400] " + testCase.expectedErrorMessage))) .andExpect(jsonPath("statusCode", is(400))) .andExpect(jsonPath("statusPhrase", is("Bad Request"))) - .andExpect(jsonPath("message", is("ERROR: [400] " + testCase.expectedErrorMessage))); + .andExpect(status().is4xxClientError()); } } -- 2.39.5 From 9531f6a881bac0b0e2f1f84862ba2d4633aa76e0 Mon Sep 17 00:00:00 2001 From: Michael Hoennig Date: Sat, 9 Nov 2024 20:54:59 +0100 Subject: [PATCH 6/6] amend RestTests according to .uuid - wrong error-message mapping in SpringValidation --- .../coopassets/HsOfficeCoopAssetsTransactionController.java | 1 - .../HsOfficeCoopAssetsTransactionControllerRestTest.java | 4 ++-- 2 files changed, 2 insertions(+), 3 deletions(-) diff --git a/src/main/java/net/hostsharing/hsadminng/hs/office/coopassets/HsOfficeCoopAssetsTransactionController.java b/src/main/java/net/hostsharing/hsadminng/hs/office/coopassets/HsOfficeCoopAssetsTransactionController.java index 9154b8f0..b8da7580 100644 --- a/src/main/java/net/hostsharing/hsadminng/hs/office/coopassets/HsOfficeCoopAssetsTransactionController.java +++ b/src/main/java/net/hostsharing/hsadminng/hs/office/coopassets/HsOfficeCoopAssetsTransactionController.java @@ -77,7 +77,6 @@ public class HsOfficeCoopAssetsTransactionController implements HsOfficeCoopAsse @Override @Transactional(readOnly = true) - public ResponseEntity getCoopAssetTransactionByUuid( final String currentSubject, final String assumedRoles, final UUID assetTransactionUuid) { diff --git a/src/test/java/net/hostsharing/hsadminng/hs/office/coopassets/HsOfficeCoopAssetsTransactionControllerRestTest.java b/src/test/java/net/hostsharing/hsadminng/hs/office/coopassets/HsOfficeCoopAssetsTransactionControllerRestTest.java index b64e0cae..b7f44b35 100644 --- a/src/test/java/net/hostsharing/hsadminng/hs/office/coopassets/HsOfficeCoopAssetsTransactionControllerRestTest.java +++ b/src/test/java/net/hostsharing/hsadminng/hs/office/coopassets/HsOfficeCoopAssetsTransactionControllerRestTest.java @@ -48,8 +48,8 @@ class HsOfficeCoopAssetsTransactionControllerRestTest { enum BadRequestTestCases { MEMBERSHIP_UUID_MISSING( - requestBody -> requestBody.without("membershipUuid"), - "[membership.uuid must not be null but is \"null\"]"), + requestBody -> requestBody.without("membership.uuid"), + "[membershipUuid must not be null but is \"null\"]"), // TODO.impl: should be membership.uuid, Spring validation-problem? TRANSACTION_TYPE_MISSING( requestBody -> requestBody.without("transactionType"), -- 2.39.5